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-volume: adds --crush-device-class flag for lvm prepare and create #19949
Conversation
src/ceph-disk/ceph_disk/main.py
Outdated
@@ -2755,7 +2758,7 @@ def create_key(self): | |||
bootstrap = self.args.prepare_key_template.format(cluster=cluster, | |||
statedir=STATEDIR) | |||
path = self.get_mount_point() | |||
secrets = LockboxSecrets(self.args) | |||
secrets = LockboxSecrets(self.args, crush_device_class=self.args.crush_device_class) |
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, i don't think we need to pass crush_device_class
a separately if it's already in self.args
.
Signed-off-by: Andrew Schoen <aschoen@redhat.com>
Signed-off-by: Andrew Schoen <aschoen@redhat.com>
Signed-off-by: Andrew Schoen <aschoen@redhat.com>
Signed-off-by: Andrew Schoen <aschoen@redhat.com>
Signed-off-by: Andrew Schoen <aschoen@redhat.com>
47087b4
to
21f4594
Compare
@alfredodeza @tchaikov I've removed the |
jenkins test ceph-volume tox |
jenkins test ceph-volume lvm centos7-bluestore-create |
1 similar comment
jenkins test ceph-volume lvm centos7-bluestore-create |
Move the terminal.success to the methods that have access to the correct osd_id so that they do not print None for the osd_id in the log message. Signed-off-by: Andrew Schoen <aschoen@redhat.com>
jenkins test ceph-volume lvm all |
luminous: ceph-volume: adds --crush-device-class flag for lvm prepare and create #19949 Reviewed-by: Alfredo Deza <adeza@redhat.com>
This supports the new method of setting the crush device class through the json given to the
osd new
command. Introduced here: #19939Needs this for ceph-ansible support and for CI testing: ceph/ceph-ansible#2318
Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1498521