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

backport v1.14: gh/workflows: Reusable workflow for ci-e2e and misc changes #27375

Merged
merged 3 commits into from Aug 11, 2023
Merged

backport v1.14: gh/workflows: Reusable workflow for ci-e2e and misc changes #27375

merged 3 commits into from Aug 11, 2023

Conversation

brb
Copy link
Member

@brb brb commented Aug 9, 2023

Manual backport of #27355.

@brb
gh/actions: Add cilium-config action
64573b0 
[ upstream commit 4be4cca ]

The action is going to be used by ci-e2e, ci-upgrade, ci-ipsec*. This
will allow those jobs to reduce the boilerplate required to derive a
Cilium configuration.

Signed-off-by: Martynas Pumputis <m@lambda.lt>
@brb brb added kind/backports This PR provides functionality previously merged into master. backport/1.14 This PR represents a backport for Cilium 1.14.x of a PR that was merged to main. labels Aug 9, 2023
@brb
Copy link
Member Author

brb commented Aug 9, 2023

/ci-e2e

@brb brb marked this pull request as ready for review August 9, 2023 10:40
@brb brb requested review from a team as code owners August 9, 2023 10:40
brb added 2 commits August 9, 2023 14:46
@brb
gh/workflows: Use cilium-config action in ci-e2e
f959907 
[ upstream commit 89004fb ]

To derive Cilium configuration.

Signed-off-by: Martynas Pumputis <m@lambda.lt>
@brb
gh/workflows: Split ci-e2e run tests
3abd67c 
[ upstream commit 1e9f619 ]

Into multiple steps for better readability of testing output.

Signed-off-by: Martynas Pumputis <m@lambda.lt>
@brb brb force-pushed the pr/brb/v1.14-ci-reuse-config branch from 3df98f9 to 3abd67c Compare August 9, 2023 12:47
@brb
Copy link
Member Author

brb commented Aug 9, 2023

/ci-e2e

@brb brb marked this pull request as draft August 9, 2023 12:48
@brb brb marked this pull request as ready for review August 9, 2023 13:19
@brb brb added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Aug 10, 2023
@ldelossa
Copy link
Contributor

@brb is only the /ci-e2e tests necessary here to merge? If so please make a note here.

@brb
Copy link
Member Author

brb commented Aug 10, 2023

@ldelossa Yep, only ci-e2e.

@ldelossa ldelossa merged commit e4716a9 into v1.14 Aug 11, 2023
61 checks passed
@ldelossa ldelossa deleted the pr/brb/v1.14-ci-reuse-config branch August 11, 2023 17:16
@michi-covalent michi-covalent mentioned this pull request Sep 9, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ldelossa ldelossa ldelossa approved these changes

@viktor-kurchenko viktor-kurchenko viktor-kurchenko approved these changes

Assignees
No one assigned
Labels
backport/1.14 This PR represents a backport for Cilium 1.14.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
@brb @ldelossa @viktor-kurchenko