Skip to content

Test: Demonstrate CI break and fix workflow for VS build#10

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

Test: Demonstrate CI break and fix workflow for VS build#10
devin-ai-integration[bot] wants to merge 4 commits intomasterfrom
devin/1768029548-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

This is a test PR created to demonstrate the workflow of:

  1. Creating a PR with a breaking change
  2. Observing the CI failure
  3. Fixing the issue based on pipeline logs

The workflow has been completed successfully - the build was broken and then fixed.

Work item tracking
  • Microsoft ADO (number only): N/A (test PR)

How I did it

Breaking the build:

  • Initial attempts with Python syntax error in scripts/github_get.py and Makefile syntax error in rules/docker-orchagent.mk did not trigger build failures (these files aren't executed during the CI check phase)
  • Successfully broke the build by adding an intentional exit 1 at the start of build_debian.sh

Fixing the build:

How to verify it

The CI pipeline failed with the intentional error in build_debian.sh:

INTENTIONAL BUILD FAILURE: Testing CI error detection

The fix removes this intentional failure. Note: The fix commit shows CI status as "skipping" due to Azure pipeline path filters not matching the changed file.

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 demonstrating CI workflow

Description for the changelog

Test PR demonstrating CI failure detection and fix workflow (do not merge)

Link to config_db schema for YANG module changes

N/A

Human Review Checklist

  • Verify build_debian.sh no longer contains the intentional exit 1 error
  • Note: Final commit CI shows "skipping" due to path filters - this is expected behavior
  • This PR is for demonstration purposes only and should not be merged

Requested by: Arthur Poon (@arthurkkp-cog)
Link to Devin run: https://cisco-demo.devinenterprise.com/sessions/a41bcb26832e4994beb99c980b1d75fc

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

devin-ai-integration bot and others added 3 commits January 10, 2026 08:20
…b_get.py fix)

Co-Authored-By: Arthur Poon <arthur.poon@windsurf.com>
Co-Authored-By: Arthur Poon <arthur.poon@windsurf.com>
Co-Authored-By: Arthur Poon <arthur.poon@windsurf.com>
@devin-ai-integration devin-ai-integration bot changed the title Test: Introduce syntax error to break VS build Test: Demonstrate CI break and fix workflow for VS build Jan 10, 2026
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.

0 participants