Skip to content

feat(templates): add configmaps for opa-trino-rules and opa-trino-dat… - #24

Merged
rpignolet merged 1 commit into
OKDP:mainfrom
alliage-io:feat/opa-configmaps
Aug 7, 2026
Merged

feat(templates): add configmaps for opa-trino-rules and opa-trino-dat…#24
rpignolet merged 1 commit into
OKDP:mainfrom
alliage-io:feat/opa-configmaps

Conversation

@SteBaum

@SteBaum SteBaum commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

…a for opa-kube-managment

Description

Two ConfigMaps have been added helm/okdp-examples/templates/configmap-opa-trino-data.yaml and helm/okdp-examples/templates/configmap-opa-trino-rules.yaml with the same policies and data of their counterparts in trino_opa_policy. They are meant to be used by OPA when using the Kube-managment side car instead of OPAL which will use the policy and data in the latter directory.

Related Issue

Fixes #23

Type of Change

  • Bug fix
  • New feature
  • Documentation update
  • Refactor / chore
  • Breaking change

How to Test

Checklist

  • I have tested my changes
  • Documentation updated if needed
  • If breaking change: migration path described above
  • I hereby declare this contribution to be licensed under the Apache License Version 2.0.
  • I hereby agree to grant TOSIT a copyright license to use my contributions.

@SteBaum SteBaum self-assigned this Aug 5, 2026
@SteBaum
SteBaum requested review from a team August 5, 2026 14:17
@jpmewenemesse

Copy link
Copy Markdown

Please take a look at the PR description in order to provide information about what this PR is about

@SteBaum SteBaum mentioned this pull request Aug 5, 2026
10 tasks

@abir-oumghar abir-oumghar left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed together with OKDP/platform-packages#45, which consumes these ConfigMaps through the kube-mgmt sidecar. Tested both on a local kind sandbox: kube-mgmt loads the policy and the data. Two small things before merge, both left inline as suggestions.

Comment thread helm/okdp-examples/templates/configmap-opa-trino-rules.yaml Outdated
Comment thread helm/okdp-examples/templates/configmap-opa-trino-data.yaml Outdated
@SteBaum
SteBaum force-pushed the feat/opa-configmaps branch from 4667c31 to 67aea1d Compare August 6, 2026 12:33
@SteBaum
SteBaum requested a review from abir-oumghar August 6, 2026 12:39
@SteBaum
SteBaum force-pushed the feat/opa-configmaps branch from cab3159 to 26fade9 Compare August 6, 2026 12:40
@rpignolet
rpignolet merged commit 5cc8c77 into OKDP:main Aug 7, 2026
4 checks passed
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.

Add configmaps for trino opa rules and trino opa data to be used by opa kube-managment

5 participants