Skip to content

ci: fix pr number check for base branch retrieval#45024

Merged
aanm merged 1 commit intomainfrom
pr/artyop/fix-44914
Mar 27, 2026
Merged

ci: fix pr number check for base branch retrieval#45024
aanm merged 1 commit intomainfrom
pr/artyop/fix-44914

Conversation

@Artyop
Copy link
Copy Markdown
Contributor

@Artyop Artyop commented Mar 27, 2026

Fixing bad changes in #45019 pr-number doesn't have the value of the stable branch but a replacement using hyphens instead of the standard dot.

Instead of comparing pr-number and context-ref, verify that pr-number is an actual pr-number, if it isn't, assuming the dispatch isn't coming from a PR but Ariane's dispatch, so use the context-ref instead.

Fixes #44914

Fixing bad changes in #45019
pr-number doesn't have the value of the stable branch but a replacement
using hyphens instead of the standard dot.

Instead of comparing pr-number and context-ref, verify that pr-number
is an actual pr-number, if it isn't, assuming the dispatch isn't coming
from a PR but Ariane's dispatch, so use the context-ref instead.

Fixes #44914

Signed-off-by: Antony Reynaud <antony.reynaud@isovalent.com>
@Artyop Artyop requested review from a team as code owners March 27, 2026 16:18
@Artyop Artyop requested review from liyihuang and smagnani96 March 27, 2026 16:18
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Mar 27, 2026
@Artyop Artyop requested review from aanm and nebril March 27, 2026 16:18
@Artyop Artyop added area/CI Continuous Integration testing issue or flake release-note/ci This PR makes changes to the CI. labels Mar 27, 2026
@maintainer-s-little-helper maintainer-s-little-helper bot removed dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. labels Mar 27, 2026
@Artyop Artyop added the needs-backport/1.19 This PR / issue needs backporting to the v1.19 branch label Mar 27, 2026
@Artyop Artyop requested a review from aanm March 27, 2026 16:50
@Artyop
Copy link
Copy Markdown
Contributor Author

Artyop commented Mar 27, 2026

/test

@aanm aanm added this pull request to the merge queue Mar 27, 2026
Merged via the queue into main with commit 0b72c00 Mar 27, 2026
118 of 119 checks passed
@aanm aanm deleted the pr/artyop/fix-44914 branch March 27, 2026 18:17
@viktor-kurchenko viktor-kurchenko added backport-pending/1.19 The backport for Cilium 1.19.x for this PR is in progress. and removed needs-backport/1.19 This PR / issue needs backporting to the v1.19 branch labels Mar 30, 2026
@github-actions github-actions bot added backport-done/1.19 The backport for Cilium 1.19.x for this PR is done. and removed backport-pending/1.19 The backport for Cilium 1.19.x for this PR is in progress. labels Apr 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/CI Continuous Integration testing issue or flake backport-done/1.19 The backport for Cilium 1.19.x for this PR is done. release-note/ci This PR makes changes to the CI.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ci: ci-eks - Get PR base branch fails on ariane-scheduled dispatches

4 participants