Skip to content

infra: default cloud-instance.sh to latest CentOS 9 AMI#3248

Merged
mergify[bot] merged 1 commit intomainfrom
latest-centos-9-ami
Apr 2, 2025
Merged

infra: default cloud-instance.sh to latest CentOS 9 AMI#3248
mergify[bot] merged 1 commit intomainfrom
latest-centos-9-ami

Conversation

@ktdreyer
Copy link
Contributor

If the cloud-init.sh user has not hard-coded an AMI ID in ~/.instructlab/cloud-config, default to using the newest CentOS 9 AMI that the CPE team has published to the AWS marketplace.

The purpose of this change is to make it easier to launch the latest cloud images with fewer security vulnerabilities.


To test this, I've run cloud-instance.sh with the following scenarios:

  1. Setting an old EC2_AMI_ID="ami-09fc0e32ec75bfe3f" in ~/.instructlab/cloud-config continues to launch the old AMI
  2. Deleting EC2_AMI_ID from in ~/.instructlab/cloud-config properly selects the latest CentOS image, and dnf shows no pending updates on the node.

@ktdreyer ktdreyer requested review from booxter and danmcp March 27, 2025 22:09
If the cloud-init.sh user has not hard-coded an AMI ID in
~/.instructlab/cloud-config, default to using the newest CentOS 9 AMI
that the CPE team has published to the AWS marketplace.

The purpose of this change is to make it easier to launch the latest
cloud images with fewer security vulnerabilities.

Signed-off-by: Ken Dreyer <kdreyer@redhat.com>
@ktdreyer ktdreyer force-pushed the latest-centos-9-ami branch from d46dfd5 to 6515813 Compare March 31, 2025 19:38
@mergify mergify bot added the one-approval PR has one approval from a maintainer label Apr 2, 2025
@danmcp
Copy link
Member

danmcp commented Apr 2, 2025

Nice change!

@mergify mergify bot merged commit 9d2afec into main Apr 2, 2025
7 checks passed
@mergify mergify bot removed the one-approval PR has one approval from a maintainer label Apr 2, 2025
@mergify mergify bot deleted the latest-centos-9-ami branch April 2, 2025 16:40
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