Skip to content

Adds a pod list rbac - #21832

Merged
vmuzikar merged 1 commit into
keycloak:mainfrom
shawkins:iss21814
Jul 20, 2023
Merged

Adds a pod list rbac#21832
vmuzikar merged 1 commit into
keycloak:mainfrom
shawkins:iss21814

Conversation

@shawkins

Copy link
Copy Markdown
Contributor

Unfortunately the minikube remote tests still pass even without this permission - I wasn't aware of this difference openshift handling.

I know fabric8 is still stuck at integration testing with openshift 3.11 - is there any desire around pushing to have something 4.x based runnable in a github action?

Closes #21814

@ahus1

ahus1 commented Jul 20, 2023

Copy link
Copy Markdown
Member

@shawkins - the Keycloak Benchmark sub-project has some scripts on how to run OpenShift on AWS. It has its downsides: it takes ~45 minutes to spin up a cluster. With the new architecture of OpenShift on AWS this might be down to 15 minutes, and there could be options to not spin up a cluster for everything. Let's talk.

@vmuzikar

Copy link
Copy Markdown
Contributor

I think we should be able to hook up to OpenShift CI / clusterbot in some way.

@vmuzikar

Copy link
Copy Markdown
Contributor

In any case, QE is testing the Operator with OCP internally. I wonder why it wasn't caught before. @miquelsi

@vmuzikar
vmuzikar enabled auto-merge (squash) July 20, 2023 08:28
@vmuzikar
vmuzikar merged commit 1fa6f6a into keycloak:main Jul 20, 2023
@shawkins

Copy link
Copy Markdown
Contributor Author

In any case, QE is testing the Operator with OCP internally. I wonder why it wasn't caught before. @miquelsi

It takes running a failing scenario - where the statefulset is failing to rollout to trigger this logic.

@vmuzikar

Copy link
Copy Markdown
Contributor

QE is AFAIK running the full remote testsuite on OCP nightly. @miquelsi Is that correct?

@vmuzikar

Copy link
Copy Markdown
Contributor

@shawkins BTW, we should backport this to 22. Can you please look into it?

@shawkins shawkins mentioned this pull request Jul 20, 2023
@shawkins

Copy link
Copy Markdown
Contributor Author

@shawkins BTW, we should backport this to 22. Can you please look into it?

#21850

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Keycloak operator lacks RBAC for Pods

3 participants