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

Optionally configure internal service via its load balancer #52

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

ivanmb
Copy link

@ivanmb ivanmb commented Mar 3, 2020

This PR adds the option to configure a service that's not internet facing.
It will use an internal load balancer instead of internet facing, living in the private subnets instead of public ones

@jlsan92
Copy link
Contributor

jlsan92 commented Mar 3, 2020

This looks amazing 💪, sadly I'm no longer working for STRV, so I don't have permissions over this repo anymore.

Actually, I'm not sure what would be the maintenance of the module overall, probably @robertrossmann can give us some insights regarding this 🙏🍻

@jlsan92
Copy link
Contributor

jlsan92 commented Mar 3, 2020

Btw, I'd be more than happy to continue giving support on this!

@robertrossmann
Copy link
Member

Hey @jlsan92 so much thanks for expressing your interest in keeping this module going! ❤️ I have added you as a maintainer which should give you access to everything you might need in terms of maintenance or making new releases. If you discover anything missing just let me know on any channel, I'll do my best to help you! 👍

@mellster2012
Copy link

Is this still maintained? It's a great module but there doesn't seem much activity lately wrt outstanding PRs. We can keep developing on a fork but it would be nice to have one source of truth. Thanks!

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.

4 participants