-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Feature: Native SSA Support #3183
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
Comments
cc @JoelSpeed |
@sbueringer Could you describe what's the gap with SSA in controller-runtime? I was under the impression SSA is already supported with |
Yep that works, but we want it to be a first-class citizen. Please see the initial issue for a list of missing things. |
I think I ran into a bug with SSA: #347 (comment) Let me know if this is related to the issue being fixed. |
One gap that we have is the extraction functions for the ApplyConfigurations, which require openapi.json to be able to generate. The first step of which is to use kube-openapi to generate the openapi definitions as generated Go code. This code is then imported into models-schema which then generated an This openapi.json is then fed into the OpenAPISchemaFilePath argument to the applyconfiguration gen, which allows it to generate the extraction functions. I'd be interested to see if there was a way that we could combine the first two stages so that we don't have to output the intermediate generated go code, and could put the openapi.json file in a known location or even a temp location for creating the extract functions |
Tasks:
Tasks controller-tools:
The text was updated successfully, but these errors were encountered: