-
Notifications
You must be signed in to change notification settings - Fork 239
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
A38: Admin Interface API #218
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for doing this! Please let me know if you have any questions.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the detailed review. The content of this design doc has been trimmed down as suggested.
@markdroth PTALA.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for doing this!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the detailed review. I have resolved most comments. But not sure about the more languages one. I can add Python for sure, but for Ruby PHP Node.js, I might want to involve language owners to see if the design works.
A38-admin-interface-api.md
Outdated
|
||
## Abstract | ||
|
||
This proposal describes a convenience API in each gRPC language to improve the usability of creating a gRPC server with admin services to expose states in the gRPC library. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: s/convenience/convenient/ ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a couple of remaining comments here. Overall, looks very good!
@markdroth PTALAAA. I think all open comments are resolved in this gRFC, and the configuration dump part is split into another gRFC. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great!
gRFC: https://github.com/lidizheng/proposal/blob/admin-interface/A38-admin-interface-api.md
This proposal describes a convenience API in each gRPC language to improve the usability of creating a gRPC server with admin services.