Skip to content

test: fixed performance pipeline#26071

Merged
javiergarciavera merged 1 commit intomainfrom
MMQA-XX-fix-permissions-release-branch
Feb 13, 2026
Merged

test: fixed performance pipeline#26071
javiergarciavera merged 1 commit intomainfrom
MMQA-XX-fix-permissions-release-branch

Conversation

@javiergarciavera
Copy link
Copy Markdown
Contributor

@javiergarciavera javiergarciavera commented Feb 13, 2026

Description

Changelog

CHANGELOG entry:

Related issues

Fixes:

Manual testing steps

Feature: my feature name

  Scenario: user [verb for user action]
    Given [describe expected initial app state]

    When user [verb for user action]
    Then [describe expected outcome]

Screenshots/Recordings

Before

After

Pre-merge author checklist

Pre-merge reviewer checklist

  • I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
  • I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

Note

Low Risk
Workflow-only changes to GitHub Actions permissions and concurrency; low code risk but could affect CI run frequency/queuing if misconfigured.

Overview
Fixes the release performance E2E trigger workflow by adding explicit top-level permissions so the reusable run-performance-e2e.yml workflow can request id-token and actions access.

Updates run-performance-e2e.yml concurrency grouping to include inputs.build_variant (defaulting to rc), preventing experimental and release runs from sharing a queue and canceling each other while still leaving cancel-in-progress: false.

Written by Cursor Bugbot for commit d578778. This will update automatically on new commits. Configure here.

@javiergarciavera javiergarciavera requested a review from a team as a code owner February 13, 2026 13:59
@github-actions
Copy link
Copy Markdown
Contributor

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

@metamaskbot metamaskbot added the team-qa QA team label Feb 13, 2026
@github-actions
Copy link
Copy Markdown
Contributor

🔍 Smart E2E Test Selection

  • Selected E2E tags: None (no tests recommended)
  • Selected Performance tags: None (no tests recommended)
  • Risk Level: low
  • AI Confidence: 95%
click to see 🤖 AI reasoning details

E2E Test Selection:
The changes are purely GitHub Actions workflow configuration changes:

  1. run-performance-e2e-release.yml: Added permissions block (contents: read, id-token: write, actions: write) to enable the reusable workflow to use id-token and actions properly.

  2. run-performance-e2e.yml: Modified the concurrency group to include build_variant parameter, preventing experimental (exp) and release/direct (rc) builds from sharing the same queue and canceling each other.

These are CI infrastructure improvements that:

  • Do not modify any application code
  • Do not modify any test code or test infrastructure
  • Do not affect how tests execute or what they test
  • Are purely about workflow permissions and concurrency management

No E2E tests are needed because there's no application behavior change to validate. The workflow changes would be validated by the CI system itself running successfully, not by E2E tests.

Performance Test Selection:
While these changes are to performance test workflow files, they don't modify the actual performance test code or any application code that would affect performance. The changes are about GitHub Actions permissions and concurrency group naming - purely CI infrastructure concerns. Running performance tests wouldn't validate these workflow configuration changes; the workflow changes are validated by the CI system running successfully. No performance impact to the application.

View GitHub Actions results

@javiergarciavera javiergarciavera added the no-changelog no-changelog Indicates no external facing user changes, therefore no changelog documentation needed label Feb 13, 2026
@javiergarciavera javiergarciavera added this pull request to the merge queue Feb 13, 2026
Merged via the queue into main with commit 2750a7a Feb 13, 2026
78 of 82 checks passed
@javiergarciavera javiergarciavera deleted the MMQA-XX-fix-permissions-release-branch branch February 13, 2026 15:38
@github-actions github-actions bot locked and limited conversation to collaborators Feb 13, 2026
@metamaskbot metamaskbot added the release-7.67.0 Issue or pull request that will be included in release 7.67.0 label Feb 13, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

no-changelog no-changelog Indicates no external facing user changes, therefore no changelog documentation needed release-7.67.0 Issue or pull request that will be included in release 7.67.0 size-S team-qa QA team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants