-
Notifications
You must be signed in to change notification settings - Fork 288
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
Define Initial APIs #50
Comments
+1 for defining APIs using protos
|
@skriss @youngnick I'm interested in your feedback here since Contour does not use protos. It would be great to achieve consensus so we can start building out the APIs. |
xref Bootstrap API: #51 |
the decision was made in the maintainers meeting to use Kubebuilder based APIs over Protobufs
|
@LukeShu per the 5/24/22 meeting notes, please document the rationale for using kube-builder/controller-gen over protobuf. |
Sorry, I didn't get to this on Wednesday, and have been on vacation since then. Will get that documentation up soon. |
BackgroundFor those outside of the know: The RationaleSo why use the
Perhaps the pain points with Protobuf in this use-case are solvable with newer/better tools (there's one @arkodg mentioned that I hadn't heard of before, and whose name is escaping me at the moment). But ultimately, we're specifying a Kubernetes API, and it makes more sense to use a tooling ecosystem that is aligned with that. Let's go with the mainstream choice; and Kubebuilder absolutely is the "default" choice for new Kubernetes projects. |
This issue has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in the next 7 days unless it is tagged "help wanted" or "no stalebot" or other activity occurs. Thank you for your contributions. |
#51 defines the initial config APIs for control and data planes. |
#16 specifies the need to define APIs, e.g. bootstrap config. To begin developing these APIs, the community should agree on:
The text was updated successfully, but these errors were encountered: