-
Notifications
You must be signed in to change notification settings - Fork 5
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
Updates to validating with Rego and CEL #13
Conversation
@santoshkal , resolve the conflicts please |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@santoshkal , Please move the build file to root and make it executable anywhere.
Instead of ./genval , This should be genval [Command]
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please look at the comment and add autocompletion as well and fix in the next PR.
While reviewing, got some conflicts, Please resolve and getback with the above mentioned changed required.
Signed-off-by: santoshkal <ksantosh@intelops.dev>
Thanks for the review @MrAzharuddin Resolved the conflicts and is now ready to be merged. |
Update the Makefile path for |
… ./bin/genval currently Signed-off-by: Santosh Kaluskar <ksantosh@intelops.dev>
Signed-off-by: Santosh Kaluskar <ksantosh@intelops.dev>
This PR adds more validation features to Genval.
.tf
resource files with rego policies - closes Implement Terraform resource validation using Rego Policies #8--showjson
to show JSON representation of the input, this would help users write custom Rego/CEL policies. Closes Add a command to show JSON representation of generated Dockerfile #10cue
mode. The generated manifests will be stored in./output
directory. - Coses Implement validation/generation of multiple JSON/YAML files in Cue mode #9@MrAzharuddin