Skip to content

chore: monitor improvements#2118

Merged
Mirko-von-Leipzig merged 6 commits into
nextfrom
santiagopittella-monitor-improvements
May 23, 2026
Merged

chore: monitor improvements#2118
Mirko-von-Leipzig merged 6 commits into
nextfrom
santiagopittella-monitor-improvements

Conversation

@SantiagoPittella
Copy link
Copy Markdown
Collaborator

@SantiagoPittella SantiagoPittella commented May 22, 2026

This PR gathers 4 monitor fixes and improvements remaining from #1907 .

Order by commit:

  1. Uses serde json for the explorer and faucet responses to simplify handling, also adds a new field added to the faucet metadata response.
  2. Fixes and issue with copy buttons being placed in a new line.
  3. Remove the persistence of account used for the network transactions tests (wallet and counter) and goes full in-memory.
  4. Adds a threshold to mark the ntxs as unhealthy, currently we were just displaying that there was a difference between the coutner value and the expected value but didn't marking the service as unhealthy. Not we have a configurable threshold difference.
  5. Changes the auth component for the counter account to use NetworkAccount and adds the increase note in the allow list

@SantiagoPittella
Copy link
Copy Markdown
Collaborator Author

SantiagoPittella commented May 22, 2026

After #2095 merge we can also move the monitor to use the NetworkAuth component allowing only our own note to be executed nvm, it was already possible

@Mirko-von-Leipzig Mirko-von-Leipzig merged commit dd09915 into next May 23, 2026
20 checks passed
@Mirko-von-Leipzig Mirko-von-Leipzig deleted the santiagopittella-monitor-improvements branch May 23, 2026 08:01
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