]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
cephadm: emit warning if daemon's image is not to be used 61721/head
authorMatthew Vernon <mvernon@wikimedia.org>
Wed, 28 Aug 2024 15:37:46 +0000 (16:37 +0100)
committerAdam King <adking@redhat.com>
Fri, 7 Feb 2025 21:30:49 +0000 (16:30 -0500)
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>
(cherry picked from commit b863c93ef1a1ce85164584dd17c5e71441bc550f)

src/cephadm/cephadm.py

index c718106400f192ed42064f2eded12e7d177e5966..ea970489663a318eb8906e7d336723a8c84badd6 100755 (executable)
@@ -2738,6 +2738,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