)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"c7a0cea9b49fa22e4a9eee985acb858690e01594","unresolved":true,"context_lines":[{"line_number":16,"context_line":"this reproducer script."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Change-Id: I33c9b5bea949d148f21175c15e9d8522fa4297bf"},{"line_number":19,"context_line":"Resolves: rhbz#2049444"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"97a2d352_8d9799e9","line":19,"range":{"start_line":19,"start_character":0,"end_line":19,"end_character":22},"updated":"2022-02-14 14:20:13.000000000","message":"As usual I strongly discourage linking bugzilla instead of launchpad. In my understanding bugzilla is for downstream stuff and is not appropriate for upstream thing, IMHO.","commit_id":"348437dfa200dff0e45c665f0bf1a2dd65121052"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"9aadd4e04ff69ca283faa4e75fc72f21bf1467ae","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"de7669a1_e3b7c1ee","updated":"2022-02-14 06:15:27.000000000","message":"+1 to Brendan comment about doc.\nIn addition, tripleo-operator-ansible will also need an update to integrate this new parameter in the associated roles.","commit_id":"348437dfa200dff0e45c665f0bf1a2dd65121052"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"c7a0cea9b49fa22e4a9eee985acb858690e01594","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"65e56fc4_5127fcaa","updated":"2022-02-14 14:20:13.000000000","message":"I\u0027ll leave some comments here. This change is being merged but it\u0027d be nice if you can address these (except the one for commit message) in a follow-up commit.","commit_id":"348437dfa200dff0e45c665f0bf1a2dd65121052"},{"author":{"_account_id":30073,"name":"Brendan Shephard","email":"bshephar@bne-home.net","username":"bshephar"},"change_message_id":"2b8065cd74897e430588fd060727bc551fafeedd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"d24384d1_4f659479","updated":"2022-02-14 05:33:37.000000000","message":"It would be nice if we can document this new feature as well with an accompanying change:\n\nhttps://docs.openstack.org/project-deploy-guide/tripleo-docs/latest/deployment/install_undercloud.html\nhttps://github.com/openstack/tripleo-docs/blob/master/deploy-guide/source/deployment/install_undercloud.rst\n\n","commit_id":"348437dfa200dff0e45c665f0bf1a2dd65121052"},{"author":{"_account_id":30073,"name":"Brendan Shephard","email":"bshephar@bne-home.net","username":"bshephar"},"change_message_id":"c4f3068bc96d5cdf110e2d120e7db89c4e0f8cf7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"67b8a519_6afa8881","updated":"2022-02-14 05:29:58.000000000","message":"Tested this, works as described. Nice addition that will hopefully help more people debug their undercloud install and upgrades:\n\n```\n[stack@undercloud-0 undercloud]$ find . -name ansible-playbook-command.sh 2\u003e/dev/null\n./undercloud-ansible-q_f1g6pd/ansible-playbook-command.sh\n\n[stack@undercloud-0 undercloud]$ find . -name ansible-playbook-command.sh -exec tail -n3 {} + 2\u003e/dev/null\nexport ANSIBLE_CONFIG\u003d\"/home/stack/tripleo-deploy/undercloud/undercloud-ansible-q_f1g6pd/ansible.cfg\"\necho -e \"Running Ansible command\"\nansible-playbook -i /home/stack/tripleo-deploy/undercloud/undercloud-ansible-q_f1g6pd/inventory.yaml --skip-tags opendev-validation deploy_steps_playbook.yaml \"$@\"\n```","commit_id":"348437dfa200dff0e45c665f0bf1a2dd65121052"},{"author":{"_account_id":26343,"name":"Jose Luis Franco","email":"jfrancoa@redhat.com","username":"jfrancoa"},"change_message_id":"07685b13f7509d19fdcf17a0414fbcd5f0ae3499","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"dc9f19b7_e8c00ecc","updated":"2022-02-11 10:06:45.000000000","message":"recheck","commit_id":"348437dfa200dff0e45c665f0bf1a2dd65121052"},{"author":{"_account_id":26343,"name":"Jose Luis Franco","email":"jfrancoa@redhat.com","username":"jfrancoa"},"change_message_id":"55242dfbc6b1bcb5458e7af8291bc27ad15dac00","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"a7c32175_22fd09eb","in_reply_to":"65e56fc4_5127fcaa","updated":"2022-02-14 15:33:13.000000000","message":"Done! https://review.opendev.org/c/openstack/python-tripleoclient/+/829060\n\nThanks for the comments.","commit_id":"348437dfa200dff0e45c665f0bf1a2dd65121052"},{"author":{"_account_id":26343,"name":"Jose Luis Franco","email":"jfrancoa@redhat.com","username":"jfrancoa"},"change_message_id":"d2a8fcbd1ffddf603cacdf7de80017bec3035bac","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"6af4efa6_aeb78cc0","in_reply_to":"de7669a1_e3b7c1ee","updated":"2022-02-14 06:38:37.000000000","message":"Done! https://review.opendev.org/c/openstack/tripleo-docs/+/828998 and https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/828999\n\nThanks for the comments folks.","commit_id":"348437dfa200dff0e45c665f0bf1a2dd65121052"}],"tripleoclient/v1/undercloud.py":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"c7a0cea9b49fa22e4a9eee985acb858690e01594","unresolved":true,"context_lines":[{"line_number":132,"context_line":"                   \u0027action may fail.\u0027)"},{"line_number":133,"context_line":"        )"},{"line_number":134,"context_line":"        parser.add_argument("},{"line_number":135,"context_line":"            \u0027--reproduce-command\u0027,"},{"line_number":136,"context_line":"            action\u003d\u0027store_true\u0027,"},{"line_number":137,"context_line":"            default\u003dFalse,"},{"line_number":138,"context_line":"            help\u003d_(\u0027Create a reproducer command with ansible command\u0027"}],"source_content_type":"text/x-python","patch_set":6,"id":"50150441_e60fa887","line":135,"range":{"start_line":135,"start_character":13,"end_line":135,"end_character":32},"updated":"2022-02-14 14:20:13.000000000","message":"Can we add a release note about this new feature ?","commit_id":"348437dfa200dff0e45c665f0bf1a2dd65121052"}],"tripleoclient/v1/undercloud_config.py":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"c7a0cea9b49fa22e4a9eee985acb858690e01594","unresolved":true,"context_lines":[{"line_number":433,"context_line":"                              verbose_level\u003d1, yes\u003dFalse,"},{"line_number":434,"context_line":"                              force_stack_update\u003dFalse, dry_run\u003dFalse,"},{"line_number":435,"context_line":"                              inflight\u003dFalse,"},{"line_number":436,"context_line":"                              reproducer\u003dFalse,"},{"line_number":437,"context_line":"                              disable_container_prepare\u003dFalse):"},{"line_number":438,"context_line":"    \"\"\"Prepare Undercloud deploy command based on undercloud.conf\"\"\""},{"line_number":439,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"94963026_5a207a43","line":436,"range":{"start_line":436,"start_character":30,"end_line":436,"end_character":40},"updated":"2022-02-14 14:20:13.000000000","message":"Can we use the consistent name, reproduce_command instead ?","commit_id":"348437dfa200dff0e45c665f0bf1a2dd65121052"}]}