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

Added google_compute_router_peer. #341

Conversation

bradkwadsworth-mw
Copy link
Contributor

@bradkwadsworth-mw bradkwadsworth-mw commented Jul 25, 2023

Description of your changes

Added and tested the google_compute_router_peer resource in this issue.

I have:

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

How has this code been tested

Applied example to existing GCP project.
Screenshot 2023-07-28 at 10 25 06 AM

Uptest: https://github.com/upbound/provider-gcp/actions/runs/5718696102/job/15495055680

Copy link
Collaborator

@turkenf turkenf left a comment

Choose a reason for hiding this comment

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

Thank you @bradkwadsworth-mw for this PR, I left a few comments to consider. In addition, if the resources are not uptestable, please add a screenshot under the How has this code been tested title. Please see: #198

config/externalname.go Outdated Show resolved Hide resolved
examples/compute/routerpeer.yaml Outdated Show resolved Hide resolved
examples/compute/routerpeer.yaml Outdated Show resolved Hide resolved
examples/compute/externalvpngateway.yaml Outdated Show resolved Hide resolved
examples/compute/routerpeer.yaml Outdated Show resolved Hide resolved
examples/compute/routerpeer.yaml Outdated Show resolved Hide resolved
examples/compute/routerpeer.yaml Outdated Show resolved Hide resolved
config/compute/config.go Outdated Show resolved Hide resolved
@bradkwadsworth-mw bradkwadsworth-mw force-pushed the feature/add-compute-router-peer branch from cc042a8 to 41b4d46 Compare July 31, 2023 19:02
@turkenf
Copy link
Collaborator

turkenf commented Jul 31, 2023

/test-examples="examples/compute/routerpeer.yaml"

Copy link
Collaborator

@turkenf turkenf left a comment

Choose a reason for hiding this comment

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

Thank you @bradkwadsworth-mw, LGTM.

@turkenf turkenf changed the title Added google_compute_router_peer. Also updated examples. Added google_compute_router_peer. Jul 31, 2023
@turkenf turkenf merged commit 1687467 into crossplane-contrib:main Jul 31, 2023
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