Skip to content

[monit] Periodically monitor route consistency#5085

Merged
renukamanavalan merged 4 commits intosonic-net:masterfrom
renukamanavalan:monit_update
Aug 4, 2020
Merged

[monit] Periodically monitor route consistency#5085
renukamanavalan merged 4 commits intosonic-net:masterfrom
renukamanavalan:monit_update

Conversation

@renukamanavalan
Copy link
Copy Markdown
Contributor

route_check.py is a tool for "Verify routes between APPL-DB & ASIC-DB are in sync".
This tool is added to monit to run the check periodically.

For any failure, the monit will raise alert based on return code.
The tool will log required details.

@jleveque jleveque changed the title Add route_check to mont [monit] Periodically monitor route consistency Aug 1, 2020
@renukamanavalan
Copy link
Copy Markdown
Contributor Author

retest mellanox please

jleveque
jleveque previously approved these changes Aug 3, 2020
Copy link
Copy Markdown
Contributor

@jleveque jleveque left a comment

Choose a reason for hiding this comment

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

Maybe we should add a comment stating what this script does?

@renukamanavalan
Copy link
Copy Markdown
Contributor Author

retest vsimage please

@renukamanavalan renukamanavalan merged commit 312771d into sonic-net:master Aug 4, 2020
@abdosi
Copy link
Copy Markdown
Contributor

abdosi commented Aug 14, 2020

@rlhui @lguohan do we need this in 201911 ? I feel this might need more regression testing.

@rlhui
Copy link
Copy Markdown
Contributor

rlhui commented Aug 14, 2020

@renukamanavalan - how was this tested? Thanks.

abdosi pushed a commit that referenced this pull request Sep 19, 2020
* Add route_check to mont.

* Switched to units of cycles per comments

* Added comments per Joe's comments.

* Added more comments per Royal's comments.
santhosh-kt pushed a commit to santhosh-kt/sonic-buildimage that referenced this pull request Feb 25, 2021
* Add route_check to mont.

* Switched to units of cycles per comments

* Added comments per Joe's comments.

* Added more comments per Royal's comments.
@renukamanavalan renukamanavalan deleted the monit_update branch April 17, 2022 22:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants