Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

feat: Introduce Amazon Linux 2023 ARM image#4780

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
dimamo5 wants to merge5 commits intogithub-aws-runners:main
base:main
Choose a base branch
Loading
fromdimamo5:amazon-linux-2023-arm

Conversation

@dimamo5
Copy link

This PR proposes to introduce a new AMI for Amazon Linux 2023 in the ARM 64 architecture. I took a similar approach to what was done for Ubuntu Jammy where the code is duplicated into a new Packer directory. I'm also open to parameterize the runner architecture and have a single Packer configuration.

@dimamo5dimamo5 requested a review froma team as acode ownerSeptember 21, 2025 13:34
@dimamo5dimamo5 changed the titleIntroduce Amazon Linux 2023 ARM imagefeat: Introduce Amazon Linux 2023 ARM imageSep 21, 2025
@npalm
Copy link
Member

To me the example looks similar to al2023 for x64. Images are just an example and not tested at all. I have no issue with adding another example. But would it not be possible to make the al2023 more generic and just inject the architecture?

Any thoughts?

@dimamo5
Copy link
Author

But would it not be possible to make the al2023 more generic and just inject the architecture?

Made these changes in the latest commits. Seems to be working well in both architectures.

@dimamo5
Copy link
Author

@npalm when you have some free time, could you take a look?

## install the runner

s3_location=${S3_LOCATION_RUNNER_DISTRIBUTION}
architecture=${RUNNER_ARCHITECTURE}
Copy link
Member

Choose a reason for hiding this comment

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

What is the reason of removing this variable? Is it not used?

@npalm
Copy link
Member

Thx for the PR, will not be able to able to get the PR merged in the next weeks. I will catch up end of the month. Sorry for tthe delay.

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@npalmnpalmnpalm left review comments

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@dimamo5@npalm

[8]ページ先頭

©2009-2025 Movatter.jp