Skip to content

VPAAMP-1346: Test-Only change. DO NOT MERGE - #297

Closed
Vinish100 wants to merge 1 commit into
developfrom
feature/VPAAMP-1346
Closed

Vinish100 wants to merge 1 commit into
developfrom
feature/VPAAMP-1346

Conversation

@Vinish100

Copy link
Copy Markdown

Reason for change: To validate L2 is running with a specified middleware commit ID. Designed to fail L2 7000 and 7001 Test Procedure: Validate via GHA that L2 7000 and 7001 fail when this middleware commit ID is specified
Risks: None

Reason for change: To validate L2 is running with a specified
middleware commit ID. Designed to fail L2 7000 and 7001
Test Procedure: Validate via GHA that L2 7000 and 7001 fail
when this middleware commit ID is specified
Risks: None

Signed-off-by: Vinish K B <vinish.balan@gmail.com>
Copilot AI balanced review requested due to automatic review settings October 12, 2026 15:09
@Vinish100
Vinish100 requested a review from a team as a code owner October 12, 2026 15:09

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

The PR is explicitly test-only, marked “DO NOT MERGE,” and designed to fail validation.

0 open findings

What changed in this PR

Updates a diagnostic log solely to verify middleware commit pinning through intentionally failing L2 tests.

Changes:

  • Shortens the stream-cap log message to trigger L2 7000/7001 failures.
File Description
InterfacePlayerRDK.cpp Changes one stream-cap log message.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@Vinish100 Vinish100 closed this Oct 13, 2026
@Vinish100
Vinish100 deleted the feature/VPAAMP-1346 branch October 13, 2026 07:01
@github-actions github-actions Bot locked and limited conversation to collaborators Oct 13, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants