run system test from private repo

[bvolovat]

Summary by CodeRabbit

• Chores
  • Updated testing infrastructure to coordinate E2E tests through external workflow orchestration with comprehensive automated monitoring
  • Implemented correlation-based test tracking, conditional re-run logic based on test outcomes, and enhanced failure diagnostics with consolidated log artifacts

[coderabbitai]

📝 Walkthrough

Walkthrough

The workflow transitions E2E test execution from in-repository to a private repository via GitHub Actions dispatch. It generates correlation IDs, dispatches test workflows, polls for completion, handles conditional re-runs, and implements comprehensive failure logging with artifact uploads.

Changes

Cohort / File(s)	Summary
E2E Test Orchestration .github/workflows/incluster-comp-pr-merged.yaml	Restructures test flow to dispatch E2E tests to a private repository instead of in-repo execution. Adds correlation ID generation, GitHub App token-based dispatch, run detection polling, conditional re-run logic (for failed/cancelled states), status monitoring loop, and failure log aggregation with artifact uploads.

Sequence Diagram(s)

sequenceDiagram
    participant GH as GitHub Actions<br/>(PR Merged)
    participant CI as Current Workflow
    participant PrivateRepo as Private E2E Repo
    participant Monitor as Status Monitor
    participant Logs as Log Aggregator

    GH->>CI: Trigger incluster-comp-pr-merged
    CI->>CI: Generate correlation_id
    CI->>PrivateRepo: Dispatch e2e test workflow<br/>(GitHub App token)
    CI->>Monitor: Start polling for run_id
    Monitor->>PrivateRepo: Query workflow runs
    Monitor->>Monitor: Wait for run_id available
    PrivateRepo->>Monitor: Return run_id
    Monitor->>PrivateRepo: Poll run status
    PrivateRepo->>Monitor: In progress...
    Monitor->>PrivateRepo: Poll again
    PrivateRepo->>Monitor: Completed (success/failed)
    alt Workflow Failed
        Monitor->>Logs: Fetch failed job logs
        Logs->>CI: Generate log artifacts
        CI->>CI: Upload artifacts
        CI->>CI: Report failure summary
    else Workflow Success
        Monitor->>CI: Report completion
    end

Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Poem

🐰 A test dispatch bound for realms afar,
Correlation IDs like a guiding star!
We poll and we wait, with patience so true,
Cross-repo orchestration, a bold thing to pursue! 🌟
Failed logs we gather with care and with grace,
E2E testing finds its rightful place!

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately summarizes the main change: refactoring test execution to dispatch E2E tests to a private repository instead of running them in-repo.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch run_test_from_private_repo

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[github-actions]

Summary:

• License scan: failure
• Credentials scan: failure
• Vulnerabilities scan: failure
• Unit test: success
• Go linting: failure

[github-actions]

Summary:

• License scan: failure
• Credentials scan: failure
• Vulnerabilities scan: failure
• Unit test: success
• Go linting: failure