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

Upgrade can delete machine without creating a new one #93

Open
dlipovetsky opened this issue Aug 12, 2018 · 4 comments
Open

Upgrade can delete machine without creating a new one #93

dlipovetsky opened this issue Aug 12, 2018 · 4 comments
Labels
kind/bug Something isn't working

Comments

@dlipovetsky
Copy link
Contributor

Let's suppose the user runs cctl machine upgrade, and deleteMachine succeeds, but createMachine fails. That means the machine has been removed from the on-disk state, and cctl machine upgrade cannot be invoked again; cctl create machine must be invoked.

@dlipovetsky
Copy link
Contributor Author

If we don't update state after delete, another invocation of machine upgrade will fail unless delete is idempotent. It isn't yet (e.g. drain/delete fail if the node isn't found).

@dlipovetsky dlipovetsky added the kind/bug Something isn't working label Aug 12, 2018
@dlipovetsky
Copy link
Contributor Author

@puneetguptanitj FYI

@puneetguptanitj
Copy link
Contributor

Thanks Daniel, once we start persisting kubeconfig into cctl-state we might be able to make drain and delete idempotent as well. (As we would be able to execute it directly from the machine where we run cctl). Will update the ticket with findings soon.

@puneetguptanitj
Copy link
Contributor

This can be better addressed through in place upgrade, that might allow us to not do drain and/or delete

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants