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

fix: Make Rego inventories ordered and deterministic #655

Merged
merged 2 commits into from
Nov 23, 2023

Conversation

viccuad
Copy link
Member

@viccuad viccuad commented Nov 23, 2023

Description

Ensure the rego inventory of Kubernetes resources is built in a deterministic way. Without that, a kwctl run outcome might fail without a valid reason.

This issue was reported by @brunorene

This depends on kubewarden/policy-evaluator#389

@viccuad viccuad self-assigned this Nov 23, 2023
@viccuad viccuad requested a review from a team as a code owner November 23, 2023 11:11
@viccuad viccuad force-pushed the deterministic-inventory branch from 82e8fc5 to cabefc7 Compare November 23, 2023 12:40
@fabriziosestito fabriziosestito force-pushed the deterministic-inventory branch 2 times, most recently from 849b75f to 441ead1 Compare November 23, 2023 15:05
Cargo.toml Outdated Show resolved Hide resolved
Copy link
Member

@flavio flavio left a comment

Choose a reason for hiding this comment

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

LGTM, of course we have to consume the final version of policy-evaluator (a tagged one, not a fix branch)

Signed-off-by: Fabrizio Sestito <fabrizio.sestito@suse.com>
Signed-off-by: Fabrizio Sestito <fabrizio.sestito@suse.com>
@fabriziosestito fabriziosestito merged commit cafb94c into kubewarden:main Nov 23, 2023
11 checks passed
@flavio flavio mentioned this pull request Nov 24, 2023
flavio added a commit to flavio/kwctl that referenced this pull request Nov 24, 2023
- fix: Make Rego inventories ordered and deterministic (kubewarden#655)
- Update many deps

Signed-off-by: Flavio Castelli <fcastelli@suse.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants