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

gh/workflows: Fix encryption installation in ci-datapath #23325

Merged
merged 2 commits into from Jan 25, 2023
Merged

gh/workflows: Fix encryption installation in ci-datapath #23325

merged 2 commits into from Jan 25, 2023

Conversation

brb
Copy link
Member

@brb brb commented Jan 25, 2023 •
edited

@brb
gh/worflows: Fix encryption installation in ci-datapath
384c790 
Reported-by: Joe Stringer <joe@cilium.io>
Fixes: 481015e ("gh/workflows: Enable node encryption tests")
Signed-off-by: Martynas Pumputis <m@lambda.lt>
@brb brb added kind/bug/CI This is a bug in the testing code. area/CI Continuous Integration testing issue or flake labels Jan 25, 2023
@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 Jan 25, 2023
@brb brb added the release-note/ci This PR makes changes to the CI. label Jan 25, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Jan 25, 2023
@brb
gh/workflows: Disable DSR when running with WireGuard
8d57c95 
Until [1] and [2] have been resolved.

[1]: #23327
[2]: #23328

Signed-off-by: Martynas Pumputis <m@lambda.lt>
@brb brb force-pushed the pr/brb/fix-ci-dp branch 2 times, most recently from 5c0b02d to 8d57c95 Compare January 25, 2023 08:26
@brb brb marked this pull request as ready for review January 25, 2023 08:27
@brb brb requested review from a team as code owners January 25, 2023 08:27
@aanm aanm merged commit 5449a9f into master Jan 25, 2023
@aanm aanm deleted the pr/brb/fix-ci-dp branch January 25, 2023 08:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aanm aanm aanm approved these changes

@dylandreimerink dylandreimerink Awaiting requested review from dylandreimerink dylandreimerink was automatically assigned from cilium/sig-datapath

@nbusseneau nbusseneau Awaiting requested review from nbusseneau nbusseneau was automatically assigned from cilium/github-sec

Assignees
No one assigned
Labels
area/CI Continuous Integration testing issue or flake kind/bug/CI This is a bug in the testing code. release-note/ci This PR makes changes to the CI.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

CI: ci-datapath: Run encryption tests fails with bash: line 9: unexpected EOF while looking for matching `"'
2 participants
@brb @aanm