Skip to content

feat: allow configuring rego engine#2314

Merged
migmartri merged 3 commits into
chainloop-dev:mainfrom
migmartri:2267-configure-domains
Aug 7, 2025
Merged

feat: allow configuring rego engine#2314
migmartri merged 3 commits into
chainloop-dev:mainfrom
migmartri:2267-configure-domains

Conversation

@migmartri

@migmartri migmartri commented Aug 6, 2025

Copy link
Copy Markdown
Member

Prepares the rego engine code to support customization, including additional domains

refs #2267

Signed-off-by: Miguel Martinez <miguel@chainloop.dev>
@migmartri migmartri requested review from Piskoo, javirln and jiparis August 6, 2025 15:24
Signed-off-by: Miguel Martinez <miguel@chainloop.dev>
@migmartri

migmartri commented Aug 6, 2025

Copy link
Copy Markdown
Member Author

Once we merge this, we might want to add support of this configuration option in the eval command cc/ @Piskoo

This is going to be used during the attestation process once #2315 is merged, but the eval command should support it too, even if it's explicit as flags

Comment thread pkg/policies/engine/rego/rego.go Outdated
Signed-off-by: Miguel Martinez <miguel@chainloop.dev>
@migmartri migmartri merged commit 052fefa into chainloop-dev:main Aug 7, 2025
12 checks passed
@migmartri migmartri deleted the 2267-configure-domains branch August 7, 2025 13:31
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

Successfully merging this pull request may close these issues.

3 participants