deckhouse / deckhouse/deckhouse
[admission-policy-engine] advanced operation policies
Nobody has claimed this yet.
- Dominant language
- Go
- Stars
- 1.3k
- Forks
- 162
- Avg merge
- 1d 5h
- Merged PRs (30d)
- 653
Description
Preflight Checklist
- I agree to follow the Code of Conduct that this project adheres to.
- I have searched the issue tracker for an issue that matches the one I want to file, without success.
Use case. Why is this important?
After #3131 issue we have a few policies left:
- PortRange - find intersection between service ports and busy node ports
- RoleBinding to the absent ServiceAccount
- Deployment/DaemonSet/StatefulSet has no VPA defined
- no PDB set
all these checks are hard to implement because they are dependent on foreign objects. For example, if you are deploying Deployment + VPA, Deployment will be deployed first and validation webhook will forbid it because VPA does not exist yet.
Proposed Solution
Maybe we should not have the validation webhook for these checks. And just check them in the cluster via OPA audit deployment
Additional Information
No response
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reading the admission-policy-engine implementation and the work from issue #3131, then compare the four proposed checks: PortRange, missing ServiceAccount RoleBinding, absent VPA, and missing PDB. Determine whether these checks belong in a validation webhook or an OPA audit deployment, and define completion as a reviewed approach that handles dependencies between foreign objects.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- kubernetes
- Domain
- infrastructure, security
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 15/100