Skip to content
This repository was archived by the owner on Jul 28, 2025. It is now read-only.

Conversation

jsaun
Copy link
Contributor

@jsaun jsaun commented May 19, 2023

Fixes #668

@jsaun jsaun marked this pull request as draft May 19, 2023 15:37
@jsaun jsaun changed the title Disable batch public ips by default and always use subnet Always use batch subnet setup with service endpoints Jun 9, 2023
@jsaun jsaun marked this pull request as ready for review June 9, 2023 20:53
@jsaun jsaun requested review from BMurri and MattMcL4475 June 9, 2023 20:53
@jsaun
Copy link
Contributor Author

jsaun commented Jun 9, 2023

I will follow up with the tes deployer PR as soon as this is approved.

Copy link
Contributor

@BMurri BMurri left a comment

Choose a reason for hiding this comment

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

Did a batch service endpoint need to be added also?

@jsaun
Copy link
Contributor Author

jsaun commented Jun 9, 2023

Did a batch service endpoint need to be added also?

No, not needed.

@jsaun jsaun merged commit f44f857 into main Jun 12, 2023
@jsaun jsaun deleted the jsaun/private-networking branch June 12, 2023 16:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Create and pass vm worker vnet with Service Endpoints to Batch to avoid Batch LB traffic bottleneck

2 participants