Test: Demonstrate VS build break and fix workflow via Dockerfile change#14
Open
devin-ai-integration[bot] wants to merge 4 commits intomasterfrom
Open
Test: Demonstrate VS build break and fix workflow via Dockerfile change#14devin-ai-integration[bot] wants to merge 4 commits intomasterfrom
devin-ai-integration[bot] wants to merge 4 commits intomasterfrom
Conversation
Co-Authored-By: Arthur Poon <arthur.poon@windsurf.com>
Author
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
Co-Authored-By: Arthur Poon <arthur.poon@windsurf.com>
Co-Authored-By: Arthur Poon <arthur.poon@windsurf.com>
Owner
❌ Build Failed:
|
Based on CI logs showing: - E: Unable to locate package nonexistent-package-that-does-not-exist-12345 - ERROR: failed to solve: process did not complete successfully: exit code: 100 Removed the intentionally broken package reference to fix the VS build. Co-Authored-By: Arthur Poon <arthur.poon@windsurf.com>
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.
Why I did it
This is a test PR to demonstrate the workflow of:
Note: This PR demonstrates the complete break-and-fix cycle. The final state should have all files restored to their original working state.
Work item tracking
How I did it
Initially attempted to introduce a Python syntax error in
platform/vs/sonic-platform-modules-vs/sonic_platform/chassis.py, but this did not break the CI because Python files are not compiled during the Debian package build process.Successfully broke the VS build by adding a non-existent package to
platform/vs/docker-syncd-vs/Dockerfile.j2:RUN apt-get install -f -y iproute2 libcap2-bin nonexistent-package-that-does-not-exist-12345How to verify it
docker-syncd-vs.gz.log:Updates since last revision
Which release branch to backport (provide reason below if selected)
N/A - This is a test PR, not intended for backporting.
Tested branch (Please provide the tested image version)
Description for the changelog
Test PR: Demonstrate VS build break/fix workflow using Dockerfile package error
Link to config_db schema for YANG module changes
N/A
A picture of a cute animal (not mandatory but encouraged)
🐛➡️🦋 (bug transformed into butterfly - fixed!)
Link to Devin run: https://cisco-demo.devinenterprise.com/sessions/9bf43bc4ec044968a47891ceb4ab1681
Requested by: Arthur Poon (@arthurkkp-cog)