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

Policy group files #518

Merged
merged 3 commits into from
Sep 9, 2024
Merged

Policy group files #518

merged 3 commits into from
Sep 9, 2024

Conversation

jvanz
Copy link
Member

@jvanz jvanz commented Sep 6, 2024

Description

Updates the Helm charts to have the files required by the controller to be able to reconcile policy group types.

Fix kubewarden/kubewarden-controller#857

Updates the kubewarden-crds Helm chart to install the new policy types.
The ClusterAdmissionPolicyGroup and AdmissionPolicyGroup.

Signed-off-by: José Guilherme Vanz <[email protected]>
Updates the RBAC roles used by the controller to include the new policy
group types. Therefore, the controller will be able to start watching
the new policies types.

Signed-off-by: José Guilherme Vanz <[email protected]>
Updates the webhooks configuration used be the controller to validate
CRDs. The webhooks now include the new policy group types.

Signed-off-by: José Guilherme Vanz <[email protected]>
@jvanz jvanz self-assigned this Sep 6, 2024
@jvanz jvanz requested a review from a team as a code owner September 6, 2024 17:29
Copy link
Member

@flavio flavio left a comment

Choose a reason for hiding this comment

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

LGTM

@flavio
Copy link
Member

flavio commented Sep 9, 2024

Moving to blocked, we have to wait for kubewarden/kubewarden-controller#872 to be fixed, then copy the correct CRDs definitions into this PR

Copy link
Member

@flavio flavio left a comment

Choose a reason for hiding this comment

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

Waiting for kubewarden/kubewarden-controller#872 to be solved

@viccuad
Copy link
Member

viccuad commented Sep 9, 2024

We have decided to merge this to unblock the E2E tests.

Once we have kubewarden/kubewarden-controller#872, we will bump the Helm charts with the CRDs accordingly.

@viccuad viccuad merged commit cfab705 into kubewarden:main Sep 9, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Add PolicyGroup to the helm chart, remove feature flag from controller
4 participants