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

indices in case of clear alert #867

Closed
ltrilety opened this issue Mar 12, 2018 · 6 comments
Closed

indices in case of clear alert #867

ltrilety opened this issue Mar 12, 2018 · 6 comments

Comments

@ltrilety
Copy link
Contributor

Indices for alerts are decreased when a clear alert comes. That's fine as long as the related alert is present however in some strange situation it could happen that the alert does not exist. That leads to state when the number of alerts could be minus number.

BTW In my case I just imported a cluster which looks like 'ready to use' before the import was altogether completed.

alert_nr

Tendrl version:
tendrl-commons-1.6.1-1.el7.centos.noarch
tendrl-api-1.6.1-1.el7.centos.noarch
tendrl-ui-1.6.1-1.el7.centos.noarch
tendrl-grafana-selinux-1.5.4-2.el7.centos.noarch
tendrl-ansible-1.6.1-1.el7.centos.noarch
tendrl-notifier-1.6.0-1.el7.centos.noarch
tendrl-node-agent-1.6.1-1.el7.centos.noarch
tendrl-api-httpd-1.6.1-1.el7.centos.noarch
tendrl-selinux-1.5.4-2.el7.centos.noarch
tendrl-grafana-plugins-1.6.1-1.el7.centos.noarch
tendrl-monitoring-integration-1.6.1-1.el7.centos.noarch

@gnehapk
Copy link
Member

gnehapk commented Mar 12, 2018

@ltrilety Please share the /clusters API response.

@ltrilety
Copy link
Contributor Author

ltrilety commented Mar 12, 2018

@gnehapk you're most probably right with your suspicion that the issue is not on UI side. Unfortunately I don't have that response anymore, as of now API is telling me I have 2 alerts for the clusters whilst all other indices says zero. Feel free to move this issue where it belongs.

@GowthamShanmugam
Copy link
Contributor

GowthamShanmugam commented Apr 7, 2018

PR is under review, Every gluster-integration sync we are recalculating alert count and updating. Please verify and close this issue Tendrl/gluster-integration#599

@gnehapk
Copy link
Member

gnehapk commented Sep 11, 2018

@GowthamShanmugam Please close this issue if fixed.

@GowthamShanmugam
Copy link
Contributor

This issue is fixed, We are now calculating alert count in each gluster-integration sync, So the alert count always matches with a number of alerts.

@gnehapk we can close this issue. @ltrilety please reopen this issue if it is reprodusable with the latest master.

@GowthamShanmugam
Copy link
Contributor

This issue is fixed in PR: Tendrl/gluster-integration#599

@gnehapk gnehapk closed this as completed Sep 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants