-
Notifications
You must be signed in to change notification settings - Fork 36
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
Add firefox automation #67
Add firefox automation #67
Conversation
Signed-off-by: Benji Visser <benji@093b.org>
Signed-off-by: Benji Visser <benji@093b.org>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work!
I left you a note for a potential bug and some suggestions on improvements that could be done to reduce the execution time (which is quite long).
Signed-off-by: Benji Visser <benji@093b.org>
Signed-off-by: Benji Visser <benji@093b.org>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. @captn3m0 do you want to take a look at this script before we merge it ?
Google cloud documentation errors out occasionally with SSL errors. We either retry the job manually, or the automation catches up in the next scheduled run in 4 hours typically.
We can add proper retries for individual scripts in a separate PR.
|
Ok, let's merge and see. Retries will have to be added if it fails often. |
We just had a timeout with the maven script. Maybe we should add a retry mechanism there too ;). |
Looks like the retry mechanism will be needed, already got a timeout in https://github.com/endoflife-date/release-data/actions/runs/3867095398. |
Fetch Firefox versions with their dates from https://www.mozilla.org/en-US/firefox/releases/
Unfortunately firefox release pages for versions <3.0 are not dated (example). Not sure where we can get this data.