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

feat: deploying admin console with the embedded license #86

Conversation

ricardomaraschini
Copy link
Contributor

if a license exists inside the binary we configure admin console to use it. if no license was found deploys admin console as we used to (without license).

if a license exists inside the binary we configure admin console to use
it. if no license was found deploys admin console as we used to (without
license).
@ricardomaraschini ricardomaraschini force-pushed the ricardomaraschini/sc-89490/embed-license-in-helmvm-binary branch from 1f579de to ee80394 Compare September 26, 2023 14:44
@ricardomaraschini ricardomaraschini marked this pull request as ready for review September 26, 2023 15:26
Comment on lines +80 to +85
helmValues["automation"] = map[string]interface{}{
"license": map[string]interface{}{
"slug": license.Spec.AppSlug,
"data": string(raw),
},
}
Copy link
Member

Choose a reason for hiding this comment

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

I would prefer if this was a safeadd (ie it doesn't wipe automation.somethingelse) but this works for now

@ricardomaraschini ricardomaraschini merged commit ef4ea79 into main Sep 27, 2023
16 checks passed
@ricardomaraschini ricardomaraschini deleted the ricardomaraschini/sc-89490/embed-license-in-helmvm-binary branch September 27, 2023 08:56
emosbaugh pushed a commit that referenced this pull request Aug 26, 2024
* watch k0s chart objects

* do not update k0s cluster object if it is still applying charts
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