feat: Harvest should monitor fabric link status#4175
Conversation
Thanks to dhirajmalkari for reporting!
There was a problem hiding this comment.
Pull request overview
Adds FabricLink status awareness to the SnapMirror REST collection so Harvest can surface a derived “fabric link healthy” label alongside existing SnapMirror health signals.
Changes:
- Extend the SnapMirror REST config to collect
fl_statusand export a new derived labelfl_healthy. - Add plugin logic to derive
fl_healthyfromhealthyand parsedfl_status.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| conf/rest/9.12.0/snapmirror.yaml | Collects fl_status and exports fl_healthy as an instance label. |
| cmd/collectors/rest/plugins/snapmirror/snapmirror.go | Computes and sets fl_healthy during SnapMirror plugin execution. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
Thanks to dhirajmalkari for reporting!
Thanks to dhirajmalkari for reporting!
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
Thanks to dhirajmalkari for reporting!