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.

Sign up for GitHub

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

Jump to bottom

Typos in AP Channel Utilization triggers #65

Closed
GMichu opened this issue Oct 17, 2022 · 2 comments
Closed

Typos in AP Channel Utilization triggers #65

GMichu opened this issue Oct 17, 2022 · 2 comments

Comments

@GMichu
Copy link

GMichu commented Oct 17, 2022

firefox_nnCAGzBgUo

  1. Operational data does not display correctly due to a missing parenthesis in the opdata expression
  2. Same goes for dependency expression - problems with severity of info and average are displayed simultaneously
  3. There is also a minor problem in the logic of the AVERAGE trigger operational data - it displays the INFO trigger threshhold
@GMichu
Copy link
Author

GMichu commented Oct 17, 2022

Also operational data should generally show item value and not threshold

@GMichu GMichu mentioned this issue Oct 17, 2022
Closed
patricegautier added a commit that referenced this issue Oct 19, 2022
@patricegautier
new item definitions for the SolarPoint devices; divorcing Unifi temp…
b43c928 
…lates from SSH dependencies to ease the load on devices; adding validation logic to mca-dump-short; dropping values if mcaDumpError is reported by mca-dump-short; fixing AP triggers as per #65
@patricegautier
Copy link
Owner

Hi GMichu,

Thanks much for reporting this - looks to me you are right on all counts.

I included your changes in the latest commit; there are however quite a few other changes in there

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@patricegautier @GMichu