-
Notifications
You must be signed in to change notification settings - Fork 62
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
Update readme #103
Comments
@nabuskey I like to take this up. |
Thanks @fighting-dreamer. Let me know if you have any questions. |
@nabuskey I am trying to understand the above mentioned tools, I have worked with k8s to some extent and have small overview about other tools. Can you suggest the right way for approaching this issue. currently, I was trying to understand the k8s sigs and how controller-gen operates or what controller-gen do for us. |
If you haven't played with this tool yet, I'd highly recommend playing with it first. Just to make sure you are doing things right to get started, here's how to build and run idpbuilder on your laptop from the main branch. Note that I've only tested this on Linux and MacOS only. I don't know if it works on Windows because I haven't tested it.
As for your question regarding controller-gen, it's an utility very commonly used by Kubernetes operators. It allows you to generate code that's usually pain to write such as deepcopy and CRDs. At its core, idpbuilder is made of a few controllers so it makes sense to use controller-gen. For this issue though, you shouldn't have to worry about code generation stuff. I am looking to document the components installed to the kind cluster, the order, and how. When you run the command, the controller are started here: idpbuilder/pkg/controllers/run.go Line 12 in 9b25a0c
Once they start, the Reconcile methods are invoked periodically. For example:
You should be able to follow the flow from here on. Let me know if you have questions and feel free to DM me on the CNCF slack. |
Can we move this text part of a CONTRIBUTING.md file (= setup and build) please as that will help a lot our contributors (aka https://github.com/quarkusio/quarkus/blob/main/CONTRIBUTING.md) ? |
Great idea and I can work on that for sure. @fighting-dreamer Do you have any more questions? Feel free to reach out via DM or in the #cnoe-interest channel. |
I am trying to understand it more. I think it will be early for me to pick it up. Not able to give proper time. @nabuskey unassigning it for now. |
@nabuskey is this closed given the latest changes to README? |
Update the readme file with:
The text was updated successfully, but these errors were encountered: