deckhouse / deckhouse/deckhouse

[admission-policy-engine] advanced operation policies

Open
#3,236 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

area/security priority/backlog source/deckhouse-team type/enhancement
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:

  1. PortRange - find intersection between service ports and busy node ports
  2. RoleBinding to the absent ServiceAccount
  3. Deployment/DaemonSet/StatefulSet has no VPA defined
  4. 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

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.