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

Updating the example for current software versions #325

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

kellyhair
Copy link

Changes to aws-kms-unseal-ha:

  • Updated versions.tf to required Terraform 0.13 or greater.
  • Vault updated to 1.6.3. URL added to the variables.tf.
  • Consul updated from 1.0.0 to 1.9.4. Additionally, extrapolated OS & CPU architecture in the userdata.tpl.
  • Moved from Ubuntu 16.04 (Xenial) LTS to 20.04 (Focal) LTS.
  • Updated README.md to update the unseal instructions.

Changes to aws-kms-unseal:

  • Added versions.tf file; set required version of Terraform to 0.13.
  • Bumped version of Vault to 1.6.3 from 1.6.0.

Notes:

Tested both plans (aws-kms-unseal & aws-kms-unseal-ha) with Terraform v0.13.0 and v0.14.8. Terraform v0.14.8 was the most recent as of commits & PR.

Kelly Hair added 10 commits March 17, 2021 20:50
* Use of Ubuntu 20.04 LTS (update from 16.04 LTS)
* Either Vault OSS or Enterprise > 1.0.0  can be used
* Updated vault init to "vault operator init" vs. older "vault init"
* Updated output for "vault operator init" & "vault status"
Returning to a default of all three instances in a single AZ
Removed white space in .gitignore
Reverted to /24 subnet /16 as a single zone is used in the example
Tested with v0.13.0 & v0.14.8
@hashicorp-cla
Copy link

CLA assistant check

Thank you for your submission! We require that all contributors sign our Contributor License Agreement ("CLA") before we can accept the contribution. Read and sign the agreement

Learn more about why HashiCorp requires a CLA and what the CLA includes


Kelly Hair seems not to be a GitHub user.
You need a GitHub account to be able to sign the CLA. If you already have a GitHub account, please add the email address used for this commit to your account.

Have you signed the CLA already but the status is still pending? Recheck it.

@kellyhair
Copy link
Author

I noticed that I left some commented lines in. Will clean up & resubmit.

@kellyhair kellyhair closed this Mar 20, 2021
@kellyhair
Copy link
Author

Updated files; reopening

@kellyhair kellyhair reopened this Mar 20, 2021
@yhyakuna
Copy link
Collaborator

@kellyhair Could you please review and agree to CLA?

CLA

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants