)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":32190,"name":"Frank Ritchie","email":"fr801x@att.com","username":"fr801x"},"change_message_id":"b963d1c889c43c4a58f0284312eaf09090c8bc11","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"ae572e6a_849c6115","updated":"2023-08-11 20:04:41.000000000","message":"Yes, but since this section already had parameter values included inline I decided to not break the precedent. Refactoring this can be a separate PS.","commit_id":"c394080738a309abc5f30584019b63692ec6a1be"}],"glance/templates/deployment-api.yaml":[{"author":{"_account_id":3009,"name":"Vladimir Kozhukalov","email":"kozhukalov@gmail.com","username":"kozhukalov"},"change_message_id":"a4b57d182b719627e93bca444e099ddc017c1c10","unresolved":true,"context_lines":[{"line_number":167,"context_line":"                - python"},{"line_number":168,"context_line":"                - -c"},{"line_number":169,"context_line":"                - \"import requests; requests.get(\u0027http://127.0.0.1:{{ tuple \"image\" \"service\" \"api\" . | include \"helm-toolkit.endpoints.endpoint_port_lookup\" }}\u0027)\""},{"line_number":170,"context_line":"            periodSeconds: 15"},{"line_number":171,"context_line":"            timeoutSeconds: 10"},{"line_number":172,"context_line":"          livenessProbe:"},{"line_number":173,"context_line":"            exec:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"fa0fa563_ebd57859","line":170,"updated":"2023-08-11 19:51:32.000000000","message":"It is better to bring these parameters out to the values.yaml. Anyway thanks for tuning this.","commit_id":"c394080738a309abc5f30584019b63692ec6a1be"},{"author":{"_account_id":32190,"name":"Frank Ritchie","email":"fr801x@att.com","username":"fr801x"},"change_message_id":"40c6295fca0a20b18ff0047eb66ea2c59e6946d6","unresolved":false,"context_lines":[{"line_number":167,"context_line":"                - python"},{"line_number":168,"context_line":"                - -c"},{"line_number":169,"context_line":"                - \"import requests; requests.get(\u0027http://127.0.0.1:{{ tuple \"image\" \"service\" \"api\" . | include \"helm-toolkit.endpoints.endpoint_port_lookup\" }}\u0027)\""},{"line_number":170,"context_line":"            periodSeconds: 15"},{"line_number":171,"context_line":"            timeoutSeconds: 10"},{"line_number":172,"context_line":"          livenessProbe:"},{"line_number":173,"context_line":"            exec:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"670f644e_ecafde0f","line":170,"in_reply_to":"5ea7419a_000c1e78","updated":"2023-08-11 20:37:28.000000000","message":"Done","commit_id":"c394080738a309abc5f30584019b63692ec6a1be"},{"author":{"_account_id":32190,"name":"Frank Ritchie","email":"fr801x@att.com","username":"fr801x"},"change_message_id":"b963d1c889c43c4a58f0284312eaf09090c8bc11","unresolved":true,"context_lines":[{"line_number":167,"context_line":"                - python"},{"line_number":168,"context_line":"                - -c"},{"line_number":169,"context_line":"                - \"import requests; requests.get(\u0027http://127.0.0.1:{{ tuple \"image\" \"service\" \"api\" . | include \"helm-toolkit.endpoints.endpoint_port_lookup\" }}\u0027)\""},{"line_number":170,"context_line":"            periodSeconds: 15"},{"line_number":171,"context_line":"            timeoutSeconds: 10"},{"line_number":172,"context_line":"          livenessProbe:"},{"line_number":173,"context_line":"            exec:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"5ea7419a_000c1e78","line":170,"in_reply_to":"fa0fa563_ebd57859","updated":"2023-08-11 20:04:41.000000000","message":"Yes, but since this section already had parameter values included inline I decided to not break the precedent. Refactoring this can be a separate PS.","commit_id":"c394080738a309abc5f30584019b63692ec6a1be"}]}
