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

leader-elect flags are not being accepted with ClusterAutoscaler 1.28.0 version #6421

Closed
vbhargav875 opened this issue Jan 5, 2024 · 9 comments
Labels
kind/bug Categorizes issue or PR as related to a bug.

Comments

@vbhargav875
Copy link
Contributor

vbhargav875 commented Jan 5, 2024

Which component are you using?:
ClusterAutoscaler - v1.28.0

Which autoscaling component hosted in this repository (cluster-autoscaler, vertical-pod-autoscaler, addon-resizer, helm charts) is the bug in?
cluster-autoscaler

What version of the component are you using?:
v1.28.0

What version of the relevant component are you using? Either the image tag or helm chart version.
cluster-autoscaler-release-1.28.0

Component version:

What k8s version are you using (kubectl version)?:
v1.28.2

kubectl version Output
$ kubectl version
`$ kubectl version
WARNING: This version information is deprecated and will be replaced with the output from kubectl version --short.  Use --output=yaml|json to get the full version.
Client Version: version.Info{Major:"1", Minor:"27", GitVersion:"v1.27.2", GitCommit:"7f6f68fdabc4df88cfea2dcf9a19b2b830f1e647", GitTreeState:"clean", BuildDate:"2023-05-17T14:20:07Z", GoVersion:"go1.20.4", Compiler:"gc", Platform:"darwin/amd64"}
Kustomize Version: v5.0.1
Server Version: version.Info{Major:"1", Minor:"28", GitVersion:"v1.28.2", GitCommit:"6f3fcf83109204e359ca38144405293dbeec2200", GitTreeState:"clean", BuildDate:"2023-10-20T09:33:45Z", GoVersion:"go1.20.8 4493 X:boringcrypto", Compiler:"gc", Platform:"linux/amd64"}`

What environment is this in?:

If you're using a cloud provider or hardware configuration as your deployment environment let us know here.
Oracle Cloud Infrastructure

What did you expect to happen?:
--leader-elect flag should be accepted.

What behaviour did you expect to see?
Should not observe the below error
unknown flag: --leader-elect

What happened instead?:
Observed below
unknown flag: --leader-elect

What behaviour did see instead?
Observed below error
unknown flag: --leader-elect

How to reproduce it (as minimally and precisely as possible):
Pass leader-elect argument to cluster-autoscaler v1.28.0

@vbhargav875 vbhargav875 added the kind/bug Categorizes issue or PR as related to a bug. label Jan 5, 2024
@MaciekPytel
Copy link
Contributor

@gjtempleton
Copy link
Member

That issue was only present in v1.28.1 as far as we were aware. Can you double check the image tag you're using where you're seeing this behaviour please @vbhargav875 ?

@vbhargav875
Copy link
Contributor Author

@gjtempleton I can confirm that the version I used was v1.28.0
Referring to the version specified in the version.go

@gjtempleton
Copy link
Member

gjtempleton commented Jan 8, 2024

Are you building the images yourselves or using an image published to registry.k8s.io?

If building yourself, can you confirm whether this commit is included: e75903e ?

@vbhargav875
Copy link
Contributor Author

I am building the image myself. Yes, this commit is included.
If it helps, I had built the image on Nov 14th 2023 with the changes in the cluster-autoscaler-release-1.28 branch.
It looks like 1.28.1 came out towards the end on Nov.

@gjtempleton
Copy link
Member

gjtempleton commented Jan 9, 2024

Yeah, in that case you've picked up the known bug from our v1.28.1 release, we only increment the version.go as part of the release and tag process itself. Building based on the head of a release branch will most likely put you in a state where your packaged code isn't matching a release cut by us.

Rebasing off the latest patch release in a given minor release series would be what I'd recommend going forward.

Any objection to us closing this as it's been patched in the release branches?

@vbhargav875
Copy link
Contributor Author

Understood, thank you.
Yes, this issue can be closed.

@gjtempleton
Copy link
Member

Thanks, and thanks for reporting it!
/close

@k8s-ci-robot
Copy link
Contributor

@gjtempleton: Closing this issue.

In response to this:

Thanks, and thanks for reporting it!
/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

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

4 participants