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

UI: Analysis cannot start because of docker image pulls either in progress or failed #792

Open
benmandrew opened this issue Mar 20, 2023 · 0 comments
Labels
context/causing-pain Limitations of ocaml-ci which hurt development workflows type/enhancement New feature or request

Comments

@benmandrew
Copy link
Contributor

Context

When the set of docker images increases on a deploy, the deployer must pull the images before any analysis jobs can start running for specific repositories. While this is happening, on the web-UI the analysis jobs appear greyed out and clicking on them results in a 500 server error.

Once the docker pulls complete the analysis jobs start and the correct behaviour ensues. However, if any pull fails behind the scenes, then the analyses remain greyed out indefinitely with no indication that something is permanently wrong.

Expected behaviour

At minimum, clicking on the analysis job should not result in an opaque 500 error. Ideally, while pulls are happening there should be no analysis job visible and there should be some indication in the web-UI that we're waiting on background stuff to process (said in user-friendly terms).

This would help users understand that they just have to wait a little bit, but also help the CI developers to diagnose when a pull has failed, as I have had multiple occasions where I have wasted time trying to debug an issue in the CI that turned out to be failed pulls for whatever reason.

As a non-UI extension, one or two failed image pulls should raise some alarms, but should not prevent the CI from starting analysis jobs for the platforms that succeeded.

@benmandrew benmandrew added type/enhancement New feature or request context/causing-pain Limitations of ocaml-ci which hurt development workflows labels Mar 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
context/causing-pain Limitations of ocaml-ci which hurt development workflows type/enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant