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.

Sign up for GitHub

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

Jump to bottom

Enhancement/data feed metrics #422

Merged
merged 10 commits into from
May 1, 2023
Merged

Enhancement/data feed metrics #422

merged 10 commits into from
May 1, 2023

Conversation

pakrentos
Copy link
Contributor

Changes

  • Added prometheus metric for counting outdated oracle symbols
  • Added prometheus metric for counting oracle symbols with invalid timestamp (greater than current timestamp)
  • Every time a new symbol appears, new dynamic metric is initialized, which holds the new symbol update timestamp
  • Datafeed metrics update each 5 minutes

@pakrentos pakrentos requested a review from vovac12 April 21, 2023 10:08
vovac12
vovac12 reviewed Apr 24, 2023
View reviewed changes
node/src/data_feed_metrics.rs Outdated Show resolved Hide resolved
node/src/data_feed_metrics.rs Outdated Show resolved Hide resolved
node/src/service.rs Outdated Show resolved Hide resolved
@pakrentos pakrentos requested a review from vovac12 April 25, 2023 09:41
@pakrentos pakrentos merged commit 01e63da into develop May 1, 2023
3 checks passed
@pakrentos pakrentos deleted the enhancement/data-feed-metrics branch May 1, 2023 09:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vovac12 vovac12 vovac12 approved these changes

@N1ghtStorm N1ghtStorm N1ghtStorm approved these changes

@baotn166 baotn166 Awaiting requested review from baotn166

@Cre-eD Cre-eD Awaiting requested review from Cre-eD

@safinsaf safinsaf Awaiting requested review from safinsaf

@BAStos525 BAStos525 Awaiting requested review from BAStos525

@wpi86 wpi86 Awaiting requested review from wpi86

@alexkg alexkg Awaiting requested review from alexkg

@RamilMus RamilMus Awaiting requested review from RamilMus

@f33r0 f33r0 Awaiting requested review from f33r0

@stefashkaa stefashkaa Awaiting requested review from stefashkaa

@Nikita-Polyakov Nikita-Polyakov Awaiting requested review from Nikita-Polyakov

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@pakrentos @vovac12 @N1ghtStorm