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.

Sign up for GitHub

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

Jump to bottom

v1.11 Backports 2023-07-13 #26801

Merged
merged 1 commit into from
Jul 13, 2023
Merged

v1.11 Backports 2023-07-13 #26801

merged 1 commit into from
Jul 13, 2023

Conversation

jibi
Copy link
Member

@jibi jibi commented Jul 13, 2023

Once this PR is merged, you can update the PR labels via:

for pr in 26715; do contrib/backporting/set-labels.py $pr done 1.11; done

or with

make add-labels BRANCH=v1.11 ISSUES=26715

@tklauser @jibi
ariane: don't skip verifier and l4lb tests on vendor/ changes
54f75b3 
[ upstream commit 1f35baf ]

Both of these workflows use binaries that are built in CI making use of
various vendored dependencies, so run them as well on PRs only changing
vendor/.

backporting conflicts:
* tests-datapath-verifier.yaml doesn't exist in the v1.11 branch

Signed-off-by: Tobias Klauser <tobias@cilium.io>
Signed-off-by: Gilberto Bertin <jibi@cilium.io>
@jibi jibi added kind/backports This PR provides functionality previously merged into master. backport/1.11 This PR represents a backport for Cilium 1.11.x of a PR that was merged to main. labels Jul 13, 2023
@jibi jibi requested a review from tklauser July 13, 2023 08:08
tklauser
tklauser approved these changes Jul 13, 2023
View reviewed changes
Copy link
Member

@tklauser tklauser left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

@jibi
Copy link
Member Author

jibi commented Jul 13, 2023

/ci-l4lb

@jibi
Copy link
Member Author

jibi commented Jul 13, 2023

l4lb test got skipped but at least we know the workflow is valid. Making as ready

@jibi jibi added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Jul 13, 2023
@jibi jibi requested a review from julianwiedmann July 13, 2023 08:27
@jibi jibi marked this pull request as ready for review July 13, 2023 08:33
@jibi jibi requested a review from a team as a code owner July 13, 2023 08:33
@julianwiedmann julianwiedmann merged commit e4b0551 into v1.11 Jul 13, 2023
37 checks passed
@julianwiedmann julianwiedmann deleted the pr/v1.11-backport-2023-07-13 branch July 13, 2023 09:23
@gentoo-root gentoo-root mentioned this pull request Jul 26, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tklauser tklauser tklauser approved these changes

@julianwiedmann julianwiedmann julianwiedmann approved these changes

Assignees
No one assigned
Labels
backport/1.11 This PR represents a backport for Cilium 1.11.x of a PR that was merged to main. kind/backports This PR provides functionality previously merged into master. ready-to-merge This PR has passed all tests and received consensus from code owners to merge.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@jibi @tklauser @julianwiedmann