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

genpolicy: add patch to filter for runtime class #504

Merged
merged 1 commit into from
Jun 6, 2024

Conversation

3u13r
Copy link
Member

@3u13r 3u13r commented May 27, 2024

Uses kata-containers/kata-containers#9725 adapted so it successfully applies to the msft fork.

@3u13r 3u13r added the no changelog PRs not listed in the release notes label May 27, 2024
@katexochen
Copy link
Member

Let's give upstream some days to review the upstream PR.

@3u13r 3u13r force-pushed the feat/genpolicy/runtime-class-filtering branch 2 times, most recently from f76ebd1 to f7c3802 Compare May 30, 2024 00:57
@3u13r 3u13r changed the title wip: genpolicy: add patch to filter for runtime class genpolicy: add patch to filter for runtime class Jun 4, 2024
@3u13r 3u13r marked this pull request as ready for review June 4, 2024 17:16
@3u13r 3u13r requested a review from katexochen as a code owner June 4, 2024 17:16
@3u13r 3u13r requested a review from burgerdev June 4, 2024 17:16
@3u13r
Copy link
Member Author

3u13r commented Jun 4, 2024

Upstream has been merged without additional changes 🎉.

@3u13r 3u13r force-pushed the feat/genpolicy/runtime-class-filtering branch 2 times, most recently from 70e1f9e to 6415dc2 Compare June 4, 2024 22:04
let yaml_contents = yaml::get_input_yaml(&config.yaml_file)?;

- for document in serde_yaml::Deserializer::from_str(&yaml_contents) {
+ 'doc: for document in serde_yaml::Deserializer::from_str(&yaml_contents) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That label is unused, isn't it?

@@ -33,6 +33,7 @@ rustPlatform.buildRustPackage rec {
url = "https://github.com/kata-containers/kata-containers/commit/f61b43777834f097fcca26864ee634125d9266ef.patch";
sha256 = "sha256-wBOyrFY4ZdWBjF5bIrHm7CFy6lVclcvwhF85wXpFZoc=";
})
./genpolicy_msft_runtime_class_filter.patch
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please create patches with git format-patch from a commit (in the future). Rules and settings patches in this dir are a bad example.

@3u13r 3u13r force-pushed the feat/genpolicy/runtime-class-filtering branch from 6415dc2 to fd4fdea Compare June 6, 2024 09:43
@3u13r 3u13r merged commit cdd6cdd into main Jun 6, 2024
8 checks passed
@3u13r 3u13r deleted the feat/genpolicy/runtime-class-filtering branch June 6, 2024 09:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no changelog PRs not listed in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants