Skip to content

Test: Introduce breaking change in VS build#12

Open
devin-ai-integration[bot] wants to merge 3 commits intomasterfrom
devin/1768072176-test-break-vs-build
Open

Test: Introduce breaking change in VS build#12
devin-ai-integration[bot] wants to merge 3 commits intomasterfrom
devin/1768072176-test-break-vs-build

Conversation

@devin-ai-integration
Copy link

@devin-ai-integration devin-ai-integration bot commented Jan 10, 2026

Why I did it

⚠️ TEST PR - DO NOT MERGE ⚠️

This PR was created to test the CI pipeline failure detection and fix workflow. The goal was to:

  1. Trigger a CI failure
  2. Review the pipeline logs from GitHub comments
  3. Fix the issue based on those logs
Work item tracking
  • Microsoft ADO (number only): N/A (test PR)

How I did it

Introduced an intentional breaking change in platform/vs/docker-sonic-vs/start.sh by changing supervisorctl to supervisorctl_broken, which is not a valid command. This will cause a runtime failure when the container starts.

Note: A build-time failure was also introduced in Dockerfile.j2 and subsequently fixed based on CI logs. The Dockerfile.j2 is now restored to its original state, and only the start.sh runtime change remains.

How to verify it

  1. The start.sh change will cause runtime failures when the docker-sonic-vs container attempts to start rsyslogd
  2. This can be verified by running the container and observing the failure

Updates since last revision

  • Fixed the Dockerfile.j2 build-time breaking change (removed invalid RUN command)
  • CI now passes (build succeeds, but runtime will fail)
  • The start.sh runtime breaking change (supervisorctl_broken) remains in the PR as intended

Which release branch to backport (provide reason below if selected)

N/A - This is a test PR and should not be backported.

  • 202305
  • 202311
  • 202405
  • 202411
  • 202505
  • 202511

Tested branch (Please provide the tested image version)

  • N/A - test PR with intentional breaking change

Description for the changelog

Test PR to verify CI pipeline failure detection and fix workflow (DO NOT MERGE)

Human Review Checklist

  • Understand this is a TEST PR that should NOT be merged
  • Note that start.sh contains supervisorctl_broken which will cause runtime failures
  • CI passes because this is a runtime error, not a build-time error

Link to Devin run

https://cisco-demo.devinenterprise.com/sessions/9426dcded3d84173a2ce74a20103ec0e

Requested by: Arthur Poon (@arthurkkp-cog)

Co-Authored-By: Arthur Poon <arthur.poon@windsurf.com>
@devin-ai-integration
Copy link
Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

arthurkkp-cog and others added 2 commits January 10, 2026 20:06
Co-Authored-By: Arthur Poon <arthur.poon@windsurf.com>
Co-Authored-By: Arthur Poon <arthur.poon@windsurf.com>
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.

1 participant