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

[release-1.25] Backports for 2023-10 release #8617

Merged
merged 21 commits into from
Oct 13, 2023

Conversation

Fixes performance issue under load, ref: etcd-io/etcd#15402 and kubernetes/kubernetes#118460

Signed-off-by: Brad Davidson <[email protected]>
(cherry picked from commit 8c73fd6)
Signed-off-by: Brad Davidson <[email protected]>
Signed-off-by: Brad Davidson <[email protected]>
(cherry picked from commit 1e38b5d)
Signed-off-by: Brad Davidson <[email protected]>
Signed-off-by: Brad Davidson <[email protected]>
(cherry picked from commit 0e5c760)
Signed-off-by: Brad Davidson <[email protected]>
@brandond brandond requested a review from a team as a code owner October 12, 2023 22:46
Signed-off-by: Brad Davidson <[email protected]>
(cherry picked from commit 5fe4f67)
Signed-off-by: Brad Davidson <[email protected]>
Signed-off-by: Brad Davidson <[email protected]>
(cherry picked from commit 9bb1ce1)
Signed-off-by: Brad Davidson <[email protected]>
Signed-off-by: Brad Davidson <[email protected]>
(cherry picked from commit 22065af)
Signed-off-by: Brad Davidson <[email protected]>
Signed-off-by: Brad Davidson <[email protected]>
(cherry picked from commit 64107b5)
Signed-off-by: Brad Davidson <[email protected]>
@brandond brandond force-pushed the 2023-10-backports_release-1.25 branch from bb987f1 to 947b84d Compare October 12, 2023 22:50
Also adds a hack go script to print the embedded CRDs, for developer use.

Signed-off-by: Brad Davidson <[email protected]>
(cherry picked from commit 500744b)
Signed-off-by: Brad Davidson <[email protected]>
Signed-off-by: Brad Davidson <[email protected]>
(cherry picked from commit 676b00a)
Signed-off-by: Brad Davidson <[email protected]>
…estEntityTooLarge

Signed-off-by: Brad Davidson <[email protected]>
(cherry picked from commit 2b0e2e8)
Signed-off-by: Brad Davidson <[email protected]>
Consistently refer to object keys as such, simplify error handling.

Signed-off-by: Brad Davidson <[email protected]>
(cherry picked from commit f1afe15)
Signed-off-by: Brad Davidson <[email protected]>
Attempt to use timestamp from creation or filename instead of file/object modification times

Signed-off-by: Brad Davidson <[email protected]>
(cherry picked from commit 8d47645)
Signed-off-by: Brad Davidson <[email protected]>
Also, don't list ONLY s3 snapshots if S3 is enabled.

Signed-off-by: Brad Davidson <[email protected]>
(cherry picked from commit 80f909d)
Signed-off-by: Brad Davidson <[email protected]>
Write the extra metadata both locally and to S3. These files are placed such that they will not be used by older versions of K3s that do not make use of them.

Signed-off-by: Brad Davidson <[email protected]>
(cherry picked from commit 7464007)
Signed-off-by: Brad Davidson <[email protected]>
Fixes snapshot list coming out in non-deterministic order

Signed-off-by: Brad Davidson <[email protected]>
(cherry picked from commit a15b804)
Signed-off-by: Brad Davidson <[email protected]>
Signed-off-by: Brad Davidson <[email protected]>
(cherry picked from commit 5cd4f69)
Signed-off-by: Brad Davidson <[email protected]>
Reconcile snapshot CRs instead of ConfigMap; manage ConfigMap downstream from CR list

Signed-off-by: Brad Davidson <[email protected]>
(cherry picked from commit 550ab36)
Signed-off-by: Brad Davidson <[email protected]>
This required pulling the token hash stuff out of the cluster package, into util.

Signed-off-by: Brad Davidson <[email protected]>
(cherry picked from commit d885162)
Signed-off-by: Brad Davidson <[email protected]>
Snapshot delete/prune tests were only working because the delete command
would report success even when deleting a snapshot that didn't exist,
and the test regex was finding the snapshot name multiple times in
the list output and deleting it twice.

Snapshot restore tests seem to have expected the deployment to be rolled out
immediately, which is not a reasonable expectation.

Signed-off-by: Brad Davidson <[email protected]>
(cherry picked from commit 7c5b69c)
Signed-off-by: Brad Davidson <[email protected]>
@brandond brandond force-pushed the 2023-10-backports_release-1.25 branch from 947b84d to 268f45b Compare October 12, 2023 23:06
* Switch build target from main.go to a package.
* Dont build with vcs

Signed-off-by: Dan Lorenc <[email protected]>
Signed-off-by: Derek Nola <[email protected]>
Co-authored-by: Derek Nola <[email protected]>
(cherry picked from commit 3d25e9f)
Signed-off-by: Brad Davidson <[email protected]>
@brandond brandond force-pushed the 2023-10-backports_release-1.25 branch from a17360c to 8200982 Compare October 13, 2023 17:08
@brandond
Copy link
Member Author

S390x is having issues, all other tests are passing. Merging.

@brandond brandond merged commit fdd8018 into k3s-io:release-1.25 Oct 13, 2023
@brandond brandond deleted the 2023-10-backports_release-1.25 branch June 6, 2024 21:07
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