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(source/alloydb-pg): add configuration for public and private IP #103

Merged
merged 3 commits into from
Dec 5, 2024

Conversation

Yuan325
Copy link
Contributor

@Yuan325 Yuan325 commented Dec 2, 2024

Allow user to set if their database uses private or public ip. The reason we add this is because the dialer require different initialization with private and public ip.

By default, toolbox will use public ip.

@Yuan325 Yuan325 requested a review from a team as a code owner December 2, 2024 19:43
Copy link
Collaborator

@kurtisvg kurtisvg left a comment

Choose a reason for hiding this comment

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

Two other comments:

  1. We should probably separate this into two PRs (one for AlloyDB and Cloud SQL)
  2. We should add test(s)

@Yuan325 Yuan325 force-pushed the add-iptype branch 2 times, most recently from 75ceb1d to bac5a28 Compare December 3, 2024 01:12
@Yuan325 Yuan325 changed the title chore: add ip_type to alloydb and cloudsql source chore: add ip_type to alloydb source Dec 3, 2024
@kurtisvg kurtisvg changed the title chore: add ip_type to alloydb source feat(source/alloydb-pg): add configuration for public and private IP Dec 4, 2024
Yuan325 and others added 2 commits December 5, 2024 14:57
Co-authored-by: Kurtis Van Gent <31518063+kurtisvg@users.noreply.github.com>
@Yuan325 Yuan325 enabled auto-merge (squash) December 5, 2024 23:04
@Yuan325 Yuan325 merged commit e88ec40 into main Dec 5, 2024
8 checks passed
@Yuan325 Yuan325 deleted the add-iptype branch December 5, 2024 23:05
duwenxin99 pushed a commit that referenced this pull request Dec 10, 2024
🤖 I have created a release *beep* *boop*
---


##
[0.0.3](v0.0.2...v0.0.3)
(2024-12-10)


### Features

* Add --log-level and --logging-format flags
([#97](#97))
([9a0f618](9a0f618))
* Add options for command
([#110](#110))
([5c690c5](5c690c5))
* Add Spanner source and tool
([#90](#90))
([890914a](890914a))
* Add std logger
([#95](#95))
([6a8feb5](6a8feb5))
* Add structured logger
([#96](#96))
([5e20417](5e20417))
* **source/alloydb-pg:** Add configuration for public and private IP
([#103](#103))
([e88ec40](e88ec40))
* **source/cloudsql-pg:** Add configuration for public and private IP
([#114](#114))
([6479c1d](6479c1d))


### Bug Fixes

* Fix go test workflow
([#84](#84))
([8c2c373](8c2c373))
* Fix issue causing client session to not close properly while closing
SDK. ([#81](#81))
([9d360e1](9d360e1))
* Fix test cases for ip_type
([#115](#115))
([5528bec](5528bec))
* Fix the errors showing up after setting up mypy type checker.
([#74](#74))
([522bbef](522bbef))
* **llamaindex-sdk:** Fix issue causing client session to not close
properly while closing SDK.
([#82](#82))
([fa03376](fa03376))


### Miscellaneous Chores

* Release 0.0.3
([#122](#122))
([626e12f](626e12f))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
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.

3 participants