)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"59fa08829eb69e057195971a850a3461d5265965","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"f1918bc9_3edef1ac","updated":"2022-08-03 12:41:29.000000000","message":"the CI is failing with\n\n    \"msg\": \"Ansible inventory does not contain the expected group prometheus-msteams for service prometheus-msteams in prometheus.\"\n\n\nwhich means you need to modify the CI inventory template as well","commit_id":"d76c79ab1bba3fc1e536bb46c3c673c638233646"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"19494ca5452c7ea702fc43d47c2f809811554cbd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"c40257f8_540813a3","in_reply_to":"f1918bc9_3edef1ac","updated":"2022-08-04 05:18:50.000000000","message":"In which log file did you see this?","commit_id":"d76c79ab1bba3fc1e536bb46c3c673c638233646"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"3e70b7ad182f568c74d21138107efaa6b7617ff3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"7827b5d2_afa6b345","updated":"2022-08-08 07:30:09.000000000","message":"Missing precheck for prometheus_msteams port\nmissing support for http proxy env variables (just like here: https://github.com/prometheus-msteams/prometheus-msteams/blob/44e217c0c003fc2c5900b4553ed528b5c2a41c0e/chart/prometheus-msteams/README.md)","commit_id":"bbd65ea81fa2884a5d0d2da0be49f64ca7e72187"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"66a5b26218bf5e753e0f92e193af7c49d1c93201","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"6b5a95d9_66746671","in_reply_to":"3768426b_2d071943","updated":"2022-08-12 16:42:30.000000000","message":"Precheck wasn\u0027t complete, it was fixed in https://review.opendev.org/c/openstack/kolla-ansible/+/852939.","commit_id":"bbd65ea81fa2884a5d0d2da0be49f64ca7e72187"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"86fc693b516ca4935112dc1ac7ae783e5cf4b928","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":10,"id":"e9c98780_4fabdcba","in_reply_to":"7827b5d2_afa6b345","updated":"2022-08-08 07:58:32.000000000","message":"I\u0027ve added the port precheck. I will push another revision once I get the chance to test the proxy configuration.","commit_id":"bbd65ea81fa2884a5d0d2da0be49f64ca7e72187"},{"author":{"_account_id":32657,"name":"Piotr Parczewski","email":"piotr@stackhpc.com","username":"piotrp"},"change_message_id":"8c45862b56b5112aacb50daf1e9409be06f64795","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":10,"id":"3768426b_2d071943","in_reply_to":"e9c98780_4fabdcba","updated":"2022-08-09 09:24:24.000000000","message":"Happy to add this as well. Just tested this in a deployment, it works.","commit_id":"bbd65ea81fa2884a5d0d2da0be49f64ca7e72187"}],"ansible/roles/prometheus/defaults/main.yml":[{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"76b31e5640a7616875d4db933bb9e3ce1050d5aa","unresolved":true,"context_lines":[{"line_number":112,"context_line":"    container_name: \"prometheus_msteams\""},{"line_number":113,"context_line":"    group: \"prometheus-msteams\""},{"line_number":114,"context_line":"    enabled: \"{{ enable_prometheus_msteams | bool }}\""},{"line_number":115,"context_line":"    environment: \"{{ container_proxy }}\""},{"line_number":116,"context_line":"    image: \"{{ prometheus_msteams_image_full }}\""},{"line_number":117,"context_line":"    volumes: \"{{ prometheus_msteams_default_volumes + prometheus_msteams_extra_volumes }}\""},{"line_number":118,"context_line":"    dimensions: \"{{ prometheus_msteams_dimensions }}\""}],"source_content_type":"text/x-yaml","patch_set":12,"id":"fb558eee_47fba2ab","line":115,"range":{"start_line":115,"start_character":21,"end_line":115,"end_character":36},"updated":"2022-08-10 11:07:02.000000000","message":"can we have this as prometheus_msteams_container_proxy just like in Magnum role (https://opendev.org/openstack/kolla-ansible/src/commit/8ef21123aea6371f23a7e86f6461a91c17bd84fd/ansible/roles/magnum/defaults/main.yml#L29)","commit_id":"95596c901ce7ef2836a20a39dd06e79479d579df"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"66a5b26218bf5e753e0f92e193af7c49d1c93201","unresolved":false,"context_lines":[{"line_number":112,"context_line":"    container_name: \"prometheus_msteams\""},{"line_number":113,"context_line":"    group: \"prometheus-msteams\""},{"line_number":114,"context_line":"    enabled: \"{{ enable_prometheus_msteams | bool }}\""},{"line_number":115,"context_line":"    environment: \"{{ container_proxy }}\""},{"line_number":116,"context_line":"    image: \"{{ prometheus_msteams_image_full }}\""},{"line_number":117,"context_line":"    volumes: \"{{ prometheus_msteams_default_volumes + prometheus_msteams_extra_volumes }}\""},{"line_number":118,"context_line":"    dimensions: \"{{ prometheus_msteams_dimensions }}\""}],"source_content_type":"text/x-yaml","patch_set":12,"id":"d7dc8fdd_1b229eea","line":115,"range":{"start_line":115,"start_character":21,"end_line":115,"end_character":36},"in_reply_to":"fb558eee_47fba2ab","updated":"2022-08-12 16:42:30.000000000","message":"Done by Piotr.","commit_id":"95596c901ce7ef2836a20a39dd06e79479d579df"}]}
