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

Metrics: Startup time #1788

Closed
stefa2k opened this issue Oct 19, 2020 · 6 comments
Closed

Metrics: Startup time #1788

stefa2k opened this issue Oct 19, 2020 · 6 comments
Labels
A1 enhancement New feature or request

Comments

@stefa2k
Copy link
Contributor

stefa2k commented Oct 19, 2020

Description

There is currently no way to tell the startup time of lighthouse components, except using OS features. Accessing this information on OS level rises security concerns.

Present Behaviour

No metrics of startup time/up time present.

Expected Behaviour

Publishing information about startup time on process_start_time_seconds like teku and prysm do.

@paulhauner paulhauner added A1 enhancement New feature or request labels Nov 9, 2020
@paulhauner
Copy link
Member

paulhauner commented Nov 9, 2020

I'm not exactly sure what we'd publish in process_start_time_seconds. It's not immediately clear to me what we're trying to measure here; startup is fairly complex and involves several components running concurrently.

@stefa2k
Copy link
Contributor Author

stefa2k commented Nov 9, 2020

To my understanding Teku and Prysm return the unix timestamp of the start of the main process.

@Yoldark34
Copy link

@stefa2k
Workaround for beacon node with validator monitor.

validator_monitor_unaggregated_attestation_total*384
Gives you the qty of seconds since restarted

Reduce to the last value if you got multiple validator. It will provide you with a good enough estimation of when your beacon node restarted.

Still looking for a reliable not resource intensive way to know when the validator process restarded

@Yoldark34
Copy link

@stefa2k
For the validator process :

vc_signed_attestations_total*384 reduced / with validator_monitor_validators_total
I dont undestand why i can't do in a single query
vc_signed_attestations_total/validator_monitor_validators_total*384

Best regards

@paulhauner
Copy link
Member

To my understanding Teku and Prysm return the unix timestamp of the start of the main process.

Oh I understand. Sorry, I was reading this as "the duration LH took to start up" instead of "the timestamp at which LH was started".

This seems quite easy to achieve.

bors bot pushed a commit that referenced this issue Mar 2, 2021
## Issue Addressed

Resolves #1788

## Proposed Changes

As per #1788, expose the time at which the process started via the `process_start_time_seconds` Prometheus metric. This will help users track uptime.

## Additional Info

NA
bors bot pushed a commit that referenced this issue Mar 2, 2021
## Issue Addressed

Resolves #1788

## Proposed Changes

As per #1788, expose the time at which the process started via the `process_start_time_seconds` Prometheus metric. This will help users track uptime.

## Additional Info

NA
bors bot pushed a commit that referenced this issue Mar 2, 2021
## Issue Addressed

Resolves #1788

## Proposed Changes

As per #1788, expose the time at which the process started via the `process_start_time_seconds` Prometheus metric. This will help users track uptime.

## Additional Info

NA
bors bot pushed a commit that referenced this issue Mar 2, 2021
## Issue Addressed

Resolves #1788

## Proposed Changes

As per #1788, expose the time at which the process started via the `process_start_time_seconds` Prometheus metric. This will help users track uptime.

## Additional Info

NA
bors bot pushed a commit that referenced this issue Mar 2, 2021
## Issue Addressed

Resolves #1788

## Proposed Changes

As per #1788, expose the time at which the process started via the `process_start_time_seconds` Prometheus metric. This will help users track uptime.

## Additional Info

NA
bors bot pushed a commit that referenced this issue Mar 2, 2021
## Issue Addressed

Resolves #1788

## Proposed Changes

As per #1788, expose the time at which the process started via the `process_start_time_seconds` Prometheus metric. This will help users track uptime.

## Additional Info

NA

Co-authored-by: Michael Sproul <[email protected]>
@Yoldark34
Copy link

Thanks <3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A1 enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants