]> git.apps.os.sepia.ceph.com Git - ceph-ci.git/commitdiff
cephadm: emit warning if daemon's image is not to be used
authorMatthew Vernon <mvernon@wikimedia.org>
Wed, 28 Aug 2024 15:37:46 +0000 (16:37 +0100)
committerMatthew Vernon <mvernon@wikimedia.org>
Wed, 28 Aug 2024 18:04:18 +0000 (19:04 +0100)
If an image is not specified, cephadm shell will use the image
corresponding to a Ceph daemon running on the host (and will log a
debug message to that effect).

However, it will only use that image if it appears in the output of:

podman images --filter label=ceph=True --filter dangling=false

This commit means that cephadm will emit a warning if the container
image it was going to use fails this check, so the operator has more
of a clue to what is going on.

Fixes: https://tracker.ceph.com/issues/67778
Signed-off-by: Matthew Vernon <mvernon@wikimedia.org>
src/cephadm/cephadm.py

index e71addf7bfa5a6a05cf2bb08b3bd4d66f35affe1..587c2c0529be2276c6bb17c3bab48b2c11f79ffb 100755 (executable)
@@ -595,6 +595,8 @@ def infer_local_ceph_image(ctx: CephadmContext, container_path: str) -> Optional
             if digest and not digest.endswith('@'):
                 logger.info(f"Using ceph image with id '{image_id}' and tag '{tag}' created on {created_date}\n{digest}")
                 return digest
+    if container_info is not None:
+        logger.warning(f"Not using image '{container_info.image_id}' as it's not in list of non-dangling images with ceph=True label")
     return None