SSL Mode and Require Connectors Support for AlloyDB #6494
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* support to create instance with ssl mode and require connectors, with acceptance tests * removed client_connection_config in basic instance example, and cleaned up the code a little bit * attempts to set the default ssl mode in the custom code * able to set default value during instance creation and addressing some comments * trying to fix updating ssl mode * fixed updating the instance with ssl mode * cleaning up code and fixing acceptance tests * Fixing spacing issues in instance.yaml * fixing spacing issues again in instance.yaml * fixing formatting in unit test * fixing the acceptance tests for other alloydb resources by explicitly adding the clientconnectionconfig * fixing spacing and removing commented out imports * commenting out createInstanceWithMandatoryFields and createPrimaryAndReadPoolInstance tests because they're failing due to an unrelated service networking issue * fix tests failing due to missing client_connection_config * removing service networking resources from tests since there's a bug in service networking * fix breaking change that causes a permadiff in terraform on update * uncomment out the TestAccAlloydbInstance_createPrimaryAndReadPoolInstance and TestAccAlloydbInstance_createInstanceWithMandatoryFields tests * remove private_ip_alloc from client connection config tests * use BootstrapSharedServiceNetworkingConnection in tests instead of BootstrapSharedTestNetwork in tests * add back in custom code * remove commented out code * fix formatting * match network name to be the same format as the other acceptance tests in the file * change network names in test * remove custom code and update e2e test * remove default test case and add more update steps to update test [upstream:06557f8d065d1b096db68d8f8a817eba13c445ee] Signed-off-by: Modular Magician <[email protected]>
modular-magician
deleted the
downstream-pr-06557f8d065d1b096db68d8f8a817eba13c445ee
branch
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description:
Add support to configure SSL mode and if the instance should require connectors (like Auth Proxy or language connectors) for AlloyDB instances. As discussed offline, this PR takes the default value from API by using
default_from_apiinstead of trying to inject it in the pre_create custom code.Issue:
Previous PR's for reference:
Release Note Template for Downstream PRs (will be copied)
Derived from GoogleCloudPlatform/magic-modules#9238