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

Controller-gen binary has bad version #7

Open
mogensen opened this issue Oct 27, 2021 · 3 comments
Open

Controller-gen binary has bad version #7

mogensen opened this issue Oct 27, 2021 · 3 comments

Comments

@mogensen
Copy link

First of, great tool!

The only issue I have encountered is that the version of controller-gen included in the bazel rules does not have the correct version set.

See:
image

This results in the following annotation on the generated CRDs:

apiVersion: apiextensions.k8s.io/v1
kind: CustomResourceDefinition
metadata:
  annotations:
    controller-gen.kubebuilder.io/version: (devel)

Because of this we can't tell which version is used to generate the different CRDs.

@ob
Copy link
Owner

ob commented Oct 28, 2021

Ah, good catch. I was really hoping that at some point controller-tools would put out binary releases, I keep waiting for #500 to be resolved :-)

At any rate, in the meantime I'll see if I can figure out how to fix it. Thanks!

@kriscfoster
Copy link

Hi @ob, I'm interested in this one too. I was thinking of trying to push a fix, if you have any pointers, thank you.

@ob
Copy link
Owner

ob commented May 9, 2022

@kriscfoster I think the problem is how controller-gen is being built. If you just clone it and build it you get this (devel) version. Whoh is what the build script is doing. I think the right fix is to actually solve kubernetes-sigs/controller-tools#500 and then just consume the binaries directly via and http_archive() rule.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants