)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":37173,"name":"Alex Fernandes Figueirêdo","display_name":"Alex Figueiredo","email":"alex.fernandesfigueiredo@windriver.com","username":"afernand"},"change_message_id":"f073de747a6620a8f9ae7d12459c7d80583c377a","unresolved":true,"context_lines":[{"line_number":7,"context_line":"Update ingress-nginx docker images reference"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The way that the Ingress-nginx Helm chart references the"},{"line_number":10,"context_line":"Docker images in the values file its not supperted by the"},{"line_number":11,"context_line":"application framework."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This review creates a patch that updates the way the Ingress"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"2965db43_33d8ad4f","line":10,"range":{"start_line":10,"start_character":41,"end_line":10,"end_character":50},"updated":"2024-12-12 14:20:35.000000000","message":"typo","commit_id":"b0896295098262ab31d5d1b289d9f9d2b425f75d"},{"author":{"_account_id":37078,"name":"Daniel Marques Caires","display_name":"Daniel Caires","email":"DanielMarques.Caires@windriver.com","username":"dcaires"},"change_message_id":"acaf8773da3e2624f0215fc7314e9e690fbe99d0","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Update ingress-nginx docker images reference"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The way that the Ingress-nginx Helm chart references the"},{"line_number":10,"context_line":"Docker images in the values file its not supperted by the"},{"line_number":11,"context_line":"application framework."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This review creates a patch that updates the way the Ingress"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"2f99ca0a_3ac35e0f","line":10,"range":{"start_line":10,"start_character":41,"end_line":10,"end_character":50},"in_reply_to":"2965db43_33d8ad4f","updated":"2024-12-12 14:28:59.000000000","message":"Done","commit_id":"b0896295098262ab31d5d1b289d9f9d2b425f75d"},{"author":{"_account_id":35446,"name":"Igor Pires Soares","display_name":"Igor Soares","email":"Igor.PiresSoares@windriver.com","username":"ipiresso"},"change_message_id":"390274fd9ab0a489dcdd5709b9cd7c638efd0b06","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The way that the Ingress-nginx Helm chart references the"},{"line_number":10,"context_line":"Docker images in the values file its not supported by the"},{"line_number":11,"context_line":"application framework."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This review creates a patch that updates the way the Ingress"},{"line_number":14,"context_line":"Helm chart deals with the docker images inside the values and"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"98897a72_6ecae629","line":11,"updated":"2024-12-16 21:08:13.000000000","message":"Was this tested after https://review.opendev.org/c/starlingx/config/+/936360 got merged?\n\nIdeally this patch shouldn\u0027t be necessary.","commit_id":"d824a9ca42aea0487a532de4a8578423ae39ae7c"},{"author":{"_account_id":37078,"name":"Daniel Marques Caires","display_name":"Daniel Caires","email":"DanielMarques.Caires@windriver.com","username":"dcaires"},"change_message_id":"9ce5bf725f7d4f066513bb728e5631fd003bd0cc","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The way that the Ingress-nginx Helm chart references the"},{"line_number":10,"context_line":"Docker images in the values file its not supported by the"},{"line_number":11,"context_line":"application framework."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"This review creates a patch that updates the way the Ingress"},{"line_number":14,"context_line":"Helm chart deals with the docker images inside the values and"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"19b30dab_d6ca7ee5","line":11,"in_reply_to":"98897a72_6ecae629","updated":"2024-12-18 16:48:48.000000000","message":"You are right! This patch is not necessary. I\u0027ll be abandoning this change as it is not necessary.","commit_id":"d824a9ca42aea0487a532de4a8578423ae39ae7c"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":37173,"name":"Alex Fernandes Figueirêdo","display_name":"Alex Figueiredo","email":"alex.fernandesfigueiredo@windriver.com","username":"afernand"},"change_message_id":"f073de747a6620a8f9ae7d12459c7d80583c377a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"658582ce_225d1d5f","updated":"2024-12-12 14:20:35.000000000","message":"LGTM. Please fix the typo mentioned in the comments.","commit_id":"b0896295098262ab31d5d1b289d9f9d2b425f75d"}]}