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

Update sgc.csv #27

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Update sgc.csv #27

wants to merge 3 commits into from

Conversation

golfvert
Copy link
Contributor

@golfvert golfvert commented Dec 7, 2024

Following discussion at WISOP meeting early December in WMO, and considering the interest of the Sensor Centre for Global Cache, I am proposing some changes to the metrics. In particular, adding one with threshold on the delay to cache. We need to agree on the threshold as well.
Suggestion here is 120s, 300s, 600s. Something not in the cache in 10 minutes is considered "missed". I also suggest creating an additional metric for data not in ANY cache.

Following discussion at WISOP meeting early December in WMO, and considering the interest of the Sensor Centre for Global Cache, I am proposing some changes to the metrics.
In particular, adding one with threshold on the delay to cache.
We need to agree on the threshold as well.
Suggestion here is 120s, 300s, 600s. Something not in the cache in 10 minutes is considered "missed".
I also suggest creating an additional metric for data not in ANY cache.
@kaiwirt
Copy link
Collaborator

kaiwirt commented Dec 12, 2024

How is the delay metric supposed to work (what does it show)? Do we need the delay as label? Do we need three different metrics delay_120_total, delay_300_total, delay_600_total?

Add delay as a label in wmo_wis2_sgc_messages_cached_delay_total 
We need to agree on acceptable values for the delay.
120, 300, 600 for example.
@golfvert
Copy link
Contributor Author

I was unclear. Sorry. Delay should be a label. And we need to agree on allowed values for the label.
The other option is to do delay_120_total, delay_300_total, delay_600_total
Not sure which one is the "good" choice.

Add centre_id as label for ...cache_delay... metric.
@golfvert
Copy link
Contributor Author

Third update. Include centre_id in ....cache_delay... metric.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants