Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

pacific: src/pybind/mgr/mirroring/fs/snapshot_mirror.py: do not assume a cephf… #42226

Merged
merged 1 commit into from
Jul 15, 2021

Conversation

cfsnyder
Copy link
Contributor

@cfsnyder cfsnyder commented Jul 7, 2021

backport tracker: https://tracker.ceph.com/issues/51547


backport of #42139
parent tracker: https://tracker.ceph.com/issues/51476

this backport was staged using ceph-backport.sh version 16.0.0.6848
find the latest version at https://github.com/ceph/ceph/blob/master/src/script/ceph-backport.sh

…s-mirror daemon is always running

We should not assume a daemon is runnning. If the daemon is not running
we get the default value of None. So let's skip the status if no daemon
is running yet.

Fixes: https://tracker.ceph.com/issues/51476
Signed-off-by: Sébastien Han <seb@redhat.com>
(cherry picked from commit dd45ec4)
@cfsnyder cfsnyder added this to the pacific milestone Jul 7, 2021
@cfsnyder cfsnyder added the pybind label Jul 7, 2021
@leseb leseb requested review from vshankar and batrick July 12, 2021 08:10
@yuriw yuriw merged commit 35d2af8 into ceph:pacific Jul 15, 2021
votdev pushed a commit to votdev/ceph that referenced this pull request Aug 2, 2021
pacific: src/pybind/mgr/mirroring/fs/snapshot_mirror.py: do not assume a cephf…

Reviewed-by: Venky Shankar <vshankar@redhat.com>
Reviewed-by: Sébastien Han <seb@redhat.com>
Reviewed-by: Jos Collin <jcollin@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants