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.

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

ci: Add matrix for bpf.tproxy and ingress-controller #31875

Merged
merged 3 commits into from
Apr 11, 2024

Conversation

sayboras
Copy link
Member

This is to sync up with conformance-e2e workflow.

Relates: #31272

@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 Apr 10, 2024
@sayboras sayboras added the release-note/ci This PR makes changes to the CI. label Apr 10, 2024
@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 Apr 10, 2024
@sayboras
Copy link
Member Author

/ci-e2e-upgrade

@sayboras sayboras added the needs-backport/1.15 This PR / issue needs backporting to the v1.15 branch label Apr 10, 2024
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from main in 1.15.4 Apr 10, 2024
@sayboras sayboras force-pushed the pr/tammach/tests-e2e-upgrade-tproxy branch from a0323bb to 87f491f Compare April 10, 2024 09:40
@sayboras
Copy link
Member Author

/ci-e2e-upgrade

@sayboras sayboras removed the needs-backport/1.15 This PR / issue needs backporting to the v1.15 branch label Apr 10, 2024
@maintainer-s-little-helper maintainer-s-little-helper bot removed this from Needs backport from main in 1.15.4 Apr 10, 2024
@sayboras
Copy link
Member Author

/ci-e2e-upgrade

@sayboras sayboras force-pushed the pr/tammach/tests-e2e-upgrade-tproxy branch from 87f491f to 34f24ad Compare April 10, 2024 10:59
@sayboras
Copy link
Member Author

/ci-e2e-upgrade

@sayboras sayboras force-pushed the pr/tammach/tests-e2e-upgrade-tproxy branch from 34f24ad to 529f5e4 Compare April 10, 2024 16:35
@sayboras
Copy link
Member Author

/test

@sayboras sayboras marked this pull request as ready for review April 10, 2024 16:36
@sayboras sayboras requested review from a team as code owners April 10, 2024 16:36
@sayboras sayboras requested a review from brlbil April 10, 2024 16:36
@sayboras
Copy link
Member Author

/ci-multi-pool

@sayboras
Copy link
Member Author

/ci-e2e-upgrade

@sayboras sayboras force-pushed the pr/tammach/tests-e2e-upgrade-tproxy branch from 529f5e4 to 575bab2 Compare April 11, 2024 00:00
@sayboras
Copy link
Member Author

/test

@sayboras
Copy link
Member Author

/ci-e2e-upgrade

This is to support passing misc value if required in other test
configuration.

Signed-off-by: Tam Mach <tam.mach@cilium.io>
When any value in the matrix's config contains an '=' sign, then it causes
cilium-cli to reject the input.

For example,
`--junit-property github_job_step="Run tests (13, cidr-match-mode=nodes)"`
fails with an invalid argument error.

Related: 973a2eb
Signed-off-by: Tam Mach <tam.mach@cilium.io>
This is to sync up with conformance-e2e workflow.

Relates: #31272
Signed-off-by: Tam Mach <tam.mach@cilium.io>
@sayboras sayboras force-pushed the pr/tammach/tests-e2e-upgrade-tproxy branch from 575bab2 to 329cdc5 Compare April 11, 2024 01:55
@sayboras
Copy link
Member Author

/test

@sayboras
Copy link
Member Author

/ci-e2e-upgrade

@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 Apr 11, 2024
@sayboras sayboras added this pull request to the merge queue Apr 11, 2024
Merged via the queue into main with commit 4da49de Apr 11, 2024
81 checks passed
@sayboras sayboras deleted the pr/tammach/tests-e2e-upgrade-tproxy branch April 11, 2024 11:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

None yet

2 participants