Skip to content

test: fix e2e tests in CI workflows and don't run the workflows on "push" events #2240

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

Merged

Conversation

thomasferrandiz
Copy link
Contributor

@thomasferrandiz thomasferrandiz commented Jun 2, 2025

  • GH runners don't load the br_netfilter by default anymore so we need to modprobe it manually
  • Running through workflows on "pull_request" should be enough and will avoid duplicate runs

Description

Todos

  • Tests
  • Documentation
  • Release note

Release Note

None required

Running them on "pull_request" should be enough and will avoid duplicate runs
@thomasferrandiz thomasferrandiz force-pushed the fix-test-workflows branch 3 times, most recently from 0d9df4b to 9f6ef19 Compare June 3, 2025 15:47
@thomasferrandiz thomasferrandiz changed the title test: don't run the workflows on "push" events test: fix e2e tests in CI workflows and don't run the workflows on "push" events Jun 3, 2025
GH runners don't load this module anymore so we need
to load it manually so that flannel works properly in the e2e tests.
@thomasferrandiz thomasferrandiz merged commit a160a7f into flannel-io:master Jun 4, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants