Skip to content

[v1.24.x] OCPBUGS-1666: Expose flag to enable/disable PodSecurity#6080

Merged
jmrodri merged 7 commits into
operator-framework:v1.24.xfrom
openshift-cherrypick-robot:cherry-pick-6062-to-v1.24.x
Oct 13, 2022
Merged

[v1.24.x] OCPBUGS-1666: Expose flag to enable/disable PodSecurity#6080
jmrodri merged 7 commits into
operator-framework:v1.24.xfrom
openshift-cherrypick-robot:cherry-pick-6062-to-v1.24.x

Conversation

@openshift-cherrypick-robot

Copy link
Copy Markdown

This is an automated cherry-pick of #6062

/assign jmrodri

jmrodri and others added 7 commits October 13, 2022 03:56
Added --security-context-config flag to enable seccompprofile.
It defaults to enabled to support k8s 1.25. You can disable it
with --security-context-config=legacy

Signed-off-by: jesus m. rodriguez <jesusr@redhat.com>
* Ignoring error from Set call in test
* Update .cncf maintainers
* Update run bundle(-upgrade) CLI docs

Signed-off-by: jesus m. rodriguez <jmrodri@gmail.com>
Signed-off-by: jesus m. rodriguez <jmrodri@gmail.com>
Signed-off-by: jesus m. rodriguez <jmrodri@gmail.com>
Signed-off-by: jesus m. rodriguez <jmrodri@gmail.com>
Signed-off-by: jesus m. rodriguez <jesusr@redhat.com>
This was being duplicated because we had it in the text but were not
setting the value to a default value. Once we set the value to the
default cobra realized this and would output "(default: restricted)".

So removing the manually entered text fixes the duplicate.

Signed-off-by: jesus m. rodriguez <jesusr@redhat.com>
@jmrodri

jmrodri commented Oct 13, 2022

Copy link
Copy Markdown
Member

/lgtm

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Oct 13, 2022

@jmrodri jmrodri left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

/lgtm

@grokspawn

Copy link
Copy Markdown
Contributor

/approve

@everettraven everettraven left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

/lgtm

@jmrodri jmrodri merged commit 0b763cd into operator-framework:v1.24.x Oct 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants