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

Replaced Linux fixed AMI source with source AMI filter #636

Merged
merged 1 commit into from Sep 24, 2019

Conversation

cawilson
Copy link
Contributor

Replaced the hard-coded AMI source when building the Linux instance (using Packer) to use a filter query instead. This way the latest AMI is always sourced and the value doesn't need to be as actively maintained (matches the Packer builder for the Windows image, which uses a filter query).

Doing this also reduces the need to apply updates at build time.

@lox
Copy link
Contributor

lox commented Sep 24, 2019

Thanks @cawilson! Appreciate it! 🙇🏼‍♂️

@lox lox merged commit db97804 into buildkite:master Sep 24, 2019
@cawilson cawilson deleted the linux-dynamic-source-ami-filter branch September 24, 2019 01:08
lox added a commit that referenced this pull request Nov 4, 2019
Replaced Linux fixed AMI source with source AMI filter
yob added a commit that referenced this pull request Feb 25, 2021
Prior to PR #636 we looked up the source AMI to use for packer builds
in the Makefile. However in #636 we instead configured packer with some
AMI filter rules and allow it to choose the base AMI itself.

These bits the Makefile are unused.
yob added a commit that referenced this pull request Feb 25, 2021
Prior to PR #636 we looked up the source AMI to use for packer builds
in the Makefile. However in #636 we instead configured packer with some
AMI filter rules and allow it to choose the base AMI itself.

These bits the Makefile are unused.
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

2 participants