Skip to content

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

Merged
merged 1 commit into from
Jun 12, 2025
Merged

Add Azure ASO CRDs #570

merged 1 commit into from
Jun 12, 2025

Conversation

afarbos
Copy link
Contributor

@afarbos afarbos commented May 15, 2025

PR Checklist

Generated with:

kind create cluster && \
  helm install cert-manager jetstack/cert-manager --namespace cert-manager --create-namespace --set installCRDs=true --wait --wait-for-jobs && \
  helm install aso2 aso2/azure-service-operator --create-namespace --namespace=azureserviceoperator-system --set crdPattern='*' --wait --wait-for-jobs && \
  ./Utilities/crd-extractor.sh && \
  cp -r $HOME/.datree/crdSchemas/ . && \
  kind delete cluster && \
  git add $(ls -1 *azure.com/* | grep -vE  '^.+storage.json$') && \
  git commit -m "Add Azure ASO CRDs"

@eyarz
Copy link
Member

eyarz commented May 20, 2025

From the naming convention, it looks like some of it is auto-generated (e.g., api_v1api20220801storage).
Do you have a link to the original CRD to check if those are validated schemas?

@afarbos
Copy link
Contributor Author

afarbos commented May 20, 2025

From the naming convention, it looks like some of it is auto-generated (e.g., api_v1api20220801storage). Do you have a link to the original CRD to check if those are validated schemas?

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?

@eyarz
Copy link
Member

eyarz commented May 25, 2025

Yes, please remove everything that is auto-generated 😅

@afarbos
Copy link
Contributor Author

afarbos commented May 25, 2025

Yes, please remove everything that is auto-generated 😅

Done! Updated the CLI to generate this also.

Copy link
Member

@eyarz eyarz left a 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.

@eyarz
Copy link
Member

eyarz commented Jun 8, 2025

@afarbos just thinking out loud here.

As you know, the Azure ASO CRDs include over 400 Custom Resources.
If the CR names are generated with a timestamp, that means every time you update those CRs, you'll need to add 400+ new CRs instead of simply updating the existing ones.

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.

@afarbos
Copy link
Contributor Author

afarbos commented Jun 11, 2025

@afarbos just thinking out loud here.

As you know, the Azure ASO CRDs include over 400 Custom Resources. If the CR names are generated with a timestamp, that means every time you update those CRs, you'll need to add 400+ new CRs instead of simply updating the existing ones.

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.

@eyarz eyarz merged commit 813ec16 into datreeio:main Jun 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants