Project

General

Profile

Actions
Copy link

Bug #55478

closed

cephadm calls podman using ".Id" which is only available on podman (Docker doesn't support this field)

Added by Redouane Kachach Elhichou about 2 years ago. Updated about 2 years ago.

Status:
Rejected
Priority:
Normal
Assignee:
Redouane Kachach Elhichou
Category:
cephadm (binary)
Target version:
-
% Done:

0%

Source:
Tags:
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

in cephadm binary:


def get_container_stats(ctx: CephadmContext, container_path: str, fsid: str, daemon_type: str, daemon_id: str) -> Tuple[str, str, int]:
    c = CephContainer.for_daemon(ctx, fsid, daemon_type, daemon_id, 'bash')
    out, err, code = '', '', -1
    for name in (c.cname, c.old_cname):
        cmd = [
            container_path, 'inspect',
            '--format', '{{.Id}},{{.Config.Image}},{{.Image}},{{.Created}},{{index .Config.Labels "io.ceph.version"}}',
            name
        ]

Actions
Copy link
 #1

Updated by Redouane Kachach Elhichou about 2 years ago

  • Status changed from New to Rejected

End up to be not an issue. This field (.Id) is supported for containers but not for images.

Actions
Copy link

Also available in: Atom PDF