Skip to content

Update datadog CRDs#348

Closed
trahim wants to merge 2 commits intodatreeio:mainfrom
trahim:add-dd-crd-update
Closed

Update datadog CRDs#348
trahim wants to merge 2 commits intodatreeio:mainfrom
trahim:add-dd-crd-update

Conversation

@trahim
Copy link
Contributor

@trahim trahim commented Jul 8, 2024

No description provided.

@userbradley
Copy link
Collaborator

Thanks for adding this. Please can you list where you got the schema from, and possibly run a validation against some of your own resources to show it works? I am not familiar with the Datadog CRD sadly

@trahim trahim force-pushed the add-dd-crd-update branch from 6aa4115 to 4cf18ec Compare August 1, 2024 15:15
@trahim
Copy link
Contributor Author

trahim commented Aug 1, 2024

Hey @userbradley apologies for the late reply I ran the following.

git clone https://github.com/DataDog/helm-charts.git
git clone https://github.com/yannh/kubeconform.git
git clone https://github.com/datreeio/CRDs-catalog.git
cd CRDs-catalog
for N in ../helm-charts/crds/*; do echo "# $N"; GROUP=$(basename "${N//_*}"); mkdir -p "$GROUP"; (cd "$GROUP"; ../../kubeconform/scripts/openapi2jsonschema.py "../$N"); done

@ElanHasson
Copy link
Collaborator

Hi @trahim,

Thanks for the PR!

Can you please recreate it using the CRD Extractor?

Thanks!

@trahim
Copy link
Contributor Author

trahim commented Mar 24, 2025

Hey @ElanHasson Sorry late reply but think this was already sorted by a previous PR so closing this

@trahim trahim closed this Mar 24, 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.

3 participants