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-disk: create deprecation warnings #18988
Conversation
doc/ceph-volume/index.rst
Outdated
@@ -17,6 +17,33 @@ that may have been deployed with ``ceph-disk``. | |||
* :ref:`ceph-volume-lvm` | |||
* :ref:`ceph-volume-simple` | |||
|
|||
|
|||
migrating |
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.
Migrating
src/ceph-disk/ceph_disk/main.py
Outdated
try: | ||
args.func(args) | ||
except Exception: | ||
# warn on any exception when not running with verbosity |
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.
is this backwards? this looks like the verbose path
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.
Yes, my wording is backwards, hah!
Signed-off-by: Alfredo Deza <adeza@redhat.com> Fixes: http://tracker.ceph.com/issues/22154
…ceph-disk Signed-off-by: Alfredo Deza <adeza@redhat.com>
75a908a
to
e433efe
Compare
@liewegas Should we backport this to luminous? (N.B. I checked the Luminous v12.2.0 and v12.2.1 release notes and didn't find any mention of ceph-disk being deprecated.) |
@smithfarm it was already backported here #18989 |
@andrewschoen OK. Would it make sense to add something to |
@smithfarm sure, that sounds reasonable. |
Adds a banner-like deprecation warning for ceph-disk, and a doc section to help ceph-disk users what the correct way of migrating or consuming ceph-volume would be, with the noted exception of encryption