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

coreos-overlay: Add support for new EKS versions #1210

Merged
merged 3 commits into from
Oct 5, 2023

Conversation

hhoover
Copy link
Contributor

@hhoover hhoover commented Oct 4, 2023

Adds S3 download locations for newer EKS versions

Flatcar out of the box only works for EKS 1.23 and older, this adds support for newer versions in the download_kubelet.sh script. This change adds download URLs for EKS 1.24, 1.25, 1.26, 1.27 and 1.28 so images can more easily be used for EKS workers.

Similar to flatcar-archive/coreos-overlay#2110

How to use

Deploy an image as an EKS worker on cluster versions 1.24-1.28 (or just verify the downloads work)

Testing done

Tested S3 download URLs using aws s3 sync as it is in the script for other versions.

  • Changelog entries added in the respective changelog/ directory (user-facing change, bug fix, security fix, update)
  • Inspected CI output for image differences: /boot and /usr size, packages, list files for any missing binaries, kernel modules, config files, kernel modules, etc.

This change adds download URLs for EKS 1.24,
1.25, 1.26, 1.27 and 1.28 so images can
more easily be used for EKS workers.

Signed-off-by: Hart Hoover <[email protected]>
@hhoover hhoover force-pushed the pr/hhoover/eks-versions branch from 2d2076a to 63f825b Compare October 4, 2023 21:27
jepio added 2 commits October 5, 2023 09:55
This helps track which versions include certain changes because these ebuild
revisions show up in the package version list that we publish alongside each
release.

Signed-off-by: Jeremi Piotrowski <[email protected]>
@jepio
Copy link
Member

jepio commented Oct 5, 2023

Thanks for this contribution @hhoover, i've pushed an extra commit to include the PR link in the changelog (i know its annoying to do because you only get the pr number after you open the pr).

@github-actions
Copy link

github-actions bot commented Oct 5, 2023

@jepio jepio temporarily deployed to development October 5, 2023 08:00 — with GitHub Actions Inactive
@hhoover
Copy link
Contributor Author

hhoover commented Oct 5, 2023

@jepio looks like builds failed not related to the PR - did I miss something?

@jepio
Copy link
Member

jepio commented Oct 5, 2023

No, you did everything right. The failing test cases don't work for PRs from forks, I just ran the CI to see that things build correctly.

Again, thanks for this PR. We appreciate when users scratch their own itch :)

@jepio jepio merged commit 6b18048 into flatcar:main Oct 5, 2023
4 of 7 checks passed
@dongsupark dongsupark added the main label Oct 6, 2023
@hhoover hhoover deleted the pr/hhoover/eks-versions branch October 9, 2023 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants