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

Master monitor and dependency on it #1453

Closed
1 task done
SAOPP opened this issue Apr 4, 2022 · 2 comments
Closed
1 task done

Master monitor and dependency on it #1453

SAOPP opened this issue Apr 4, 2022 · 2 comments
Labels
area:monitor Everything related to monitors feature-request Request for new features to be added type:new proposing to add a new monitor

Comments

@SAOPP
Copy link

SAOPP commented Apr 4, 2022

⚠️ Please verify that this feature request has NOT been suggested before.

  • I checked and didn't find similar feature request

🏷️ Feature Request Type

New Monitor, Other

🔖 Feature description

Hey!

Is it possible to implement something like a dependency for monitors on 1 master monitor?

What I mean.

For example, I have a vps with uptime kuma, I also have remote mikrotiks with their infrastructure behind them, each of these mikrotiks connects via vpn with my vps...

And for example, when the connection with some mikrotik is broken, I would like to receive only a notification about this incident, because if there is no connection with this node, then it is clear that there is no connection with the devices behind it - and these messages are no longer necessary or significant, since there is ALREADY no tunnel.

✔️ Solution

I would like such dependent monitors to report the problem only if the main monitor is up, and if this main monitor is down, depenced minitors are paused or simply not notify reported.

❓ Alternatives

I came up with an idea regarding alternatives, for example, that there would be a message from the main monitor that the connection was broken, and the rest of the monitors dependent on it would go into the UNKNOWN state - this is just an example. Because their state due to the fact that there is no tunnel in reality will not be determined correctly.

📝 Additional Context

I hope I have described my wishlist clearly. :) Thanks!

@SAOPP SAOPP added the feature-request Request for new features to be added label Apr 4, 2022
@koen20
Copy link
Contributor

koen20 commented Apr 4, 2022

Someone opened a pull request #1236 which adds this feature.
This is a duplicate of #1089

@louislam louislam closed this as completed Apr 4, 2022
@SAOPP
Copy link
Author

SAOPP commented Apr 4, 2022

Oh maaaan :( I'm sorry guys, but I'm search it before, I'm swear :) Sorry again! Thank u!

@CommanderStorm CommanderStorm added area:monitor Everything related to monitors type:new proposing to add a new monitor labels Mar 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:monitor Everything related to monitors feature-request Request for new features to be added type:new proposing to add a new monitor
Projects
None yet
Development

No branches or pull requests

4 participants