)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"56789c397c21508ec9c629bcc3b3ccdcfe681ba1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"080ffb90_94b2255e","updated":"2024-06-11 08:48:38.000000000","message":"We\u0027ve found redfish exporter usage sufficient for most recent hardware, maybe that\u0027s a better path instead of adding vendor specific exporters?","commit_id":"aaa3cd6ab971e616a6176648a1fee08a71711b82"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"be7b6654b0a1b033b01bfc8271f8432a7c6fde7b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"8c0d929b_98f8086d","updated":"2024-06-13 09:32:37.000000000","message":"-1 for visibility","commit_id":"15a970e69cd55687edfa1211574836f4c9ce3e1f"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"f272ee2748fc5930ee539ea34429155e17e8b072","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"c7713d2c_d4e36c56","updated":"2024-07-21 07:41:15.000000000","message":"LGTM, but still need some pice of documentation about unused by default \u0027prometheus_bmc_exporter_packages\u0027 variable, why and when it can be used?","commit_id":"5ef94f04329d8982a6179861b5461cb448ce61ec"},{"author":{"_account_id":32398,"name":"Gaël THEROND","display_name":"Fl1nt","email":"gael.therond@bitswalk.com","username":"Fl1nt"},"change_message_id":"a99fff03e6d5b0c2fa1474f7e7080d61cbb75b42","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"ce9d9e62_0e226bdc","in_reply_to":"5dcbd07d_1ebb4909","updated":"2024-07-29 14:59:12.000000000","message":"It was the moment I found the issue thanks to your message and review ;-) We can go ahead with this issue discussion/process I think.\n\nI still need to rework a bit the kolla-ansible deployment part of it tho, but the exporter container is at least building and working as expected (tested on lab).","commit_id":"5ef94f04329d8982a6179861b5461cb448ce61ec"},{"author":{"_account_id":32398,"name":"Gaël THEROND","display_name":"Fl1nt","email":"gael.therond@bitswalk.com","username":"Fl1nt"},"change_message_id":"4176badeadc0c4a6e6d0709de6940fa013aef833","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"c79c8e7b_70f8a903","in_reply_to":"c7713d2c_d4e36c56","updated":"2024-07-22 07:41:01.000000000","message":"It was indeed a bad copy/paste from the blackbox exporter that I used as the cookie cutter for this exporter, it\u0027s fixed now.","commit_id":"5ef94f04329d8982a6179861b5461cb448ce61ec"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"f17daf7eb3b48c94aa777b61c2924b60c876fd42","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":9,"id":"5dcbd07d_1ebb4909","in_reply_to":"c79c8e7b_70f8a903","updated":"2024-07-23 07:38:37.000000000","message":"so we should drop that block as not needed.","commit_id":"5ef94f04329d8982a6179861b5461cb448ce61ec"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"0c5d4121d1878ac1ad89db8c4543c1d3b70b56c2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"7a56df82_6c1d3e5e","updated":"2024-09-16 10:13:13.000000000","message":"recheck debian upgrade galaxy fail","commit_id":"835a8fed768e87ff828fe3526bb9e8046b02607b"},{"author":{"_account_id":32398,"name":"Gaël THEROND","display_name":"Fl1nt","email":"gael.therond@bitswalk.com","username":"Fl1nt"},"change_message_id":"105fcac4d1d52aa666d51d9841e767ec779d5137","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"8f388465_ed912e23","updated":"2024-10-01 08:52:13.000000000","message":"Alright, I\u0027ve just adjusted the naming/paths according to the IPMI naming for: https://review.opendev.org/c/openstack/kolla-ansible/+/922211 and added a link to the kolla-ansible documentation for reference.","commit_id":"0d3f5dff5fb79b6ad990043f30b87c2156e2ad1c"}],"docker/prometheus/prometheus-bmc-exporter/Dockerfile.j2":[{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"be7b6654b0a1b033b01bfc8271f8432a7c6fde7b","unresolved":true,"context_lines":[{"line_number":33,"context_line":"ARG prometheus-bmc-exporter-version\u003d0.1.8"},{"line_number":34,"context_line":"WORKDIR /opt/bmc-exporter/"},{"line_number":35,"context_line":"ADD prometheus-bmc-exporter-archive src/"},{"line_number":36,"context_line":"ENV GOPATH\u003d/opt/bmc-exporter/src/idrac_exporter-helm-idrac-exporter-${prometheus-bmc-exporter-version}/"},{"line_number":37,"context_line":"RUN go build -o /opt/bmc-exporter/bmc-exporter ./idrac_exporter-helm-idrac-exporter-${prometheus-bmc-exporter-version}/cmd/idrac_exporter \u0026\u0026 rm -rf /opt/bmc-exporter/src/"},{"line_number":38,"context_line":"{% endblock %}"},{"line_number":39,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"6a94af36_08445428","line":36,"updated":"2024-06-13 09:32:37.000000000","message":"idrac exporter is not bmc exporter - did you want to use https://github.com/gebn/bmc_exporter?","commit_id":"15a970e69cd55687edfa1211574836f4c9ce3e1f"},{"author":{"_account_id":32398,"name":"Gaël THEROND","display_name":"Fl1nt","email":"gael.therond@bitswalk.com","username":"Fl1nt"},"change_message_id":"9800b917f616fa7870fdabe0329323b969c927a0","unresolved":true,"context_lines":[{"line_number":33,"context_line":"ARG prometheus-bmc-exporter-version\u003d0.1.8"},{"line_number":34,"context_line":"WORKDIR /opt/bmc-exporter/"},{"line_number":35,"context_line":"ADD prometheus-bmc-exporter-archive src/"},{"line_number":36,"context_line":"ENV GOPATH\u003d/opt/bmc-exporter/src/idrac_exporter-helm-idrac-exporter-${prometheus-bmc-exporter-version}/"},{"line_number":37,"context_line":"RUN go build -o /opt/bmc-exporter/bmc-exporter ./idrac_exporter-helm-idrac-exporter-${prometheus-bmc-exporter-version}/cmd/idrac_exporter \u0026\u0026 rm -rf /opt/bmc-exporter/src/"},{"line_number":38,"context_line":"{% endblock %}"},{"line_number":39,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"be9bce6d_715ef845","line":36,"in_reply_to":"6a94af36_08445428","updated":"2024-06-13 09:53:58.000000000","message":"I didn\u0027t found it when I did researched available bmc exporters, I can use whatever we want as long as the exporter leverage redfish api to introspect BMCs.\n\nThe one I added is also leveraging redfish api and supporting both DELL and HPE, but I can switch to the one you suggested if you want.","commit_id":"15a970e69cd55687edfa1211574836f4c9ce3e1f"},{"author":{"_account_id":32398,"name":"Gaël THEROND","display_name":"Fl1nt","email":"gael.therond@bitswalk.com","username":"Fl1nt"},"change_message_id":"cab232f2f2f9eada5a0ef0ba36939470fdbda896","unresolved":false,"context_lines":[{"line_number":33,"context_line":"ARG prometheus-bmc-exporter-version\u003d0.1.8"},{"line_number":34,"context_line":"WORKDIR /opt/bmc-exporter/"},{"line_number":35,"context_line":"ADD prometheus-bmc-exporter-archive src/"},{"line_number":36,"context_line":"ENV GOPATH\u003d/opt/bmc-exporter/src/idrac_exporter-helm-idrac-exporter-${prometheus-bmc-exporter-version}/"},{"line_number":37,"context_line":"RUN go build -o /opt/bmc-exporter/bmc-exporter ./idrac_exporter-helm-idrac-exporter-${prometheus-bmc-exporter-version}/cmd/idrac_exporter \u0026\u0026 rm -rf /opt/bmc-exporter/src/"},{"line_number":38,"context_line":"{% endblock %}"},{"line_number":39,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"969ecc7b_56751a28","line":36,"in_reply_to":"be9bce6d_715ef845","updated":"2024-06-13 11:58:22.000000000","message":"Ok, I\u0027ve switched to the suggested exporter, it use IPMI instead of HTTPs redfish approach but it provide the metrics we were specifically looking for and this IPMI based approach do provide optimization in terms of sessions/responsiveness and traffic.","commit_id":"15a970e69cd55687edfa1211574836f4c9ce3e1f"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"800b7b2fb6289ac8c4ac797acae476262cabb6b6","unresolved":true,"context_lines":[{"line_number":6,"context_line":"{% block prometheus_bmc_exporter_header %}{% endblock %}"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"{% block prometheus_bmc_exporter_packages %}"},{"line_number":10,"context_line":"{% import \"macros.j2\" as macros with context %}"},{"line_number":11,"context_line":"{{ macros.install_packages(prometheus_bmc_exporter_packages | customizable(\"packages\")) }}"},{"line_number":12,"context_line":"{% endblock %}"}],"source_content_type":"text/x-jinja2","patch_set":6,"id":"244ae117_b00a9cf8","line":9,"range":{"start_line":9,"start_character":9,"end_line":9,"end_character":41},"updated":"2024-07-16 09:40:43.000000000","message":"rename to install_prometheus_bmc_exporter_packages since prometheus_bmc_exporter_packages used in macros","commit_id":"67cac21d1db430e4c475199168167bcd1d7a4f6e"},{"author":{"_account_id":32398,"name":"Gaël THEROND","display_name":"Fl1nt","email":"gael.therond@bitswalk.com","username":"Fl1nt"},"change_message_id":"1681f550aef3033c44e8a5c8df998026b3f1b8c4","unresolved":false,"context_lines":[{"line_number":6,"context_line":"{% block prometheus_bmc_exporter_header %}{% endblock %}"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"{% block prometheus_bmc_exporter_packages %}"},{"line_number":10,"context_line":"{% import \"macros.j2\" as macros with context %}"},{"line_number":11,"context_line":"{{ macros.install_packages(prometheus_bmc_exporter_packages | customizable(\"packages\")) }}"},{"line_number":12,"context_line":"{% endblock %}"}],"source_content_type":"text/x-jinja2","patch_set":6,"id":"073b6e5b_34c597f0","line":9,"range":{"start_line":9,"start_character":9,"end_line":9,"end_character":41},"in_reply_to":"244ae117_b00a9cf8","updated":"2024-07-16 14:22:46.000000000","message":"You\u0027re right, will do.","commit_id":"67cac21d1db430e4c475199168167bcd1d7a4f6e"},{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"800b7b2fb6289ac8c4ac797acae476262cabb6b6","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"{% block prometheus_bmc_exporter_packages %}"},{"line_number":10,"context_line":"{% import \"macros.j2\" as macros with context %}"},{"line_number":11,"context_line":"{{ macros.install_packages(prometheus_bmc_exporter_packages | customizable(\"packages\")) }}"},{"line_number":12,"context_line":"{% endblock %}"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"{% block prometheus_bmc_exporter_install %}"}],"source_content_type":"text/x-jinja2","patch_set":6,"id":"58387e0d_ade6ceec","line":11,"range":{"start_line":11,"start_character":27,"end_line":11,"end_character":59},"updated":"2024-07-16 09:40:43.000000000","message":"please add the release note and/or pice of documentation about these packages","commit_id":"67cac21d1db430e4c475199168167bcd1d7a4f6e"},{"author":{"_account_id":32398,"name":"Gaël THEROND","display_name":"Fl1nt","email":"gael.therond@bitswalk.com","username":"Fl1nt"},"change_message_id":"1681f550aef3033c44e8a5c8df998026b3f1b8c4","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"{% block prometheus_bmc_exporter_packages %}"},{"line_number":10,"context_line":"{% import \"macros.j2\" as macros with context %}"},{"line_number":11,"context_line":"{{ macros.install_packages(prometheus_bmc_exporter_packages | customizable(\"packages\")) }}"},{"line_number":12,"context_line":"{% endblock %}"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"{% block prometheus_bmc_exporter_install %}"}],"source_content_type":"text/x-jinja2","patch_set":6,"id":"991b80f6_6683e236","line":11,"range":{"start_line":11,"start_character":27,"end_line":11,"end_character":59},"in_reply_to":"58387e0d_ade6ceec","updated":"2024-07-16 14:22:46.000000000","message":"You\u0027re right, will do.","commit_id":"67cac21d1db430e4c475199168167bcd1d7a4f6e"},{"author":{"_account_id":32398,"name":"Gaël THEROND","display_name":"Fl1nt","email":"gael.therond@bitswalk.com","username":"Fl1nt"},"change_message_id":"79ae2ab9339eabad546b4c08a7dd98ec6e360ec7","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"{% block prometheus_bmc_exporter_packages %}"},{"line_number":10,"context_line":"{% import \"macros.j2\" as macros with context %}"},{"line_number":11,"context_line":"{{ macros.install_packages(prometheus_bmc_exporter_packages | customizable(\"packages\")) }}"},{"line_number":12,"context_line":"{% endblock %}"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"{% block prometheus_bmc_exporter_install %}"}],"source_content_type":"text/x-jinja2","patch_set":6,"id":"659a4fef_4dead286","line":11,"range":{"start_line":11,"start_character":27,"end_line":11,"end_character":59},"in_reply_to":"991b80f6_6683e236","updated":"2024-07-18 09:34:08.000000000","message":"Indeed, the reno already exist: add-prometheus_bmc_exporter-252e69f09d182100","commit_id":"67cac21d1db430e4c475199168167bcd1d7a4f6e"}],"kolla/common/sources.py":[{"author":{"_account_id":14200,"name":"Maksim Malchuk","email":"maksim.malchuk@gmail.com","username":"mmalchuk"},"change_message_id":"800b7b2fb6289ac8c4ac797acae476262cabb6b6","unresolved":true,"context_lines":[{"line_number":298,"context_line":"                     \u0027gebn/bmc_exporter/\u0027"},{"line_number":299,"context_line":"                     \u0027releases/download/v${version}/\u0027"},{"line_number":300,"context_line":"                     \u0027bmc_exporter\u0027"},{"line_number":301,"context_line":"                     \u0027-${version}.linux-${debian_arch}.tar.gz\u0027)},"},{"line_number":302,"context_line":"    \u0027prometheus-cadvisor\u0027: {"},{"line_number":303,"context_line":"        \u0027version\u0027: \u00270.49.1\u0027,"},{"line_number":304,"context_line":"        \u0027type\u0027: \u0027url\u0027,"}],"source_content_type":"text/x-python","patch_set":6,"id":"4d4b754f_64f93416","line":301,"updated":"2024-07-16 09:40:43.000000000","message":"please add the sha256 checksums from related *.tar.gz.sha256 files","commit_id":"67cac21d1db430e4c475199168167bcd1d7a4f6e"},{"author":{"_account_id":32398,"name":"Gaël THEROND","display_name":"Fl1nt","email":"gael.therond@bitswalk.com","username":"Fl1nt"},"change_message_id":"1681f550aef3033c44e8a5c8df998026b3f1b8c4","unresolved":false,"context_lines":[{"line_number":298,"context_line":"                     \u0027gebn/bmc_exporter/\u0027"},{"line_number":299,"context_line":"                     \u0027releases/download/v${version}/\u0027"},{"line_number":300,"context_line":"                     \u0027bmc_exporter\u0027"},{"line_number":301,"context_line":"                     \u0027-${version}.linux-${debian_arch}.tar.gz\u0027)},"},{"line_number":302,"context_line":"    \u0027prometheus-cadvisor\u0027: {"},{"line_number":303,"context_line":"        \u0027version\u0027: \u00270.49.1\u0027,"},{"line_number":304,"context_line":"        \u0027type\u0027: \u0027url\u0027,"}],"source_content_type":"text/x-python","patch_set":6,"id":"e653b324_e52fbed4","line":301,"in_reply_to":"4d4b754f_64f93416","updated":"2024-07-16 14:22:46.000000000","message":"You\u0027re right, will do.","commit_id":"67cac21d1db430e4c475199168167bcd1d7a4f6e"}]}