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: kill some broken links #15203

Merged
merged 1 commit into from Jun 8, 2017

Conversation

Projects
None yet
3 participants
@Liuchang0812
Contributor

Liuchang0812 commented May 22, 2017

Signed-off-by: liuchang0812 liuchang0812@gmail.com

@@ -30,7 +30,7 @@ API`_ provides a complete example. It is pulled into Sphinx by
`librados.rst`_, which is rendered at :doc:`/rados/api/librados`.
.. _`librados C API`: https://github.com/ceph/ceph/blob/master/src/include/rados/librados.h
.. _`librados.rst`: https://raw.github.com/ceph/ceph/master/doc/api/librados.rst
.. _`librados.rst`: https://github.com/ceph/ceph/raw/master/doc/rados/api/librados.rst

This comment has been minimized.

@tchaikov

tchaikov May 22, 2017

Contributor

can we reference the local copy of librados.rst using relative path?

This comment has been minimized.

@Liuchang0812

Liuchang0812 May 22, 2017

Contributor

i will have a try

This comment has been minimized.

@Liuchang0812

Liuchang0812 Jun 5, 2017

Contributor

we have to keep this URI.

➜  WorkSpace curl -q -I  http://localhost:8080/rados/api/librados/
HTTP/1.0 200 OK
Server: SimpleHTTP/0.6 Python/2.7.12
Date: Mon, 05 Jun 2017 08:34:54 GMT
Content-type: text/html
Content-Length: 377413
Last-Modified: Mon, 05 Jun 2017 08:30:42 GMT

➜  WorkSpace curl -q -I  http://localhost:8080/rados/api/librados.rst
HTTP/1.0 404 File not found
Server: SimpleHTTP/0.6 Python/2.7.12
Date: Mon, 05 Jun 2017 08:34:59 GMT
Connection: close
Content-Type: text/html
are pseudo-randomly evenly distributed to the PGs. So a PG explicitly
does NOT represent a fixed amount of storage; it represents 1/pg_num
'th of the storage you happen to have on your OSDs.
number of PGs you want, number of OSDs * 100 is a good starting point

This comment has been minimized.

@tchaikov

tchaikov May 22, 2017

Contributor

is it intentional to drop the link?

This comment has been minimized.

@Liuchang0812

Liuchang0812 May 22, 2017

Contributor

i searched it, but i can't find it in our wiki. I think it's ok to remove this link

@@ -15,7 +15,6 @@ Locations
These mirrors are available on the following locations:
- **EU: Netherlands**: http://eu.ceph.com/
- **AU: Australia**: http://au.ceph.com/

This comment has been minimized.

@tchaikov

tchaikov May 22, 2017

Contributor

@wido could you help confirm this? is this transient issue ? DNS still resolves, but seems the web server is not accessible anymore.

This comment has been minimized.

This comment has been minimized.

@Liuchang0812

Liuchang0812 Jun 6, 2017

Contributor

🐒 a magical site....

@@ -6,7 +6,7 @@ The *jerasure* plugin is the most generic and flexible plugin, it is
also the default for Ceph erasure coded pools.
The *jerasure* plugin encapsulates the `Jerasure
<https://bitbucket.org/jimplank/jerasure/>`_ library. It is
<https://github.com/tsuraan/Jerasure/>`_ library. It is

This comment has been minimized.

@tchaikov

tchaikov May 22, 2017

Contributor

i am under the impression that jerasure's upstream web is jerasure.org, but seems the domain name is expired. we failed to renew it?

it should be http://jerasure.org or http://lab.jerasure.org/jerasure/jerasure

This comment has been minimized.

@Liuchang0812

Liuchang0812 Jun 5, 2017

Contributor

done

@@ -12,11 +12,11 @@ These are exciting times in the Ceph community! Get involved!
| **Blog** | Check the Ceph Blog_ periodically to keep track | http://ceph.com/community/blog/ |
| | of Ceph progress and important announcements. | |
+----------------------+-------------------------------------------------+-----------------------------------------------+
| **Planet Ceph** | Check the blog aggregation on Planet Ceph for | http://ceph.com/community/planet-ceph/ |
| **Planet Ceph** | Check the blog aggregation on Planet Ceph for | http://ceph.com/category/planet/ |

This comment has been minimized.

@tchaikov

tchaikov May 22, 2017

Contributor

i am able to access https://ceph.com/category/planet/, so probably we can s/http/https/ ?

This comment has been minimized.

@Liuchang0812

Liuchang0812 May 22, 2017

Contributor

good! seems that some sub-sites under ceph.com is HTTP only(wiki.ceph.com) it would be better to support HTTPS.

This comment has been minimized.

@Liuchang0812

Liuchang0812 Jun 5, 2017

Contributor

done

This comment has been minimized.

@tchaikov

tchaikov Jun 6, 2017

Contributor

@Liuchang0812 it's not addressed.

This comment has been minimized.

@Liuchang0812

Liuchang0812 Jun 6, 2017

Contributor

.... I made some mistakes when i merged code. sorry for that. I will check this PR again.

@@ -1136,7 +1136,7 @@ files. For instance, all tests in the `rados/thrash suite
<https://github.com/ceph/ceph/tree/master/qa/suites/rados/thrash>`_
run for ``xfs``, ``btrfs`` and ``ext4`` because they are combined (via
special file ``%``) with the `fs directory
<https://github.com/ceph/ceph/tree/master/qa/suites/rados/thrash/fs>`_
<https://github.com/ceph/ceph/tree/master/qa/suites/fs/>`_

This comment has been minimized.

@Liuchang0812

Liuchang0812 Jun 6, 2017

Contributor

I'm not sure whether it is correct

This comment has been minimized.

@tchaikov

tchaikov Jun 6, 2017

Contributor

#15511 should address this, probably you could drop this change from your PR.

This comment has been minimized.

@Liuchang0812

Liuchang0812 Jun 6, 2017

Contributor

great!

@@ -135,7 +135,7 @@ in the body of the message.
There are also `other Ceph-related mailing lists`_.
.. _`other Ceph-related mailing lists`: https://ceph.com/resources/mailing-list-irc/
.. _`other Ceph-related mailing lists`: http://ceph.com/irc/

This comment has been minimized.

@tchaikov

tchaikov Jun 6, 2017

Contributor

s/http/https/

@@ -145,7 +145,7 @@ time using `Internet Relay Chat`_.
.. _`Internet Relay Chat`: http://www.irchelp.org/
See https://ceph.com/resources/mailing-list-irc/ for how to set up your IRC
See http://ceph.com/irc/ for how to set up your IRC

This comment has been minimized.

@tchaikov

tchaikov Jun 6, 2017

Contributor

ditto.

@@ -750,7 +750,7 @@ The results of the nightlies are published at http://pulpito.ceph.com/ and
http://pulpito.ovh.sepia.ceph.com:8081/. The developer nick shows in the
test results URL and in the first column of the Pulpito dashboard. The
results are also reported on the `ceph-qa mailing list
<http://ceph.com/resources/mailing-list-irc/>`_ for analysis.
<http://ceph.com/irc/>`_ for analysis.

This comment has been minimized.

@tchaikov

tchaikov Jun 6, 2017

Contributor

ditto.

@@ -12,11 +12,11 @@ These are exciting times in the Ceph community! Get involved!
| **Blog** | Check the Ceph Blog_ periodically to keep track | http://ceph.com/community/blog/ |
| | of Ceph progress and important announcements. | |
+----------------------+-------------------------------------------------+-----------------------------------------------+
| **Planet Ceph** | Check the blog aggregation on Planet Ceph for | http://ceph.com/community/planet-ceph/ |
| **Planet Ceph** | Check the blog aggregation on Planet Ceph for | http://ceph.com/category/planet/ |

This comment has been minimized.

@tchaikov

tchaikov Jun 6, 2017

Contributor

@Liuchang0812 it's not addressed.

@@ -1201,7 +1201,7 @@ ceph-qa-suite`_ tool, you will need to install that first. It is designed
to be installed via Docker, so if you don't have Docker running on your
development machine, take care of that first. The Docker project has a good
tutorial called `Get Started with Docker Engine for Linux
<https://docs.docker.com/linux/>`_ if you unsure how to proceed.
<https://docs.docker.com/get-started/>`_ if you unsure how to proceed.

This comment has been minimized.

@Liuchang0812

Liuchang0812 Jun 6, 2017

Contributor

The Docker organization removed this tutorial, and I can't find any mirror. Maybe we could delete this sentence or suggest other tutorial

This comment has been minimized.

@tchaikov

tchaikov Jun 6, 2017

Contributor

probably you can use https://docs.docker.com/engine/installation/ , and update the text for the hyper link to reflect the change?

doc: kill some broken links
Signed-off-by: liuchang0812 <liuchang0812@gmail.com>
@Liuchang0812

This comment has been minimized.

Contributor

Liuchang0812 commented Jun 8, 2017

repushed(addressed all your comments carefully). @tchaikov

@tchaikov tchaikov merged commit d010f6a into ceph:master Jun 8, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment