]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
mgr/cephadm: use fixed config file path as argument
authorPatrick Seidensal <pseidensal@suse.com>
Wed, 5 Aug 2020 14:47:13 +0000 (16:47 +0200)
committerSebastian Wagner <sebastian.wagner@suse.com>
Fri, 21 Aug 2020 11:04:01 +0000 (13:04 +0200)
Fixes other alertmanager containers which do not default to look for a
configuration file in `/etc/alertmanager/alertmanager.yml`.

Signed-off-by: Patrick Seidensal <pseidensal@suse.com>
(cherry picked from commit cdd0ba13a5109e4319ffdaff8e8a17212e78dd15)

src/cephadm/cephadm

index 829f56bb9e8c8c243ba8bdcaaa619808955d69d9..d6f85f1ad8811a0ef2120cde2f6f8e4be8c1b79c 100755 (executable)
@@ -1544,6 +1544,8 @@ def get_daemon_args(fsid, daemon_type, daemon_id):
             peers = config.get('peers', list())  # type: ignore
             for peer in peers:
                 r += ["--cluster.peer={}".format(peer)]
+            # some alertmanager, by default, look elsewhere for a config
+            r += ["--config.file=/etc/alertmanager/alertmanager.yml"]
     elif daemon_type == NFSGanesha.daemon_type:
         nfs_ganesha = NFSGanesha.init(fsid, daemon_id)
         r += nfs_ganesha.get_daemon_args()
@@ -1840,7 +1842,7 @@ def get_container(fsid, daemon_type, daemon_id,
 
 
 def extract_uid_gid(img='', file_path='/var/lib/ceph'):
-    # type: (str, Union[str, Iterabe[str]]) -> Tuple[int, int]
+    # type: (str, Union[str, List[str]]) -> Tuple[int, int]
 
     if not img:
         img = args.image