-
Notifications
You must be signed in to change notification settings - Fork 9.9k
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
[3.5] Backport e2e tests for livez/readyz. #17083
[3.5] Backport e2e tests for livez/readyz. #17083
Conversation
Signed-off-by: Siyuan Zhang <sizhang@google.com>
c4bf59b
to
dc79aec
Compare
cc @serathius |
0e226df
to
fab5df2
Compare
cc @ahrtr |
Makefile
Outdated
GOFAIL_VERSION = $(shell cd tools/mod && go list -m -f {{.Version}} go.etcd.io/gofail) | ||
.PHONY: install-gofail | ||
install-gofail: | ||
cd tools/mod; go install go.etcd.io/gofail@${GOFAIL_VERSION} |
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.
Let's backport #17088 as well, in this PR or in a followup PR. Thanks.
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.
done
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.
LGTM with a minor comment.
Thanks
Signed-off-by: Siyuan Zhang <sizhang@google.com>
fab5df2
to
cb65c1f
Compare
Please read https://github.com/etcd-io/etcd/blob/main/CONTRIBUTING.md#contribution-flow.
Part of #16007
Followup PR of #17039
Original PR #16698