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

Update documentation for Packer to include minimum operational requirements #2241

Merged
merged 4 commits into from
Feb 14, 2024

Conversation

tpdownes
Copy link
Member

Building images with Packer is a significant challenge because there are minimum requirements that are not always fully captured in a blueprint. These include:

  1. the IAM roles of the user invoking Packer
  2. the IAM roles of the service account used by the build VM
  3. inbound internet access to the build VM
  4. outbound internet access from the build VM

We document these requirements and capture several of them in an updated copy of the Packer image-builder.yaml example.

I suggest reviewing this commit-by-commit as the last commit includes a number of formatting changes made automatically by mdformat that do not include functionality changes.

Submission Checklist

Please take the following actions before submitting this pull request.

  • Fork your PR branch from the Toolkit "develop" branch (not main)
  • Test all changes with pre-commit in a local branch #
  • Confirm that "make tests" passes all tests
  • Add or modify unit tests to cover code changes
  • Ensure that unit test coverage remains above 80%
  • Update all applicable documentation
  • Follow Cloud HPC Toolkit Contribution guidelines #

@tpdownes tpdownes added the release-module-improvements Added to release notes under the "Module Improvements" heading. label Feb 14, 2024
@tpdownes tpdownes requested a review from rohitramu February 14, 2024 16:52
@rohitramu rohitramu assigned tpdownes and unassigned rohitramu Feb 14, 2024
@tpdownes tpdownes merged commit ec8ad2d into GoogleCloudPlatform:develop Feb 14, 2024
10 of 35 checks passed
@tpdownes tpdownes deleted the packer_docs_example branch February 14, 2024 21:50
@alyssa-sm alyssa-sm mentioned this pull request Mar 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-module-improvements Added to release notes under the "Module Improvements" heading.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants