Maintenance required on sphinx build bots (original) (raw)

September 21, 2023, 7:01am 1

Hey folks.
We need to update the sphinx build bots so that we can switch the markdown parser we use from recommonmark which has been deprecated and is no longer maintained to MyST.

The following command can be used to install the required dependendencies:

pip3 install --user -r ./llvm/docs/requirements.txt

from the root of the git repo.

Going forward, this file will be updated if we start depending on new sphinx extensions or themes.

The PR operating the switch is there https://github.com/llvm/llvm-project/pull/65664

My understanding is that some of the bots have been updated, so this mostly concerns polly-x86_64-gce1 and gribozavr3 and these jobs

Thanks!

Corentin

cor3ntin September 25, 2023, 12:03pm 2

cor3ntin October 4, 2023, 2:16pm 3

@tonic I was told you might be able to help with the sphinx bots. At the moment we do not have any bot validating the documentation before it’s build for publishing, it’s not an ideal situation. Thanks!

mib October 4, 2023, 5:25pm 4

@cor3ntin I think it’s @gkistanova that maintains the sphinx bots these days.

I do not maintain builders mentioned above.

mib October 4, 2023, 9:36pm 6

Oh, my bad!

If we cannot locate owners of those builders, and nobody steps up, we probably should remove them from the build bot.

At the moment we do not have any bot validating the documentation before it’s build for publishing, it’s not an ideal situation.

Sounds like that should be a part of pre-merge checks?

Seems to me, if a change is committed, and Sphinx documentation could be built for that revision, it should be published for the current development version of the docs, unless I’m missing something about validation @cor3ntin has in mind.

If we cannot locate owners of those builders, and nobody steps up, we probably should remove them from the build bot.

I have removed abandoned red Sphinx builders from llvm-zorg.