-
Notifications
You must be signed in to change notification settings - Fork 9
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
Conversation
Let's give upstream some days to review the upstream PR. |
f76ebd1
to
f7c3802
Compare
Upstream has been merged without additional changes 🎉. |
70e1f9e
to
6415dc2
Compare
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) { |
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.
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 |
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.
Please create patches with git format-patch
from a commit (in the future). Rules and settings patches in this dir are a bad example.
6415dc2
to
fd4fdea
Compare
Uses kata-containers/kata-containers#9725 adapted so it successfully applies to the msft fork.