Project

General

Profile

Actions
Copy link

Bug #44559

closed

cephadm logs an invalid stat command

Added by Nathan Cutler about 4 years ago. Updated about 3 years ago.

Status:
Can't reproduce
Priority:
Low
Assignee:
-
Category:
-
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

When I run "cephadm bootstrap", I see the following in the log:

INFO:cephadm:Extracting ceph user uid/gid from container image...
DEBUG:cephadm:['/usr/bin/podman', 'run', '--rm', '--net=host', '-e', 'CONTAINER_IMAGE=registry.opensuse.org/filesystems/ceph/octopus/images/ceph/ceph', '-e', 'NODE_NAME=admin', '--entrypoint', 'stat', 'registry.opensuse.org/filesystems/ceph/octopus/images/ceph/ceph', '-c', '%u %g', '/var/lib/ceph']
DEBUG:cephadm:Running command: /usr/bin/podman run --rm --net=host -e CONTAINER_IMAGE=registry.opensuse.org/filesystems/ceph/octopus/images/ceph/ceph -e NODE_NAME=admin --entrypoint stat registry.opensuse.org/filesystems/ceph/octopus/images/ceph/ceph -c %u %g /var/lib/ceph

But the actual command being run is:

"/usr/bin/podman" "run" "--rm" "--net=host" "-e" "CONTAINER_IMAGE=registry.opensuse.org/filesystems/ceph/octopus/images/ceph/ceph" "-e" "NODE_NAME=admin" "--entrypoint stat registry.opensuse.org/filesystems/ceph/octopus/images/ceph/ceph" "-c" "%u %g" "/var/lib/ceph"

This is important because stat -c %u %g /var/lib/ceph is syntactically incorrect and will produce an error if you try to run it by copying the command as cephadm logs it.

Actions
Copy link
 #1

Updated by Nathan Cutler about 4 years ago

  • Description updated (diff)
  • Status changed from New to Fix Under Review
  • Pull request ID set to 33881
Actions
Copy link
 #2

Updated by Kristoffer Gronlund about 4 years ago

Shouldn't that be

stat -c "%u %g" /var/lib/ceph

?

Seems like a strange thing to differ between distributions..

Actions
Copy link
 #3

Updated by Nathan Cutler about 4 years ago

  • Subject changed from cephadm tries to run invalid "stat" command inside the ceph container to cephadm logs an invalid stat command
  • Description updated (diff)
Actions
Copy link
 #4

Updated by Nathan Cutler about 4 years ago

Thanks Kris - updated the bug description.

Actions
Copy link
 #5

Updated by Nathan Cutler about 4 years ago

  • Status changed from Fix Under Review to New
  • Pull request ID deleted (33881)
Actions
Copy link
 #6

Updated by Sebastian Wagner about 4 years ago

just to clarify,

stat -c %u %g /var/lib/ceph

is incorrect and

stat -c "%u %g" /var/lib/ceph

is correct?

Actions
Copy link
 #7

Updated by Sebastian Wagner about 4 years ago

  • Status changed from New to Need More Info
Actions
Copy link
 #8

Updated by Nathan Cutler almost 4 years ago

  • Status changed from Need More Info to In Progress
  • Priority changed from Normal to Low

I still have this on my plate. The "fix" here is to include the double quotes in the log message. This isn't a bug per se (cephadm issues the command with double quotes, but logs it without).

Actions
Copy link
 #9

Updated by Nathan Cutler about 3 years ago

  • Status changed from In Progress to New
  • Assignee deleted (Nathan Cutler)

(no longer on my plate)

Actions
Copy link
 #10

Updated by Sebastian Wagner about 3 years ago

  • Status changed from New to Can't reproduce

please reopen, if you see this again

Actions
Copy link

Also available in: Atom PDF