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

[SPPSP-2925] add timeout to ListPodNodes #506

Merged
merged 1 commit into from
Nov 22, 2021

Conversation

DuodenumL
Copy link
Contributor

给ListPodNodes加了个timeout参数,默认是10s。

@DuodenumL
Copy link
Contributor Author

发现我本地的protoc-gen-go是上古版本,生成的pb文件里还是"github.com/golang/protobuf"这个已经过期了的库,赶紧更新了下本地环境。奇怪的是看了下我之前的commit 6d3c4ea938806ed679b7d4355fbd15b064211c34,这个commit里也改了proto,但是生成的pb竟然是对的...?

@@ -338,6 +338,7 @@ message ListNodesOptions {
string podname = 1;
bool all = 2;
map<string, string> labels = 3;
int32 timeout = 4;
Copy link
Member

Choose a reason for hiding this comment

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

叫 timeout_in_second 吧

Copy link
Contributor Author

Choose a reason for hiding this comment

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

改啦

@jschwinger233 jschwinger233 merged commit 00fb8bd into projecteru2:master Nov 22, 2021
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.

2 participants