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(networkconnectivity): add Service Connection Policy support #624

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

rickard-von-essen
Copy link

@rickard-von-essen rickard-von-essen commented Sep 26, 2024

Description of your changes

This adds support for google_network_connectivity_service_connection_policy Terraform resource, see Terraform - Google - Service Connection Policy.

Includes #631

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

@rickard-von-essen
Copy link
Author

Working on a fix for this in upjet, here rickard-von-essen/upjet @ bug/ref-subnet, see this line.

This includes PR 441 which fixes scraping of Refs which contains lists. See upjet v1.4.1-0.20241007154751-5cdf36996e4a

Signed-off-by: Rickard von Essen <[email protected]>
Signed-off-by: Rickard von Essen <[email protected]>
Signed-off-by: Rickard von Essen <[email protected]>
@rickard-von-essen
Copy link
Author

Depends on fix included in #631

@turkenf
Copy link
Collaborator

turkenf commented Oct 9, 2024

Depends on fix included in #631

It might be better to test the Upjet update in this PR.

@rickard-von-essen
Copy link
Author

Depends on fix included in #631

It might be better to test the Upjet update in this PR.

Sure I'll rebase this onto that. brb

@rickard-von-essen
Copy link
Author

Rebased onto #631

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