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.

Sign up for GitHub

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

Jump to bottom

ci: set nodes config to recommended values #879

Merged
merged 1 commit into from
Jun 22, 2023
Merged

ci: set nodes config to recommended values #879

merged 1 commit into from
Jun 22, 2023

Conversation

ldevulder
Copy link
Contributor

@ldevulder ldevulder commented Jun 21, 2023 •
edited
Loading

It's better to test with the recommended values for both K3s and RKE2 instead of the minimal ones.

K3s HW requirements: https://docs.k3s.io/installation/requirements#hardware
RKE2 HW requirements: https://docs.rke2.io/install/requirements#hardware

Verification runs:

@ldevulder ldevulder self-assigned this Jun 21, 2023
@ldevulder ldevulder requested a review from a team as a code owner June 21, 2023 13:09
@ldevulder ldevulder marked this pull request as draft June 21, 2023 13:09
@ldevulder ldevulder force-pushed the ci-set-4gb-k3s-test branch from 55a497c to 7e6c937 Compare June 21, 2023 14:38
@ldevulder ldevulder marked this pull request as ready for review June 21, 2023 14:54
@ldevulder ldevulder force-pushed the ci-set-4gb-k3s-test branch 2 times, most recently from 882aa91 to 24fa947 Compare June 21, 2023 16:22
@ldevulder
ci: set nodes config to recommended values
0fd69c6 
It's better to test with the recommended values for both K3s and RKE2
instead of the minimal ones.

Signed-off-by: Loic Devulder <ldevulder@suse.com>
@ldevulder ldevulder force-pushed the ci-set-4gb-k3s-test branch from 24fa947 to 0fd69c6 Compare June 21, 2023 19:21
@ldevulder ldevulder merged commit 53a817b into main Jun 22, 2023
@ldevulder ldevulder deleted the ci-set-4gb-k3s-test branch June 22, 2023 08:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@juadk juadk juadk approved these changes

Assignees

@ldevulder ldevulder

Labels
kind/QA
Projects
Elemental
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ldevulder @juadk