Skip to content

Run test from private repo#76

Merged
bvolovat merged 2 commits intomainfrom
run_test_from_private_repo
Feb 2, 2026
Merged

Run test from private repo#76
bvolovat merged 2 commits intomainfrom
run_test_from_private_repo

Conversation

@bvolovat
Copy link
Contributor

@bvolovat bvolovat commented Feb 2, 2026

Summary by CodeRabbit

Release Notes

  • New Features

    • Enables manual workflow triggering with extensive input parameters for test configuration, image selection, and deployment options.
    • Added support for uploading failed test logs as workflow artifacts for improved debugging visibility.
  • Chores

    • Enhanced test orchestration with improved failure logging, monitoring capabilities, and automatic retry mechanisms for failed tests.

@coderabbitai
Copy link

coderabbitai bot commented Feb 2, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

The PR adds workflow dispatch capability and private repository E2E test orchestration to the incluster-comp-pr-merged workflow, including GitHub App token creation, system test dispatching with correlation tracking, workflow run discovery, rerun logic, test result polling, and comprehensive failure logging.

Changes

Cohort / File(s) Summary
Workflow Dispatch & E2E Test Orchestration
.github/workflows/incluster-comp-pr-merged.yaml
Adds workflow_dispatch trigger with extensive input parameters; extends run-tests job with private-repo dispatch flow including GitHub App token creation, correlation_id-based workflow run discovery, rerun logic, test polling (up to 1 hour), and comprehensive failure logging with artifact uploads.

Sequence Diagram

sequenceDiagram
    participant Workflow as Current Workflow
    participant GitHub as GitHub API
    participant PrivateRepo as Private Repository
    participant SharedWF as Shared Workflows
    participant Artifacts as Artifact Storage

    Workflow->>GitHub: Create GitHub App token
    Workflow->>PrivateRepo: Dispatch system tests event
    Workflow->>PrivateRepo: Poll for workflow run (by correlation_id)
    alt Run not found
        Workflow->>Workflow: Retry polling (multiple attempts)
    end
    Workflow->>PrivateRepo: Check test results
    alt Tests failed/cancelled
        Workflow->>PrivateRepo: Fetch failed jobs logs
        Workflow->>Artifacts: Upload failed test logs
        Workflow->>PrivateRepo: Rerun failed/all jobs
        Workflow->>PrivateRepo: Poll for new run completion
    else Tests passed
        Workflow->>Workflow: Proceed to release
    end
    Workflow->>GitHub: Report final status
Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

  • run system test from private repo #75: Implements the matching cross-repo E2E test dispatching infrastructure with correlation_id-based run discovery, rerun logic, polling, and failed-job log collection that this workflow now depends on.

Suggested reviewers

  • Naor-Armo

Poem

🐰 A workflows leap, so bold and true,
With tokens, tests, and retries too,
Correlation IDs guide the way,
E2E checks pass the day!
Logs captured, failures logged with care, 🎯

✨ 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

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

@bvolovat bvolovat merged commit 669f559 into main Feb 2, 2026
3 of 5 checks passed
@github-actions
Copy link

github-actions bot commented Feb 2, 2026

Summary:

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

1 similar comment
@github-actions
Copy link

github-actions bot commented Feb 2, 2026

Summary:

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

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