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

feat!: replace Amazon Linux 2 by Amazon Linux 2023 as default Linux AMI #3437

Merged
merged 17 commits into from
Oct 19, 2023

Conversation

eabrouwer3
Copy link
Contributor

Add runner_amazon_linux_2022 boolean variable

Get rid of amazon linux 2022 config option

Make PR changes even smaller

Update all references to amzn2. Also bump to al2023 (latest supported version)

Add the al2023 packer file

@eabrouwer3
Copy link
Contributor Author

Working right now on addressing the latest comment from #3293.

Add `runner_amazon_linux_2022` boolean variable

Get rid of amazon linux 2022 config option

Make PR changes even smaller

Update all references to amzn2. Also bump to al2023 (latest supported version)

Add the al2023 packer file

curl comes shipped with al2023
@eabrouwer3
Copy link
Contributor Author

@GuptaNavdeep1983, I've fixed the packer build issue and rebased master onto my branch, so I think I've addressed any issues you had on the old PR.

@npalm, tagging you on this as well, as you were the "reviewer" on the old PR.

@npalm npalm self-requested a review August 17, 2023 20:05
@npalm npalm changed the title feat: Add Support for Amazon Linux 2022 feat!: replace Amazon Linux 2 by Amazon Linux 2023 as default Linux AMI Aug 17, 2023
modules/runners/main.tf Outdated Show resolved Hide resolved
@Chobito
Copy link

Chobito commented Sep 13, 2023

any update on this? seems that packer check failed due rate limit

@eabrouwer3
Copy link
Contributor Author

I've pushed up a change that resolves the two PR comments. Thanks @wangzw!

Copy link
Member

@npalm npalm left a comment

Choose a reason for hiding this comment

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

Just checked the PR with mult-runner example. Both the arm64 and amazon linux runners are not coming online. On the x64 runner looks like the curl installation is failing.

@wangzw
Copy link

wangzw commented Sep 15, 2023

Just checked the PR with mult-runner example. Both the arm64 and amazon linux runners are not coming online. On the x64 runner looks like the curl installation is failing.

You are right, curl-minimal is installed by default, and conflict with curl, I forget to mention that.

@eabrouwer3

@eabrouwer3
Copy link
Contributor Author

Just checked the PR with mult-runner example. Both the arm64 and amazon linux runners are not coming online. On the x64 runner looks like the curl installation is failing.

You are right, curl-minimal is installed by default, and conflict with curl, I forget to mention that.

So should we remove the curl install then? In modules/runners/templates/user-data.sh. I think we still need it in the ubuntu files where we install it, cause this shouldn't affect any of those.

@wangzw
Copy link

wangzw commented Sep 19, 2023

Just checked the PR with mult-runner example. Both the arm64 and amazon linux runners are not coming online. On the x64 runner looks like the curl installation is failing.

You are right, curl-minimal is installed by default, and conflict with curl, I forget to mention that.

So should we remove the curl install then? In modules/runners/templates/user-data.sh. I think we still need it in the ubuntu files where we install it, cause this shouldn't affect any of those.

I do not think yum install -y curl works for ubuntu, so just remove it. Anyone who use ubuntu has to customized a user_data.sh

Default user_data.sh only needs to match your default image.

@npalm npalm self-requested a review September 19, 2023 20:09
@wangzw
Copy link

wangzw commented Sep 23, 2023

Looks good to me, but you need to fix the conflicts.

@aktur
Copy link

aktur commented Sep 24, 2023

@eabrouwer3 I was updating a runner to Linux 2023 recently too and have two comments:

  • you should use dnf instead of yum on Linux 2023;
  • to install curl without conflicts, use the following line dnf install curl -y --allowerasing.

@npalm
Copy link
Member

npalm commented Oct 5, 2023

Will test the PR asap. Would like to get the PR merged with other breaking changes planned.

Copy link
Member

@npalm npalm left a comment

Choose a reason for hiding this comment

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

Validatinging examples

  • default
  • multi runners
  • pre-build ami

@Mogost
Copy link

Mogost commented Oct 10, 2023

Looks like this is connected to #3534

@aktur
Copy link

aktur commented Oct 10, 2023

Looks like this is connected to #3534

Yes, it is. I successfully ran the PR's runner, and the error has been resolved.

@garethallen-daemon
Copy link

Any update on this?

@npalm npalm changed the base branch from main to next October 19, 2023 11:52
@npalm npalm changed the base branch from next to main October 19, 2023 11:53
@npalm npalm changed the base branch from main to next October 19, 2023 12:06
Copy link
Member

@npalm npalm left a comment

Choose a reason for hiding this comment

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

Tested an all good. I will merge the PR into next. We planning to combine a few breaking changes and release next week.

@npalm npalm merged commit fb0f7e3 into philips-labs:next Oct 19, 2023
28 checks passed
@eabrouwer3 eabrouwer3 deleted the eb/2848-al2023-support branch October 19, 2023 17:31
This was referenced Oct 20, 2023
npalm added a commit that referenced this pull request Oct 26, 2023
…MI (#3437)

Replaces Amazon lLnux 2 (default) by Amazon Linux 2023. Impact for anyone relying on the default setup provided by the module.

---------

Co-authored-by: Niek Palm <npalm@users.noreply.github.com>
npalm added a commit that referenced this pull request Oct 27, 2023
…MI (#3437)

Replaces Amazon lLnux 2 (default) by Amazon Linux 2023. Impact for anyone relying on the default setup provided by the module.

---------

Co-authored-by: Niek Palm <npalm@users.noreply.github.com>
npalm pushed a commit that referenced this pull request Oct 27, 2023
🤖 I have created a release *beep* *boop*
---


##
[5.0.0](v4.7.0...v5.0.0)
(2023-10-27)

### Migration directions

See
[PR](#3552)
for migration directions.

### ⚠ BREAKING CHANGES

* remove depcrecated input and output variables
([#3567](#3567))
* Change runner labels to list[string] and apply TFLint
([#3410](#3410))
* replace Amazon Linux 2 by Amazon Linux 2023 as default Linux AMI
([#3437](#3437))

### Features

* replace Amazon Linux 2 by Amazon Linux 2023 as default Linux AMI
([#3437](#3437))
([35219f5](35219f5))


### Bug Fixes

* Change runner labels to list[string] and apply TFLint
([#3410](#3410))
([0b398f2](0b398f2))
* remove depcrecated input and output variables
([#3567](#3567))
([89c8575](89c8575))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: forest-releaser[bot] <80285352+forest-releaser[bot]@users.noreply.github.com>
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.