Skip to content

doc: drop obsolete cache tier options#35015

Merged
smithfarm merged 2 commits intoceph:masterfrom
smithfarm:wip-45514
May 18, 2020
Merged

doc: drop obsolete cache tier options#35015
smithfarm merged 2 commits intoceph:masterfrom
smithfarm:wip-45514

Conversation

@smithfarm
Copy link
Contributor

Fixes: https://tracker.ceph.com/issues/45514
Signed-off-by: Nathan Cutler ncutler@suse.com

@smithfarm
Copy link
Contributor Author

@jdurgin Thanks for reviewing. I just found another instance of it - could you take a look at the new commit I just added?

@smithfarm smithfarm changed the title doc: drop obsolete cache tier options from man page doc: drop obsolete cache tier options May 13, 2020
@smithfarm
Copy link
Contributor Author

since

  • this is a documentation thing
  • it passed "make check"
  • I did not find any test under qa/ that examines the contextual help message

I'll merge this now

@smithfarm smithfarm merged commit ec13e8d into ceph:master May 18, 2020
@smithfarm smithfarm deleted the wip-45514 branch May 18, 2020 13:18
@neha-ojha
Copy link
Member

@smithfarm
Copy link
Contributor Author

@neha-ojha Sorry about that! #35139 is open to fix.

@neha-ojha
Copy link
Member

@neha-ojha Sorry about that! #35139 is open to fix.

Thanks, let's use https://tracker.ceph.com/issues/45614 to track it.

@tchaikov
Copy link
Contributor

since

  • this is a documentation thing
  • it passed "make check"
  • I did not find any test under qa/ that examines the contextual help message

I'll merge this now

i encourage to add the test result before merging a PR. we've been breaking the tests recently.

@NUABO NUABO mentioned this pull request Mar 16, 2022
14 tasks
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.

4 participants