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-e2e: Sync jobs list between tests-e2e-upgrade and conformance-e2e #31592

Closed
wants to merge 1 commit into from

Conversation

qmonnet
Copy link
Member

@qmonnet qmonnet commented Mar 25, 2024

We want to keep the list in both files synchronised, but over time we let the two versions diverge. Let's synchronise them now.

Fixes: 0b4d94e ("ci-e2e: add job testing node cidr feature")
Fixes: 9311af0 ("ci-e2e: Enable Ingress Controller test for more setup")
Fixes: 0e67360 ("conformance-e2e: Explicitly set devices to fix e2e tests")
Fixes: 5884af1 ("ci-e2e: Add matrix for bpf.tproxy")

@qmonnet qmonnet added kind/bug/CI This is a bug in the testing code. area/CI Continuous Integration testing issue or flake release-note/ci This PR makes changes to the CI. labels Mar 25, 2024
@qmonnet qmonnet requested review from a team as code owners March 25, 2024 15:54
@qmonnet qmonnet requested a review from brlbil March 25, 2024 15:54
@qmonnet
Copy link
Member Author

qmonnet commented Mar 25, 2024

/test

@qmonnet
Copy link
Member Author

qmonnet commented Mar 25, 2024

/ci-e2e-upgrade

@qmonnet
Copy link
Member Author

qmonnet commented Mar 25, 2024

Hmm it doesn't look like I managed to trigger the workflow 🤔

[EDIT] Apparently the workflow has been manually disabled.

@qmonnet qmonnet marked this pull request as draft March 25, 2024 17:11
We want to keep the list in both files synchronised, but over time we
let the two versions diverge. Let's synchronise them now.

Fixes: 0b4d94e ("ci-e2e: add job testing node cidr feature")
Fixes: 9311af0 ("ci-e2e: Enable Ingress Controller test for more setup")
Fixes: 0e67360 ("conformance-e2e: Explicitly set devices to fix e2e tests")
Fixes: 5884af1 ("ci-e2e: Add matrix for bpf.tproxy")
Signed-off-by: Quentin Monnet <qmo@qmon.net>
@qmonnet
Copy link
Member Author

qmonnet commented Apr 2, 2024

/test

@qmonnet
Copy link
Member Author

qmonnet commented Apr 2, 2024

/ci-e2e-upgrade

@sayboras
Copy link
Member

sayboras commented Apr 8, 2024

Just a note #31658

Copy link

github-actions bot commented May 9, 2024

This pull request has been automatically marked as stale because it
has not had recent activity. It will be closed if no further activity
occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale The stale bot thinks this issue is old. Add "pinned" label to prevent this from becoming stale. label May 9, 2024
Copy link

This pull request has not seen any activity since it was marked stale.
Closing.

@github-actions github-actions bot closed this May 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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. stale The stale bot thinks this issue is old. Add "pinned" label to prevent this from becoming stale.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants