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

generate: add flag for aks reference values #612

Merged
merged 2 commits into from
Jun 25, 2024

Conversation

davidweisse
Copy link
Contributor

@davidweisse davidweisse commented Jun 20, 2024

This allows for fields in the manifest to have the value "null", in which case the attestation will fail when trying to parse the validation options from the manifest. This is helpful when generating the manifest for the first time with contrast generate where the SNP reference values are now left empty ("null") by default. The default reference values used for AKS can then optionally be generated with the --reference-values aks flag.

@davidweisse davidweisse added the changelog PRs that should be part of the release notes label Jun 20, 2024
@davidweisse davidweisse requested a review from burgerdev June 20, 2024 15:10
cli/cmd/generate.go Outdated Show resolved Hide resolved
cli/cmd/generate.go Outdated Show resolved Hide resolved
internal/manifest/manifest.go Outdated Show resolved Hide resolved
Base automatically changed from dav/use-manifest-ref-values to main June 21, 2024 12:24
@davidweisse davidweisse force-pushed the dav/manifest-aks-ref-values-flag branch from 223ade8 to f9bd971 Compare June 21, 2024 12:25
@davidweisse davidweisse marked this pull request as ready for review June 21, 2024 12:26
@davidweisse davidweisse requested a review from katexochen as a code owner June 21, 2024 12:26
cli/cmd/generate.go Outdated Show resolved Hide resolved
The manifest will be generated with invalid null values when the flag is
not specified which must be filled out by the user.
@davidweisse davidweisse force-pushed the dav/manifest-aks-ref-values-flag branch from f9bd971 to 41a4804 Compare June 21, 2024 14:23
@davidweisse davidweisse requested a review from burgerdev June 25, 2024 07:20
cli/cmd/generate.go Show resolved Hide resolved
@davidweisse davidweisse merged commit 2b7c9e4 into main Jun 25, 2024
8 checks passed
@davidweisse davidweisse deleted the dav/manifest-aks-ref-values-flag branch June 25, 2024 13:51
@katexochen katexochen added breaking change A user-affecting breaking change and removed changelog PRs that should be part of the release notes labels Jul 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking change A user-affecting breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants