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

Bootstrap project using kubebuilder 3.1.0 #1

Merged
merged 2 commits into from
Aug 17, 2021

Conversation

baijum
Copy link
Member

@baijum baijum commented Aug 6, 2021

Commands used to bootstrap:

kubebuilder init --domain="service.binding" --repo="github.com/kubernetes-sigs/service-binding" --owner="The Kubernetes Authors" --project-name="service-binding-operator"
kubebuilder edit --multigroup=true
kubebuilder create api --crd-version="v1" --group="" --version="v1alpha2" --kind="ServiceBinding" --resource --controller
kubebuilder create api --crd-version="v1" --group="" --version="v1alpha2" --kind ClusterApplicationResourceMapping --namespaced=false --resource --controller
make generate

@sbose78
Copy link

sbose78 commented Aug 12, 2021

Are these inter-operable with kubebuilder?

PROJECT Outdated Show resolved Hide resolved
.dockerignore Outdated Show resolved Hide resolved
PROJECT Outdated Show resolved Hide resolved
Commands used to bootstrap:

kubebuilder init --domain="service.binding" --repo="github.com/kubernetes-sigs/service-binding" --owner="The Kubernetes Authors" --project-name="service-binding-operator"
kubebuilder edit --multigroup=true
kubebuilder create api --crd-version="v1" --group="" --version="v1alpha2" --kind="ServiceBinding" --resource --controller
kubebuilder create api --crd-version="v1" --group="" --version="v1alpha2" --kind ClusterApplicationResourceMapping --resource --controller
make generate
@baijum baijum changed the title Bootstrap project using operator-sdk v1.10.1 Bootstrap project using kubebuilder 3.1.0 Aug 17, 2021
@baijum
Copy link
Member Author

baijum commented Aug 17, 2021

I have bootstrapped the project using kubebuilder. Force pushed the commit as the old commit is obsolete.

PROJECT Outdated
version: v1alpha2
- api:
crdVersion: v1
namespaced: true
Copy link
Contributor

Choose a reason for hiding this comment

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

should not be namespaced

Copy link
Member Author

Choose a reason for hiding this comment

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

Fixed. 295e16d

@@ -0,0 +1,24 @@
# permissions for end users to edit clusterapplicationresourcemappings.
Copy link
Contributor

Choose a reason for hiding this comment

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

is this file used?

Copy link
Member Author

Choose a reason for hiding this comment

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

The RBAC-related files are auto-generated by kubebuilder. Probably helpful for those who are packing this project (say Helm charts).

Copy link
Contributor

Choose a reason for hiding this comment

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

ok. I was a bit surprised to not see it listed in the kustomization.yaml file, even if commented

@@ -0,0 +1,20 @@
# permissions for end users to view clusterapplicationresourcemappings.
Copy link
Contributor

Choose a reason for hiding this comment

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

is this file used?

@@ -0,0 +1,24 @@
# permissions for end users to edit servicebindings.
Copy link
Contributor

Choose a reason for hiding this comment

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

is this file used?

@@ -0,0 +1,20 @@
# permissions for end users to view servicebindings.
Copy link
Contributor

Choose a reason for hiding this comment

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

is this file used?

@scothis scothis requested a review from a team August 17, 2021 14:00
@scothis scothis merged commit 3fd7947 into servicebinding:main Aug 17, 2021
@baijum baijum deleted the bootstrap branch August 18, 2021 02:32
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