-
Notifications
You must be signed in to change notification settings - Fork 227
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
Add a new server with localhost handlers #2635
Conversation
efd7b95
to
2dcabad
Compare
44c2cf5
to
1b501f4
Compare
96623df
to
bda18db
Compare
@paulyufan2, can you update me on this? My understanding is that we’re starting two servers within CNS, with the echo one handling local requests. Can we move all localhost handlers to this new server in this PR for clarity? |
@nddq This PR has moved all localhost handlers to the new echo server and next PR will remove localhost handlers from restserver and start two listeners. |
/azp run Azure Container Networking PR |
Azure Pipelines successfully started running 1 pipeline(s). |
bda18db
to
ca9d863
Compare
/azp run Azure Container Networking PR |
Azure Pipelines successfully started running 1 pipeline(s). |
Reason for Change:
The AKS listeners for the CNI to talk to CNS should be on localhost
This PR is to move all handlers that should be on localhost to a echo server
Next PR will start this server, separate handlers from restserver and start two listeners
Issue Fixed:
Requirements:
Notes: