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

Unnecessary messages during "kops reconcile cluster" #17287

Open
stl-victor-sudakov opened this issue Feb 25, 2025 · 4 comments
Open

Unnecessary messages during "kops reconcile cluster" #17287

stl-victor-sudakov opened this issue Feb 25, 2025 · 4 comments
Labels
kind/bug Categorizes issue or PR as related to a bug.

Comments

@stl-victor-sudakov
Copy link

/kind bug

1. What kops version are you running? The command kops version, will display
this information.

v1.31.0

2. What Kubernetes version are you running? kubectl version will print the
version if a cluster is running or provide the Kubernetes version specified as
a kops flag.

v1.31.6

3. What cloud provider are you using?

AWS

4. What commands did you run? What is the simplest way to reproduce this issue?

kops upgrade cluster --yes && kops reconcile cluster --yes

5. What happened after the commands executed?

The execution was successful but the kops console output was full of warnings like Changes may require instances to restart: kops rolling-update cluster.

6. What did you expect to happen?

I think these messages should not be printed because the "kops reconcile cluster" command is expected tto be self-sufficient and running other commands after it is not required. So these messages can be misleading, especially to new users.

7. Please provide your cluster manifest. Execute
kops get --name my.example.com -o yaml to display your cluster manifest.
You may want to remove your cluster name and other sensitive information.

N/A

@k8s-ci-robot k8s-ci-robot added the kind/bug Categorizes issue or PR as related to a bug. label Feb 25, 2025
@rifelpet
Copy link
Member

I improved this in #17282 which will be in the next batch of patch releases

@stl-victor-sudakov
Copy link
Author

I improved this in #17282 which will be in the next batch of patch releases

Great news!

Out of curiosity, why was the "Detected single-control-plane cluster; won't detach before draining" message removed?

@rifelpet
Copy link
Member

That was unrelated to reconcile specifically, it just wasn't particularly useful.

@stl-victor-sudakov
Copy link
Author

stl-victor-sudakov commented Feb 26, 2025

That was unrelated to reconcile specifically, it just wasn't particularly useful.

As a person running some single-control-plane clusters since the time when there was no special treatment of such clusters, I used to find the message comforting :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug.
Projects
None yet
Development

No branches or pull requests

3 participants