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

CMP-2666: Added SPO 0.8.4 release notes #79305

Merged
merged 1 commit into from
Jul 25, 2024
Merged

Conversation

sheriff-rh
Copy link
Contributor

@sheriff-rh sheriff-rh commented Jul 23, 2024

@openshift-ci-robot
Copy link

openshift-ci-robot commented Jul 23, 2024

@sheriff-rh: This pull request references CMP-2666 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the sub-task to target the "4.17.0" version, but no target version was set.

In response to this:

Version(s):
4.12+

Issue:
https://issues.redhat.com/browse/CMP-2666

Link to docs preview:
In Progress

QE review:

  • QE has approved this change.

Additional information:
None

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Jul 23, 2024
@openshift-ci-robot
Copy link

openshift-ci-robot commented Jul 23, 2024

@sheriff-rh: This pull request references CMP-2666 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the sub-task to target the "4.17.0" version, but no target version was set.

In response to this:

Version(s):
4.12+

Issue:
https://issues.redhat.com/browse/CMP-2666

Link to docs preview:
In Progress

QE review:

  • QE has approved this change.

Additional information:
None

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

@openshift-ci openshift-ci bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Jul 23, 2024
@openshift-ci-robot
Copy link

openshift-ci-robot commented Jul 23, 2024

@sheriff-rh: This pull request references CMP-2666 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the sub-task to target the "4.17.0" version, but no target version was set.

In response to this:

Version(s):
4.12+

Issue:
https://issues.redhat.com/browse/CMP-2666

Link to docs preview:
Security Profiles Operator 0.8.4

QE review:

  • QE has approved this change.

Additional information:
None

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

@openshift-ci openshift-ci bot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Jul 24, 2024
Copy link

@Vincent056 Vincent056 left a comment

Choose a reason for hiding this comment

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

Just some small nits

modules/spo-binding-workloads.adoc Show resolved Hide resolved
@sheriff-rh sheriff-rh force-pushed the CMP-2666 branch 7 times, most recently from 4f98cce to a797fd1 Compare July 24, 2024 18:16
@sheriff-rh sheriff-rh added the peer-review-needed Signifies that the peer review team needs to review this PR label Jul 24, 2024
@@ -35,10 +35,16 @@ spec:
profileRef:
kind: {kind} <1>
name: profile <2>
image: quay.io/security-profiles-operator/test-nginx-unprivileged:1.21
image: quay.io/security-profiles-operator/test-nginx-unprivileged:1.21 <3>
----
<1> The `kind:` variable refers to the name of the profile.
Copy link

@xiaojiey xiaojiey Jul 25, 2024

Choose a reason for hiding this comment

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

Maybe better to be:
The kind variable refers to the kind of the profile.

@@ -35,10 +35,16 @@ spec:
profileRef:
kind: {kind} <1>
name: profile <2>
image: quay.io/security-profiles-operator/test-nginx-unprivileged:1.21
image: quay.io/security-profiles-operator/test-nginx-unprivileged:1.21 <3>
----
<1> The `kind:` variable refers to the name of the profile.
<2> The `name:` variable refers to the name of the profile.

Choose a reason for hiding this comment

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

Maybe better to be:
The name variable refers to the name of the profile.

+
[IMPORTANT]
====
Using the `image: "*"` wildcard attribute binds all pods with a default security profile in a given namespace.

Choose a reason for hiding this comment

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

@Vincent056 I am unsure about two points. Could you please help to double confirm? Thanks.
Firstly, I tested this function. Actually, it only works for newly created pods(it doesn't work for the existing pods before profilebinding created). Maybe we can say
Using the image: "*" wildcard attribute binds all new pods with a default security profile in a given namespace.

Secondly, there is a Import severity bug when trying to delete the profilebinding: https://issues.redhat.com/browse/OCPBUGS-37557. Do we need to add it to doc? Thanks.

Choose a reason for hiding this comment

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

@Vincent056 I am unsure about two points. Could you please help to double confirm? Thanks. Firstly, I tested this function. Actually, it only works for newly created pods(it doesn't work for the existing pods before profilebinding created). Maybe we can say Using the image: "*" wildcard attribute binds all new pods with a default security profile in a given namespace.

Secondly, there is a Import severity bug when trying to delete the profilebinding: https://issues.redhat.com/browse/OCPBUGS-37557. Do we need to add it to doc? Thanks.

that's correct, it will not work for the old pods.

for second, have you removed all the pods that uses the binding before you remove the profile @xiaojiey

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I addressed @xiaojiey's feedback, updating the text in the callouts and changing the note to "binds new pods...".

Anything else I can add? @Vincent056 @BhargaviGudi

Copy link

openshift-ci bot commented Jul 25, 2024

@sheriff-rh: all tests passed!

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

Copy link

@Vincent056 Vincent056 left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jul 25, 2024
@sheriff-rh
Copy link
Contributor Author

All feedback has been addressed, Vincent was unable to duplicate the bug Xiaojie ran into. I will publish this for now so that customers have the information pertaining to the release.

If we need to make further corrections, we can do tomorrow or next week. Thanks all!

Merging to 4.12+.

@sheriff-rh sheriff-rh merged commit 4cd43bb into openshift:main Jul 25, 2024
3 checks passed
@sheriff-rh sheriff-rh deleted the CMP-2666 branch July 25, 2024 20:35
@sheriff-rh
Copy link
Contributor Author

/cherrypick enterprise-4.17

@sheriff-rh
Copy link
Contributor Author

/cherrypick enterprise-4.16

@sheriff-rh
Copy link
Contributor Author

/cherrypick enterprise-4.15

@sheriff-rh
Copy link
Contributor Author

/cherrypick enterprise-4.14

@sheriff-rh
Copy link
Contributor Author

/cherrypick enterprise-4.13

@sheriff-rh
Copy link
Contributor Author

/cherrypick enterprise-4.12

@openshift-cherrypick-robot

@sheriff-rh: new pull request created: #79514

In response to this:

/cherrypick enterprise-4.17

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@openshift-cherrypick-robot

@sheriff-rh: new pull request created: #79515

In response to this:

/cherrypick enterprise-4.16

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@openshift-cherrypick-robot

@sheriff-rh: new pull request created: #79516

In response to this:

/cherrypick enterprise-4.15

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@openshift-cherrypick-robot

@sheriff-rh: new pull request created: #79517

In response to this:

/cherrypick enterprise-4.14

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@openshift-cherrypick-robot

@sheriff-rh: new pull request created: #79518

In response to this:

/cherrypick enterprise-4.13

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@openshift-cherrypick-robot

@sheriff-rh: new pull request created: #79519

In response to this:

/cherrypick enterprise-4.12

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
branch/enterprise-4.12 branch/enterprise-4.13 branch/enterprise-4.14 branch/enterprise-4.15 branch/enterprise-4.16 branch/enterprise-4.17 jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. lgtm Indicates that a PR is ready to be merged. peer-review-done Signifies that the peer review team has reviewed this PR size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants