Standard Liveness Endpoint

[Gua00va]

Issue Addressed

Solves #4522

Proposed Changes

Validator Client now uses standard liveness endpoint /eth/v1/validator/liveness/{epoch} implemented here #4343

[Gua00va]

@jimmygchen , I think because #4343 is not yet merged into unstable, so the endpoint is throwing a 404 error.

[paulhauner]

I think because #4343 is not yet merged into unstable

I've just triggered bors on #4343, so hopefully it'll merge in the next couple of hours.

[paulhauner]

#4343 has merged and there's a little conflict here now 🙏

[Gua00va]

@paulhauner , I've fixed the conflict. Should work now !!

[paulhauner]

Thanks for fixing the conflicts! We have some unhappy tests now though :( Feel free to fix at your leisure ☺️

[paulhauner]

Hey @Gua00va, we're hoping to include this in v4.4.0 which means we'd need to have this merged this week (the week of the 21st).

If you'd like to fix up the test compile errors, please go ahead! If you don't have the time to finish it up (which is totally OK), just let me know and we'll finish it up.

If I don't hear from you I'll probably end up just pushing some fixes to this branch ☺️

[Gua00va]

Hey @Gua00va, we're hoping to include this in v4.4.0 which means we'd need to have this merged this week (the week of the 21st).

If you'd like to fix up the test compile errors, please go ahead! If you don't have the time to finish it up (which is totally OK), just let me know and we'll finish it up.

If I don't hear from you I'll probably end up just pushing some fixes to this branch ☺️

I will try to complete this in a day or two

[paulhauner]

I'm thinking that we push this to the next release. It'll take the urgency off and there's also merits to delaying it to the next release (see comments in #4522).

So no rush on this one @Gua00va 🙏

[Gua00va]

@paulhauner, The tests should work fine now. Sorry for the delay !

[michaelsproul]

@Gua00va Did you mean to close this?

[Gua00va]

@Gua00va Did you mean to close this?

Yes. I am working on it in a seperate PR.