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: fix the links to http://ceph.com/docs #13976

Merged
merged 1 commit into from Mar 15, 2017

Conversation

tchaikov
Copy link
Contributor

they should point to http://docs.ceph.com/docs/master/.. instead

Signed-off-by: Kefu Chai kchai@redhat.com

Copy link
Member

@joscollin joscollin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it required to modify the following files, as it is changelogs? Please check and confirm.

  1. doc/changelog/v0.61.3.txt
  2. doc/changelog/v0.94.3.txt

The following files are missing:

  1. doc/rados/configuration/auth-config-ref.rst
  2. doc/rbd/rbd-openstack.rst
  3. doc/start/hardware-recommendations.rst

You can include the details of Cleanup #19090 too in the heading, so that we can close that too.

they should point to http://docs.ceph.com/docs/master/.. instead

Fixes: http://tracker.ceph.com/issues/19090
Signed-off-by: Kefu Chai <kchai@redhat.com>
@tchaikov
Copy link
Contributor Author

Is it required to modify the following files, as it is changelogs? Please check and confirm.

not required, but i think it would be better of have a reachable link even we've released 0.61.3 and 0.94.3

fixed and repushed.

Copy link
Member

@joscollin joscollin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved

@tchaikov tchaikov merged commit 7f1e575 into ceph:master Mar 15, 2017
@tchaikov tchaikov deleted the wip-fix-docs-links branch March 15, 2017 10:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants