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

Adding Nomad packer template for Azure, updated versions #61

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

Conversation

errygg
Copy link

@errygg errygg commented Aug 28, 2018

No description provided.

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

@bensojona bensojona left a comment

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
Copy link
Author

errygg commented Aug 29, 2018

Should I bump the RELEASE_VERSION as well?

@bensojona
Copy link
Contributor

bensojona commented Aug 29, 2018 via email

@bensojona
Copy link
Contributor

bensojona commented Aug 29, 2018 via email

@errygg
Copy link
Author

errygg commented Aug 29, 2018

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.

Copy link
Contributor

@bensojona bensojona left a comment

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.

Copy link
Contributor

@bensojona bensojona left a comment

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.

Copy link
Contributor

@bensojona bensojona left a comment

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
Copy link

hashicorp-cla commented Mar 12, 2022

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.

None yet

3 participants