-
Notifications
You must be signed in to change notification settings - Fork 1
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: node public key retrieval #45
Conversation
🦋 Changeset detectedLatest commit: 055231b The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
The latest Buf updates on your PR. Results from workflow Regenerate Protobuf Files / buf (pull_request).
|
545fcc8
to
d1167e1
Compare
d1167e1
to
b65f833
Compare
@@ -172,6 +173,7 @@ message ListNodesRequest { | |||
repeated string ids = 1; | |||
EnableState enabled = 2; | |||
repeated api.label.Selector selectors = 3; | |||
repeated string public_keys = 4; |
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.
Update the comment for ListNodesRequest
?
job-distributor/v1/node/node.proto
Outdated
@@ -143,6 +143,7 @@ message RegisterNodeResponse { | |||
// GetNodeRequest is the request to retrieve a single node by its ID. |
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.
need to update the comment here too
8cf7677
to
c227105
Compare
Add public key to ListNodesRequest Filter and GetNodeRequest