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

tools: Allow setting policy rego file via environment variable #123

Merged
merged 4 commits into from
Dec 20, 2023

Conversation

ms-mahuber
Copy link

@ms-mahuber ms-mahuber commented Dec 13, 2023

This change introduces a new variable to rootfs.sh that allows setting a different rego policy file for the UVM.
The current behavior is unchanged as allow-all.rego is still set by default.

This change also introduces a new file, set-policy-allowed.rego alongside allow-all.rego to enable setting a restrictive initial policy file, requiring all pod YAMLs to carry proper policy annotations.

@ms-mahuber ms-mahuber changed the title Set policy file via env var runtime: Allow setting policy rego file via environment variable Dec 15, 2023
@ms-mahuber ms-mahuber changed the title runtime: Allow setting policy rego file via environment variable tools: Allow setting policy rego file via environment variable Dec 15, 2023
@ms-mahuber ms-mahuber marked this pull request as ready for review December 15, 2023 17:31
sprt added a commit that referenced this pull request Dec 19, 2023
all-allowed.rego now lives under src/kata-opa/ and
set-policy-allowed.rego will be reintroduced by #123 in the same
location.
sprt added a commit that referenced this pull request Dec 19, 2023
all-allowed.rego now lives under src/kata-opa/ and
set-policy-allowed.rego will be reintroduced by #123 in the same
location.
@ms-mahuber ms-mahuber changed the base branch from sprt/upstream-main-fork to msft-main December 19, 2023 19:48
@ms-mahuber ms-mahuber changed the base branch from msft-main to sprt/upstream-main-fork December 19, 2023 19:48
@ms-mahuber ms-mahuber changed the base branch from sprt/upstream-main-fork to msft-main December 19, 2023 19:54
@ms-mahuber ms-mahuber changed the base branch from msft-main to sprt/upstream-main-fork December 19, 2023 19:55
@ms-mahuber ms-mahuber changed the base branch from sprt/upstream-main-fork to msft-main December 19, 2023 20:02
@ms-mahuber ms-mahuber force-pushed the mahuber/sprt/upstream-main-fork branch from 43200cf to 1a5105d Compare December 19, 2023 20:07
@ms-mahuber ms-mahuber merged commit 1f96bd6 into msft-main Dec 20, 2023
38 of 50 checks passed
sprt added a commit that referenced this pull request Dec 20, 2023
all-allowed.rego now lives under src/kata-opa/ and
set-policy-allowed.rego will be reintroduced by #123 in the same
location.
sprt pushed a commit that referenced this pull request Dec 20, 2023
* Set policy file via env var

* Add restrictive policy file to kata-opa folder

* Change restrictive policy file name

* Change relative default path location
@sprt sprt added the upstream/missing PRs that are yet to be upstreamed label Jan 22, 2024
Redent0r pushed a commit that referenced this pull request Jun 25, 2024
environment variable (#123)

* Set policy file via env var

* Add restrictive policy file to kata-opa folder

* Change restrictive policy file name

* Change relative default path location
Redent0r pushed a commit that referenced this pull request Jun 27, 2024
environment variable (#123)

* Set policy file via env var

* Add restrictive policy file to kata-opa folder

* Change restrictive policy file name

* Change relative default path location

* Add license headers
Redent0r pushed a commit that referenced this pull request Jun 27, 2024
environment variable (#123)

* Set policy file via env var

* Add restrictive policy file to kata-opa folder

* Change restrictive policy file name

* Change relative default path location

* Add license headers

Signed-off-by: Saul Paredes <[email protected]>
@Redent0r Redent0r added upstream/merged PRs that have been merged upstream and removed upstream/missing PRs that are yet to be upstreamed labels Jul 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
upstream/merged PRs that have been merged upstream
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants