Skip to content

OLMv0 delegated Operator installation warnings - #119174

Open
michaelryanpeter wants to merge 1 commit into
openshift:mainfrom
michaelryanpeter:olm-cve-mitigation-1
Open

OLMv0 delegated Operator installation warnings#119174
michaelryanpeter wants to merge 1 commit into
openshift:mainfrom
michaelryanpeter:olm-cve-mitigation-1

Conversation

@michaelryanpeter

@michaelryanpeter michaelryanpeter commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Version(s):

NOTE: I still need to place these admonitions/snippets in the appropriate sections of the docs. I am looking for initial feedback on the wording of these warnings while I work on the proper placement.

Issue:

Link to docs preview:

QE review:

  • QE has approved this change.

Additional information:

@openshift-ci openshift-ci Bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Sep 1, 2026
@ocpdocs-previewbot

Copy link
Copy Markdown

🤖 Tue Sep 01 20:29:36 - Prow CI generated the docs preview:

https://119174--ocpdocs-pr.netlify.app/

@openshift-ci

openshift-ci Bot commented Sep 1, 2026

Copy link
Copy Markdown

@michaelryanpeter: all tests passed!

Full PR test history. Your PR dashboard.

Details

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.


* Default `edit` cluster role
* Service account token creation
* Pod creation

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.

Maybe Workload creation (e.g. Pod, ReplicaSet, Deployment, Job, CronJob, DaemonSet)?

But this is a slippery slope and is hard to describe because those are just some of the APIs that enable running a pod with a certain service account. CRs for arbitrary CRDs might also lead to pods running in a way where the user can control the service account and code execution.

@michaelryanpeter michaelryanpeter Sep 1, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Suggested change
* Pod creation
* Workload creation, for example:
** Pods
** Replica sets
** Deployments
** Jobs
** Cron jobs
** Daemon sets

Seeing it all listed out and knowing that this list is not exhaustive, perhaps we should take a different approach.

Could we simplify by saying:
"Do not install Operators that require cluster-scoped permissions in namespaces where non-administrator users have delegated Operator install permissions."

I understand that might include false positives, but that seems better than giving a false sense of security.

WDYT?
Is there another way to describe the risks more accurately?

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

Labels

size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants