ci: improve E2E preflight with version mismatch detection #32

Merged
ghost merged 1 commits from fix/e2e-version-preflight into main 2026-03-09 17:16:16 +00:00
ghost commented 2026-03-09 13:07:13 +00:00 (Migrated from github.com)

Summary

  • Enhances E2E preflight step to detect plugin version mismatches between the repo and the deployed Headlamp instance
  • Fetches the deployed plugin's package.json to compare versions
  • Emits ::warning:: annotation when there's a mismatch — immediately visible in CI summary
  • Uses artifacthub plugin name for correct path resolution

This would have immediately surfaced the v0.6.0 → v0.7.0 mismatch that caused 14/16 test timeouts (PRI-28), saving debugging time.

Test plan

  • Verify preflight step runs and detects version mismatch on current Headlamp
  • Confirm warning annotation appears in CI summary
  • Verify no regression in E2E test execution

🤖 Generated with Claude Code

## Summary - Enhances E2E preflight step to detect plugin version mismatches between the repo and the deployed Headlamp instance - Fetches the deployed plugin's package.json to compare versions - Emits `::warning::` annotation when there's a mismatch — immediately visible in CI summary - Uses artifacthub plugin name for correct path resolution This would have immediately surfaced the v0.6.0 → v0.7.0 mismatch that caused 14/16 test timeouts (PRI-28), saving debugging time. ## Test plan - [ ] Verify preflight step runs and detects version mismatch on current Headlamp - [ ] Confirm warning annotation appears in CI summary - [ ] Verify no regression in E2E test execution 🤖 Generated with [Claude Code](https://claude.com/claude-code)
Chris Farhood approved these changes 2026-03-09 17:11:40 +00:00
Chris Farhood left a comment

QA Review: APPROVED

Summary

CI-only change that enhances the E2E preflight step to detect plugin version mismatches between the repo and the deployed Headlamp instance.

Testing Results

  • Unit tests: PASS (77/77)
  • TypeScript: Pre-existing errors (same failure on main branch - Issue #29)
  • CI: PASS
  • E2E Tests: Pre-existing failures (Issue #29 - fails on main too)

Code Review

  • Only modifies .github/workflows/e2e.yaml
  • Adds version mismatch detection to preflight
  • Fetches deployed plugin package.json to compare versions
  • Emits warning annotation when mismatch detected
  • No new code paths to test - this is a CI workflow change

Notes

  • The E2E test failures and TypeScript errors are pre-existing issues in the repo, not caused by this PR
  • No regressions - this is purely a CI enhancement
  • The change is straightforward and well-implemented

Recommendation: Approve and merge

## QA Review: ✅ APPROVED ### Summary CI-only change that enhances the E2E preflight step to detect plugin version mismatches between the repo and the deployed Headlamp instance. ### Testing Results - **Unit tests**: ✅ PASS (77/77) - **TypeScript**: Pre-existing errors (same failure on main branch - Issue #29) - **CI**: ✅ PASS - **E2E Tests**: Pre-existing failures (Issue #29 - fails on main too) ### Code Review - Only modifies `.github/workflows/e2e.yaml` - Adds version mismatch detection to preflight - Fetches deployed plugin package.json to compare versions - Emits warning annotation when mismatch detected - No new code paths to test - this is a CI workflow change ### Notes - The E2E test failures and TypeScript errors are pre-existing issues in the repo, not caused by this PR - No regressions - this is purely a CI enhancement - The change is straightforward and well-implemented **Recommendation**: Approve and merge
Sign in to join this conversation.
No Reviewers
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: privilegedescalation/headlamp-polaris-plugin#32