Skip to content

Conversation

@sergiupopescu199
Copy link
Contributor

Description of change

To enable better monitoring with Grafana for catching regressions and performance issues, this PR expands on the reliability requirements for the iota-indexer.

Links to any relevant issues

fixes #6374

Type of change

  • Enhancement (a non-breaking change which adds functionality)

How the change has been tested

No code change was done so as test only basic lists & formatters were used like dprint fmt when creating the file.

Note

The MD file does not influence the actual Indexer performance, no further tests were performed.

Change checklist

Tick the boxes that are relevant to your changes, and delete any items that are not.

  • I have followed the contribution guidelines for this project
  • I have performed a self-review of my own code

Release Notes

  • Protocol:
  • Nodes (Validators and Full nodes):
  • Indexer:
  • JSON-RPC:
  • GraphQL:
  • CLI:
  • Rust SDK:
  • REST API:

@sergiupopescu199 sergiupopescu199 requested a review from a team as a code owner April 30, 2025 13:56
@vercel
Copy link

vercel bot commented Apr 30, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

4 Skipped Deployments
Name Status Preview Comments Updated (UTC)
apps-backend ⬜️ Ignored (Inspect) Visit Preview May 7, 2025 0:59am
apps-ui-kit ⬜️ Ignored (Inspect) Visit Preview May 7, 2025 0:59am
rebased-explorer ⬜️ Ignored (Inspect) Visit Preview May 7, 2025 0:59am
wallet-dashboard ⬜️ Ignored (Inspect) Visit Preview May 7, 2025 0:59am

@sergiupopescu199 sergiupopescu199 added the infrastructure Issues related to the Infrastructure Team label Apr 30, 2025
@iota-ci iota-ci added the sc-platform Issues related to the Smart Contract Platform group. label Apr 30, 2025
@tomxey tomxey requested a review from a team May 5, 2025 14:03
@tomxey tomxey requested a review from a team May 6, 2025 07:05
Copy link
Contributor

@kodemartin kodemartin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @sergiupopescu199, I think some refinements are in order. Please see more specific comments.

Also could you also add a section Metrics with a table mapping a short description to each metric used in the requirements?

Copy link
Contributor

@kodemartin kodemartin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm with a few typo corrections and a nitpick

sergiupopescu199 and others added 3 commits May 7, 2025 14:38
Co-authored-by: Konstantinos Demartinos <konstantinos.demartinos@iota.org>
Co-authored-by: Konstantinos Demartinos <konstantinos.demartinos@iota.org>
@sergiupopescu199 sergiupopescu199 merged commit eb65e42 into develop May 8, 2025
35 checks passed
@sergiupopescu199 sergiupopescu199 deleted the sc-platform/issue-6374 branch May 8, 2025 08:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

infrastructure Issues related to the Infrastructure Team sc-platform Issues related to the Smart Contract Platform group.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[iota-indexer]: create draft of reliability requirements

5 participants