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.

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

Create a PoC for moving builtin rules to rego #1559

Open
lcarva opened this issue Apr 24, 2024 · 1 comment
Open

Create a PoC for moving builtin rules to rego #1559

lcarva opened this issue Apr 24, 2024 · 1 comment

Comments

@lcarva
Copy link
Member

lcarva commented Apr 24, 2024

In the previous community meeting, we talked about exploring the option of making the builtin rules more similar to other policy rules.

This issue is about creating a proof-of-concept so we can evaluate the idea further.

Proposal (as a starting point; PoC can, and likely will, deviate):

Considerations:

  • How would this impact the report if one of the builtin policy rules fails? How does it differ from the current behavior? Maybe wider use of depends_on is necessary.
  • How could someone create a policy config that explicitly includes the builtin rules? What if someone does it? Can the auto-injection detect and skip injecting them? Could someone use this as mechanism to exclude them?

Acceptance Criteria

  • A PoC is created the illustrates the ideas above. This should be working code but it does not have to be perfect nor handle different use cases. Hard code things if it makes it easier.
  • Demo the PoC with EC maintainers and maybe a wider audience.
  • Help decide next steps. (Abandoning the idea is also a perfectly acceptable next step). File issues accordingly.
@simonbaird
Copy link
Member

I wrote some notes about the motivation here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants