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

Deploy documentation through Buildkite. #1051

Closed
ali-ramadhan opened this issue Oct 13, 2020 · 3 comments · Fixed by #1209
Closed

Deploy documentation through Buildkite. #1051

ali-ramadhan opened this issue Oct 13, 2020 · 3 comments · Fixed by #1209
Labels
documentation 📜 The sacred scrolls testing 🧪 Tests get priority in case of emergency evacuation

Comments

@ali-ramadhan
Copy link
Member

Right now it's done through Travis which is very slow. We can build and deploy docs much faster on Buildkite as Tartarus has lots of CPU resources.

@ali-ramadhan ali-ramadhan added testing 🧪 Tests get priority in case of emergency evacuation documentation 📜 The sacred scrolls labels Oct 13, 2020
@navidcy
Copy link
Collaborator

navidcy commented Oct 13, 2020

Why not via Github actions? (just saying..)

Do you need GPU functionality for some doc testing? That could be, e.g., a good reason why not Github actions.

@ali-ramadhan
Copy link
Member Author

Ah yeah no that's a good question.

Initially it's just because the CPUs on Buildkite (running on our machine) are more powerful than the ones on Travis so we can reduce documentation build times from 30~40 minutes down to ~20 minutes.

But we also want to run higher-resolution examples and more of them, which is where more powerful CPUs is helpful, and since we have many cores available we can hopefully run/test/build many examples in parallel (#1053).

And yeah maybe at some point we'll also have MPI (+ GPU?) examples which would have to run on more powerful machines that we can probably access through Buildkite.

@navidcy
Copy link
Collaborator

navidcy commented Nov 26, 2020

I guess v0.44.2 is missing from docs because this was merged after the v0.44.2 tag, correct?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation 📜 The sacred scrolls testing 🧪 Tests get priority in case of emergency evacuation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants