Cherry-pick of #3945 to release-6.3 - #3947
Merged
yasserfaraazkhan merged 2 commits intoAug 14, 2026
Merged
Conversation
Contributor
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
amyblais
approved these changes
Aug 14, 2026
) (cherry picked from commit d794838)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Cherry pick of #3945 on release-6.3.
Restores per-OS GitHub commit statuses (
e2e/linux,e2e/macos,e2e/windows, pluse2e/macos-policy/e2e/windows-policyon PR/master). CMT still keeps thee2e/compatibility-matrix-testingumbrella.Also cherry-picks #3923 so required
e2e/<os>checks can pass on this branch.release-6.3still has the pre-11.10 MM-T4054 fixture; CMT onrelease-6.3HEAD already fails MM-T4054 on server 11.10.0 only. Master is green because it already has #3923. This is a test-only change (e2e/specs/mattermost/media_preview.test.ts).Summary
e2e/utils/cmt-channel-notify.js(needed for per-OS aggregation).e2e/<os>checks from TSIO.Ticket Link
Cherry-pick of #3945
Also includes #3923 (needed for required E2E checks vs server 11.10.0)
Checklist
npm run lint:jsfor proper code formattingE2E/RunDevice Information
This PR was tested on: unit tests via
node --test e2e/utils/cmt-channel-notify.test.js e2e/utils/tsio-report-status.test.js(40 passed)Release Note
Backport notes
Automated cherry-pick conflicted because
release-6.3does not have channel-notify wiring from #3902.What this PR includes:
cmt-channel-notify.jshelpers required to aggregate TSIO results by OS (parseCmtJobName,fetchPerJobCountsFromConsolidated,buildIndividualReportUrl)e2e/linux|macos|windowswere red on the previous HEAD for a pre-existing 11.10.0 incompatibility, not because of ci(e2e): unique CMT failure counts + restore per-OS commit statuses #3945)What this PR does not enable:
MATTERMOST_*_WEBHOOK_URLnot wired on this branch; unique CMT headline counts only apply once channel notify is present)Merge gate: after this push, re-add the
E2E/Runlabel so Matterwick dispatches Playwright against the new HEAD (e733a294). Do not merge untile2e/linux,e2e/macos, ande2e/windowsare successful on that SHA.