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

cli: add an Upgrade command #2556

Closed
olix0r opened this issue Mar 25, 2019 · 3 comments
Closed

cli: add an Upgrade command #2556

olix0r opened this issue Mar 25, 2019 · 3 comments
Assignees
@olix0r
Labels
area/cli area/install needs/docs priority/P0 Release Blocker

Comments

@olix0r
Copy link
Member

olix0r commented Mar 25, 2019

Now that install fails if a cluster exists, we need an upgrade command that fetches the controller's config to use to configure the newer installation. This command should probably NOT accept override configuration (i.e. as is accepted by install and inject), since config updates and software upgrades should, hopefully, be distinct.
@olix0r olix0r self-assigned this Mar 25, 2019
@olix0r
Copy link
Member Author

olix0r commented Mar 25, 2019

i have a branch that adds the basis for the command...

This was referenced Mar 25, 2019
Merged
Closed
Merged
@grampelberg grampelberg mentioned this issue Mar 27, 2019
Closed
@alpeb
Copy link
Member

alpeb commented Mar 29, 2019

I just wanted to clarify that according to #2564 we are accepting override config flags after all.

@olix0r
Copy link
Member Author

olix0r commented Mar 29, 2019

@alpeb correct. we realized there was no way to update the configuration otherwise.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 18, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@olix0r olix0r

Labels
area/cli area/install needs/docs priority/P0 Release Blocker
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@grampelberg @olix0r @alpeb