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

doc: remove deprecated 'scrubq' from ceph(8) #23959

Merged

Conversation

rubenk
Copy link
Contributor

@rubenk rubenk commented Sep 6, 2018

The option was removed in a30cbe9

Signed-off-by: Ruben Kerkhof ruben@rubenkerkhof.com

  • References tracker ticket
  • Updates documentation if necessary
  • Includes tests for new functionality or reproducer for bug

@tchaikov
Copy link
Contributor

tchaikov commented Sep 6, 2018

strictly speaking, we should backport this change to mimic and luminous.

@tchaikov
Copy link
Contributor

tchaikov commented Sep 6, 2018

@rubenk could you add a "Fixed: http://tracker.ceph.com/issues/35813" line in your commit message? so it'd be easier for the release team to backport this change.

The option was removed in a30cbe9

Fixes: http://tracker.ceph.com/issues/35813

Signed-off-by: Ruben Kerkhof <ruben@rubenkerkhof.com>
@rubenk rubenk force-pushed the doc-remove-unknown-option-from-manpage branch from 9fa11cf to d05cd1a Compare September 6, 2018 14:24
@rubenk
Copy link
Contributor Author

rubenk commented Sep 6, 2018

Sure thing, done.

@tchaikov
Copy link
Contributor

tchaikov commented Sep 7, 2018

retest this please.

1 similar comment
@tchaikov
Copy link
Contributor

tchaikov commented Sep 7, 2018

retest this please.

@tchaikov tchaikov merged commit d4a7814 into ceph:master Sep 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants