-
Notifications
You must be signed in to change notification settings - Fork 740
pkg/*: Use endpoint status for probes #2029
Conversation
Can one of the admins verify this patch? |
2 similar comments
Can one of the admins verify this patch? |
Can one of the admins verify this patch? |
@etcd-bot ok to test |
@hexfusion Thanks for the quick response. Hope this is ok. |
@avorima OK we are really close, but we need to get rid of the merge commit. And we would like all commit messages to be try
Thanks! |
@hexfusion What about the merge conflict? |
@avorima I see you need to rebase against master. |
lgtm, thanks @avorima! |
@avorima I have run into an issue merging your commit because it has no email address. please update with interactive rebase.
you will see something like pick 7c7141550 pkg/*: Use endpoint status for probes
pick a580ec454 CHANGELOG: updated to reflect probe fix change pick to edit for both and save this will start the rebase.
when done git push -f Then please make sure to update your email address for future commits. Once this is done I can merge, thanks! |
My bad, here you go. |
endpoint status
doesn't require a username when authentication is enabled.Fixes #1957