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

Operator for AlloyDB? #453

Closed
adamstrawson opened this issue Oct 3, 2023 · 5 comments
Closed

Operator for AlloyDB? #453

adamstrawson opened this issue Oct 3, 2023 · 5 comments
Assignees
Labels
priority: p2 Moderately-important priority. Fix may not be included in next release. type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design.

Comments

@adamstrawson
Copy link

Are there any plans to create an operator for alloydb-auth-proxy too?

If it's on the roadmap, but no plans yet, I'm happy to take on the work needed to make it happen, (Potentially just fork this operator and tailor it more for AlloyDB)

@enocom
Copy link
Member

enocom commented Oct 3, 2023

Thanks for the request. We do have it on the roadmap, but not until next year given the current load.

We split the Connectors into two code bases because we anticipate that they'll diverge over time. For the operator here, though, I suspect we'll possibly want to extend this project to support AlloyDB using the same code paths because the two Proxy interfaces are almost identical. How we do that without any breaking changes and with updating the name to be more generic (maybe db-proxy-operator?) is still an open question.

@enocom enocom added type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design. priority: p2 Moderately-important priority. Fix may not be included in next release. labels Oct 3, 2023
@hessjcg hessjcg pinned this issue Oct 3, 2023
@hessjcg
Copy link
Collaborator

hessjcg commented Oct 3, 2023

Hi all. Please comment or upvote if this is important to you. I have pinned this issue to the top of the Issues list.

@chobostar
Copy link

chobostar commented Feb 22, 2024

Since AlloyDB doesn't support non-RFC 1918 IP ranges for access: https://issuetracker.google.com/issues/291966450
and by today AlloyDB doesn't support Public IP.

For some environments alloydb-auth-proxy could be the only proper way to connect.

Operator could simplify workloads migration, manual work of alloydb-auth-proxy adopting and keep unify method for both CloudSQL/AlloyDB client applications

UPD. If you approve and OK with using the same operator for cloudsql and alloydb, I could make PR proposal.

@ttosta-google
Copy link
Contributor

Thanks for the interest. We're still considering this work, but aren't accepting PRs for it.

AlloyDB now supports Public IP. See documentation.

@hessjcg
Copy link
Collaborator

hessjcg commented Feb 22, 2024

We are tracking AlloyDB Proxy feature in #250. I'll close this as a duplicate. I'll note your interest on that issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: p2 Moderately-important priority. Fix may not be included in next release. type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design.
Projects
None yet
Development

No branches or pull requests

5 participants