-
Notifications
You must be signed in to change notification settings - Fork 74
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
adding a crossplane.yaml to the template #32
Conversation
Signed-off-by: salaboy <[email protected]>
Signed-off-by: salaboy <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice, thanks @salaboy! Do you want to also add some machinery to build the package image as well?
I was thinking of improving the Readme file with the instructions to do that. |
@hasheddan hmm.. now that I am looking at this comment again.. what do you mean by
and then to install the new provider
So back to my previous comment, was more like adding those steps to the README in this repo. |
package/crossplane.yaml
Outdated
|
||
spec: | ||
controller: | ||
image: crossplane/provider-template-controller:v0.0.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nitpick:
image: crossplane/provider-template-controller:v0.0.1 | |
image: crossplane/provider-template-controller:latest |
@salaboy maybe best to match the Makefile
for now?
@salaboy are you still interested in making this PR ready to merge? |
@muvaf I am .. let me get it done tomorrow early morning. |
@muvaf is there something else needed for this PR to be merged? |
Closing some very old PRs. :) |
This PR adds a crossplane.yaml example file to be able to package and distribute the provider.
This PR also includes a change in the name of the docker image generated by the make file adding
-controller
because I've seen in other providers and also in online tutorials that this pattern is being used.. For me, it was more clear to be able to differentiate the -controller image and the provider image.