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

Fix code generation #1536

Merged
merged 5 commits into from
May 23, 2024
Merged

Conversation

alemorcuq
Copy link
Collaborator

Description of the change
Code generation wasn't working. This PR fixes it and updates the generated code:

  • Use latest version of controller-gen to generate the manifests for the CRD
  • Use kube_codegen.sh to generate the client code. The generate_groups.sh script we were using previously has been deprecated and removed.
  • Code generation is now done from a hack directory following the Kubernetes sample-controller example.
  • make manifests has been run to update the manifests.
  • make generate has been run to update the generated code.

Signed-off-by: Alejandro Moreno <amorenoc@vmware.com>
Signed-off-by: Alejandro Moreno <amorenoc@vmware.com>
Signed-off-by: Alejandro Moreno <amorenoc@vmware.com>
alvneiayu
alvneiayu previously approved these changes May 23, 2024
Copy link
Collaborator

@alvneiayu alvneiayu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

agarcia-oss
agarcia-oss previously approved these changes May 23, 2024
Signed-off-by: Alejandro Moreno <amorenoc@vmware.com>
@alemorcuq alemorcuq dismissed stale reviews from agarcia-oss and alvneiayu via 4ccbe8a May 23, 2024 11:28
Signed-off-by: Alejandro Moreno <amorenoc@vmware.com>
@alemorcuq alemorcuq merged commit fcecd38 into bitnami-labs:main May 23, 2024
16 checks passed
@alemorcuq alemorcuq deleted the fix-code-generation branch May 23, 2024 11:40
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

Successfully merging this pull request may close these issues.

3 participants