Skip to content

[test] Check subnet isolation through interface routes - #637

Merged
AshleyDumaine merged 3 commits into
linode:mainfrom
komer3:fix-subnet-e2e-assertions
Oct 2, 2026
Merged

AshleyDumaine merged 3 commits into
linode:mainfrom
komer3:fix-subnet-e2e-assertions

Conversation

@komer3

@komer3 komer3 commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

Summary

The subnet test now checks VPC interface routes instead of searching CCM logs. It requires each node's Pod CIDR route and rejects routes from the other cluster.

Missing routes are polled every five seconds, with a shared three-minute deadline. Foreign routes and API errors fail immediately. The testing docs describe this behavior.

Validation

  • All CI workflows passed, including the live E2E suite and subnet test.
  • Local Chainsaw lint and 11 synthetic runtime scenarios passed.

General:

  • No sensitive information.
  • Checked for duplicate PRs.

Pull Request Guidelines:

  • Tests pass.
  • New tests added (updated the existing subnet scenario).
  • Single focused change.
  • Atomic implementation commit.
  • Follows language conventions.
  • No unrelated formatting changes.
  • Rationale explained above.
  • Public issue linked.

@github-actions github-actions Bot added the testing for updates to the testing suite in the changelog. label Oct 1, 2026
@komer3
komer3 deployed to prod-external October 1, 2026 16:44 — with GitHub Actions Active
@komer3
komer3 deployed to prod-external October 1, 2026 16:44 — with GitHub Actions Active
@codecov

codecov Bot commented Oct 1, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.40%. Comparing base (2fa074b) to head (03e9437).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #637   +/-   ##
=======================================
  Coverage   75.40%   75.40%           
=======================================
  Files          19       19           
  Lines        2887     2887           
=======================================
  Hits         2177     2177           
  Misses        481      481           
  Partials      229      229           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@komer3
komer3 marked this pull request as ready for review October 1, 2026 18:00
Comment thread e2e/subnet-test/chainsaw-test.yaml
@komer3
komer3 deployed to prod-external October 1, 2026 19:41 — with GitHub Actions Active
@komer3
komer3 deployed to prod-external October 1, 2026 19:41 — with GitHub Actions Active
@AshleyDumaine
AshleyDumaine merged commit aa151ed into linode:main Oct 2, 2026
12 checks passed

This branch was successfully deployed

1 active deployment
prod-external — 03e94372 Deployed Oct 1, 2026 by komer3 via e2e-tests #2074
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

testing for updates to the testing suite in the changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants