Skip to content

Adding Nomad packer template for Azure, updated versions - #61

Open
errygg wants to merge 7 commits into
masterfrom
nomad_azure
Open

errygg wants to merge 7 commits into
masterfrom
nomad_azure

Conversation

@errygg

@errygg errygg commented Aug 28, 2018

Copy link
Copy Markdown

No description provided.

@errygg
errygg requested a review from bensojona August 28, 2018 21:32

@bensojona bensojona left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

NICE

I’m on my phone at the moment so can’t link to it, but there’s 2 more places that need to be updated when changing product versions in versions.sh - 1 in the README at the bottom (Image Version Table), and the other is the *_VERSIONS env vars at the bottom of versions.sh (so we can publish the images publicly).

@errygg

errygg commented Aug 29, 2018

Copy link
Copy Markdown
Author

Should I bump the RELEASE_VERSION as well?

@bensojona

bensojona commented Aug 29, 2018 via email

Copy link
Copy Markdown
Contributor

@bensojona

bensojona commented Aug 29, 2018 via email

Copy link
Copy Markdown
Contributor

@errygg

errygg commented Aug 29, 2018

Copy link
Copy Markdown
Author

A thing I noticed when testing, I don't see the *-ent binaries in hc-enterprise-binaries s3 bucket. I modified the version to only reference the non-*-ent version. For example, s3://hc-enterprise-binaries/nomad-enterprise/ is where I found the enterprise binaries for Nomad.

@bensojona bensojona left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hey Erik, you still need to add another version for NOMAD_VERSIONS as well (0.8.4), even if the version # hasn’t changed (each of the list lengths need to be the same).

Also - set the Boolean map for 0.1.4 in RELEASE_VERSIONS to ‘true’ so the AMIs get published publicly. Setting to ‘false’ keeps the AMIs private where you would need to add specific account IDs to access them. Since no enterprise binaries are placed on the AMIs, they can be publicly accessible for all. I can show you how to add account ids to private AMIs if you’d like to bake Enterprise AMIs as well.

Regarding where the ent binaries are - I’ll ping you.

@bensojona bensojona left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I’d still set the 0.1.4 RELEASE_VERSION map to true.

@bensojona bensojona left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good, once you get the image building successfully in Azure, just flip RUN_BUILD back to false and merge to master. We don’t currently have a strategy for publishing Azure images publicly like we do for AMIs, we’ll need to investigate the proper way to do that.

@hashicorp-cla

hashicorp-cla commented Mar 12, 2022

Copy link
Copy Markdown

CLA assistant check
All committers have signed the 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