* refs/pull/33190/head:
cephadm: report image name for stopped daemons
cephadm: record the image name to unit.image
Reviewed-by: Michael Fritch <mfritch@suse.com>
When deploying a crash container, the crash keyring is the crash keyring,
and there is not additional crash_keyring for the implicit crash
container.
Signed-off-by: Sage Weil <sage@redhat.com>
Use the normal unit file, with crash.$hostname as the name. This makes
the crash agent behave like all of the other services instead of being
special. The exception is that the crash agent is still implicitly
deployed alongside other containers.
Signed-off-by: Sage Weil <sage@redhat.com>
* refs/pull/33162/head:
cephadm: KillMode=none in unit file
cephadm: clean up unit string syntax a bit
Reviewed-by: Sebastian Wagner <swagner@suse.com>
Other parts of this script leave OSDs reweighted, which can make this test
fail to go fully clean.
0 ssd 0.08789 osd.0 up 0.63213 1.00000
1 ssd 0.08789 osd.1 up 0.63213 1.00000
2 ssd 0.08789 osd.2 up 1.00000 1.00000
35.0 raw ([2,1,2147483647], p2) up ([2,1,2147483647], p2) acting ([2,1,2], p2)
Fix by just deleting this pool when we're done.
Fixes: https://tracker.ceph.com/issues/44067
Signed-off-by: Sage Weil <sage@redhat.com>
The mirroring site name is stored in the MON config which requires
higher privledges than the standard "client.mirror" user.
Fixes: https://tracker.ceph.com/issues/44066
Signed-off-by: Jason Dillaman <dillaman@redhat.com>
otherwise some packages which uses six.ensure_text will fail, as it
was introduced by 1.12. and 1.14 is the latest stable version of
six. so let's use it instead.
Fixes: https://tracker.ceph.com/issues/44063
Signed-off-by: Kefu Chai <kchai@redhat.com>
When snapshot-based mirroring is enabled, the primary image
should always have at least one mirror-based snapshot to help
distinguish it from an incomplete non-primary image.
Signed-off-by: Jason Dillaman <dillaman@redhat.com>
Address "Deprecation notice for authentication via URL query parameters"
and other recent GitHub API behavior changes.
Signed-off-by: Nathan Cutler <ncutler@suse.com>
- add "Tell $type commands" heading
- 'ceph tell mon.a -h' now works
- 'ceph tell mon.a prefix -h' also works
Signed-off-by: Sage Weil <sage@redhat.com>
if "ccache" is used, compiler[0] would be "ccache", and compiler[1]
would be "clang", in this case, we should search for "clang" until
an argument starting with "-" is found. it signals be the first
cflag passed to the compiler.
Signed-off-by: Kefu Chai <kchai@redhat.com>
otherwise we are always looking at the default compiler on 'unix' --
'cc'. and will fail to filter cflags not supported by clang, if we
are compiling the python bindings using clang.
Signed-off-by: Kefu Chai <kchai@redhat.com>
`copy.deepcopy` doesn't work to cross sub-interpreter boundaries.
Fixes: https://tracker.ceph.com/issues/43913
Signed-off-by: Sebastian Wagner <sebastian.wagner@suse.com>
Sometimes the PR body has multiple newlines which do not need
to be quoted. Also, the way we interact with the GitHub API, we
were getting some newline duplication.
This patch reduces all clumps of multiple blank lines down to a single
blank line.
Signed-off-by: Nathan Cutler <ncutler@suse.com>
when ceph-mon starts, it checks to see if it's listed in the monmap, if
not it complains
```
no public_addr or public_network specified, and mon.a not present in
monmap or ceph.conf.
```
then bails out. normally, the monitor will try to rename its name in
monmap when performing "mkfs", but in our case, we are merely using the
"mkfs" monmap for passing the monmap built by ceph-monstore-tools, and
we don't actually go through the "mkfs" process. so, ceph-mon won't
rename when booting up.
in this change, user is allowed to specify the mon-ids in command line
when rebuilding mondb, the default mon-ids would be a,b,c,... if not
specified.
Signed-off-by: Kefu Chai <kchai@redhat.com>
Reset the progress each time we make an adjustment, and track progress
from that initial state to that new target. Previously we were also
using the wrong target: the current pg_num_target, not the new value
(pg_num_final) that we set.
Look up the pool by name, not id, in _maybe_adjust(), since that is how it is
retrieved by osdmap.get_pools_by_name().
Dedupe some logic into PgAdjustmentProgress to simplify things.
Signed-off-by: Josh Durgin <jdurgin@redhat.com>
