Skip to content

Conversation

@albertsola
Copy link
Contributor

@albertsola albertsola commented Feb 3, 2026

Closes MPT-16824

  • Updated e2e cronjob GitHub workflow to configure Report Portal launch attributes
  • Added Report Portal launch attributes argument to e2e command: -o rp_launch_attributes="$RP_LAUNCH_ATTR"
  • Changed RP_LAUNCH from dynamic github-ref-based value to static value: mpt-api-client-e2e
  • Introduced new environment variable RP_LAUNCH_ATTR containing ref and event name: ref:${{ github.ref }} event_name:${{ github.event_name }}

@albertsola albertsola requested a review from a team as a code owner February 3, 2026 12:03
@albertsola albertsola requested review from d3rky and robcsegal February 3, 2026 12:03
@coderabbitai
Copy link

coderabbitai bot commented Feb 3, 2026

📝 Walkthrough

Walkthrough

This pull request modifies the GitHub Actions e2e workflow to update ReportPortal integration. It replaces a dynamic RP_LAUNCH value with a static identifier, introduces a new RP_LAUNCH_ATTR environment variable containing git reference and event metadata, and adds the launch attributes argument to the e2e command invocation.

Changes

Cohort / File(s) Summary
GitHub Actions Workflow
.github/workflows/cron-main-e2e.yml
Updated ReportPortal configuration: changed RP_LAUNCH from dynamic github-ref string to static mpt-api-client-e2e, added RP_LAUNCH_ATTR environment variable, and added -o rp_launch_attributes="$RP_LAUNCH_ATTR" argument to e2e command.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Jira Issue Key In Title ✅ Passed The PR title contains exactly one Jira issue key in the required format MPT-XXXX.
Test Coverage Required ✅ Passed The pull request only modifies a GitHub Actions workflow configuration file (.github/workflows/cron-main-e2e.yml), which is CI/CD automation rather than application source code. Test coverage requirements do not apply to workflow configuration files.
Single Commit Required ✅ Passed The PR contains exactly one commit (e6adb62 MPT-16824 Update e2e cronjob report portal setup), which meets the single commit requirement.

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

Warning

Review ran into problems

🔥 Problems

Errors were encountered while retrieving linked issues.

Errors (1)
  • MPT-16824: Cannot read properties of undefined (reading 'map')

Important

Action Needed: IP Allowlist Update

If your organization protects your Git platform with IP whitelisting, please add the new CodeRabbit IP address to your allowlist:

  • 136.113.208.247/32 (new)
  • 34.170.211.100/32
  • 35.222.179.152/32

Failure to add the new IP will result in interrupted reviews.


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

@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 3, 2026

@albertsola albertsola merged commit 74a40e8 into main Feb 3, 2026
4 checks passed
@albertsola albertsola deleted the MPT-16824/Update-e2e-cronjob-report-portal-setup branch February 3, 2026 14:15
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.

3 participants