QA » openQA Project » openQA Infrastructure

After https://gitlab.suse.de/openqa/salt-states-openqa/-/merge_requests/1093 being merged the pipeline itself was able to test the pillars properly again which enabled us to rebase https://gitlab.suse.de/openqa/salt-states-openqa/-/merge_requests/1092 to pass the pipeline there. Unfortunately with my revert (https://gitlab.suse.de/openqa/salt-states-openqa/-/merge_requests/1089) I did not delete the according, generated telegraf-config in /etc/telegraf/telegraf.d which left us with the old and wrong API key and therefore telegraf was still failing. With MR!1092 merged this also got fixed and we now stumble over a new issue: https://gitlab.suse.de/openqa/salt-states-openqa/-/jobs/2186381#L129

This seems to be caused by https://github.com/rakoenig/qe-yam-backlog-assistant/blob/master/queries.yaml#L31 which triggers a missing escaping implementation in out backlogger tool. I looked up the rules and implemented https://github.com/openSUSE/backlogger/pull/28 as permanent fix. As urgency mitigation I adjusted /etc/telegraf/scripts/tools-yam-backlog/queries.yaml manually and added a backslash before every comma. I'd assume salt overwrites this repo but maybe due to an unclean state it doesn't and apparently seems to work for now.

good. https://github.com/openSUSE/backlogger/pull/28 is now merged as well. What do you plan as next step?

I checked and my changes are already deployed on the monitor host. I reverted the config and ran the script again - this time successful. Metrics are available in grafana as well: