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
ceph_objectstore_tool.py: Don't use btrfs on FreeBSD #10507
ceph_objectstore_tool.py: Don't use btrfs on FreeBSD #10507
Conversation
@@ -1909,6 +1909,7 @@ def remove_btrfs_subvolumes(path): | |||
status = main(sys.argv[1:]) | |||
finally: | |||
kill_daemons() | |||
remove_btrfs_subvolumes(CEPH_DIR) | |||
if platform.system() != "FreeBSD": |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit, could you move this check into remove_btrfs_subvolumes()
?
605356e
to
71d4a1c
Compare
@tchaikov |
lgtm @dzafman what do you think? |
@@ -1889,6 +1889,8 @@ def main(argv): | |||
|
|||
|
|||
def remove_btrfs_subvolumes(path): | |||
if platform.system() != "FreeBSD": |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This shouldn't be: if platform.system() == "FreeBSD": ?
@tchaikov I don't understand this change. This looks like the purpose of the remove_btrfs_subvolumes() is exclusively for FreeBSD. See my comment on the code. |
@dzafman sorry, i should have checked this change more closely. yes, it should be if platform.system() == "FreeBSD" @wjwithagen could you fix this? |
Signed-off-by: Willem Jan Withagen <wjw@digiware.nl>
71d4a1c
to
bc03507
Compare
please retest this |
@tchaikov |
retest this please. |
test this please |
No description provided.