Skip to content

fix(ci): run e2e tests on pull requests#114

Merged
privateip merged 3 commits into
mainfrom
fix/e2e-run-on-pr
Jun 16, 2026
Merged

fix(ci): run e2e tests on pull requests#114
privateip merged 3 commits into
mainfrom
fix/e2e-run-on-pr

Conversation

@privateip

Copy link
Copy Markdown
Contributor

Summary

  • Remove the if: condition that restricted the test-e2e job to main branch and tags only
  • E2E tests now run on every PR after lint, unit tests, and build pass

Test plan

  • Open a PR and confirm the E2E Tests job appears and runs in CI

🤖 Generated with Claude Code

Remove the condition that restricted the test-e2e job to main branch
and tags only, so integration tests run on every PR.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@privateip privateip requested a review from a team as a code owner June 16, 2026 16:59
@privateip privateip requested a review from felixwidjaja June 16, 2026 16:59
privateip and others added 2 commits June 16, 2026 16:56
The TestKernelCapabilities/vrf_module e2e test creates a VRF device
inside a privileged pod to verify the host kernel supports VRF. On
GitHub Actions ubuntu-24.04 runners the vrf module is available but
not loaded by default, causing `ip link add ... type vrf` to fail with
"RTNETLINK answers: Operation not supported".

Load the module explicitly before creating the Kind cluster so the test
environment matches what galactic requires at runtime.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
The vrf kernel module is not preinstalled on GitHub-hosted Ubuntu runners.
On Azure-optimized kernels, vrf.ko lives in linux-modules-extra-azure which
must be explicitly installed before modprobe vrf can succeed. Using the
unversioned metapackage avoids version-mismatch failures as the runner
kernel ages between image rebuilds.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@privateip privateip merged commit d0f263d into main Jun 16, 2026
5 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.

2 participants