Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

feat: Add psc configs for mssql#783

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
msharkey wants to merge5 commits intoterraform-google-modules:main
base:main
Choose a base branch
Loading
frommsharkey:add_psc_configs_mssql

Conversation

@msharkey
Copy link

Add Private Service Connect (PSC) Configuration Support for MSSQL Module

This pull request adds Private Service Connect (PSC) configuration support to the MSSQL module, bringing it in line with the existing MySQL and PostgreSQL modules that already support PSC.

  • This is a non-breaking change - existing configurations will continue to work unchanged
  • PSC is disabled by default, requiring explicit opt-in
  • Follows the same pattern established by the MySQL and PostgreSQL modules
  • This enhancement completes the PSC feature parity across all SQL database modules in this repository.

@google-cla
Copy link

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View thisfailed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@imrannayer
Copy link
Collaborator

/gcbrun

@imrannayer
Copy link
Collaborator

@msharkey thanks for the PR. Can you plz fix lint issues? Followthis doc

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@ayushmjainayushmjainAwaiting requested review from ayushmjainayushmjain is a code owner

@imrannayerimrannayerAwaiting requested review from imrannayerimrannayer is a code owner

@q2wq2wAwaiting requested review from q2wq2w is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@msharkey@imrannayer

[8]ページ先頭

©2009-2025 Movatter.jp