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

feat: add a submodule to support proxy based regional external load balancer #75

Conversation

jeffling-google
Copy link
Contributor

The feature request is documented in b/303348819

@jeffling-google jeffling-google requested review from imrannayer and a team as code owners October 16, 2023 19:52
@g-awmalik
Copy link
Contributor

/gcbrun

@g-awmalik g-awmalik changed the title Add a submodule to support proxy based regional external loal balancer feat: add a submodule to support proxy based regional external load balancer Oct 17, 2023
@g-awmalik
Copy link
Contributor

@jeffling-google - PTAL at the int test failure. Seems like you're using a network that's already in use.

@jeffling-google
Copy link
Contributor Author

Changing the network name of the example/regional_proxy_lb to avoid conflicting with the network names in other example(s).

@g-awmalik
Copy link
Contributor

/gcbrun

@g-awmalik
Copy link
Contributor

@imrannayer - can you take a look at this?

@imrannayer
Copy link
Collaborator

@jeffling-google We are not using old tetsing framework. Can you plz create tests using new framework?
You can also see example here

@imrannayer
Copy link
Collaborator

/gcbrun

@imrannayer
Copy link
Collaborator

/gcbrun

kitchen.yml Outdated Show resolved Hide resolved
@imrannayer
Copy link
Collaborator

/gcbrun

@imrannayer imrannayer self-requested a review November 14, 2023 17:16
@imrannayer
Copy link
Collaborator

/gcbrun

@imrannayer
Copy link
Collaborator

Fixed in #77

@imrannayer imrannayer closed this Dec 4, 2023
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.

3 participants