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

no history of previous opam.ocaml.org builds #190

Open
avsm opened this issue Jun 6, 2023 · 3 comments
Open

no history of previous opam.ocaml.org builds #190

avsm opened this issue Jun 6, 2023 · 3 comments

Comments

@avsm
Copy link
Member

avsm commented Jun 6, 2023

image

While trying to debug why opam.ocaml.org isn't updating, I see this:
image

But no history of builds. Without the history, it's impossible to tell what's causing the build failure.
@patricoferris noted in another conversation that this may be due to the shape of the ocurrent DAG changing via binds, and so each run of the pipeline is actually a different graph.

In the meanwhile, the deployer is showing orange all the time, and the Hub shows that there hasn't been a push to opam.ocaml.org for 2 days... /cc @tmcgilchrist

@talex5
Copy link
Contributor

talex5 commented Jun 9, 2023

Note that OCurrent's built-in history only works for builds with the same (primary) key. For example, the history of an ocaml-ci analyse job only shows builds for a single commit (but includes all manual rebuilds and opam-repository updates, where the branch being tested didn't change).

ocaml-ci also has its own web UI, which provides its own history based on the branch name or PR. It would be good to get that supported in OCurrent itself.

@benmandrew
Copy link
Contributor

@talex5 I've been very interested in this myself*, but my impression was that OCurrent is application-agnostic and so has no idea about branches or PRs for instance.

I briefly looked at something along the lines of allowing the injection of extra data fields for jobs and search/sort functions into the web UI by a functor, but it seems like a big job.

*It was requested for:

@talex5
Copy link
Contributor

talex5 commented Jun 9, 2023

Yes, probably is quite a big job. As well as allowing customisation of the UI, it also needs the code to record the history (e.g. something that combines https://github.com/ocurrent/ocaml-ci/blob/master/lib/index.ml and https://github.com/ocurrent/opam-repo-ci/blob/master/lib/index.ml). Maybe a new sub-library like current.index (might need a better name).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants