Skip to content

Conversation

aanm
Copy link
Member

@aanm aanm commented Oct 7, 2024

In the case of push events the git rev-list command will not work since there isn't a "base" and a "head" SHA available. With this commit we will retrieve the commit of the push event and test the code from that commit.

Fixes: b73cce3 (".github: recreate caches on push to main")

In the case of push events the git rev-list command will not work since
there isn't a "base" and a "head" SHA available. With this commit we
will retrieve the commit of the push event and test the code from that
commit.

Fixes: b73cce3 (".github: recreate caches on push to main")
Signed-off-by: André Martins <andre@cilium.io>
@aanm aanm added kind/bug/CI This is a bug in the testing code. release-note/ci This PR makes changes to the CI. labels Oct 7, 2024
@aanm aanm requested review from a team as code owners October 7, 2024 10:07
@aanm aanm requested a review from brlbil October 7, 2024 10:07
@aanm aanm added needs-backport/1.14 needs-backport/1.16 This PR / issue needs backporting to the v1.16 branch labels Oct 7, 2024
@aanm aanm enabled auto-merge October 7, 2024 10:07
@aanm
Copy link
Member Author

aanm commented Oct 7, 2024

/test

@aanm aanm added this pull request to the merge queue Oct 7, 2024
@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Oct 7, 2024
Merged via the queue into main with commit f5a1427 Oct 7, 2024
71 checks passed
@aanm aanm deleted the pr/fix-build-commits-main branch October 7, 2024 10:35
@giorio94 giorio94 mentioned this pull request Oct 7, 2024
15 tasks
@giorio94 giorio94 added backport-pending/1.16 The backport for Cilium 1.16.x for this PR is in progress. and removed needs-backport/1.16 This PR / issue needs backporting to the v1.16 branch labels Oct 7, 2024
@giorio94 giorio94 mentioned this pull request Oct 7, 2024
9 tasks
@giorio94 giorio94 added backport-pending/1.15 The backport for Cilium 1.15.x for this PR is in progress. and removed needs-backport/1.15 labels Oct 7, 2024
@giorio94 giorio94 mentioned this pull request Oct 7, 2024
5 tasks
@giorio94 giorio94 added backport-pending/1.14 The backport for Cilium 1.14.x for this PR is in progress. and removed needs-backport/1.14 labels Oct 7, 2024
@github-actions github-actions bot added backport-done/1.14 The backport for Cilium 1.14.x for this PR is done. backport-done/1.15 The backport for Cilium 1.15.x for this PR is done. and removed backport-pending/1.14 The backport for Cilium 1.14.x for this PR is in progress. backport-pending/1.15 The backport for Cilium 1.15.x for this PR is in progress. labels Oct 8, 2024
@github-actions github-actions bot added backport-done/1.16 The backport for Cilium 1.16.x for this PR is done. and removed backport-pending/1.16 The backport for Cilium 1.16.x for this PR is in progress. labels Oct 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-done/1.14 The backport for Cilium 1.14.x for this PR is done. backport-done/1.15 The backport for Cilium 1.15.x for this PR is done. backport-done/1.16 The backport for Cilium 1.16.x for this PR is done. kind/bug/CI This is a bug in the testing code. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/ci This PR makes changes to the CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants