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

ci: disable metrics-server, bump k8s versions, increase cpu/ram of nodes #1092

Merged
merged 6 commits into from
Nov 29, 2023

Conversation

ldevulder
Copy link
Contributor

@ldevulder ldevulder commented Nov 22, 2023

metrics-server is not useful for QA, so better to remove it to save some resources. This also remove/workaround some sporadic issues we have with the installation of CertManager.

This also fixes some other sporadic issues. It should fix #1087.

Verification runs:

@ldevulder ldevulder self-assigned this Nov 22, 2023
@ldevulder ldevulder requested a review from a team as a code owner November 22, 2023 15:59
@ldevulder ldevulder force-pushed the ci-disable-metrics-server branch 3 times, most recently from ea2fc18 to 3df0f90 Compare November 22, 2023 17:44
Copy link
Contributor

@frelon frelon left a comment

Choose a reason for hiding this comment

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

LGTM!

@ldevulder ldevulder marked this pull request as draft November 27, 2023 09:31
Signed-off-by: Loic Devulder <ldevulder@suse.com>
metrics-server is not useful for QA, so better to remove it to save some
resources. This also remove/workaround some sporadic issues we have
with the installation of CertManager.

Signed-off-by: Loic Devulder <ldevulder@suse.com>
Signed-off-by: Loic Devulder <ldevulder@suse.com>
Signed-off-by: Loic Devulder <ldevulder@suse.com>
Increase values to:
- 4 vCPUs, 8GB of RAM for K3s
- 6 vCPUs, 10GB of RAM for RKE2

Signed-off-by: Loic Devulder <ldevulder@suse.com>
@ldevulder ldevulder force-pushed the ci-disable-metrics-server branch from f430ef7 to 51a2135 Compare November 29, 2023 09:48
Sometimes `GetFileFromURL` fails because the URL cannot be joined on
first try.

Signed-off-by: Loic Devulder <ldevulder@suse.com>
@ldevulder ldevulder marked this pull request as ready for review November 29, 2023 11:39
@ldevulder ldevulder changed the title ci: disable metrics-server on upstream cluster ci: disable metrics-server, bump k8s versions, increase cpu/ram of nodes Nov 29, 2023
@ldevulder ldevulder merged commit 2731138 into main Nov 29, 2023
@ldevulder ldevulder deleted the ci-disable-metrics-server branch November 29, 2023 15:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

e2e/ci: check failures in RKE2 multi-cluster tests
3 participants