build: drop kata-containers - #18792
Merged
Christopher Co (christopherco) merged 1 commit intoSep 11, 2026
Merged
Conversation
Copilot started reviewing on behalf of
Christopher Co (christopherco)
September 11, 2026 06:59
View session
Contributor
There was a problem hiding this comment.
🟡 Changes recommended
Stale systemd preset configuration and reverse-dependency documentation still reference the removed component.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
Removes Kata Containers from Azure Linux build and publication inputs.
Changes:
- Deletes component definition, lock, and publication entry.
- Removes all rendered package sources and support files.
File summaries
| File | Description |
|---|---|
base/comps/kata-containers/kata-containers.comp.toml |
Removes component definition. |
base/comps/components-publish-channels.toml |
Removes publication entry. |
locks/kata-containers.lock |
Removes component lock. |
specs/k/kata-containers/kata-containers.spec |
Removes rendered spec. |
specs/k/kata-containers/sources |
Removes source checksums. |
specs/k/kata-containers/kata-osbuilder.sh |
Removes image-builder script. |
specs/k/kata-containers/kata-osbuilder-generate.service |
Removes systemd unit. |
specs/k/kata-containers/50-kata |
Removes CRI-O configuration. |
specs/k/kata-containers/15-dracut.conf |
Removes dracut configuration. |
specs/k/kata-containers/1001-Remove-warnings-as-compilation-errors.patch |
Removes build patch. |
specs/k/kata-containers/0999-osbuilder-Adjust-agent_version-for-our-builds.patch |
Removes osbuilder patch. |
Review details
Suppressed comments (1)
base/comps/kata-containers/kata-containers.comp.toml:1
- Removing this component makes the reverse-dependency note in
base/comps/rust-libc/rust-libc.comp.toml:7-10inaccurate: it still counts five in-tree consumers and nameskata-containers. Update that comment to describe the remaining four consumers so its compatibility rationale remains reliable.
- Files reviewed: 10/11 changed files
- Comments generated: 1
- Review effort level: Balanced
💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Tobias Brick (tobiasb-ms)
requested changes
Sep 11, 2026
Tobias Brick (tobiasb-ms)
left a comment
Tobias Brick (tobiasb-ms)
left a comment
Contributor
There was a problem hiding this comment.
Generally looks good but the copilot comment #18792 (comment) is worth looking into before merging.
Stop building and publishing kata-containers and remove its component definition, lock and rendered sources. Drop the obsolete preset and bump azurelinux-release to 4.0-29 so upgrades receive the cleanup. Correct the rust-libc consumer count. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 1fd8ffad-c9e1-4d63-978a-dcab02131c50
Christopher Co (christopherco)
force-pushed
the
chrco/remove-kata-containers
branch
from
September 11, 2026 15:10
60ffe8d to
2c19768
Compare
Copilot started reviewing on behalf of
Christopher Co (christopherco)
September 11, 2026 15:11
View session
Contributor
There was a problem hiding this comment.
🟢 Approval recommended
The component and its publication, lock, rendered artifacts, service preset, and remaining references are consistently removed.
Review details
- Files reviewed: 15/17 changed files
- Comments generated: 0 new
- Review effort level: Balanced
Collaborator
Author
|
/azp run |
Christopher Co (christopherco)
marked this pull request as ready for review
September 11, 2026 15:50
Christopher Co (christopherco)
requested a review
from a team
as a code owner
September 11, 2026 15:50
|
Azure Pipelines: 2 pipeline(s) were filtered out due to trigger conditions. |
Tobias Brick (tobiasb-ms)
approved these changes
Sep 11, 2026
Christopher Co (christopherco)
merged commit Sep 11, 2026
9666700
into
microsoft:4.0
11 checks passed
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.
Stop building and publishing kata-containers. Remove its component definition, lock and rendered sources to avoid stale build inputs.
Fixes: AB#23671