Skip to content

Remove CBL-Mariner Dockerfiles and infrastructure#1617

Draft
Copilot wants to merge 3 commits intomainfrom
copilot/remove-cbl-mariner-dockerfiles
Draft

Remove CBL-Mariner Dockerfiles and infrastructure#1617
Copilot wants to merge 3 commits intomainfrom
copilot/remove-cbl-mariner-dockerfiles

Conversation

Copy link
Contributor

Copilot AI commented Mar 19, 2026

  • Remove src/cbl-mariner/ directory (all Dockerfiles and associated files)
  • Remove src/cbl-mariner/manifest.json include from root manifest.json
  • Update README.md examples that reference CBL-Mariner images
  • Remove CBL-Mariner entries from CODEOWNERS (fixes CodeOwnersTests.PathsAreUsed test)

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

Co-authored-by: mthalman <15789599+mthalman@users.noreply.github.com>
Updated image digest references and Dockerfile paths in the README.
@mthalman mthalman marked this pull request as ready for review March 19, 2026 14:22
@mthalman mthalman requested review from a team as code owners March 19, 2026 14:22
@mthalman
Copy link
Member

@copilot - Fix the failing tests in the Run Tests workflow.

Co-authored-by: mthalman <15789599+mthalman@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Mar 19, 2026

@copilot - Fix the failing tests in the Run Tests workflow.

Fixed in 6e103d1. The PathsAreUsed test was failing because CODEOWNERS still had 4 CBL-Mariner entries that no longer matched any Dockerfiles. Removed the # cbl-mariner section from CODEOWNERS. All 6 tests now pass.

@mthalman mthalman marked this pull request as draft March 19, 2026 20:28
@mthalman
Copy link
Member

Wait to merge this until after April .NET release.

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.

2 participants