)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"fd4a5ed963dbe3c135e47c102573d0c33e9a5f81","unresolved":true,"context_lines":[{"line_number":11,"context_line":"building tripleo-ansible-ee container[2]."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"[1].https://docs.openstack.org/project-deploy-guide/tripleo-docs/latest/deployment/3rd_party.html#tips-and-tricks-with-tripleo-container-image-build"},{"line_number":14,"context_line":"[2]. https://review.opendev.org/c/openstack/tripleo-common/+/776674"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":""},{"line_number":17,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":14,"id":"c1e9d27a_a2cd9f69","line":14,"range":{"start_line":14,"start_character":0,"end_line":14,"end_character":67},"updated":"2022-09-05 14:12:49.000000000","message":"I\u0027ve checked this change but could not find usage of extra_config in this patch. Will you update that or am I missing something ?","commit_id":"8aa02c90d93d77525695d5c6b8c1105290f829ca"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"dc31bb03d8ca3d9bc777bbc0a2353eb2419028c3","unresolved":false,"context_lines":[{"line_number":11,"context_line":"building tripleo-ansible-ee container[2]."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"[1].https://docs.openstack.org/project-deploy-guide/tripleo-docs/latest/deployment/3rd_party.html#tips-and-tricks-with-tripleo-container-image-build"},{"line_number":14,"context_line":"[2]. https://review.opendev.org/c/openstack/tripleo-common/+/776674"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":""},{"line_number":17,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":14,"id":"fc04b8ef_786f3988","line":14,"range":{"start_line":14,"start_character":0,"end_line":14,"end_character":67},"in_reply_to":"8d2511f4_955f6cab","updated":"2022-09-13 08:17:58.000000000","message":"OK. I also found this mentioned in the commit message of tripleo-common.\n\nhttps://review.opendev.org/c/openstack/tripleo-ci/+/843836","commit_id":"8aa02c90d93d77525695d5c6b8c1105290f829ca"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"9a97eaae6a7bf59087308395db16de2931e8131a","unresolved":true,"context_lines":[{"line_number":11,"context_line":"building tripleo-ansible-ee container[2]."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"[1].https://docs.openstack.org/project-deploy-guide/tripleo-docs/latest/deployment/3rd_party.html#tips-and-tricks-with-tripleo-container-image-build"},{"line_number":14,"context_line":"[2]. https://review.opendev.org/c/openstack/tripleo-common/+/776674"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":""},{"line_number":17,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":14,"id":"8d2511f4_955f6cab","line":14,"range":{"start_line":14,"start_character":0,"end_line":14,"end_character":67},"in_reply_to":"c1e9d27a_a2cd9f69","updated":"2022-09-13 08:02:50.000000000","message":"This role is used in periodic container build job.\n\nOn this testproject https://review.rdoproject.org/r/c/testproject/+/44079, we have tested it and below are the logs:\n1. https://logserver.rdoproject.org/79/44079/11/check/tripleo-centos-9-tripleo-ansible-ee-container-master/67890f6/logs/build_containers.sh\n```\n     --tcib-extra tcib_release\u003d9 \\\n     --tcib-extra tcib_python_version\u003d3.9 \\\n     --extra-config /home/zuul/workspace/extra_config.yaml \\\n     --debug\n```\nand\nhere is the tripleo-ansible-ee container build log:\nhttps://logserver.rdoproject.org/79/44079/11/check/tripleo-centos-9-tripleo-ansible-ee-container-master/67890f6/logs/container-builds/806b204a-35b9-4c56-a79e-dbae4b9bf23e/tripleo-ansible-ee/tripleo-ansible-ee-build.log","commit_id":"8aa02c90d93d77525695d5c6b8c1105290f829ca"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"fd4a5ed963dbe3c135e47c102573d0c33e9a5f81","unresolved":true,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"[1].https://docs.openstack.org/project-deploy-guide/tripleo-docs/latest/deployment/3rd_party.html#tips-and-tricks-with-tripleo-container-image-build"},{"line_number":14,"context_line":"[2]. https://review.opendev.org/c/openstack/tripleo-common/+/776674"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":""},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Signed-off-by: Chandan Kumar (raukadah) \u003cchkumar@redhat.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":14,"id":"ce67daaf_9184ec87","line":15,"updated":"2022-09-05 14:12:49.000000000","message":"(nit) redundant blank lines.","commit_id":"8aa02c90d93d77525695d5c6b8c1105290f829ca"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"a45e68efcb3280d9f8f26e2a35fe5e9e1a906bc5","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"[1].https://docs.openstack.org/project-deploy-guide/tripleo-docs/latest/deployment/3rd_party.html#tips-and-tricks-with-tripleo-container-image-build"},{"line_number":14,"context_line":"[2]. https://review.opendev.org/c/openstack/tripleo-common/+/776674"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":""},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Signed-off-by: Chandan Kumar (raukadah) \u003cchkumar@redhat.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":14,"id":"0e194839_581d49fa","line":15,"in_reply_to":"ce67daaf_9184ec87","updated":"2022-09-12 05:50:52.000000000","message":"Done","commit_id":"8aa02c90d93d77525695d5c6b8c1105290f829ca"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"fd4a5ed963dbe3c135e47c102573d0c33e9a5f81","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"cf642273_2c9a4997","updated":"2022-09-05 14:12:49.000000000","message":"I\u0027ll vote my +2 once I get my questions cleared, or any other cores can approve this if the change looks good for them","commit_id":"8aa02c90d93d77525695d5c6b8c1105290f829ca"}],"roles/build-containers/tasks/main.yaml":[{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"93474abba97666f9a2c9105abbdd8f5ba8295144","unresolved":true,"context_lines":[{"line_number":206,"context_line":"    dest: \"{{ workspace }}/extra_config.yaml\""},{"line_number":207,"context_line":"    content: |"},{"line_number":208,"context_line":"      {{ tcib_extra_config | to_nice_yaml }}"},{"line_number":209,"context_line":"  when: tcib_extra_config is defined"},{"line_number":210,"context_line":""},{"line_number":211,"context_line":"- name: Generate kolla building script"},{"line_number":212,"context_line":"  template:"}],"source_content_type":"text/x-yaml","patch_set":11,"id":"e25a9579_7d778935","line":209,"updated":"2022-09-01 13:26:54.000000000","message":"this is in defaults - won\u0027t it always be defined?","commit_id":"67fd2b3f44ab31a066b95d285225bc86ead36b55"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"a45e68efcb3280d9f8f26e2a35fe5e9e1a906bc5","unresolved":false,"context_lines":[{"line_number":206,"context_line":"    dest: \"{{ workspace }}/extra_config.yaml\""},{"line_number":207,"context_line":"    content: |"},{"line_number":208,"context_line":"      {{ tcib_extra_config | to_nice_yaml }}"},{"line_number":209,"context_line":"  when: tcib_extra_config is defined"},{"line_number":210,"context_line":""},{"line_number":211,"context_line":"- name: Generate kolla building script"},{"line_number":212,"context_line":"  template:"}],"source_content_type":"text/x-yaml","patch_set":11,"id":"7b5fd8d0_9b1508fc","line":209,"in_reply_to":"c93a6a29_fb1955f7","updated":"2022-09-12 05:50:52.000000000","message":"Done","commit_id":"67fd2b3f44ab31a066b95d285225bc86ead36b55"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"c0b7831de944cae79f23fc53eb875c6f905a47ec","unresolved":true,"context_lines":[{"line_number":206,"context_line":"    dest: \"{{ workspace }}/extra_config.yaml\""},{"line_number":207,"context_line":"    content: |"},{"line_number":208,"context_line":"      {{ tcib_extra_config | to_nice_yaml }}"},{"line_number":209,"context_line":"  when: tcib_extra_config is defined"},{"line_number":210,"context_line":""},{"line_number":211,"context_line":"- name: Generate kolla building script"},{"line_number":212,"context_line":"  template:"}],"source_content_type":"text/x-yaml","patch_set":11,"id":"c93a6a29_fb1955f7","line":209,"in_reply_to":"e25a9579_7d778935","updated":"2022-09-01 13:28:15.000000000","message":"yes, good catch!","commit_id":"67fd2b3f44ab31a066b95d285225bc86ead36b55"}],"roles/build-containers/templates/tripleo-build.sh.j2":[{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"314b6584c99e165b4356524fce86d6b2ef957ccd","unresolved":true,"context_lines":[{"line_number":83,"context_line":"     --label {{ item }} \\"},{"line_number":84,"context_line":"{%   endfor %}"},{"line_number":85,"context_line":"{%   endif %}"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"{%   if volume_mounts is defined %}"},{"line_number":88,"context_line":"{%   for item in volume_mounts %}"},{"line_number":89,"context_line":"     --volume {{ item }} \\"}],"source_content_type":"text/x-jinja2","patch_set":5,"id":"70787216_b17c9750","line":86,"updated":"2022-07-22 11:45:37.000000000","message":"this blank line is breaking the script","commit_id":"be3972261d034934ae0bb683796b420e30430bcf"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"47769f4afded3b5655028e6327eede86017dc43b","unresolved":false,"context_lines":[{"line_number":83,"context_line":"     --label {{ item }} \\"},{"line_number":84,"context_line":"{%   endfor %}"},{"line_number":85,"context_line":"{%   endif %}"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"{%   if volume_mounts is defined %}"},{"line_number":88,"context_line":"{%   for item in volume_mounts %}"},{"line_number":89,"context_line":"     --volume {{ item }} \\"}],"source_content_type":"text/x-jinja2","patch_set":5,"id":"0a3b7434_28dc6f9a","line":86,"in_reply_to":"70787216_b17c9750","updated":"2022-07-22 11:49:37.000000000","message":"Done","commit_id":"be3972261d034934ae0bb683796b420e30430bcf"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"0d181a61e357004bbe05cbf249b9b583e3880ef2","unresolved":false,"context_lines":[{"line_number":83,"context_line":"     --label {{ item }} \\"},{"line_number":84,"context_line":"{%   endfor %}"},{"line_number":85,"context_line":"{%   endif %}"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"{%   if volume_mounts is defined %}"},{"line_number":88,"context_line":"{%   for item in volume_mounts %}"},{"line_number":89,"context_line":"     --volume {{ item }} \\"}],"source_content_type":"text/x-jinja2","patch_set":5,"id":"4c45b486_1b5fc2f3","line":86,"in_reply_to":"70787216_b17c9750","updated":"2022-07-22 11:46:59.000000000","message":"Done","commit_id":"be3972261d034934ae0bb683796b420e30430bcf"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"fd4a5ed963dbe3c135e47c102573d0c33e9a5f81","unresolved":true,"context_lines":[{"line_number":92,"context_line":"     --tcib-extra tcib_release\u003d{{ ansible_distribution_major_version }} \\"},{"line_number":93,"context_line":"     --tcib-extra tcib_python_version\u003d{{ (ansible_distribution_major_version is version(\u00279\u0027, \u0027\u003c\u0027)) | ternary (\u00273.6\u0027, \u00273.9\u0027) }} \\"},{"line_number":94,"context_line":"{%   endif %}"},{"line_number":95,"context_line":"{% if (tcib_release_file is defined) and (release|default(branch) not  in [\"train\", \"wallaby\"]) %}"},{"line_number":96,"context_line":"     --extra-config {{ workspace }}/extra_config.yaml \\"},{"line_number":97,"context_line":"{% endif %}"},{"line_number":98,"context_line":"     --debug"}],"source_content_type":"text/x-jinja2","patch_set":14,"id":"d8e15bd8_1584b790","line":95,"range":{"start_line":95,"start_character":71,"end_line":95,"end_character":94},"updated":"2022-09-05 14:12:49.000000000","message":"the --extra-config option is also available in train and wallaby. Why are we excluding these two branches ?","commit_id":"8aa02c90d93d77525695d5c6b8c1105290f829ca"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"a45e68efcb3280d9f8f26e2a35fe5e9e1a906bc5","unresolved":false,"context_lines":[{"line_number":92,"context_line":"     --tcib-extra tcib_release\u003d{{ ansible_distribution_major_version }} \\"},{"line_number":93,"context_line":"     --tcib-extra tcib_python_version\u003d{{ (ansible_distribution_major_version is version(\u00279\u0027, \u0027\u003c\u0027)) | ternary (\u00273.6\u0027, \u00273.9\u0027) }} \\"},{"line_number":94,"context_line":"{%   endif %}"},{"line_number":95,"context_line":"{% if (tcib_release_file is defined) and (release|default(branch) not  in [\"train\", \"wallaby\"]) %}"},{"line_number":96,"context_line":"     --extra-config {{ workspace }}/extra_config.yaml \\"},{"line_number":97,"context_line":"{% endif %}"},{"line_number":98,"context_line":"     --debug"}],"source_content_type":"text/x-jinja2","patch_set":14,"id":"f01ba615_7b1b0ad2","line":95,"range":{"start_line":95,"start_character":71,"end_line":95,"end_character":94},"in_reply_to":"d8e15bd8_1584b790","updated":"2022-09-12 05:50:52.000000000","message":"Done","commit_id":"8aa02c90d93d77525695d5c6b8c1105290f829ca"}]}