-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
ci/ipsec: Fix version retrieval for downgrades to closest patch release #30503
ci/ipsec: Fix version retrieval for downgrades to closest patch release #30503
Conversation
This commit brings two fixes to the script that we use to determine to which version we should upgrade/downgrade in some CI workflows. The first fix is the most important one. When looking for the closest patch version, make the script return the value in VERSION instead of decrementing it. The rationale is that for stable branches, VERSION already points to the latest patch release, there is no need to decrease it further! This fix does not affect the output for the calculation of the previous minor version number. The second fix is simply the addition of an error message in case the minor version number is 0, to get some explicit error instead of a silent failure if we ever reach Cilium 2.0.0. Updated samples of numbers from VERSION and the corresponding values returned: VERSION Previous minor Previous patch release 1.14.3 v1.13 v1.14.3 1.14.1 v1.13 v1.14.1 1.14.0 v1.13 <error> 1.14.1-dev v1.13 v1.14.1 1.15.0-dev v1.14 <error> 1.13.90 v1.12 <error> 2.0.1 <error> v2.0.1 Fixes: 56dfec2 ("contrib/scripts: Support patch releases in print-downgrade-version.sh") Signed-off-by: Quentin Monnet <quentin@isovalent.com>
/ci-ipsec-upgrade |
cc @cilium/sig-clustermesh , as |
/test |
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.
ta!
Thanks for the ping! The clustermesh workflow is not strictly affected because it only tests downgrades to the previous minor version, not to the patch one. |
Julian reports that the IPsec upgrade/downgrade workflow still has a flake that we expected to be gone, now that the fix is released patch version 1.13.11. As it turns out, the workflow does not pick and downgrade to the latest patch release, but to the one before that, because the script would decrement the value found in file
VERSION
. We don't need this,VERSION
already contains the last patch release that was published, not the one to come.Fixes: 56dfec2 ("contrib/scripts: Support patch releases in print-downgrade-version.sh")
Fixes: #28815
Cc @pchaigno just FYI