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

Review permission check #5210

Closed
lburgazzoli opened this issue Mar 4, 2024 · 1 comment · Fixed by #5479
Closed

Review permission check #5210

lburgazzoli opened this issue Mar 4, 2024 · 1 comment · Fixed by #5479
Assignees
@lburgazzoli
Labels
kind/feature New feature or request

Comments

@lburgazzoli
Copy link
Contributor

Requirement

As today the operator performs some permission checks i.e. for events and leases but according to the RBAC defined for the operator, the operator should have access to such resources.

See:

Problem

Those check add additional complexity and cognitive load to the code.

Proposal

Evaluate if those permission checks are still needed

Open questions

No response
@lburgazzoli lburgazzoli added the kind/feature New feature or request label Mar 4, 2024
@lburgazzoli
Copy link
Contributor Author

Relates to kubernetes-sigs/controller-runtime#2694

@valdar valdar mentioned this issue Mar 4, 2024
Open
@lburgazzoli lburgazzoli self-assigned this May 7, 2024
lburgazzoli added a commit to lburgazzoli/apache-camel-k that referenced this issue May 8, 2024
@lburgazzoli
Review permission check apache#5210
92e215e
lburgazzoli added a commit to lburgazzoli/apache-camel-k that referenced this issue May 8, 2024
@lburgazzoli
Review permission check apache#5210
44d559d
@lburgazzoli lburgazzoli mentioned this issue May 8, 2024
Merged
lburgazzoli added a commit to lburgazzoli/apache-camel-k that referenced this issue May 8, 2024
@lburgazzoli
Review permission check apache#5210
f2f6531
lburgazzoli added a commit to lburgazzoli/apache-camel-k that referenced this issue May 8, 2024
@lburgazzoli
Review permission check apache#5210
4dc7f8f
lburgazzoli added a commit that referenced this issue May 9, 2024
@lburgazzoli
Review permission check #5210
1fd641a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lburgazzoli lburgazzoli

Labels
kind/feature New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Review permission check lburgazzoli/apache-camel-k
1 participant
@lburgazzoli