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

Liqo Gateway & Auth Service Loadbalancer Static IP #2013

Conversation

Sharathmk99
Copy link
Contributor

@Sharathmk99 Sharathmk99 commented Sep 12, 2023

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

When exposing Liqo Gateway & Auth service as LoadBalancer currently it's only possible to change the type to LoadBalancer, but in some cloud provided we are able to create Static LoadBalancer and use IP in loadBalancerIP field. This PR will allow user to set custom LoadBalancer IP.

Fixes #(issue)

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Please also note any relevant details for your test configuration.

  • Helm template and install

@adamjensenbot
Copy link
Collaborator

Hi @Sharathmk99. Thanks for your PR!

I am @adamjensenbot.
You can interact with me issuing a slash command in the first line of a comment.
Currently, I understand the following commands:

  • /rebase: Rebase this PR onto the master branch (You can add the option test=true to launch the tests
    when the rebase operation is completed)
  • /merge: Merge this PR into the master branch
  • /build Build Liqo components
  • /test Launch the E2E and Unit tests
  • /hold, /unhold Add/remove the hold label to prevent merging with /merge

Make sure this PR appears in the liqo changelog, adding one of the following labels:

  • kind/breaking: 💥 Breaking Change
  • kind/feature: 🚀 New Feature
  • kind/bug: 🐛 Bug Fix
  • kind/cleanup: 🧹 Code Refactoring
  • kind/docs: 📝 Documentation

deployments/liqo/README.md Outdated Show resolved Hide resolved
deployments/liqo/README.md Outdated Show resolved Hide resolved
@Sharathmk99 Sharathmk99 force-pushed the gateway-service-loadbalancer-static-ip branch from 39abce7 to 2e9a14d Compare September 14, 2023 10:46
@Sharathmk99 Sharathmk99 changed the title Liqo Gateway Service Loadbalancer Static IP Liqo Gateway & Auth Service Loadbalancer Static IP Sep 14, 2023
@Sharathmk99 Sharathmk99 force-pushed the gateway-service-loadbalancer-static-ip branch from 2e9a14d to 5fb179d Compare September 14, 2023 10:47
@Sharathmk99
Copy link
Contributor Author

Ready for review. Thanks.

@Sharathmk99 Sharathmk99 force-pushed the gateway-service-loadbalancer-static-ip branch from 5fb179d to a13c0b4 Compare September 14, 2023 14:36
deployments/liqo/README.md Outdated Show resolved Hide resolved
deployments/liqo/README.md Outdated Show resolved Hide resolved
@frisso
Copy link
Member

frisso commented Sep 17, 2023

@Sharathmk99 This looks good to me. Just commit my suggestions, if you agree, and we're all set.

@Sharathmk99 Sharathmk99 force-pushed the gateway-service-loadbalancer-static-ip branch from 3b67a88 to 50d14ac Compare September 18, 2023 12:26
@Sharathmk99 Sharathmk99 force-pushed the gateway-service-loadbalancer-static-ip branch from 50d14ac to c17b7ea Compare September 18, 2023 12:33
@cheina97
Copy link
Member

/rebase test=true

@cheina97
Copy link
Member

/merge

@adamjensenbot adamjensenbot added the merge-requested Request bot merging (automatically managed) label Sep 18, 2023
@adamjensenbot adamjensenbot merged commit f0449d6 into liqotech:master Sep 18, 2023
8 checks passed
@adamjensenbot adamjensenbot removed the merge-requested Request bot merging (automatically managed) label Sep 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants