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

worker profiles (#1915) #1974

Open
wants to merge 45 commits into
base: main
Choose a base branch
from
Open

worker profiles (#1915) #1974

wants to merge 45 commits into from

Conversation

hedge-sparrow
Copy link
Member

@hedge-sparrow hedge-sparrow commented Mar 17, 2025

What this PR does / why we need it:

The current state of this PR is that worker profiles can now be defined in unsupported overrides, and the first defined worker profile is always applied on install.

TODO:

  • Investigate kots changes required to enable worker profiles being set on join
  • confirm config UX

Which issue(s) this PR fixes:

Does this PR require a test?

Does this PR require a release note?


Does this PR require documentation?

* always enable last defined worker profile

* handle errors

* unmarshal to clusterconfig

* rename var

* extract k0s patch from unsupportedOverrides

* oops

* try using patch function

* debug

* pass overrides through to install flags

* simplify worker profile determination code

* just read config from disk

* choose worker profile based on node role

* fix crds

* log profiles

* f

* add worker profiles to spec

* workerprofiles in config

* generate

* split override functions

* include worker profiles in cfg on join

* move to unsupported overrides

* always apply first defined worker profile if any are set

* regenerate static

* manifests

---------

Co-authored-by: hedge-sparrow <[email protected]>
Copy link

github-actions bot commented Mar 17, 2025

This PR has been released (on staging) and is available for download with a embedded-cluster-smoke-test-staging-app license ID.

Online Installer:

curl "https://staging.replicated.app/embedded/embedded-cluster-smoke-test-staging-app/ci/appver-dev-1a4eadc" -H "Authorization: $EC_SMOKE_TEST_LICENSE_ID" -o embedded-cluster-smoke-test-staging-app-ci.tgz

Airgap Installer (may take a few minutes before the airgap bundle is built):

curl "https://staging.replicated.app/embedded/embedded-cluster-smoke-test-staging-app/ci-airgap/appver-dev-1a4eadc?airgap=true" -H "Authorization: $EC_SMOKE_TEST_LICENSE_ID" -o embedded-cluster-smoke-test-staging-app-ci.tgz

Happy debugging!

@hedge-sparrow hedge-sparrow marked this pull request as ready for review March 19, 2025 09:28
@hedge-sparrow hedge-sparrow requested a review from laverya March 24, 2025 14:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants