DUNE-DAQ / daq-release

Scripts and configuration files for the DUNE DAQ release
https://dune-daq-sw.readthedocs.io/en/latest/packages/daq-release/
2 stars 0 forks source link

It should be an error if `make-release-repo.py` can't find an expected tag #366

Closed jcfreeman2 closed 6 months ago

jcfreeman2 commented 7 months ago

The motivation for this Issue is an attempted candidate build I tried to perform over the weekend, https://github.com/DUNE-DAQ/daq-release/actions/runs/8868482226. The immediate cause of that build failing is that hsilibs couldn't compile, but the reason for that is that hsilibs, despite beings listed in the relevant release.yaml as being v4.1.0 didn't yet have that tag. Indeed, on line 433 of the output you see

Info: hsilibs              | v4.1.0               | b6e0904

...where what's noteworthy is that the commit b6e0904 represented the head of the production/v4 branch at the time (which aligned with v3.1.0, modulo some Workflow *.yaml changes).

In such a situation - i.e., there's not actually a git tag in the repo corresponding to the requested tag - make-release-repo.py should immediately exit with a failure. We were lucky that hsilibs didn't happen to compile, but this falls under the category of "Better to fail loudly than silently and unnoticed".

andrewmogan commented 6 months ago

Resolved as of PR #367