-
Notifications
You must be signed in to change notification settings - Fork 272
Add Azure ASO CRDs #570
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
Add Azure ASO CRDs #570
Conversation
From the naming convention, it looks like some of it is auto-generated (e.g., |
I found the documentation https://azure.github.io/azure-service-operator/design/versioning/ So yes storage are auto-generated. Do you want me to remove them? |
Yes, please remove everything that is auto-generated 😅 |
Done! Updated the CLI to generate this also. |
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.
It looks like there are still some auto-generated CRs included in the PR.
@afarbos just thinking out loud here. As you know, the Azure ASO CRDs include over 400 Custom Resources. After just a few updates, we could end up with thousands of CRs just for Azure ASO. This could significantly impact the entire repo—making it much heavier and potentially causing issues for anyone trying to pull CRs due to network limitations or GitHub performance. Just something to consider as we look at long-term scalability. |
Are you referring to Azure/azure-service-operator#2920, I don't think it would apply to git here no? I would say they do generate more versions, however this is only with new features meaning they don't just generate a new timestamp just without purpose. This is linked with api versions. |
PR Checklist
Generated with: