Skip to content
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

feat(datacatalog): add taxonomy & policy tags resources #637

Conversation

nilan3
Copy link
Contributor

@nilan3 nilan3 commented Oct 22, 2024

Description of your changes

Add resources for the following terraform resources:

  • google_data_catalog_taxonomy
  • google_data_catalog_policy_tag

I have:

  • Read and followed Crossplane's contribution process.
  • Run make reviewable to ensure this PR is ready for review.
  • Added backport release-x.y labels to auto-backport this PR if necessary.

How has this code been tested

Running Uptest locally against a personal sandbox GCP project

UPTEST_EXAMPLE_LIST="examples/datacatalog/v1beta1/policytag.yaml"

--- PASS: kuttl (50.28s)
    --- PASS: kuttl/harness (0.00s)
        --- PASS: kuttl/harness/case (49.02s)
PASS
10:45:53 [ OK ] running automated tests
image image

@nilan3 nilan3 changed the title Feature/support taxonomy policy tags Feature/support taxonomy policy tags & tag template IAM Oct 23, 2024
@nilan3 nilan3 changed the title Feature/support taxonomy policy tags & tag template IAM Feature/Data Catalog - support taxonomy policy tags & tag template IAM Oct 23, 2024
@nilan3 nilan3 changed the title Feature/Data Catalog - support taxonomy policy tags & tag template IAM feat(datacatalog): add taxonomy policy tags & tag template IAM Oct 23, 2024
@nilan3 nilan3 changed the title feat(datacatalog): add taxonomy policy tags & tag template IAM feat(datacatalog): add taxonomy & policy tags resources Oct 28, 2024
@nilan3 nilan3 marked this pull request as ready for review October 28, 2024 11:49
@nilan3
Copy link
Contributor Author

nilan3 commented Oct 28, 2024

/test-examples="examples/datacatalog/v1beta1/policytag.yaml"

@nilan3
Copy link
Contributor Author

nilan3 commented Oct 28, 2024

can I please request to be a contributor for this repo?

@jeanduplessis
Copy link
Collaborator

can I please request to be a contributor for this repo?

Upon successful merging of your first PR you will automatically become a contributor.

@jeanduplessis
Copy link
Collaborator

/test-examples="examples/datacatalog/v1beta1/policytag.yaml"

@nilan3
Copy link
Contributor Author

nilan3 commented Oct 28, 2024

/test-examples="examples/datacatalog/v1beta1/policytag.yaml"

could you re run this test pls? I've just added a validation rule to the CRD

@turkenf
Copy link
Collaborator

turkenf commented Oct 28, 2024

/test-examples="examples/datacatalog/v1beta1/policytag.yaml"

@turkenf
Copy link
Collaborator

turkenf commented Oct 28, 2024

Could you please remove the API files of the resources you did not add?

nilan3

This comment was marked as off-topic.

@nilan3
Copy link
Contributor Author

nilan3 commented Oct 28, 2024

@turkenf thanks for the comment, those files are cleaned up

@jeanduplessis
Copy link
Collaborator

/test-examples="examples/datacatalog/v1beta1/policytag.yaml"

@jeanduplessis
Copy link
Collaborator

/test-examples="examples/datacatalog/v1beta1/taxonomy.yaml"

@nilan3 nilan3 force-pushed the feature/support_taxonomy_policy_tags branch from f664ca6 to a114497 Compare October 29, 2024 17:24
@nilan3
Copy link
Contributor Author

nilan3 commented Oct 29, 2024

/test-examples="examples/datacatalog/v1beta1/taxonomy.yaml"

@nilan3
Copy link
Contributor Author

nilan3 commented Oct 29, 2024

/test-examples="examples/datacatalog/v1beta1/taxonomy.yaml"

@jeanduplessis could you help with running this test again. I've just done a complete clean up

@jeanduplessis
Copy link
Collaborator

/test-examples="examples/datacatalog/v1beta1/taxonomy.yaml"

@nilan3
Copy link
Contributor Author

nilan3 commented Oct 30, 2024

/test-examples="examples/datacatalog/v1beta1/taxonomy.yaml"

@jeanduplessis do you know why the CI checks could be failing? It's failing on 'Vendor dependencies' stage

@turkenf
Copy link
Collaborator

turkenf commented Oct 30, 2024

@jeanduplessis do you know why the CI checks could be failing? It's failing on 'Vendor dependencies' stage

@nilan3, could you please run go mod tidy then run make generate and push the changes?

Screenshot 2024-10-30 at 14 16 27

@nilan3
Copy link
Contributor Author

nilan3 commented Oct 31, 2024

@turkenf it looks like make crddiff is failing with the following error:

09:59:40 [ .. ] installing uptest /workspaces/provider-upjet-gcp-1/.cache/tools/linux_x86_64/uptest-v0.11.1
curl: (22) The requested URL returned error: 404
09:59:41 [FAIL]
make: *** [build/makelib/k8s_tools.mk:204: /workspaces/provider-upjet-gcp-1/.cache/tools/linux_x86_64/uptest-v0.11.1] Error 1

Also I've had to modify the CRD conditions after running make generate. The diff shows the changes being reverted by the make generate
image

@turkenf
Copy link
Collaborator

turkenf commented Oct 31, 2024

Hi @nilan3 , thank you for your efforts.

The issue we got in the report-breaking changes pipeline seems meaningless/temporary; Similarly, here is passed. If you have time, can you reset your branch to main, add only the external name configurations of the resources you will add, run make generate, then add the examples and push with the new commit(s). This can help us get rid of temporary/meaningless errors.

Also, please do not make any manual changes to the generated files and CRDs and please sign the DCO.

Thanks in advance!

@nilan3 nilan3 closed this Oct 31, 2024
@nilan3 nilan3 force-pushed the feature/support_taxonomy_policy_tags branch from 8e34bde to dc0fbe5 Compare October 31, 2024 12:31
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