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

add the monitoring piece to the monitoring. #238

Merged
merged 2 commits into from
Jan 17, 2024

Conversation

larry-aptos
Copy link
Contributor

@larry-aptos larry-aptos commented Jan 16, 2024

  • Adds e2e latency monitoring for processors.

Test Plan

  • locally tested, looks good.

  • Example response

{
  "processor_status": [
    {
      "processor": "objects_processor",
      "last_updated": "2024-01-16T21:59:54.425376",
      "last_transaction_timestamp": "2024-01-16T21:59:54.425376",
      "last_success_version": 853440763
    },
    {
      "processor": "token_v2_processor",
      "last_updated": "2024-01-16T21:59:54.407772",
      "last_transaction_timestamp": "2024-01-16T21:59:54.407772",
      "last_success_version": 853440763
    },
    {
      "processor": "account_transactions_processor",
      "last_updated": "2024-01-16T21:59:54.392648",
      "last_transaction_timestamp": "2024-01-16T21:59:54.392648",
      "last_success_version": 853440763
    },
    {
      "processor": "fungible_asset_processor",
      "last_updated": "2024-01-16T21:59:54.40526",
      "last_transaction_timestamp": "2024-01-16T21:59:54.40526",
      "last_success_version": 853440763
    },
    {
      "processor": "ans_processor",
      "last_updated": "2024-01-16T21:59:54.422825",
      "last_transaction_timestamp": "2024-01-16T21:59:54.422825",
      "last_success_version": 853440763
    },
    {
      "processor": "token_processor",
      "last_updated": "2024-01-16T21:59:54.438418",
      "last_transaction_timestamp": "2024-01-16T21:59:54.438418",
      "last_success_version": 853440763
    },
    {
      "processor": "user_transaction_processor",
      "last_updated": "2024-01-16T21:59:54.386354",
      "last_transaction_timestamp": "2024-01-16T21:59:54.386354",
      "last_success_version": 853440763
    },
    {
      "processor": "events_processor",
      "last_updated": "2024-01-16T21:59:54.419373",
      "last_transaction_timestamp": "2024-01-16T21:59:54.419373",
      "last_success_version": 853440763
    },
    {
      "processor": "default_processor",
      "last_updated": "2024-01-16T21:59:54.40956",
      "last_transaction_timestamp": "2024-01-16T21:59:54.40956",
      "last_success_version": 853440763
    },
    {
      "processor": "nft_metadata_processor",
      "last_updated": "2024-01-16T21:59:54.556318",
      "last_transaction_timestamp": "2024-01-16T21:59:54.556318",
      "last_success_version": 853440763
    },
    {
      "processor": "stake_processor",
      "last_updated": "2024-01-16T21:59:54.42362",
      "last_transaction_timestamp": "2024-01-16T21:59:54.42362",
      "last_success_version": 853440763
    },
    {
      "processor": "coin_processor",
      "last_updated": "2024-01-16T21:59:54.425671",
      "last_transaction_timestamp": "2024-01-16T21:59:54.425671",
      "last_success_version": 853440763
    }
  ]
}
  • Testing Result

image

  • Metrics result

image

@rtso rtso requested a review from a team January 17, 2024 18:48
@rustielin
Copy link
Contributor

@larry-aptos from the screenshot in the test plan, the metric values for indexer_metrics_hasura_latest_transaction_timestamp_secs don't look to be in seconds

@larry-aptos
Copy link
Contributor Author

larry-aptos commented Jan 17, 2024

@larry-aptos from the screenshot in the test plan, the metric values for indexer_metrics_hasura_latest_transaction_timestamp_secs don't look to be in seconds

Updated! true... braindead yesterday lol

@larry-aptos larry-aptos merged commit edc9fd8 into main Jan 17, 2024
7 checks passed
@larry-aptos larry-aptos deleted the add-the-probing-part-to-status branch January 17, 2024 21:23
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.

3 participants