-
Notifications
You must be signed in to change notification settings - Fork 260
ci: backport relevant signing changes to release/v1.6 #3862
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
* Remove File Copy * Add Image Builds to Regular Pipeline Execution
58fd455 to
9e446a7
Compare
9e446a7 to
59f6b31
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR enables NPM (Network Policy Manager) builds for Linux platforms and backports signing-related changes to the v1.6 release branch. It removes commented-out NPM build configurations and updates the CI pipeline structure.
Key changes:
- Uncommented NPM build configurations for Linux AMD64 and ARM64 platforms
- Reorganized CI pipeline stages to ensure proper build order
- Added provenance flag and updated manifest generation logic
Reviewed Changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
.pipelines/templates/run-unit-tests.stages.yaml |
Removed coverage file copy operation |
.pipelines/run-pipeline.yaml |
Enabled NPM builds for Linux platforms and reorganized pipeline stages |
.pipelines/build/ob-prepare.steps.yaml |
Uncommented NPM Linux dockerfile preparation steps |
.pipelines/build/images.jobs.yaml |
Enabled ARM64 host architecture and removed QEMU flags |
.pipelines/build/image.steps.yaml |
Added provenance false flag to build arguments |
.pipelines/build/generate-manifest.steps.yaml |
Fixed manifest annotation argument joining |
|
/azp run Azure Container Networking PR |
|
Azure Pipelines successfully started running 1 pipeline(s). |
apontejaj
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Reason for Change:
Branch is built on top of #3859 and is intended to contain
Issue Fixed:
Requirements:
Notes: