)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"34b5f501777d5faa33463f7f1db270665339de24","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"4e18476c_23e856f4","updated":"2022-05-18 00:33:53.000000000","message":"The implementation looks good to me.\n\n-1 because of a few suggestions. These are not huge blockers but it\u0027d be nice if you can fix them.","commit_id":"c8a483d1862f2ffc43a14e5bbe50f96fb9ab998c"},{"author":{"_account_id":34429,"name":"Tom Weininger","email":"dienste@weinimo.de","username":"tweining"},"change_message_id":"72f4ee83975c96a6237ba9be7f614a3f0db52deb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"d44eefe3_10481d23","updated":"2022-05-17 15:03:53.000000000","message":"recheck download error could be temporary","commit_id":"c8a483d1862f2ffc43a14e5bbe50f96fb9ab998c"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"b8d57d3d457b6b6e9de3ea0e198bebf8807af0fb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"68cce38e_47012669","updated":"2022-05-23 16:34:14.000000000","message":"recheck\n\nchecking whether the failure in dnf update -y is consistent.","commit_id":"b5ab3de4bcf122d4228173eca0d0cd733a0e1ed9"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"9a81898b0943decb3e525dc8a3358f7c438e9125","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"3ca9b2c0_157d597e","updated":"2022-05-22 14:43:00.000000000","message":"recheck\n\ndeployment failed during package installation, which seems to be a temporal issue.","commit_id":"b5ab3de4bcf122d4228173eca0d0cd733a0e1ed9"}],"manifests/controller.pp":[{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"003c6a49e637b50b4ddb9ebdadaf1968d026395d","unresolved":true,"context_lines":[{"line_number":68,"context_line":"# [*amp_timezone*]"},{"line_number":69,"context_line":"#   (optional) Defines the timezone to use as represented in"},{"line_number":70,"context_line":"#   /usr/share/zoneinfo."},{"line_number":71,"context_line":"#   Default is \u0027UTC\u0027"},{"line_number":72,"context_line":"#"},{"line_number":73,"context_line":"# [*enable_ssh_access*]"},{"line_number":74,"context_line":"#   (optional) Enable SSH key configuration for amphorae. Note that setting"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"3e0a89e9_b269d885","line":71,"range":{"start_line":71,"start_character":14,"end_line":71,"end_character":20},"updated":"2022-05-17 10:37:59.000000000","message":"I think it should be $::os_service_default","commit_id":"cc6d6dfe2537f441928ac40102f56570d5d22c2b"},{"author":{"_account_id":34429,"name":"Tom Weininger","email":"dienste@weinimo.de","username":"tweining"},"change_message_id":"f6183b1e41df4ddfaadd0e3564d7e83c4c11c443","unresolved":false,"context_lines":[{"line_number":68,"context_line":"# [*amp_timezone*]"},{"line_number":69,"context_line":"#   (optional) Defines the timezone to use as represented in"},{"line_number":70,"context_line":"#   /usr/share/zoneinfo."},{"line_number":71,"context_line":"#   Default is \u0027UTC\u0027"},{"line_number":72,"context_line":"#"},{"line_number":73,"context_line":"# [*enable_ssh_access*]"},{"line_number":74,"context_line":"#   (optional) Enable SSH key configuration for amphorae. Note that setting"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"724a4639_0cc9b35f","line":71,"range":{"start_line":71,"start_character":14,"end_line":71,"end_character":20},"in_reply_to":"3e0a89e9_b269d885","updated":"2022-05-17 10:45:45.000000000","message":"Done","commit_id":"cc6d6dfe2537f441928ac40102f56570d5d22c2b"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"34b5f501777d5faa33463f7f1db270665339de24","unresolved":true,"context_lines":[{"line_number":289,"context_line":"    \u0027controller_worker/amp_image_tag\u0027                    : value \u003d\u003e $amp_image_tag;"},{"line_number":290,"context_line":"    \u0027controller_worker/amp_image_owner_id\u0027               : value \u003d\u003e $amp_image_owner_id;"},{"line_number":291,"context_line":"    \u0027controller_worker/amp_secgroup_list\u0027                : value \u003d\u003e join(any2array($amp_secgroup_list), \u0027,\u0027);"},{"line_number":292,"context_line":"    \u0027controller_worker/amp_timezone\u0027                     : value \u003d\u003e $amp_timezone;"},{"line_number":293,"context_line":"    \u0027controller_worker/amp_boot_network_list\u0027            : value \u003d\u003e join(any2array($amp_boot_network_list), \u0027,\u0027);"},{"line_number":294,"context_line":"    \u0027controller_worker/loadbalancer_topology\u0027            : value \u003d\u003e $loadbalancer_topology;"},{"line_number":295,"context_line":"    \u0027controller_worker/amphora_driver\u0027                   : value \u003d\u003e $amphora_driver;"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"f1920123_37ea3996","line":292,"range":{"start_line":292,"start_character":4,"end_line":292,"end_character":82},"updated":"2022-05-18 00:33:53.000000000","message":"this is not the consistent position IMO. (it should be added after image_driver)\n\nPlacing this parameter below amp_boot_network_list(in all place) would be better.","commit_id":"c8a483d1862f2ffc43a14e5bbe50f96fb9ab998c"},{"author":{"_account_id":34429,"name":"Tom Weininger","email":"dienste@weinimo.de","username":"tweining"},"change_message_id":"941a56e6f335546002135ae88991b13b758bc019","unresolved":false,"context_lines":[{"line_number":289,"context_line":"    \u0027controller_worker/amp_image_tag\u0027                    : value \u003d\u003e $amp_image_tag;"},{"line_number":290,"context_line":"    \u0027controller_worker/amp_image_owner_id\u0027               : value \u003d\u003e $amp_image_owner_id;"},{"line_number":291,"context_line":"    \u0027controller_worker/amp_secgroup_list\u0027                : value \u003d\u003e join(any2array($amp_secgroup_list), \u0027,\u0027);"},{"line_number":292,"context_line":"    \u0027controller_worker/amp_timezone\u0027                     : value \u003d\u003e $amp_timezone;"},{"line_number":293,"context_line":"    \u0027controller_worker/amp_boot_network_list\u0027            : value \u003d\u003e join(any2array($amp_boot_network_list), \u0027,\u0027);"},{"line_number":294,"context_line":"    \u0027controller_worker/loadbalancer_topology\u0027            : value \u003d\u003e $loadbalancer_topology;"},{"line_number":295,"context_line":"    \u0027controller_worker/amphora_driver\u0027                   : value \u003d\u003e $amphora_driver;"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"2769346b_f59f7584","line":292,"range":{"start_line":292,"start_character":4,"end_line":292,"end_character":82},"in_reply_to":"f1920123_37ea3996","updated":"2022-05-19 09:54:18.000000000","message":"Done","commit_id":"c8a483d1862f2ffc43a14e5bbe50f96fb9ab998c"}],"releasenotes/notes/set-amphora-timezone-103b078b31ab7aeb.yaml":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"34b5f501777d5faa33463f7f1db270665339de24","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Adds the controller_worker/amp_timezone configuration parameter. It allows"},{"line_number":5,"context_line":"    to configure the timezone of the amphora."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"e122fc97_b8556dd4","line":4,"range":{"start_line":4,"start_character":13,"end_line":4,"end_character":43},"updated":"2022-05-18 00:33:53.000000000","message":"This is not the puppet parameter but octavia parameter so might be better to say;\n\nAdds support for the ``[controller_worker] amp_timezone`` configuration parameter.","commit_id":"c8a483d1862f2ffc43a14e5bbe50f96fb9ab998c"},{"author":{"_account_id":34429,"name":"Tom Weininger","email":"dienste@weinimo.de","username":"tweining"},"change_message_id":"941a56e6f335546002135ae88991b13b758bc019","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Adds the controller_worker/amp_timezone configuration parameter. It allows"},{"line_number":5,"context_line":"    to configure the timezone of the amphora."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"447b32a0_d1bf79f6","line":4,"range":{"start_line":4,"start_character":13,"end_line":4,"end_character":43},"in_reply_to":"e122fc97_b8556dd4","updated":"2022-05-19 09:54:18.000000000","message":"Done","commit_id":"c8a483d1862f2ffc43a14e5bbe50f96fb9ab998c"}]}