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

Adding google_apigee_nat_address, google_artifact_registry_repository, google_beyondcorp_app_connection, google_beyondcorp_app_connector, google_beyondcorp_app_gateway, google_bigquery_analytics_hub_data_exchange, google_bigquery_analytics_hub_listing to v1beta1 version #194

Merged
merged 1 commit into from
Feb 1, 2023

Conversation

mykolalosev
Copy link
Contributor

@mykolalosev mykolalosev commented Jan 31, 2023

Description of your changes

Adding apigee (1) resources to v1beta1 version

Adding artifactregistry (1) resources to v1beta1 version

Adding beyondcorp (3) resources to v1beta1 version

Adding bigqueryanalyticshub (2) resources to v1beta1 version

Fixes #173

I have:

  • Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

Manual:
google_apigee_nat_address - An organization resource requires a real project ID
google_apigee_nat_address

google_beyondcorp_app_connection - The resource requires a real Project ID
google_beyondcorp_app_connection

Manual & Uptest:
google_artifact_registry_repository - https://github.com/upbound/provider-gcp/actions/runs/4055433119
google_beyondcorp_app_connector - https://github.com/upbound/provider-gcp/actions/runs/4055438632
google_beyondcorp_app_gateway - https://github.com/upbound/provider-gcp/actions/runs/4055446952
google_bigquery_analytics_hub_data_exchange - https://github.com/upbound/provider-gcp/actions/runs/4055423814
google_bigquery_analytics_hub_listing - https://github.com/upbound/provider-gcp/actions/runs/4055511542

@mykolalosev
Copy link
Contributor Author

/test-examples="examples/bigquery/analyticshubdataexchange.yaml"

@mykolalosev
Copy link
Contributor Author

/test-examples="examples/artifact/registryrepository.yaml"

@mykolalosev
Copy link
Contributor Author

/test-examples="examples/beyondcorp/appconnector.yaml"

@mykolalosev
Copy link
Contributor Author

/test-examples="examples/beyondcorp/appgateway.yaml"

@mykolalosev
Copy link
Contributor Author

/test-examples="examples/bigquery/analyticshublisting.yaml"

Copy link
Collaborator

@sergenyalcin sergenyalcin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @mykolalosev LGTM!

@mykolalosev mykolalosev merged commit bd4d72e into crossplane-contrib:main Feb 1, 2023
@mykolalosev mykolalosev deleted the issue-173 branch February 1, 2023 12:52
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.

Moving apigee(1), artifactregistry(1), beyondcorp(3), bigqueryanalyticshub(2) resources to v1beta1 version
2 participants