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.

Sign up for GitHub

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

Jump to bottom

[#266] Updated Pod security standards #268

Merged
merged 1 commit into from
Mar 29, 2023
Merged

[#266] Updated Pod security standards #268

merged 1 commit into from
Mar 29, 2023

Conversation

jmesnil
Copy link
Member

@jmesnil jmesnil commented Mar 13, 2023

  • Add a default Security Context if the user does not specify one from the WildFlyServerSpec.
  • In the CSV, updates the operator's own deployment to comply with the security standards.

This fixes #266

@jmesnil jmesnil added this to the 1.0.0 milestone Mar 13, 2023
@jmesnil jmesnil added the Basic Install (I) Features for Phase I of the Operator capability levels label Mar 13, 2023
@jmesnil jmesnil mentioned this pull request Mar 13, 2023
Closed
@jmesnil jmesnil marked this pull request as draft March 13, 2023 17:08
@jmesnil jmesnil force-pushed the 266_security-context branch 4 times, most recently from 4982f64 to 32ccffe Compare March 14, 2023 09:51
@jmesnil jmesnil marked this pull request as ready for review March 15, 2023 13:55
@dcihak
Copy link

dcihak commented Mar 28, 2023

Hello, is there anything which prevents this from being merged? Issues caused by the new pod security standards cause failures of the OCP PIT tests. Thanks.

@jmesnil jmesnil requested a review from yersan March 28, 2023 08:50
@jmesnil
[wildfly#266] Updated Pod security standards
a591440 
* Add a default Security Context if the user does not specify one from the
WildFlyServerSpec.
* In the CSV, updates the operator's own deployment to comply with the
  security standards.

This fixes wildfly#266

Signed-off-by: Jeff Mesnil <[email protected]>
@jmesnil jmesnil force-pushed the 266_security-context branch from 32ccffe to a591440 Compare March 29, 2023 11:50
@yersan
Copy link
Collaborator

yersan commented Mar 29, 2023

@jmesnil can you rebase this?

@jmesnil jmesnil merged commit e7bd2a4 into wildfly:main Mar 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yersan yersan yersan approved these changes

Assignees
No one assigned
Labels
Basic Install (I) Features for Phase I of the Operator capability levels
Projects
None yet
Milestone
1.0.0
Development

Successfully merging this pull request may close these issues.

Update Pod Security Standards
3 participants
@jmesnil @dcihak @yersan