Skip to content
This repository has been archived by the owner on Mar 9, 2024. It is now read-only.

Add: python monero health checker module #70

Closed
normoes opened this issue Sep 24, 2020 · 8 comments
Closed

Add: python monero health checker module #70

normoes opened this issue Sep 24, 2020 · 8 comments
Labels
Approved consensus consensus needed for approval/rejection (see governance system) Repo proposal

Comments

@normoes
Copy link

normoes commented Sep 24, 2020

I would like to introduce monero_health to you. It is written in python and is a module to return health or status information about monerod.

https://github.com/normoes/monero_health

The README describes its usage and responses.

This module is used on the XMR.to community hub - The module provides the running serice with the JSON result.

The idea is to make the monero daemon status available in a readable format, right now only JSON.

Definition of status (see README):

  • There are the P2P and the RPC status - This is essentially about connectivity + information like the hardfork version. Both are combined in the status found at /monerod/health.
  • There is the last_block status, which defines an age limit (12 [minutes]) of a block and returns OK if the block is younger than that or ERROR if the block is older than that.

What do you think?

@Lafudoci
Copy link

This is good. A nice python module which could be easily integrated. Remote node health is important for Monero service project. I vote ACK.

@erciccione erciccione added consensus consensus needed for approval/rejection (see governance system) Repo proposal labels Sep 25, 2020
@erciccione
Copy link
Collaborator

Cool project. ACK for me as well, but @normoes could you add a license? I think it's important that for the nature of the Ecosystem project itself, all projects should be licensed, to not make contribution/distribution hard.

@normoes
Copy link
Author

normoes commented Sep 29, 2020

Good point. LICENSE added.

@erciccione
Copy link
Collaborator

@monero-ecosystem/maintainers any opinion/vote about this?

@emesik
Copy link

emesik commented Oct 21, 2020

ACK from me, definitely. This seems very useful, especially when monerod is not a super reliable piece of software.

@erciccione
Copy link
Collaborator

Thank you for voting. We can consider this accepted. @normoes you may start the transfer whenever you are ready. Please ping me when it's done :)

@normoes
Copy link
Author

normoes commented Nov 23, 2020

@erciccione
Copy link
Collaborator

Added to the README with #73

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Approved consensus consensus needed for approval/rejection (see governance system) Repo proposal
Projects
None yet
Development

No branches or pull requests

4 participants