ci: remove helm/kubectl Polaris deploy steps from E2E workflow #38

2026-03-12T14:17:12Z

ghost commented

2026-03-12 14:17:12 +00:00

(Migrated from github.com)

Summary

Removes 5 unnecessary CI steps from the E2E workflow: Setup Helm, Setup kubectl, Deploy Polaris to CI cluster, Apply RBAC, Wait for Polaris dashboard readiness
Polaris is already installed on the CI cluster — these steps fail because the runner SA lacks RBAC to deploy to the polaris namespace
E2E tests are browser-based Playwright tests hitting HEADLAMP_URL — no cluster-level Polaris deployment needed from CI

What changed

Workflow now goes: Checkout → Setup Node → npm ci → Preflight → Install Playwright → Run E2E → Upload artifacts

Resolves PRI-28 / PRI-109

## Summary - Removes 5 unnecessary CI steps from the E2E workflow: Setup Helm, Setup kubectl, Deploy Polaris to CI cluster, Apply RBAC, Wait for Polaris dashboard readiness - Polaris is already installed on the CI cluster — these steps fail because the runner SA lacks RBAC to deploy to the `polaris` namespace - E2E tests are browser-based Playwright tests hitting `HEADLAMP_URL` — no cluster-level Polaris deployment needed from CI ## What changed Workflow now goes: Checkout → Setup Node → npm ci → Preflight → Install Playwright → Run E2E → Upload artifacts Resolves PRI-28 / PRI-109

Chris Farhood approved these changes 2026-03-12 19:38:34 +00:00

Chris Farhood left a comment

QA Review: PR #38

Summary: Removes 5 unnecessary CI steps from the E2E workflow (Setup Helm, Setup kubectl, Deploy Polaris, Apply RBAC, Wait for readiness).

Testing:

Cloned repo and checked out branch fix/remove-polaris-deploy-from-e2e
Ran unit tests: pnpm test - 77 tests passed

Analysis:

This is a CI-only change (workflow YAML modification)
Removes deployment steps that fail because the runner SA lacks RBAC to deploy to the polaris namespace
E2E tests are browser-based Playwright tests hitting HEADLAMP_URL - no cluster-level Polaris deployment needed from CI
Confirmed via CI history: E2E Tests consistently failing on main due to these deployment steps, while CI passes

Verdict: Tests pass. Change is safe and addresses the root cause of E2E failures.

## QA Review: PR #38 **Summary:** Removes 5 unnecessary CI steps from the E2E workflow (Setup Helm, Setup kubectl, Deploy Polaris, Apply RBAC, Wait for readiness). **Testing:** - Cloned repo and checked out branch `fix/remove-polaris-deploy-from-e2e` - Ran unit tests: `pnpm test` - **77 tests passed** **Analysis:** - This is a CI-only change (workflow YAML modification) - Removes deployment steps that fail because the runner SA lacks RBAC to deploy to the `polaris` namespace - E2E tests are browser-based Playwright tests hitting `HEADLAMP_URL` - no cluster-level Polaris deployment needed from CI - Confirmed via CI history: E2E Tests consistently failing on main due to these deployment steps, while CI passes **Verdict:** Tests pass. Change is safe and addresses the root cause of E2E failures.

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: privilegedescalation/headlamp-polaris-plugin#38