docs-community icon indicating copy to clipboard operation
docs-community copied to clipboard

Host the Python docs on ReadTheDocs

Open JulienPalard opened this issue 4 years ago • 40 comments
trafficstars

From time to time the discussion arise about moving docs.python.org to readthedocs, there's even an experimental python.readthedocs.io.

I think there's many pros and cons of using readthedocs.

I'd personally be in favor of it if we (the PSF) support them (the cpython Doc is a big one with a lot of traffic, it take around 24h of CPU to build all versions × languages with all PDF A4, PDF letter, HTML, plaintext, epub).

In the other hand it's not an easy task, among other things docs.python.org is not only about generating the docs but also hosting history:

  • https://docs.python.org/release/ (this is all the tagged commits in the cpython repo)
  • https://docs.python.org/2.0/ (yes we keep old versions, and I'm not willing to see this disapear)

JulienPalard avatar May 11 '21 09:05 JulienPalard

ping @pradyunsg because you're speaking a lot about readthedocs :) ping @humitos because you're from readthedocs :)

JulienPalard avatar May 11 '21 09:05 JulienPalard

Disclaimer: I work at Read the Docs.

I think there's many pros and cons of using readthedocs.

I'm not going to start selling Read the Docs because that's not my topic 😄 --Currently, I'm more interested in making sure that "the migration is possible from the technical aspect" and mention some notes to consider.

I'd personally be in favor of it if we (the PSF) support them

What kind of support do you refer to here? Take into account that Read the Docs by default adds EthicalAds on the documentation it hosts. I suppose there may be some different opinions about this and I think it worth having a conversation about the different possibilities we can explore.

the cpython Doc is a big one with a lot of traffic, it take around 24h of CPU to build all versions × languages with all PDF A4, PDF letter, HTML, plaintext, epub

We are currently using Read the Docs for the Spanish translation of the CPython documentation. It takes ~500 seconds to build the HTML with our smallest builder (2 CPU, 2Gb RAM, -j auto). It takes ~500 seconds more to build the PDF

Each commit merged into an active branch from English, should trigger a re-build for all the translations to update them as well. So, one single merge will trigger 1 (version) x (8 languages) = 8 builds * ~1000 seconds = ~2 hours (is my math correct? 🤔 )

Edit: I put 8 versions originally, but that's not correct. Only one version is required to be built. The version where the merge was done. If it was done in 3.10, only 3.10 + its translations need to be re-built.

Edit: the unit of the result was incorrect. I changed "2 minutes" by "2 hours" thanks to @hugovk for noting this.

Take into account that translations may require some extra work to be able to clone CPython repository to get the source files. In the Spanish translation we are using a git submode for this and then changing the Sphinx.srcdir to point to it in the conf.py file.

Note that Read the Docs only support one PDF output file. So, all the pages are together in one big PDF file, instead of split into the tutorial and others as the current behavior. There is an issue to discuss supporting multiple PDF output at https://github.com/readthedocs/readthedocs.org/issues/2045

among other things docs.python.org is not only about generating the docs but also hosting history: https://docs.python.org/release/ (this is all the tagged commits in the cpython repo)

This page looks like a simple index of a directory containing all the versions. There is not a direct match of this to a feature on RTD that allows this listing. I'd suppose that you may need to create an RST file or include an HTML page that lists all the current releases.

Also, a URL like /release/ is not supported either. It needs to have a prefix of /lang/version/ on a multi-version project. However, a redirect could be created: /release/ -> /en/stable/release/.

https://docs.python.org/2.0/ (yes we keep old versions, and I'm not willing to see this disappear)

This shouldn't be a problem if there is a way to re-generate them (e.g. running sphinx-build from branch 2.0 works). Unfortunately, Read the Does does not support uploading pre-built documentation. See https://github.com/readthedocs/readthedocs.org/issues/1083

To summarize, I think there some initial things bumped here:

  • Ads on documentation
  • building time (resource usage)
  • multiple PDF output
  • plaintext format not supported
  • build old releases
  • page listing all versions
  • translations may need extra work

Hopefully, this helps to understand some of the technical requirements and provides some value regarding the work effort required conversation.

humitos avatar May 11 '21 10:05 humitos

Each commit merged into an active branch from English, should trigger a re-build for all the translations to update them as well. So, one single merge will trigger 1 (version) x (8 languages) = 8 builds * ~1000 seconds = ~2 minutes (is my math correct? 🤔 )

~8,000 seconds = ~133 minutes = ~2h13m

Edit: I put 8 versions originally, but that's not correct. Only one version is required to be built. The version where the merge was done. If it was done in 3.10, only 3.10 + its translations need to be re-built.

~1,000 seconds = ~17 minutes

hugovk avatar May 11 '21 15:05 hugovk

Also, a URL like /release/ is not supported either. It needs to have a prefix of /lang/version/ on a multi-version project. However, a redirect could be created: /release/ -> /en/stable/release/.

We can mostly support custom URL's like this now. It's a beta feature, but if it's a blocker we can certainly manage it. That said, we don't currently support directory indexing, but we could create a version of the page thats branded properly in the docs theme.

What kind of support do you refer to here? Take into account that Read the Docs by default adds EthicalAds on the documentation it hosts. I suppose there may be some different opinions about this and I think it worth having a conversation about the different possibilities we can explore.

Just to be explicit, we're happy to host the Python docs without ads. I know there has been some discussion about having PSF sponsors included on the pages, and we've had separate conversations with the PSF team about us enabling that with our open source EthicalAds platform. We can definitely discuss having an additional business relationship with the PSF for hosting the docs, but I don't consider it a prerequisite. We get a lot of value out of Python, and are happy to support the projects we use heavily ad-free if needed (we already do this with Sphinx).

Unfortunately, Read the Does does not support uploading pre-built documentation. See readthedocs/readthedocs.org#1083

We could upload pre-built docs if they don't ever change as a one-off. It's not supported on the platform, but we can manually do it if needed.

I'm not going to start selling Read the Docs because that's not my topic 😄 --Currently, I'm more interested in making sure that "the migration is possible from the technical aspect" and mention some notes to consider.

Agreed. In the past there were some technical issues that would have prevented this, but I think we've solved most of them. We can do a bit more work to fully replicate the docs.python.org setup with existing URL's on our side if you'd like to see a tech demo, but we'd hope to do that after there was an agreement on moving to RTD from the team.

My primary non-technical selling point to hosting docs on RTD is that any work that goes into building features will now be shared with the entire Python community via RTD, instead of work done on a custom docs hosting tool. Similarly, any work the community does to improve RTD will automatically flow back to Python itself. Hopefully these additional resources will make doc hosting better for the whole ecosystem.

Please let us know if you have any questions about the product features or feasibility of hosting, as this progresses.

ericholscher avatar May 11 '21 17:05 ericholscher

ping @pradyunsg because you're speaking a lot about readthedocs :)

Only because a lot of what the docs build scripts are doing, is something provided by RTD as well. 😅

And RTD has the benefit of being a platform, and thus being able to provide things like PR previews.

pradyunsg avatar May 11 '21 17:05 pradyunsg

Just wanted to note that this might also be worth thinking about while we're doing #1. It would be awesome if we could get a new theme and not have to implement all the various Python-specific code to support it. We'd be interested in helping with this, if it was of interest.

Not sure what kind of 👍 we'd need to move forward on it, but I'd love to get more effort going into RTD integration instead of 1-off integrations of custom build logic. We've made good progress on a few of the above issues, and would be happy to do custom stuff for Python in the places where we don't have platform support.

ericholscher avatar Nov 29 '21 19:11 ericholscher

I think it's a great idea to host the docs on Read the Docs:

  • transparency of builds for all, no need to bug someone to SSH in to a server
  • PR previews (we're using an RTD site just for this, it's extremely useful)
  • can have builds for each merge to the branch, no need to wait 24h+ for the cron
  • the RTD team are very responsive and helpful
  • RTD is open source and Python

We've been discussing this in docs-community monthly meetings and Discord, and there's seems to be support on moving forward.


We have a large build matrix of languages. versions, and output formats (HTML, PDF, EPUB, etc). I would suggest an incremental approach, starting with just English, dev and and HTML.

Once we're happy with it, we can remove that particular build from the server and add another language/version/output.

Perhaps doing English+HTML first for the main builds (currently /dev, /3 and maybe /3.11) will give the biggest initial benefit. Then other languages+HTML. It might be that we keep some builds on the server for much longer, like PDF/EPUB etc. I think this is fine, the reduced load of moving HTML builds off the server will improve the queue time for other builds.


More concretely, I think the first thing to do is deal with the HTML language/version switcher. Currently this is added on the server builds via https://github.com/python/docsbuild-scripts.

We'll need a similar switcher for RTD builds.

I don't think it necessarily needs to be identical to the legacy one, but should be similar because people will be switching to/from the old and new sites.

We would develop it on something like https://cpython-previews.readthedocs.io (which is used for PR previews) and when happy, switch https://docs.python.org to it, stop the old server HTML build, and move to the next increment.

Thoughts?

hugovk avatar Jan 19 '24 16:01 hugovk

Thoughts?

I don't know how RTD survives financially today (yes I know it has been very hard).

Building more often that we build now (for every push, with PDF US letter, PDF A3, epub, txt, downloadable HTML) will take a lot of resources, more than the 4 virtual CPU that we already use 24h/24h.

I also don't know how RTD pays for the bandwidth, but I bet docs.python.org consumes some bandwidth (@ewdurbin any numbers?), it's probably nothing compared to PyPI but yet, let's check that. PSF side the bandwidth is sponsored by Fastly, but RTD side I don't know...

I think that the PSF and RTD should talk about money here: the PSF have to ensure, at the very least, that we don't penalize RTD. But it's not enough, I think the PSF should back up RTD in case RTD looses a vital sponsor (CPU/bandwidth provider or something) to ensure the future. @ericholscher @ewdurbin.

My non-financial view on this: RTD has proven they are the Python documentation hosting platform. Using RTD will make cpython a good citizen of the ecosystem. In other words, There should be one-- and preferably only one --obvious way to do it., and it's clearly not our little snowflake.

JulienPalard avatar Jan 19 '24 20:01 JulienPalard

Building more often:

If we don't want to build every merge, we can write a custom command to skip builds. For example, we should only do so for Doc/ changes, and could come up with extra logic to limit builds.

Docs: https://docs.readthedocs.io/en/stable/build-customization.html#cancel-build-based-on-a-condition

Some numbers:

During the 30-day Plausible trial, we had 6.5 million total visits and 10.9 million pageviews to the 3.11 and 3=3.12 English sites.

hugovk avatar Jan 19 '24 20:01 hugovk

We could feasibly add a proxy/cache via Fastly to read the docs, we do this for PEPs. That might also solve keeping /2.0/ &c.

A

AA-Turner avatar Jan 20 '24 12:01 AA-Turner

Echoing support for the proxy/cache concept, I don't think there was ever a philosophical objection to hosting on RTD, just technical and sustainability questions. Both the PSF and RTD are much better equipped these days to have the sustainability discussion directly between PSF and RTD staff than either were back when RTD was still relatively new, which leaves the technical side of things for the docs community to consider.

If the existing Fastly + Nginx docs endpoint is retained, with just the build and hosting of the version specific docs for actively maintained versions shifted to RTD, then it may be feasible to migrate new docs builds without needing to worry about too many one-off CPython-docs-specific hosting capabilities in RTD (instead leaving those in the existing PSF infrastructure).

I do think it would be important to be explicit about the intended benefits of the hosting change, though. As far as I am aware, the main CPython docs builds are still just periodic (daily?), which would make the commit triggered builds in RTD one of the biggest improvements on offer.

ncoghlan avatar Jan 23 '24 03:01 ncoghlan

Another benefit of using Read the Docs is the server side search.

methane avatar Jan 23 '24 06:01 methane

Just wanted to chime in here from the RTD perspective. RTD has a similar relationship with sponsored CDN hosting, where we aren't paying for bandwidth. So we don't have much worry about traffic or CDN support there. If it's a benefit to y'all to be able to put your own CDN in front, that's something we can support, but we already have our CDN configured to purge automatically on build, redirects, and other settings change, which won't flow through. That's likely not a huge deal, but might lead to delays in content changes, so something to consider.

I don't want to block this discussion on sustainability -- RTD is able to host the Python docs without any specific payment -- as noted above we're in a much better place these days, with our server costs and CDN sponsored and a full-time team working on the platform. We would still love some kind of support from the PSF, but it's not required. We'd just ask that the fact that RTD is hosting it is shown somewhere on the pages, perhaps in the footer.

Overall, we're excited to be able to host the official Python docs, and want to work with y'all to make it successful. I know @humitos has already chimed in here, but we're both available to answer any questions that y'all might have, as needed.

ericholscher avatar Jan 23 '24 18:01 ericholscher

Thanks for jumping in @ericholscher.

From PSF infra side, I'm a huge +1 on moving away from bespoke docs build infra as much as is possible. Given that RTD's CDN is sponsored as well I see no reason to involve our CDN in front of RTD's CDN (except perhaps as a mechanism for transition).

From PSF general side, we always like to acknowledge our hosting providers so assuming it is doable from the docs side of things adding a hosted by is 100% fine. Additional considerations would be another conversation with our Director of Resource Development. I'm happy to start that conversation once we have established a technical plan and committed to RTD.

ewdurbin avatar Jan 23 '24 18:01 ewdurbin

~~The only "hmmmmmm" I can think of off top of head is that we currently have good access-logging of docs.python.org requests that have been used in the past for some analysis of "top pages" and "top content" particularly by translations teams. Does RTD provide any insight into traffic?~~

Edit: Yes. With a 30 day lookback. I believe this is generally sufficient.

ewdurbin avatar Jan 23 '24 18:01 ewdurbin

we currently have good access-logging of docs.python.org requests that have been used in the past for some analysis of "top pages" and "top content" particularly by translations teams

Hmm, do we? Hugo & co. recently ran a Plausible trial to get some metrics, after failing to get them from server logs. If b.p.o metrics are better but getting them to the relevant people involves asking busy server admins, I'd say basic monthly stats would be an improvement :)

encukou avatar Jan 24 '24 12:01 encukou

we currently have good access-logging of docs.python.org requests that have been used in the past for some analysis of "top pages" and "top content" particularly by translations teams

Hmm, do we? Hugo & co. recently ran a Plausible trial to get some metrics, after failing to get them from server logs. If b.p.o metrics are better but getting them to the relevant people involves asking busy server admins, I'd say basic monthly stats would be an improvement :)

Yes, currently Fastly logs are streamed to a server in real time for "right now" analysis and rotated off after 7 days. All logs are archived in segments to S3 for three years.

Getting some basics from the real-time is pretty straightforward, we have only used the archives a couple times (more often for other projects like python.org).

If b.p.o metrics are better but getting them to the relevant people involves asking busy server admins, I'd say basic monthly stats would be an improvement :)

I agree! I will investigate automating pulling and storing the basic stats long-term if/when we switch over to RTD.

ewdurbin avatar Jan 24 '24 12:01 ewdurbin

From PSF general side, we always like to acknowledge our hosting providers so assuming it is doable from the docs side of things adding a hosted by is 100% fine.

This should be doable from the docs side. The footer is defined in python-docs-theme, we can add a variable that adds the acknowledgement for RTD builds.

hugovk avatar Jan 24 '24 12:01 hugovk

This should be doable from the docs side. The footer is defined in python-docs-theme, we can add a variable that adds the acknowledgement for RTD builds.

If I recall from a past discussion of adding sponsor logos to the docs, the rub there is that we only would want to include it in the online builds and not the offline builds. Is that possible?

ewdurbin avatar Jan 24 '24 12:01 ewdurbin

Good point, we can use the READTHEDOCS environment variable or define our own in RTD builds.

hugovk avatar Jan 24 '24 12:01 hugovk

Please see PR to add a "Hosted by" link to the theme's footer if a variable is defined: https://github.com/python/python-docs-theme/pull/165.

hugovk avatar Jan 26 '24 14:01 hugovk

we currently have good access-logging of docs.python.org requests that have been used in the past for some analysis of "top pages" and "top content" particularly by translations teams

Hmm, do we? Hugo & co. recently ran a Plausible trial to get some metrics, after failing to get them from server logs. If b.p.o metrics are better but getting them to the relevant people involves asking busy server admins, I'd say basic monthly stats would be an improvement :)

Yes, currently Fastly logs are streamed to a server in real time for "right now" analysis and rotated off after 7 days. All logs are archived in segments to S3 for three years.

Getting some basics from the real-time is pretty straightforward, we have only used the archives a couple times (more often for other projects like python.org).

Of note, RTD does have some basic analytics functionality built in. It's not got a ton of features, but it might be useful for the basics like "what are the top pages people are reading?": https://docs.readthedocs.io/en/stable/analytics.html

ericholscher avatar Jan 29 '24 19:01 ericholscher

There's no sense of urgency here, but to avoid missing the boat: could I request a stay (also known as: a delay, a pause) on any decision here for another year (+1y) or so?

After getting involved in Sphinx development a year or so ago (-1yr), I think I'm nearing effective-contributor status for the client-side HTML search functionality it contains - and which could be considered a competitor to what I perceive as the most clearly-stated and agreed-with benefit to RTD that I find in this thread so far (server-side search).

Given another six months I believe it'll be possible to improve the performance and relevance of client-side search in Sphinx noticeably, without sacrificing user resource consumption (bandwidth, CPU, battery-life, user-experience), privacy, or the ability for enthusiasts to inspect, learn from and/or tweak the code locally or themselves. Another six months beyond that would allow time for evaluation, feedback, adjustments and decision-making.

jayaddison avatar Jul 17 '24 23:07 jayaddison

Hello, great to hear Sphinx search is being improved and thank you for your work on Sphinx! But I'm not too keen on delaying by a year because the docs build is currently very slow: https://github.com/python/docsbuild-scripts/issues/169 from October talks of ~27 hours for a complete rebuild.

I recently set up a repo to monitor time between each version's HTML deploy, and it's now stretching up to 50 hours:

Time between deploys; last one is time since last deploy: Version 3.12: ['38.8 hours', '39.6 hours', '58.5 hours', '39.7 hours', '39.2 hours', '41.3 hours', '45.0 hours', '42.8 hours', '41.0 hours', '38.1 hours', '44.0 hours', '44.1 hours', '40.9 hours', '47.0 hours', '46.4 hours', '45.2 hours', '43.0 hours', '46.5 hours', '48.0 hours', '49.1 hours', '42.2 hours', '4.6 hours ago'] Version 3.13: ['39.0 hours', '40.4 hours', '57.5 hours', '40.4 hours', '40.1 hours', '42.6 hours', '44.2 hours', '42.2 hours', '39.7 hours', '40.3 hours', '43.7 hours', '44.4 hours', '40.7 hours', '47.8 hours', '46.1 hours', '44.4 hours', '43.6 hours', '47.4 hours', '49.1 hours', '47.5 hours', '31.1 hours ago'] Version 3.14: ['36.3 hours', '39.2 hours', '37.7 hours', '24.4 hours', '37.2 hours', '39.0 hours', '41.1 hours', '44.6 hours', '42.6 hours', '42.1 hours', '37.1 hours', '43.1 hours', '43.9 hours', '43.5 hours', '43.1 hours', '47.6 hours', '45.5 hours', '42.9 hours', '45.2 hours', '47.5 hours', '49.7 hours', '46.0 hours', '16.4 hours ago']

I disagree that server-side search is the most-wanted benefit in this thread (it was mentioned once), for me it's definitely build and deploy time.

hugovk avatar Jul 18 '24 12:07 hugovk

Thank you for the context @hugovk - I'll follow along with the PDF build-time concern and will offer any assistance if I can (although I'll also be cautious not to get in the way or distract from existing progress on it; neither PDF nor PDF in Sphinx are areas of expertise for me).

jayaddison avatar Jul 18 '24 12:07 jayaddison

Also note that hosting the documentation on Read the Docs doesn't mean that you have to use the Read the Docs search, you can stick with the Sphinx's default search if it works better for you. Even, you can start with Read the Docs search first and then swap to the Sphinx's default search once it's improved as well.

Regarding with the migration, do we have a set of "next steps" already defined? Let me know if there is anything else I can do to help with this process.

humitos avatar Jul 18 '24 12:07 humitos

First a summary of where we're up to:

  • We have the 3.11-3.13 and main branches building on RTD at https://cpython-previews.readthedocs.io and they automatically show in the version selector, via an RTD API.

  • We've linked a Spanish translation project on RTD to the main one, and it similarly shows in the language selector.

Next steps:

  • The version selector at https://docs.python.org goes all the way back to 2.6. I doubt it's worth importing all those to RTD (some might not even build), and they're no longer updated (2.6-3.7) or rarely updated (3.8-3.11).

  • And similarly for languages, we'll still want to link to the existing ones before importing them to RTD, so we can do an incremental rollout.

  • Do we need something to define which language/versions are on RTD, and which are not?

hugovk avatar Jul 19 '24 18:07 hugovk

I'd like to apologize for my recent comment where I misrepresented the gist of this thread so far, and also to withdraw my request for a pause.

I did not take the time that I should have done to read and understand the existing commentary, and I am sorry for that. I'd like to be a constructive and collaborative contributor, and will unsubscribe and distance myself from this thread until I can learn from that mistake and figure out how to proceed.

jayaddison avatar Jul 19 '24 22:07 jayaddison

@jayaddison No problem whatsoever! You're very welcome to contribute here, and I greatly appreciate your work on Sphinx, and across the Python ecosystem. 👍

hugovk avatar Jul 20 '24 13:07 hugovk

And similarly for languages, we'll still want to link to the existing ones before importing them to RTD, so we can do an incremental rollout.

I can help you with this by opening a PR that adds hard-coded URLs in https://github.com/python/cpython/blob/main/Doc/tools/static/rtd_switcher.js#L27. Sounds good?

Do we need something to define which language/versions are on RTD, and which are not?

I'd start by hosting one version/language on Read the Docs first and keep all the other ones in the current hosting. This will minimize any initial risk and will help maintainers to get familiar with the platform (general usage, build times, build output logs, workflow, etc) before performing the full migration.

I'm sure during this period there will be different opinions (good and bad ones 😅 ), limitations found, workflow changes and others that we will need to continue talking, adapting and documenting as well. However, once the initial setup is done, all the following versions and translations will be a lot easier to add.

In case we follow this idea of having one version/language on Read the Docs, should we setup cpython.readthedocs.io (or python.readthedocs.io) and create a proxy on your side from docs.python.org/{language}/{version}/ to point the cpython.readthedocs.io/{language}/{version}/ URL? Does this make sense?

Example: let's say we define Spanish 3.12 to be hosted on Read the Docs. Then https://docs.python.org/es/3.12/ should proxy to https://cpython.readthedocs.io/es/3.12/

Then we can keep adding more versions/languages using the same technique. Finally, when we are all migrated, we can just add docs.python.org as a canonical domain on Read the Docs' project and there won't be need to keep the proxy up anymore.

humitos avatar Jul 22 '24 09:07 humitos