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
gofail: show error messages returned by gofail HTTP endpoints #17842
gofail: show error messages returned by gofail HTTP endpoints #17842
Conversation
Hi @henrybear327. Thanks for your PR. I'm waiting for a etcd-io member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/assign |
82fb546
to
b3bba0d
Compare
b3bba0d
to
1488d0c
Compare
/ok-to-test |
de7b70a
to
a9485dd
Compare
a9485dd
to
d692cdf
Compare
Upon HTTP request error from fetchFailpointsBody, PUT and DELETE, all of them will return an error message, which would be nice for debugging if displayed. Simplify `Enabled()` logic (linter suggestion) Signed-off-by: Chun-Hung Tseng <henrybear327@gmail.com>
d692cdf
to
1335768
Compare
This PR introduces showing the error message returned by gofail HTTP endpoints, making debugging easier.
Also apply simplified
Enabled()
logic, as suggested by the linterPlease read https://github.com/etcd-io/etcd/blob/main/CONTRIBUTING.md#contribution-flow.