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

Support local routing in transit gateway when PowerVS and VPC are from same region #1763

Closed
dharaneeshvrd opened this issue May 7, 2024 · 11 comments · Fixed by #1777
Closed
Assignees
Labels
area/provider/ibmcloud Issues or PRs related to ibmcloud provider kind/feature Categorizes issue or PR as related to a new feature.
Milestone

Comments

@dharaneeshvrd
Copy link
Contributor

/kind feature
/area provider/ibmcloud

Describe the solution you'd like
[A clear and concise description of what you want to happen.]

Currently global routing is enabled by default, would like to enable local routing when PowerVS and VPC are from same region. This will reduce some $$$

Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]

@k8s-ci-robot k8s-ci-robot added kind/feature Categorizes issue or PR as related to a new feature. area/provider/ibmcloud Issues or PRs related to ibmcloud provider labels May 7, 2024
@mkumatag
Copy link
Member

mkumatag commented May 7, 2024

what's the different amount for these settings?

Currently global routing is enabled by default, would like to enable local routing when PowerVS and VPC are from same region. This will reduce some $$$

@mkumatag
Copy link
Member

mkumatag commented May 7, 2024

Need to add an additional flag to this spec here like routing - global/local

@dharaneeshvrd
Copy link
Contributor Author

Still need to investigate that, will run two transit gateways with local and global routing for a couple of days and come back with the cost here!

@dharaneeshvrd
Copy link
Contributor Author

Just created this issue as a placeholder, in case if we are fixing it in hypershift, we can fix here too.

@prb112
Copy link
Contributor

prb112 commented May 7, 2024

Hey @dharaneeshvrd FYI, you can model it based on the cost estimator. https://www.ibm.com/cloud/cloud-calculator Thanks, Paul

@dharaneeshvrd
Copy link
Contributor Author

dharaneeshvrd commented May 9, 2024

Thanks @prb112 for the link!

Seems global routing costs double than the local routing for data transfer per GB. So it's better to use local routing in possible situations.

Other thing I found is that reusing transit gateway also could save some dollars, it costs separately whenever we create and establish new a connection. If we can reuse the existing transit gateways with the combination of PowerVS and VPC that would save some $ as well. This would help in avoid creating multiple TGs during development phases.

@dharaneeshvrd
Copy link
Contributor Author

/assign

@Karthik-K-N
Copy link
Contributor

I think if user provided PowerVS zone and vpc.Region are within same region we can by default go for local routing and introduce a new field in api to allow user to configure it to use global.

@mkumatag
Copy link
Member

mkumatag commented May 9, 2024

This needs:

  • Additional param in the TransitGateway struct
  • Validation web book change to make sure user don't use the irrelevant values(e.g: different region requires the global routing)
  • Defaulting to use local as routing if powervs and vpc belongs to same region

@Karthik-K-N @dharaneeshvrd feel free to add If I'm missing something

@dharaneeshvrd
Copy link
Contributor Author

Do we really need a param to set the route? Not sure in which case user needs it, can't we just decide local or global based on the powervs and vpc region? What's the point in setting global routing even in cases where it only needs local routing?
@mkumatag @Karthik-K-N Please let me know is this making sense!

@mkumatag
Copy link
Member

Do we really need a param to set the route? Not sure in which case user needs it, can't we just decide local or global based on the powervs and vpc region? What's the point in setting global routing even in cases where it only needs local routing? @mkumatag @Karthik-K-N Please let me know is this making sense!

Its always good to give option for user to override, they can set it explicitly for any specific scenario.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/provider/ibmcloud Issues or PRs related to ibmcloud provider kind/feature Categorizes issue or PR as related to a new feature.
Projects
None yet
5 participants