action #159402
closed"SSL expiration" panel on https://stats.openqa-monitor.qa.suse.de/d/WebuiDb/webui-summary shows problems size:S
0%
Description
Observation¶
https://stats.openqa-monitor.qa.suse.de/d/WebuiDb/webui-summary at the bottom shows in "SSL expiration"
B
Status: 500. Message: InfluxDB returned error: error parsing query: found FROM, expected identifier, string, number, bool at line 1, char 9
Maybe the problem is related due to low data collection periods and trying to show data for a small period, like 1h, by default in grafana? Valid data is still shown if a time period like 7 days is selected but the influxdb error still shows up
Acceptance criteria¶
- AC1: Panel shows valid data independant of the time period selection
- AC2: no InfluxDB error on the panel
- AC3: SSL expiration alert has been crosschecked for validity
Suggestions¶
- Go into definition of the panel, fix the error
- Crosscheck the validity of the data as soon it shows up again
- Ensure that a valid minimum time period is seelected at all times
- Crosscheck validity of the related alert
Updated by jbaier_cz 8 months ago
- Subject changed from "SSL expiration" panel on https://stats.openqa-monitor.qa.suse.de/d/WebuiDb/webui-summary shows problems to "SSL expiration" panel on https://stats.openqa-monitor.qa.suse.de/d/WebuiDb/webui-summary shows problems size:S
- Status changed from New to Workable
Updated by livdywan 8 months ago
- Copied to action #159666: Lack of alerts about error messages related to InfluxDB and SSL expiration panel added
Updated by nicksinger 8 months ago
- Status changed from Workable to In Progress
- Assignee set to nicksinger
Updated by nicksinger 8 months ago
- Status changed from In Progress to Feedback
The superfluous query got introduced with https://gitlab.suse.de/openqa/salt-states-openqa/-/merge_requests/1051/diffs#83363be13fc620ae79e6efa4c1e0cca5f9e930cb_7458_7475 and was most likely not critical back in the days. I assume with some update grafana started to report these issues and shows us this ugly exclamation mark on the panel itself. I created https://gitlab.suse.de/openqa/salt-states-openqa/-/merge_requests/1163 to remove it, adjust the interval on the graph and crosschecked that we use the correct query in the alert (yes we do). So all ACs covered it the MR gets merged.
Updated by nicksinger 8 months ago
- Status changed from Feedback to Resolved
Verified the panel and the exclamation mark is now gone while still showing our data.