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

Add option --single for single node #737

Merged
merged 6 commits into from
Mar 23, 2021
Merged

Add option --single for single node #737

merged 6 commits into from
Mar 23, 2021

Conversation

ncopa
Copy link
Collaborator

@ncopa ncopa commented Feb 25, 2021

disables konnectivity, and implies --enable-worker

Signed-off-by: Natanael Copa ncopa@mirantis.com


name: Pull Request
about: Create a Pull Request
title: ''
labels: ''
assignees: ''


Issue
Fixes #720

@ncopa ncopa requested a review from a team as a code owner February 25, 2021 14:53
@ncopa ncopa requested review from kke and trawler February 25, 2021 14:53
@mikhail-sakhnov
Copy link
Contributor

May be in a single node set up we could also automatically use sqlite storage instead of etcd, to save some local machine resources?

Copy link
Collaborator

@jnummelin jnummelin left a comment

Choose a reason for hiding this comment

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

If we do not start konnectivity, we also need to remove the konnectivity related configs from kube-apiserver

@trawler trawler mentioned this pull request Feb 26, 2021
cmd/controller.go Outdated Show resolved Hide resolved
@ncopa
Copy link
Collaborator Author

ncopa commented Feb 26, 2021

May be in a single node set up we could also automatically use sqlite storage instead of etcd, to save some local machine resources?

yes, absolutely, but solving that is trickier that it may seem.

@ncopa
Copy link
Collaborator Author

ncopa commented Feb 26, 2021

also join api can be disabled

@ncopa ncopa changed the title Add option --single for single node WIP: Add option --single for single node Mar 1, 2021
@ncopa ncopa changed the title WIP: Add option --single for single node Add option --single for single node Mar 9, 2021
@ncopa ncopa force-pushed the single branch 2 times, most recently from b10e832 to 1d05482 Compare March 19, 2021 09:22
disables konnectivity, and implies --enable-worker

Signed-off-by: Natanael Copa <ncopa@mirantis.com>
The k0sVars is currently a package scoped variable in the cmd package.
Use that instead of passing it as an argument.

Also use the k0sVars instead of constant.GetConfig, so we only get a
single call to GetConfig() from a central place.

Signed-off-by: Natanael Copa <ncopa@mirantis.com>
Refactor so we can pass the k0sVars with the default storage type.

Signed-off-by: Natanael Copa <ncopa@mirantis.com>
Signed-off-by: Natanael Copa <ncopa@mirantis.com>
Wait til k0s has terminated before restarting it. This should hopefully
fix the hacontrolplan test flakiness.

Signed-off-by: Natanael Copa <ncopa@mirantis.com>
jnummelin
jnummelin previously approved these changes Mar 23, 2021
jasmingacic
jasmingacic previously approved these changes Mar 23, 2021
trawler
trawler previously approved these changes Mar 23, 2021
Copy link
Contributor

@trawler trawler left a comment

Choose a reason for hiding this comment

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

lgtm 👍 (but we should also update our single node guide appropriately)

Signed-off-by: Natanael Copa <ncopa@mirantis.com>
@ncopa ncopa dismissed stale reviews from trawler, jasmingacic, and jnummelin via 503294a March 23, 2021 10:27
@ncopa ncopa merged commit 43282a8 into k0sproject:main Mar 23, 2021
@ncopa ncopa deleted the single branch March 23, 2021 10:49
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.

add option for --single
5 participants