Skip to content
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

rootfs: Stop building and shipping OPA #187

Merged
merged 1 commit into from
May 2, 2024

Conversation

Redent0r
Copy link

Merge Checklist
  • Followed patch format from upstream recommendation: https://github.com/kata-containers/community/blob/main/CONTRIBUTING.md#patch-format
    • Included a single commit in a given PR - at least unless there are related commits and each makes sense as a change on its own.
  • Aware about the PR to be merged using "create a merge commit" rather than "squash and merge" (or similar)
  • genPolicy only: Ensured the tool still builds on Windows
  • genPolicy only: Updated sample YAMLs' policy annotations, if applicable
  • The upstream-missing label (or upstream-not-needed) has been set on the PR.
Summary

Cherry pick from kata-containers#9559

Note: No need to worry about kata-containers@7dd2fde since we don't have the original commit that this commit is trying to revert

Since OPA binary was replaced by the regorus crate, we can finally stop building and shipping the binary.

Test Methodology

Since OPA binary was replaced by the regorus crate, we can finally stop
building and shipping the binary.

Signed-off-by: Fabiano Fidêncio <[email protected]>
@Redent0r Redent0r added the upstream/merged PRs that have been merged upstream label Apr 30, 2024
@Redent0r
Copy link
Author

@Redent0r Redent0r marked this pull request as ready for review May 2, 2024 21:40
@Redent0r Redent0r requested review from a team as code owners May 2, 2024 21:40
@Redent0r
Copy link
Author

Redent0r commented May 2, 2024

https://dev.azure.com/mariner-org/mariner/_build/results?buildId=561904&view=results: Passed all internal e2e, except for 3 that use busybox sample. Verified busybox sample just needs to be updated

@Redent0r Redent0r merged commit dda2c28 into msft-main May 2, 2024
43 of 58 checks passed
@Redent0r Redent0r deleted the saulparedes/remove_opa2 branch May 2, 2024 21:41
Redent0r added a commit to microsoft/azurelinux that referenced this pull request May 2, 2024
Redent0r added a commit to microsoft/azurelinux that referenced this pull request May 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
upstream/merged PRs that have been merged upstream
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants