-
Notifications
You must be signed in to change notification settings - Fork 519
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
3p packages update 1.15.0 #3362
3p packages update 1.15.0 #3362
Conversation
Following four commands are newly added in 2.39 version of util_linux( https://github.com/util-linux/util-linux/blob/master/Documentation/releases/v2.39-ReleaseNotes). These have been excluded for now:
|
105da99
to
2a501be
Compare
packages/ecs-agent/0002-bottlerocket-remove-unsupported-capabilities.patch
Outdated
Show resolved
Hide resolved
packages/ecs-agent/0002-bottlerocket-remove-unsupported-capabilities.patch
Outdated
Show resolved
Hide resolved
packages/ecs-agent/0002-bottlerocket-remove-unsupported-capabilities.patch
Outdated
Show resolved
Hide resolved
@@ -296,7 +296,7 @@ mv %{vpccni_gorepo}-%{vpccni_gitrev}/vendor go-vendor/%{vpccni_gorepo} | |||
# ├── LICENSE.amazon-ecs-cni-plugins | |||
# ├── LICENSE.amazon-vpc-cni-plugins | |||
# ├── NOTICE | |||
# ├── THIRD-PARTY | |||
# ├── THIRD_PARTY.md |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would like to double check that the contents have not changed meaningfully. I don't expect they have, but when paths and extensions change, its worth confirming nothing has changed under us.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This has been validated, the contents are same as earlier.
28f013e
to
5291c19
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good to have some answers to Matt's questions yet, but built an aws-k8s-1.27
node and poked around. Everything happy and no errors seen. Ran a sonobuoy quick test which passed. Functionally, everything looks good to me.
8f861ad
to
95a29ae
Compare
packages/ecs-agent/0001-bottlerocket-default-filesystem-locations.patch
Outdated
Show resolved
Hide resolved
As iproute2 is backward compatible to kernel
95d1bb0
to
be9f742
Compare
- Update amazon-ecs-cni-plugins dependency - Update amazon-vpc-cni-plugins dependency - Fix patches that was not applicable - Remove defaultCredentialsAuditLogFile, defaultRuntimeStatsLogFile defaultCredentialsAuditLogFile, defaultRuntimeStatsLogFile settings from 0001-bottlerocket-default-filesystem-locations.patch and do these settings using configuration instead. - Change THIRD-PARTY path form / to ecs-agent/THIRD_PARTY
be9f742
to
8d4cd96
Compare
Remove the excluded *.la files as these files do not exist anymore in version 2.39.2. Also added new added command files fadvise, pipesz, waitpid, blkpr to the excluded files
Exclude cni/bin/*.debug and cni/bin/tap
8d4cd96
to
1f0e2cc
Compare
Ran quick test for k8s-1.24 and ecs-1 variants again. Test passes. |
Issue number:
Closes #3170
Description of changes:
Regenerated ecs-agent patches that no longer cleanly apply.
Updates all third party packages except for the following:
binutils 2.38 -> 2.40 - bottlerocket-sdk has to move to 2.40 first.
docker related packages(docker-cli, docker-engine) 20.12.21 -> 20.12.23 - To sync with ECS-optimized AMI packaged docker version listed in its version table.
wicked 0.6.68 -> 0.6.70 - See update wicked to 0.6.70+ #2591
Testing done:
nvme1n1 259:1 0 10G 0 disk
nvme1n1p1 259:16 0 10G 0 part /.bottlerocket/rootfs/local
nvme1n1 259:1 0 100G 0 disk
nvme1n1p1 259:16 0 10G 0 part /var
Terms of contribution:
By submitting this pull request, I agree that this contribution is dual-licensed under the terms of both the Apache License, version 2.0, and the MIT license.