Skip to content

Docs: invert comment lines for ImageSignatureDB and ImageForbiddenSignaturesDB #148

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

Merged
merged 1 commit into from
Apr 13, 2023

Conversation

zuc
Copy link
Contributor

@zuc zuc commented Feb 20, 2023

It seems comment lines for those two config fields were inverted.
This PR fixes it.

@zuc zuc requested a review from a team as a code owner February 20, 2023 16:25
@hashicorp-cla
Copy link

hashicorp-cla commented Feb 20, 2023

CLA assistant check
All committers have signed the CLA.

//A database of certificates that are trusted and can be used to sign boot files. You may specify single or multiple comma-separated values for this value.
ImageSignaturesDB []string `mapstructure:"image_signatures_db"`
//A database of certificates that have been revoked and will cause the system to stop booting if a boot file is signed with one of them. You may specify single or multiple comma-separated values for this value.
Copy link
Contributor

Choose a reason for hiding this comment

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

this was my bad, thanks for fixing !

Copy link
Contributor

@nywilken nywilken left a comment

Choose a reason for hiding this comment

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

Good catch - thank you.

@nywilken nywilken added the docs label Apr 13, 2023
@nywilken nywilken merged commit b9bd34c into hashicorp:main Apr 13, 2023
smCloudInTheSky pushed a commit to smCloudInTheSky/packer-plugin-googlecompute that referenced this pull request Apr 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants