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

Issues (possibly) caused by the current duration of the site generation script #2227

Closed
JJ opened this issue Jul 29, 2018 · 2 comments
Closed
Labels
big Issue consisting of many subissues

Comments

@JJ
Copy link
Contributor

JJ commented Jul 29, 2018

The problem

There are sometimes subtle problems of the type search.js is not still updated with the latest changes to the documentation; this does not take long, however, and is rarely noticed except by the person that has done the change.

But the main problem is that site regeneration takes now ~13 minutes, while the cron job that regenerates the site is programmed to work every 5 minutes. If two push are followed closely by each other, theoretically there's a lock on some files so it shouldn't go ahead, but the fact that there are subtle errors tells me lock does not work all across the files, or there's something else we are missing.

Suggestions

Obvious suggestion would be to change the cronjob so that it is more in sync with the time it takes to regenerate the site, but that's going to work as long as the generation time does not increase. Also revise the process of generation and synchronisation to avoid these problems. And, of course, try to make the generation process faster by using caches or other devices.
This would be related to #1823 mainly.

@JJ JJ added build big Issue consisting of many subissues labels Jul 29, 2018
@JJ JJ changed the title Problems with duration of site generation Issues (possibly) caused by the current duration of the site generation script Jul 29, 2018
@coke
Copy link
Collaborator

coke commented Jul 31, 2018

It's not theoretical; there's a lock:

*/5     *       *       *       * PATH=/usr/local/bin:/usr/bin:/bin
                flock -n ~/update.lock -c ./doc/util/update-and-sync > update.log 2>&1

@JJ
Copy link
Contributor Author

JJ commented Dec 29, 2020

Closing this since it's no longer generated this way.

@JJ JJ closed this as completed Dec 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
big Issue consisting of many subissues
Projects
None yet
Development

No branches or pull requests

2 participants