)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":31668,"deleted":true},"change_message_id":"1cecf6e51e57999f739cf9fd73bced9225b4a7d6","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Ken Fujimoto \u003cfujimoto@valinux.co.jp\u003e"},{"line_number":5,"context_line":"CommitDate: 2023-01-18 02:09:08 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Placement enhancement enable to retry AZ selection"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch enable availability zone reselected by stack retry"},{"line_number":10,"context_line":"when \"instantiate\" \"scale\" \"heal\" operations in v2 API fails"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":15,"id":"4302d060_f51882da","line":7,"updated":"2023-01-25 09:55:01.000000000","message":"typo: enable -\u003e enables","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"fddf6d33a0c1889e6d89ca1487e3de9cae60d614","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Ken Fujimoto \u003cfujimoto@valinux.co.jp\u003e"},{"line_number":5,"context_line":"CommitDate: 2023-01-18 02:09:08 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Placement enhancement enable to retry AZ selection"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch enable availability zone reselected by stack retry"},{"line_number":10,"context_line":"when \"instantiate\" \"scale\" \"heal\" operations in v2 API fails"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":15,"id":"a2fc4735_65e8a979","line":7,"in_reply_to":"4302d060_f51882da","updated":"2023-01-26 04:05:28.000000000","message":"I will fix.","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"d72660d82a43f079fa7367212d5bfbe1f7d82fa4","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Ken Fujimoto \u003cfujimoto@valinux.co.jp\u003e"},{"line_number":5,"context_line":"CommitDate: 2023-01-18 02:09:08 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Placement enhancement enable to retry AZ selection"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch enable availability zone reselected by stack retry"},{"line_number":10,"context_line":"when \"instantiate\" \"scale\" \"heal\" operations in v2 API fails"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":15,"id":"63d7789b_9fecbee9","line":7,"in_reply_to":"a2fc4735_65e8a979","updated":"2023-01-31 07:45:42.000000000","message":"Done","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"1cecf6e51e57999f739cf9fd73bced9225b4a7d6","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Placement enhancement enable to retry AZ selection"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch enable availability zone reselected by stack retry"},{"line_number":10,"context_line":"when \"instantiate\" \"scale\" \"heal\" operations in v2 API fails"},{"line_number":11,"context_line":"due to availability zone is unavailable."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":15,"id":"2b8006cd_23fb9b26","line":9,"updated":"2023-01-25 09:55:01.000000000","message":"typo: enable -\u003e enables","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"fddf6d33a0c1889e6d89ca1487e3de9cae60d614","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Placement enhancement enable to retry AZ selection"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch enable availability zone reselected by stack retry"},{"line_number":10,"context_line":"when \"instantiate\" \"scale\" \"heal\" operations in v2 API fails"},{"line_number":11,"context_line":"due to availability zone is unavailable."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":15,"id":"88cd7df5_35c3a26f","line":9,"in_reply_to":"2b8006cd_23fb9b26","updated":"2023-01-26 04:05:28.000000000","message":"I will fix.","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"d72660d82a43f079fa7367212d5bfbe1f7d82fa4","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Placement enhancement enable to retry AZ selection"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch enable availability zone reselected by stack retry"},{"line_number":10,"context_line":"when \"instantiate\" \"scale\" \"heal\" operations in v2 API fails"},{"line_number":11,"context_line":"due to availability zone is unavailable."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":15,"id":"9a1a0fef_2d1d7de2","line":9,"in_reply_to":"88cd7df5_35c3a26f","updated":"2023-01-31 07:45:42.000000000","message":"Done","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"1cecf6e51e57999f739cf9fd73bced9225b4a7d6","unresolved":true,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Note that precondition of using these functions is to use"},{"line_number":14,"context_line":"StandardUserData as UserData class."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: Icc9eb7a8fffbf35d2e005a9703dcefe66f097584"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":15,"id":"1f816f77_6a229e95","line":15,"updated":"2023-01-25 09:55:01.000000000","message":"Please add the following.\nImplements: blueprint enhance-placement\n\nAlso, please change the Topic to the following.\nbp/enhance-placement","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"fddf6d33a0c1889e6d89ca1487e3de9cae60d614","unresolved":true,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Note that precondition of using these functions is to use"},{"line_number":14,"context_line":"StandardUserData as UserData class."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: Icc9eb7a8fffbf35d2e005a9703dcefe66f097584"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":15,"id":"a62a6bbe_d01540ec","line":15,"in_reply_to":"1f816f77_6a229e95","updated":"2023-01-26 04:05:28.000000000","message":"I will add. Also I will change the Topic.","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"d72660d82a43f079fa7367212d5bfbe1f7d82fa4","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Note that precondition of using these functions is to use"},{"line_number":14,"context_line":"StandardUserData as UserData class."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: Icc9eb7a8fffbf35d2e005a9703dcefe66f097584"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":15,"id":"2bbb5ace_6345537b","line":15,"in_reply_to":"a62a6bbe_d01540ec","updated":"2023-01-31 07:45:42.000000000","message":"Done","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"fa709a98408baa0c86e80d00f9c63fcc0d75f512","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":13,"id":"fbda9382_17a85b71","updated":"2022-12-28 01:50:58.000000000","message":"Thank you for the review.","commit_id":"166ce18bacb6d1aac7b4335bc219e4911f88f337"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"3a6ed3272d192c9ddb846c1aec7061bae9d4a402","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"56034181_b166e36d","updated":"2023-01-10 05:32:14.000000000","message":"Thank you for your rework. LGTM.","commit_id":"8a6644fcdfae005dc052c3be85ef5fa826fc1ede"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"a62fb635346ed1368d70401e17e17662e2053b99","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":15,"id":"067d4cd4_8ef3176d","updated":"2023-01-26 05:47:18.000000000","message":"Thank you for the review.","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"fddf6d33a0c1889e6d89ca1487e3de9cae60d614","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":15,"id":"7566c59d_615dde69","updated":"2023-01-26 04:05:28.000000000","message":"Thank you for the review.","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"5afd8d6c7e57941a0c7812e22158b283ff62f103","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"5581bc92_cbaa5542","updated":"2023-01-27 11:51:17.000000000","message":"Thank you for the patch.\n\nPlease kindly find my comments.","commit_id":"17b6a1ab8f79ec95f6014ab4c03422b67adffcd9"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"4734a4b67ea4c7e9c40eb0d54c59d4d331a4233f","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":16,"id":"c62ea47f_7aad3d98","updated":"2023-01-30 06:32:15.000000000","message":"Thank you for the review.","commit_id":"17b6a1ab8f79ec95f6014ab4c03422b67adffcd9"},{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"a84fff5e020633c8f359096a52da968123fc14b4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"5f347b16_d342635d","updated":"2023-01-27 12:56:21.000000000","message":"Thanks for your patch. Please kindly check my comments.","commit_id":"17b6a1ab8f79ec95f6014ab4c03422b67adffcd9"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"c2e6ecf14afcce5535cb39d3c3b40e7da0da8fd6","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":17,"id":"14870448_20a6ae76","updated":"2023-02-01 03:49:46.000000000","message":"Thank you for the review.","commit_id":"cb069e5dbbcb3b6df6ecb35c43bcc732c40d1453"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"8cf2ca37f8c4d2293e32b6d180508ff88ddf2c7f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":18,"id":"792624d9_f6103224","updated":"2023-02-02 07:51:38.000000000","message":"Thank you for the review.","commit_id":"dbc01d134dee5049efb91351bc34fb4aefa8ed19"},{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"147be22d20092060822e75ead2e6e61b6bf3cdd5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":22,"id":"9e0ad381_58abfb60","updated":"2023-02-13 07:06:37.000000000","message":"Thank you for re-work! I added nits comment, so please kindly check it.","commit_id":"12c5aec784f0b386d990cd66285876d3f575952f"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"14d590d0253a21e42bb30969cf84c8efe64c5257","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":22,"id":"a01d69b5_a6b65e34","updated":"2023-02-13 07:30:23.000000000","message":"Thank you for the review.","commit_id":"12c5aec784f0b386d990cd66285876d3f575952f"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"03a49e09f16413c71b9ffa59ec7fba7a761eb417","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":23,"id":"d1cdd747_4711fb81","updated":"2023-02-15 23:30:15.000000000","message":"LGTM","commit_id":"124c9d1d86bed2f2439ad4dec92343304586fa01"},{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"90b4ba5c2f81c83c695509e15d09910493ee0973","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":23,"id":"8645b194_5b1c8dc3","updated":"2023-02-16 00:29:03.000000000","message":"Thanks for your re-work! LGTM","commit_id":"124c9d1d86bed2f2439ad4dec92343304586fa01"},{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"053e4f57fcbb36d9e5ae9921bf42dbe0876f672a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":24,"id":"9ab42c60_7e6c0d14","updated":"2023-02-17 00:08:04.000000000","message":"Hi, Cores. Please kindly review the patch. Thanks.","commit_id":"25459c257161ab2c408553052e85019d28caa3df"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"a84870091532b56c3499917e4f71dece62770920","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":24,"id":"20451927_47b7fb0b","updated":"2023-02-20 01:54:45.000000000","message":"Thanks for your patch, LGTM.","commit_id":"25459c257161ab2c408553052e85019d28caa3df"}],"roles/setup-multi-az/defaults/main.yaml":[{"author":{"_account_id":31668,"deleted":true},"change_message_id":"d72660d82a43f079fa7367212d5bfbe1f7d82fa4","unresolved":true,"context_lines":[{"line_number":1,"context_line":"aggregate_name: aggregate-1"},{"line_number":2,"context_line":"zone_name: az-1"},{"line_number":3,"context_line":"flavor_name: sample4G"}],"source_content_type":"text/x-yaml","patch_set":17,"id":"f14c3b23_440200a6","line":3,"updated":"2023-01-31 07:45:42.000000000","message":"How about using m1.medium(RAM:4096/Disk:20/VCPUs:2)?\nShould the Disk value be 4 instead of 20?","commit_id":"cb069e5dbbcb3b6df6ecb35c43bcc732c40d1453"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"2e3382b5c6c702ca4f101235799a12c43facc22d","unresolved":false,"context_lines":[{"line_number":1,"context_line":"aggregate_name: aggregate-1"},{"line_number":2,"context_line":"zone_name: az-1"},{"line_number":3,"context_line":"flavor_name: sample4G"}],"source_content_type":"text/x-yaml","patch_set":17,"id":"fe7c3997_a3e12a33","line":3,"in_reply_to":"5a86de8a_56983199","updated":"2023-02-02 02:17:02.000000000","message":"Thank you for your explanation.\nI understand.","commit_id":"cb069e5dbbcb3b6df6ecb35c43bcc732c40d1453"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"c2e6ecf14afcce5535cb39d3c3b40e7da0da8fd6","unresolved":true,"context_lines":[{"line_number":1,"context_line":"aggregate_name: aggregate-1"},{"line_number":2,"context_line":"zone_name: az-1"},{"line_number":3,"context_line":"flavor_name: sample4G"}],"source_content_type":"text/x-yaml","patch_set":17,"id":"5a86de8a_56983199","line":3,"in_reply_to":"f14c3b23_440200a6","updated":"2023-02-01 03:49:46.000000000","message":"m1.medium is fine, but I wanted to cause a memory resource shortage, so I created the flavor with a smaller Disk size.","commit_id":"cb069e5dbbcb3b6df6ecb35c43bcc732c40d1453"}],"roles/setup-multi-az/tasks/main.yaml":[{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"a84fff5e020633c8f359096a52da968123fc14b4","unresolved":true,"context_lines":[{"line_number":11,"context_line":"  - name: Create OpenStack flavor"},{"line_number":12,"context_line":"    shell: |"},{"line_number":13,"context_line":"       openstack --os-cloud devstack-admin \\"},{"line_number":14,"context_line":"       flavor create --ram 4096 --disk 4 --vcpus 2 {{ flavor_name }}"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"  when:"},{"line_number":17,"context_line":"    - inventory_hostname \u003d\u003d \u0027controller\u0027"}],"source_content_type":"text/x-yaml","patch_set":15,"id":"86f0e5b4_d68cce1f","line":14,"updated":"2023-01-27 12:56:21.000000000","message":"Please add Note that why the ram needs 4096.\nI think that this is to generate an insufficient resource error, and it is useful for developers if the compute resources of Zuul envs will change in the future.","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"4734a4b67ea4c7e9c40eb0d54c59d4d331a4233f","unresolved":true,"context_lines":[{"line_number":11,"context_line":"  - name: Create OpenStack flavor"},{"line_number":12,"context_line":"    shell: |"},{"line_number":13,"context_line":"       openstack --os-cloud devstack-admin \\"},{"line_number":14,"context_line":"       flavor create --ram 4096 --disk 4 --vcpus 2 {{ flavor_name }}"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"  when:"},{"line_number":17,"context_line":"    - inventory_hostname \u003d\u003d \u0027controller\u0027"}],"source_content_type":"text/x-yaml","patch_set":15,"id":"b296fc0b_dd782f75","line":14,"in_reply_to":"86f0e5b4_d68cce1f","updated":"2023-01-30 06:32:15.000000000","message":"I wil add comments.","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"147be22d20092060822e75ead2e6e61b6bf3cdd5","unresolved":false,"context_lines":[{"line_number":11,"context_line":"  - name: Create OpenStack flavor"},{"line_number":12,"context_line":"    shell: |"},{"line_number":13,"context_line":"       openstack --os-cloud devstack-admin \\"},{"line_number":14,"context_line":"       flavor create --ram 4096 --disk 4 --vcpus 2 {{ flavor_name }}"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"  when:"},{"line_number":17,"context_line":"    - inventory_hostname \u003d\u003d \u0027controller\u0027"}],"source_content_type":"text/x-yaml","patch_set":15,"id":"20cd3d5b_15aa7083","line":14,"in_reply_to":"b296fc0b_dd782f75","updated":"2023-02-13 07:06:37.000000000","message":"Done","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"}],"tacker/sol_refactored/common/config.py":[{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"9927ca63510ab668438d5bb6b256d2371daaf31b","unresolved":true,"context_lines":[{"line_number":34,"context_line":"               help\u003d_(\u0027Max content length for list APIs.\u0027)),"},{"line_number":35,"context_line":"    cfg.IntOpt(\u0027openstack_vim_stack_create_timeout\u0027,"},{"line_number":36,"context_line":"               default\u003d20,"},{"line_number":37,"context_line":"               help\u003d_(\u0027Timeout (in minuts) of heat stack creation.\u0027)),"},{"line_number":38,"context_line":"    cfg.IntOpt(\u0027kubernetes_vim_rsc_wait_timeout\u0027,"},{"line_number":39,"context_line":"               default\u003d500,"},{"line_number":40,"context_line":"               help\u003d_(\u0027Timeout (second) of k8s res creation.\u0027)),"}],"source_content_type":"text/x-python","patch_set":13,"id":"574c5a2d_d4de78e1","line":37,"range":{"start_line":37,"start_character":35,"end_line":37,"end_character":41},"updated":"2022-12-27 05:50:39.000000000","message":"Could you please fix the typo?\nminuts \u003d\u003e minutes","commit_id":"166ce18bacb6d1aac7b4335bc219e4911f88f337"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"3a6ed3272d192c9ddb846c1aec7061bae9d4a402","unresolved":false,"context_lines":[{"line_number":34,"context_line":"               help\u003d_(\u0027Max content length for list APIs.\u0027)),"},{"line_number":35,"context_line":"    cfg.IntOpt(\u0027openstack_vim_stack_create_timeout\u0027,"},{"line_number":36,"context_line":"               default\u003d20,"},{"line_number":37,"context_line":"               help\u003d_(\u0027Timeout (in minuts) of heat stack creation.\u0027)),"},{"line_number":38,"context_line":"    cfg.IntOpt(\u0027kubernetes_vim_rsc_wait_timeout\u0027,"},{"line_number":39,"context_line":"               default\u003d500,"},{"line_number":40,"context_line":"               help\u003d_(\u0027Timeout (second) of k8s res creation.\u0027)),"}],"source_content_type":"text/x-python","patch_set":13,"id":"2a3eeef0_4a3a7fc1","line":37,"range":{"start_line":37,"start_character":35,"end_line":37,"end_character":41},"in_reply_to":"328991ba_0369b80b","updated":"2023-01-10 05:32:14.000000000","message":"Done","commit_id":"166ce18bacb6d1aac7b4335bc219e4911f88f337"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"fa709a98408baa0c86e80d00f9c63fcc0d75f512","unresolved":true,"context_lines":[{"line_number":34,"context_line":"               help\u003d_(\u0027Max content length for list APIs.\u0027)),"},{"line_number":35,"context_line":"    cfg.IntOpt(\u0027openstack_vim_stack_create_timeout\u0027,"},{"line_number":36,"context_line":"               default\u003d20,"},{"line_number":37,"context_line":"               help\u003d_(\u0027Timeout (in minuts) of heat stack creation.\u0027)),"},{"line_number":38,"context_line":"    cfg.IntOpt(\u0027kubernetes_vim_rsc_wait_timeout\u0027,"},{"line_number":39,"context_line":"               default\u003d500,"},{"line_number":40,"context_line":"               help\u003d_(\u0027Timeout (second) of k8s res creation.\u0027)),"}],"source_content_type":"text/x-python","patch_set":13,"id":"328991ba_0369b80b","line":37,"range":{"start_line":37,"start_character":35,"end_line":37,"end_character":41},"in_reply_to":"574c5a2d_d4de78e1","updated":"2022-12-28 01:50:58.000000000","message":"I will fix.","commit_id":"166ce18bacb6d1aac7b4335bc219e4911f88f337"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"1cecf6e51e57999f739cf9fd73bced9225b4a7d6","unresolved":true,"context_lines":[{"line_number":67,"context_line":"    cfg.BoolOpt(\u0027placement_fallback_best_effort\u0027,"},{"line_number":68,"context_line":"               default\u003dFalse,"},{"line_number":69,"context_line":"               help\u003d_(\u0027If True, fallbackBestEffort setting is enabled \u0027"},{"line_number":70,"context_line":"                      \u0027and run a Availability Zone reselection.\u0027)),"},{"line_number":71,"context_line":"    cfg.IntOpt(\u0027placement_az_select_retry\u0027,"},{"line_number":72,"context_line":"               default\u003d0,  # 0 means unlimited number of retries"},{"line_number":73,"context_line":"               help\u003d_(\u0027Number of retries to reselect Availability Zone.\u0027)),"}],"source_content_type":"text/x-python","patch_set":15,"id":"ae088be0_d66d9a7a","line":70,"updated":"2023-01-25 09:55:01.000000000","message":"It is better not to have \"a\" to match the others.\na Availability Zone reselection -\u003e Availability Zone reselection","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"d72660d82a43f079fa7367212d5bfbe1f7d82fa4","unresolved":false,"context_lines":[{"line_number":67,"context_line":"    cfg.BoolOpt(\u0027placement_fallback_best_effort\u0027,"},{"line_number":68,"context_line":"               default\u003dFalse,"},{"line_number":69,"context_line":"               help\u003d_(\u0027If True, fallbackBestEffort setting is enabled \u0027"},{"line_number":70,"context_line":"                      \u0027and run a Availability Zone reselection.\u0027)),"},{"line_number":71,"context_line":"    cfg.IntOpt(\u0027placement_az_select_retry\u0027,"},{"line_number":72,"context_line":"               default\u003d0,  # 0 means unlimited number of retries"},{"line_number":73,"context_line":"               help\u003d_(\u0027Number of retries to reselect Availability Zone.\u0027)),"}],"source_content_type":"text/x-python","patch_set":15,"id":"2918e513_f6280395","line":70,"in_reply_to":"69a6d1e5_7eeb1c68","updated":"2023-01-31 07:45:42.000000000","message":"Done","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"fddf6d33a0c1889e6d89ca1487e3de9cae60d614","unresolved":true,"context_lines":[{"line_number":67,"context_line":"    cfg.BoolOpt(\u0027placement_fallback_best_effort\u0027,"},{"line_number":68,"context_line":"               default\u003dFalse,"},{"line_number":69,"context_line":"               help\u003d_(\u0027If True, fallbackBestEffort setting is enabled \u0027"},{"line_number":70,"context_line":"                      \u0027and run a Availability Zone reselection.\u0027)),"},{"line_number":71,"context_line":"    cfg.IntOpt(\u0027placement_az_select_retry\u0027,"},{"line_number":72,"context_line":"               default\u003d0,  # 0 means unlimited number of retries"},{"line_number":73,"context_line":"               help\u003d_(\u0027Number of retries to reselect Availability Zone.\u0027)),"}],"source_content_type":"text/x-python","patch_set":15,"id":"69a6d1e5_7eeb1c68","line":70,"in_reply_to":"ae088be0_d66d9a7a","updated":"2023-01-26 04:05:28.000000000","message":"I will fix.","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"a84fff5e020633c8f359096a52da968123fc14b4","unresolved":true,"context_lines":[{"line_number":77,"context_line":"                        r\u0027Resource UPDATE failed: resources\\.(.*): \u0027"},{"line_number":78,"context_line":"                        r\u0027Resource CREATE failed: ResourceInError: \u0027"},{"line_number":79,"context_line":"                        r\u0027resources\\.(.*): (.*)\u0027),"},{"line_number":80,"context_line":"               help\u003d_(\u0027Error message for Availability Zone reselection.\u0027)),"},{"line_number":81,"context_line":"    # NOTE: This is for test use since it is convenient to be able to delete"},{"line_number":82,"context_line":"    # under development."},{"line_number":83,"context_line":"    cfg.BoolOpt(\u0027test_enable_lcm_op_occ_delete\u0027,"}],"source_content_type":"text/x-python","patch_set":15,"id":"29f3e989_ba08dc12","line":80,"range":{"start_line":80,"start_character":23,"end_line":80,"end_character":71},"updated":"2023-01-27 12:56:21.000000000","message":"Please add `These configs are regular expressions to detect error messages from OpenStack Heat.` to the help message.","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"4734a4b67ea4c7e9c40eb0d54c59d4d331a4233f","unresolved":true,"context_lines":[{"line_number":77,"context_line":"                        r\u0027Resource UPDATE failed: resources\\.(.*): \u0027"},{"line_number":78,"context_line":"                        r\u0027Resource CREATE failed: ResourceInError: \u0027"},{"line_number":79,"context_line":"                        r\u0027resources\\.(.*): (.*)\u0027),"},{"line_number":80,"context_line":"               help\u003d_(\u0027Error message for Availability Zone reselection.\u0027)),"},{"line_number":81,"context_line":"    # NOTE: This is for test use since it is convenient to be able to delete"},{"line_number":82,"context_line":"    # under development."},{"line_number":83,"context_line":"    cfg.BoolOpt(\u0027test_enable_lcm_op_occ_delete\u0027,"}],"source_content_type":"text/x-python","patch_set":15,"id":"b51a6fd2_3b9ddd4a","line":80,"range":{"start_line":80,"start_character":23,"end_line":80,"end_character":71},"in_reply_to":"29f3e989_ba08dc12","updated":"2023-01-30 06:32:15.000000000","message":"I will add a message.","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"147be22d20092060822e75ead2e6e61b6bf3cdd5","unresolved":false,"context_lines":[{"line_number":77,"context_line":"                        r\u0027Resource UPDATE failed: resources\\.(.*): \u0027"},{"line_number":78,"context_line":"                        r\u0027Resource CREATE failed: ResourceInError: \u0027"},{"line_number":79,"context_line":"                        r\u0027resources\\.(.*): (.*)\u0027),"},{"line_number":80,"context_line":"               help\u003d_(\u0027Error message for Availability Zone reselection.\u0027)),"},{"line_number":81,"context_line":"    # NOTE: This is for test use since it is convenient to be able to delete"},{"line_number":82,"context_line":"    # under development."},{"line_number":83,"context_line":"    cfg.BoolOpt(\u0027test_enable_lcm_op_occ_delete\u0027,"}],"source_content_type":"text/x-python","patch_set":15,"id":"0330b6ff_05ca5464","line":80,"range":{"start_line":80,"start_character":23,"end_line":80,"end_character":71},"in_reply_to":"b51a6fd2_3b9ddd4a","updated":"2023-02-13 07:06:37.000000000","message":"Done","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"2e3382b5c6c702ca4f101235799a12c43facc22d","unresolved":true,"context_lines":[{"line_number":77,"context_line":"                        r\u0027Resource UPDATE failed: resources\\.(.*): \u0027"},{"line_number":78,"context_line":"                        r\u0027Resource CREATE failed: ResourceInError: \u0027"},{"line_number":79,"context_line":"                        r\u0027resources\\.(.*): (.*)\u0027),"},{"line_number":80,"context_line":"               help\u003d_(\u0027Error message for Availability Zone reselection.\u0027"},{"line_number":81,"context_line":"                      \u0027These configs are regular expressions to detect \u0027"},{"line_number":82,"context_line":"                      \u0027error messages from OpenStack Heat.\u0027)),"},{"line_number":83,"context_line":"    # NOTE: This is for test use since it is convenient to be able to delete"}],"source_content_type":"text/x-python","patch_set":18,"id":"583721ab_c6e03b15","line":80,"updated":"2023-02-02 02:17:02.000000000","message":"It is better to add a space at the end of the sentence.","commit_id":"dbc01d134dee5049efb91351bc34fb4aefa8ed19"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"79e99466068b89b3557a5ef41f1e9b0dd8c70036","unresolved":false,"context_lines":[{"line_number":77,"context_line":"                        r\u0027Resource UPDATE failed: resources\\.(.*): \u0027"},{"line_number":78,"context_line":"                        r\u0027Resource CREATE failed: ResourceInError: \u0027"},{"line_number":79,"context_line":"                        r\u0027resources\\.(.*): (.*)\u0027),"},{"line_number":80,"context_line":"               help\u003d_(\u0027Error message for Availability Zone reselection.\u0027"},{"line_number":81,"context_line":"                      \u0027These configs are regular expressions to detect \u0027"},{"line_number":82,"context_line":"                      \u0027error messages from OpenStack Heat.\u0027)),"},{"line_number":83,"context_line":"    # NOTE: This is for test use since it is convenient to be able to delete"}],"source_content_type":"text/x-python","patch_set":18,"id":"af31bd07_45742744","line":80,"in_reply_to":"1aa710ca_122c93d6","updated":"2023-02-03 02:12:54.000000000","message":"Done","commit_id":"dbc01d134dee5049efb91351bc34fb4aefa8ed19"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"8cf2ca37f8c4d2293e32b6d180508ff88ddf2c7f","unresolved":true,"context_lines":[{"line_number":77,"context_line":"                        r\u0027Resource UPDATE failed: resources\\.(.*): \u0027"},{"line_number":78,"context_line":"                        r\u0027Resource CREATE failed: ResourceInError: \u0027"},{"line_number":79,"context_line":"                        r\u0027resources\\.(.*): (.*)\u0027),"},{"line_number":80,"context_line":"               help\u003d_(\u0027Error message for Availability Zone reselection.\u0027"},{"line_number":81,"context_line":"                      \u0027These configs are regular expressions to detect \u0027"},{"line_number":82,"context_line":"                      \u0027error messages from OpenStack Heat.\u0027)),"},{"line_number":83,"context_line":"    # NOTE: This is for test use since it is convenient to be able to delete"}],"source_content_type":"text/x-python","patch_set":18,"id":"1aa710ca_122c93d6","line":80,"in_reply_to":"583721ab_c6e03b15","updated":"2023-02-02 07:51:38.000000000","message":"I will fix.","commit_id":"dbc01d134dee5049efb91351bc34fb4aefa8ed19"},{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"147be22d20092060822e75ead2e6e61b6bf3cdd5","unresolved":true,"context_lines":[{"line_number":70,"context_line":"                      \u0027and run Availability Zone reselection.\u0027)),"},{"line_number":71,"context_line":"    cfg.IntOpt(\u0027placement_az_select_retry\u0027,"},{"line_number":72,"context_line":"               default\u003d0,  # 0 means unlimited number of retries"},{"line_number":73,"context_line":"               help\u003d_(\u0027Number of retries to reselect Availability Zone.\u0027)),"},{"line_number":74,"context_line":"    cfg.StrOpt(\u0027placement_az_resource_error\u0027,"},{"line_number":75,"context_line":"               default\u003d(r\u0027Resource CREATE failed: ResourceInError: \u0027"},{"line_number":76,"context_line":"                        r\u0027resources\\.(.*)\\.(.*): (.*)|\u0027"}],"source_content_type":"text/x-python","patch_set":21,"id":"b2b1caa8_8211f898","line":73,"updated":"2023-02-13 07:06:37.000000000","message":"I think that default value of \"0\" means specially to execute the function by unlimited number of retries, not to execute the function.\nTherefore, please add `Default value \"0\" means unlimited number of retries` to the help message.\n(I understood you wrote the comment in the codes, but it is more helpful for users to add the help message.)","commit_id":"0309368d3ab774d2370fef8e74f9c0c0a3c44140"},{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"90b4ba5c2f81c83c695509e15d09910493ee0973","unresolved":false,"context_lines":[{"line_number":70,"context_line":"                      \u0027and run Availability Zone reselection.\u0027)),"},{"line_number":71,"context_line":"    cfg.IntOpt(\u0027placement_az_select_retry\u0027,"},{"line_number":72,"context_line":"               default\u003d0,  # 0 means unlimited number of retries"},{"line_number":73,"context_line":"               help\u003d_(\u0027Number of retries to reselect Availability Zone.\u0027)),"},{"line_number":74,"context_line":"    cfg.StrOpt(\u0027placement_az_resource_error\u0027,"},{"line_number":75,"context_line":"               default\u003d(r\u0027Resource CREATE failed: ResourceInError: \u0027"},{"line_number":76,"context_line":"                        r\u0027resources\\.(.*)\\.(.*): (.*)|\u0027"}],"source_content_type":"text/x-python","patch_set":21,"id":"ffdc43b2_6ba10544","line":73,"in_reply_to":"3984090c_3e3220dc","updated":"2023-02-16 00:29:03.000000000","message":"Done","commit_id":"0309368d3ab774d2370fef8e74f9c0c0a3c44140"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"14d590d0253a21e42bb30969cf84c8efe64c5257","unresolved":true,"context_lines":[{"line_number":70,"context_line":"                      \u0027and run Availability Zone reselection.\u0027)),"},{"line_number":71,"context_line":"    cfg.IntOpt(\u0027placement_az_select_retry\u0027,"},{"line_number":72,"context_line":"               default\u003d0,  # 0 means unlimited number of retries"},{"line_number":73,"context_line":"               help\u003d_(\u0027Number of retries to reselect Availability Zone.\u0027)),"},{"line_number":74,"context_line":"    cfg.StrOpt(\u0027placement_az_resource_error\u0027,"},{"line_number":75,"context_line":"               default\u003d(r\u0027Resource CREATE failed: ResourceInError: \u0027"},{"line_number":76,"context_line":"                        r\u0027resources\\.(.*)\\.(.*): (.*)|\u0027"}],"source_content_type":"text/x-python","patch_set":21,"id":"3984090c_3e3220dc","line":73,"in_reply_to":"b2b1caa8_8211f898","updated":"2023-02-13 07:30:23.000000000","message":"I see. I will add.","commit_id":"0309368d3ab774d2370fef8e74f9c0c0a3c44140"}],"tacker/sol_refactored/infra_drivers/openstack/openstack.py":[{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"9927ca63510ab668438d5bb6b256d2371daaf31b","unresolved":true,"context_lines":[{"line_number":152,"context_line":"    def _check_and_get_failed_zone(self, ex_detail, vdu_dict):"},{"line_number":153,"context_line":"        if re.match(CONF.v2_vnfm.placement_az_resource_error, ex_detail):"},{"line_number":154,"context_line":"            match_result \u003d re.search(r\u0027resources\\.((.*)-([0-9]+))\u0027, ex_detail)"},{"line_number":155,"context_line":"            vdu_id \u003d match_result.group(1)"},{"line_number":156,"context_line":"            return vdu_dict.get(vdu_id, {}).get(\u0027locationConstraints\u0027)"},{"line_number":157,"context_line":""},{"line_number":158,"context_line":"    def _get_vdu_id_from_fields(self, fields):"}],"source_content_type":"text/x-python","patch_set":13,"id":"f73cf49e_70475149","line":155,"updated":"2022-12-27 05:50:39.000000000","message":"\"placement_az_resource_error\" is user specified.\nAn error occurs if \"resources\\.\" is not included in \"ex_detail\".\nI think validation is necessary before processing.","commit_id":"166ce18bacb6d1aac7b4335bc219e4911f88f337"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"3a6ed3272d192c9ddb846c1aec7061bae9d4a402","unresolved":false,"context_lines":[{"line_number":152,"context_line":"    def _check_and_get_failed_zone(self, ex_detail, vdu_dict):"},{"line_number":153,"context_line":"        if re.match(CONF.v2_vnfm.placement_az_resource_error, ex_detail):"},{"line_number":154,"context_line":"            match_result \u003d re.search(r\u0027resources\\.((.*)-([0-9]+))\u0027, ex_detail)"},{"line_number":155,"context_line":"            vdu_id \u003d match_result.group(1)"},{"line_number":156,"context_line":"            return vdu_dict.get(vdu_id, {}).get(\u0027locationConstraints\u0027)"},{"line_number":157,"context_line":""},{"line_number":158,"context_line":"    def _get_vdu_id_from_fields(self, fields):"}],"source_content_type":"text/x-python","patch_set":13,"id":"7d5cf214_8c115270","line":155,"in_reply_to":"ef3028e4_53141be3","updated":"2023-01-10 05:32:14.000000000","message":"Done","commit_id":"166ce18bacb6d1aac7b4335bc219e4911f88f337"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"fa709a98408baa0c86e80d00f9c63fcc0d75f512","unresolved":true,"context_lines":[{"line_number":152,"context_line":"    def _check_and_get_failed_zone(self, ex_detail, vdu_dict):"},{"line_number":153,"context_line":"        if re.match(CONF.v2_vnfm.placement_az_resource_error, ex_detail):"},{"line_number":154,"context_line":"            match_result \u003d re.search(r\u0027resources\\.((.*)-([0-9]+))\u0027, ex_detail)"},{"line_number":155,"context_line":"            vdu_id \u003d match_result.group(1)"},{"line_number":156,"context_line":"            return vdu_dict.get(vdu_id, {}).get(\u0027locationConstraints\u0027)"},{"line_number":157,"context_line":""},{"line_number":158,"context_line":"    def _get_vdu_id_from_fields(self, fields):"}],"source_content_type":"text/x-python","patch_set":13,"id":"ef3028e4_53141be3","line":155,"in_reply_to":"f73cf49e_70475149","updated":"2022-12-28 01:50:58.000000000","message":"I see. I will add a check for the value of \"match_result\".","commit_id":"166ce18bacb6d1aac7b4335bc219e4911f88f337"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"1cecf6e51e57999f739cf9fd73bced9225b4a7d6","unresolved":true,"context_lines":[{"line_number":106,"context_line":"            except sol_ex.StackOperationFailed as ex:"},{"line_number":107,"context_line":"                self._update_stack_retry(heat_client, fields, inst, stack_id,"},{"line_number":108,"context_line":"                    ex, vim_info, vdu_ids)"},{"line_number":109,"context_line":"                stack_id \u003d heat_client.get_stack_id(stack_name)"},{"line_number":110,"context_line":"        else:"},{"line_number":111,"context_line":"            try:"},{"line_number":112,"context_line":"                heat_client.update_stack(f\u0027{stack_name}/{stack_id}\u0027, fields)"}],"source_content_type":"text/x-python","patch_set":15,"id":"b90d1615_9c0dce4d","line":109,"updated":"2023-01-25 09:55:01.000000000","message":"Will the stack_id in L.105 and L.107 be changed to different values?\nIf the values are the same, I think this process is unnecessary...\nIs it assumed that None is set to stack_id like heal L.307?","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"a62fb635346ed1368d70401e17e17662e2053b99","unresolved":true,"context_lines":[{"line_number":106,"context_line":"            except sol_ex.StackOperationFailed as ex:"},{"line_number":107,"context_line":"                self._update_stack_retry(heat_client, fields, inst, stack_id,"},{"line_number":108,"context_line":"                    ex, vim_info, vdu_ids)"},{"line_number":109,"context_line":"                stack_id \u003d heat_client.get_stack_id(stack_name)"},{"line_number":110,"context_line":"        else:"},{"line_number":111,"context_line":"            try:"},{"line_number":112,"context_line":"                heat_client.update_stack(f\u0027{stack_name}/{stack_id}\u0027, fields)"}],"source_content_type":"text/x-python","patch_set":15,"id":"93fde6e3_a0c01b72","line":109,"in_reply_to":"0e36d31f_7a1bc576","updated":"2023-01-26 05:47:18.000000000","message":"As a supplement, the stack_id\u0027s of L.105 and L.107 are the same.\nBut in the L.107, stack_id is None, so stack_id is retrieved after processing.\n\"_update_stack_retry\" allows stack_id to be \"None\".","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"2e3382b5c6c702ca4f101235799a12c43facc22d","unresolved":false,"context_lines":[{"line_number":106,"context_line":"            except sol_ex.StackOperationFailed as ex:"},{"line_number":107,"context_line":"                self._update_stack_retry(heat_client, fields, inst, stack_id,"},{"line_number":108,"context_line":"                    ex, vim_info, vdu_ids)"},{"line_number":109,"context_line":"                stack_id \u003d heat_client.get_stack_id(stack_name)"},{"line_number":110,"context_line":"        else:"},{"line_number":111,"context_line":"            try:"},{"line_number":112,"context_line":"                heat_client.update_stack(f\u0027{stack_name}/{stack_id}\u0027, fields)"}],"source_content_type":"text/x-python","patch_set":15,"id":"870a7353_9af49de8","line":109,"in_reply_to":"5899c22f_689cc48e","updated":"2023-02-02 02:17:02.000000000","message":"Thank you for your explanation and fix.\nI understand.","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"c2e6ecf14afcce5535cb39d3c3b40e7da0da8fd6","unresolved":true,"context_lines":[{"line_number":106,"context_line":"            except sol_ex.StackOperationFailed as ex:"},{"line_number":107,"context_line":"                self._update_stack_retry(heat_client, fields, inst, stack_id,"},{"line_number":108,"context_line":"                    ex, vim_info, vdu_ids)"},{"line_number":109,"context_line":"                stack_id \u003d heat_client.get_stack_id(stack_name)"},{"line_number":110,"context_line":"        else:"},{"line_number":111,"context_line":"            try:"},{"line_number":112,"context_line":"                heat_client.update_stack(f\u0027{stack_name}/{stack_id}\u0027, fields)"}],"source_content_type":"text/x-python","patch_set":15,"id":"5899c22f_689cc48e","line":109,"in_reply_to":"7389ffb9_c4d9a391","updated":"2023-02-01 03:49:46.000000000","message":"I specified to set the stack_id to \"_update_stack_retry\" so that it is easy to understand, but it is better to set it to None.\nI will fix it to set None.","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"d72660d82a43f079fa7367212d5bfbe1f7d82fa4","unresolved":true,"context_lines":[{"line_number":106,"context_line":"            except sol_ex.StackOperationFailed as ex:"},{"line_number":107,"context_line":"                self._update_stack_retry(heat_client, fields, inst, stack_id,"},{"line_number":108,"context_line":"                    ex, vim_info, vdu_ids)"},{"line_number":109,"context_line":"                stack_id \u003d heat_client.get_stack_id(stack_name)"},{"line_number":110,"context_line":"        else:"},{"line_number":111,"context_line":"            try:"},{"line_number":112,"context_line":"                heat_client.update_stack(f\u0027{stack_name}/{stack_id}\u0027, fields)"}],"source_content_type":"text/x-python","patch_set":15,"id":"7389ffb9_c4d9a391","line":109,"in_reply_to":"93fde6e3_a0c01b72","updated":"2023-01-31 07:45:42.000000000","message":"Thank you for your explanation. Let me ask you a few more questions.\nIs the stack_id set because there are cases where the stack_id of L.105 is not None?\nOr is it just setting stack_id according to L.114 and it will always be None?\n(I didn\u0027t understand why you didn\u0027t explicitly set None...)","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"fddf6d33a0c1889e6d89ca1487e3de9cae60d614","unresolved":true,"context_lines":[{"line_number":106,"context_line":"            except sol_ex.StackOperationFailed as ex:"},{"line_number":107,"context_line":"                self._update_stack_retry(heat_client, fields, inst, stack_id,"},{"line_number":108,"context_line":"                    ex, vim_info, vdu_ids)"},{"line_number":109,"context_line":"                stack_id \u003d heat_client.get_stack_id(stack_name)"},{"line_number":110,"context_line":"        else:"},{"line_number":111,"context_line":"            try:"},{"line_number":112,"context_line":"                heat_client.update_stack(f\u0027{stack_name}/{stack_id}\u0027, fields)"}],"source_content_type":"text/x-python","patch_set":15,"id":"0e36d31f_7a1bc576","line":109,"in_reply_to":"b90d1615_9c0dce4d","updated":"2023-01-26 04:05:28.000000000","message":"The first time stack_create is run, stack_id is None and must be set if it fails.","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"a84fff5e020633c8f359096a52da968123fc14b4","unresolved":true,"context_lines":[{"line_number":637,"context_line":"    def _update_stack_retry(self, heat_client, fields, inst, stack_id,"},{"line_number":638,"context_line":"            error_ex, vim_info, vdu_ids):"},{"line_number":639,"context_line":"        if not CONF.v2_vnfm.placement_fallback_best_effort:"},{"line_number":640,"context_line":"            raise error_ex"},{"line_number":641,"context_line":""},{"line_number":642,"context_line":"        vdu_dict \u003d fields[\u0027parameters\u0027][\u0027nfv\u0027][\u0027VDU\u0027]"},{"line_number":643,"context_line":"        failed_zone \u003d self._check_and_get_failed_zone("}],"source_content_type":"text/x-python","patch_set":15,"id":"6a3f9d79_94748639","line":640,"updated":"2023-01-27 12:56:21.000000000","message":"It is easily understood to add Note that `If fallback_best_effort is False, AZ reselection is not executed.`","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":34712,"name":"Yuta Kazato","display_name":"Yuta Kazato","email":"yuta.kazato.nw@hco.ntt.co.jp","username":"kazatoy-ntt"},"change_message_id":"147be22d20092060822e75ead2e6e61b6bf3cdd5","unresolved":false,"context_lines":[{"line_number":637,"context_line":"    def _update_stack_retry(self, heat_client, fields, inst, stack_id,"},{"line_number":638,"context_line":"            error_ex, vim_info, vdu_ids):"},{"line_number":639,"context_line":"        if not CONF.v2_vnfm.placement_fallback_best_effort:"},{"line_number":640,"context_line":"            raise error_ex"},{"line_number":641,"context_line":""},{"line_number":642,"context_line":"        vdu_dict \u003d fields[\u0027parameters\u0027][\u0027nfv\u0027][\u0027VDU\u0027]"},{"line_number":643,"context_line":"        failed_zone \u003d self._check_and_get_failed_zone("}],"source_content_type":"text/x-python","patch_set":15,"id":"b983b2ca_6afecbe9","line":640,"in_reply_to":"2e8fd619_d5961557","updated":"2023-02-13 07:06:37.000000000","message":"Done","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"4734a4b67ea4c7e9c40eb0d54c59d4d331a4233f","unresolved":true,"context_lines":[{"line_number":637,"context_line":"    def _update_stack_retry(self, heat_client, fields, inst, stack_id,"},{"line_number":638,"context_line":"            error_ex, vim_info, vdu_ids):"},{"line_number":639,"context_line":"        if not CONF.v2_vnfm.placement_fallback_best_effort:"},{"line_number":640,"context_line":"            raise error_ex"},{"line_number":641,"context_line":""},{"line_number":642,"context_line":"        vdu_dict \u003d fields[\u0027parameters\u0027][\u0027nfv\u0027][\u0027VDU\u0027]"},{"line_number":643,"context_line":"        failed_zone \u003d self._check_and_get_failed_zone("}],"source_content_type":"text/x-python","patch_set":15,"id":"2e8fd619_d5961557","line":640,"in_reply_to":"6a3f9d79_94748639","updated":"2023-01-30 06:32:15.000000000","message":"I will add.","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"5afd8d6c7e57941a0c7812e22158b283ff62f103","unresolved":true,"context_lines":[{"line_number":642,"context_line":"            heat_client, is_rollback\u003dTrue)"},{"line_number":643,"context_line":""},{"line_number":644,"context_line":"    def _update_stack_retry(self, heat_client, fields, inst, stack_id,"},{"line_number":645,"context_line":"            error_ex, vim_info, vdu_ids):"},{"line_number":646,"context_line":"        if not CONF.v2_vnfm.placement_fallback_best_effort:"},{"line_number":647,"context_line":"            raise error_ex"},{"line_number":648,"context_line":""}],"source_content_type":"text/x-python","patch_set":16,"id":"b752a03f_7cc09574","line":645,"updated":"2023-01-27 11:51:17.000000000","message":"It might be better to add a description for this function as a NOTE. There might be implicit policies behind what\u0027s happening here. For example, why do we have to separate used_zone and zone and why do we use them all in the end? There must be thoughts like \"try non_used_zone first and then try used_zone\". I think it\u0027s better to explain that explicitly.","commit_id":"17b6a1ab8f79ec95f6014ab4c03422b67adffcd9"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"4734a4b67ea4c7e9c40eb0d54c59d4d331a4233f","unresolved":true,"context_lines":[{"line_number":642,"context_line":"            heat_client, is_rollback\u003dTrue)"},{"line_number":643,"context_line":""},{"line_number":644,"context_line":"    def _update_stack_retry(self, heat_client, fields, inst, stack_id,"},{"line_number":645,"context_line":"            error_ex, vim_info, vdu_ids):"},{"line_number":646,"context_line":"        if not CONF.v2_vnfm.placement_fallback_best_effort:"},{"line_number":647,"context_line":"            raise error_ex"},{"line_number":648,"context_line":""}],"source_content_type":"text/x-python","patch_set":16,"id":"8b58475b_f3bbd5e5","line":645,"in_reply_to":"b752a03f_7cc09574","updated":"2023-01-30 06:32:15.000000000","message":"I will add a discription.","commit_id":"17b6a1ab8f79ec95f6014ab4c03422b67adffcd9"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"5afd8d6c7e57941a0c7812e22158b283ff62f103","unresolved":true,"context_lines":[{"line_number":654,"context_line":""},{"line_number":655,"context_line":"        stack_name \u003d heat_utils.get_stack_name(inst, stack_id)"},{"line_number":656,"context_line":"        nova_client \u003d nova_utils.NovaClient(vim_info)"},{"line_number":657,"context_line":"        zone_list \u003d nova_client.get_zone()"},{"line_number":658,"context_line":"        used_zone_list \u003d {parameters.get(\u0027locationConstraints\u0027)"},{"line_number":659,"context_line":"                          for parameters in vdu_dict.values()"},{"line_number":660,"context_line":"                          if parameters.get(\u0027locationConstraints\u0027) is not None}"}],"source_content_type":"text/x-python","patch_set":16,"id":"794604d4_24b8e3d6","line":657,"updated":"2023-01-27 11:51:17.000000000","message":"nits.\n\nNaming \"sets\" as \"*_list\" is a little bit confusing as they have different methods. How about just `zones`.\nSame for others.","commit_id":"17b6a1ab8f79ec95f6014ab4c03422b67adffcd9"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"4734a4b67ea4c7e9c40eb0d54c59d4d331a4233f","unresolved":true,"context_lines":[{"line_number":654,"context_line":""},{"line_number":655,"context_line":"        stack_name \u003d heat_utils.get_stack_name(inst, stack_id)"},{"line_number":656,"context_line":"        nova_client \u003d nova_utils.NovaClient(vim_info)"},{"line_number":657,"context_line":"        zone_list \u003d nova_client.get_zone()"},{"line_number":658,"context_line":"        used_zone_list \u003d {parameters.get(\u0027locationConstraints\u0027)"},{"line_number":659,"context_line":"                          for parameters in vdu_dict.values()"},{"line_number":660,"context_line":"                          if parameters.get(\u0027locationConstraints\u0027) is not None}"}],"source_content_type":"text/x-python","patch_set":16,"id":"99a6782a_0568ffb8","line":657,"in_reply_to":"794604d4_24b8e3d6","updated":"2023-01-30 06:32:15.000000000","message":"It may be better, but this valiable name \"list\" is used as a common noun, not as a python class. Is this all right with you?","commit_id":"17b6a1ab8f79ec95f6014ab4c03422b67adffcd9"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"5447061cab96b5b1b5586cc9a4cc5932875a988c","unresolved":true,"context_lines":[{"line_number":654,"context_line":""},{"line_number":655,"context_line":"        stack_name \u003d heat_utils.get_stack_name(inst, stack_id)"},{"line_number":656,"context_line":"        nova_client \u003d nova_utils.NovaClient(vim_info)"},{"line_number":657,"context_line":"        zone_list \u003d nova_client.get_zone()"},{"line_number":658,"context_line":"        used_zone_list \u003d {parameters.get(\u0027locationConstraints\u0027)"},{"line_number":659,"context_line":"                          for parameters in vdu_dict.values()"},{"line_number":660,"context_line":"                          if parameters.get(\u0027locationConstraints\u0027) is not None}"}],"source_content_type":"text/x-python","patch_set":16,"id":"8b646db5_00e436ea","line":657,"in_reply_to":"99a6782a_0568ffb8","updated":"2023-02-03 16:08:16.000000000","message":"hmm...\nI found this as the Hungarian notation, but other people may not. I don\u0027t have a strong opinion, so I\u0027ll follow PTL\u0027s opinion.\n\nWhat do you think Ogawa-san?\n\nI think either way is okey as long as there is a consistent rule. I found the following example that names a \"set\" as \"_list\" in existing codes (under `sol_refactored`), but it\u0027s not common yet. \n\nhttps://github.com/openstack/tacker/blob/master/tacker/sol_refactored/infra_drivers/kubernetes/kubernetes_common.py#L338","commit_id":"17b6a1ab8f79ec95f6014ab4c03422b67adffcd9"}],"tacker/tests/functional/sol_v2_az_retry/test_az_retry.py":[{"author":{"_account_id":31668,"deleted":true},"change_message_id":"1cecf6e51e57999f739cf9fd73bced9225b4a7d6","unresolved":true,"context_lines":[{"line_number":142,"context_line":""},{"line_number":143,"context_line":"        if self._get_vnfc_zone("},{"line_number":144,"context_line":"                inst_result[vdu_idx], \u0027VDU1\u0027, vdu_idx) !\u003d \u0027nova\u0027:"},{"line_number":145,"context_line":"            while vdu_idx \u003c MAX_SCALE_COUNT:"},{"line_number":146,"context_line":"                vdu_idx +\u003d 1"},{"line_number":147,"context_line":"                expect_vdu_idx_num.add(vdu_idx)"},{"line_number":148,"context_line":""}],"source_content_type":"text/x-python","patch_set":15,"id":"f4ee7a87_91707df2","line":145,"updated":"2023-01-25 09:55:01.000000000","message":"How about changing the following and deleting L.143-144 and L.166-168?\n```\nwhile self._get_vnfc_zone(inst_result[vdu_idx], \u0027VDU1\u0027, vdu_idx) !\u003d \u0027nova\u0027 and vdu_idx \u003c MAX_SCALE_COUNT:\n```","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"d72660d82a43f079fa7367212d5bfbe1f7d82fa4","unresolved":false,"context_lines":[{"line_number":142,"context_line":""},{"line_number":143,"context_line":"        if self._get_vnfc_zone("},{"line_number":144,"context_line":"                inst_result[vdu_idx], \u0027VDU1\u0027, vdu_idx) !\u003d \u0027nova\u0027:"},{"line_number":145,"context_line":"            while vdu_idx \u003c MAX_SCALE_COUNT:"},{"line_number":146,"context_line":"                vdu_idx +\u003d 1"},{"line_number":147,"context_line":"                expect_vdu_idx_num.add(vdu_idx)"},{"line_number":148,"context_line":""}],"source_content_type":"text/x-python","patch_set":15,"id":"e7d0bfe3_8155efa1","line":145,"in_reply_to":"1762e641_94aa5c3a","updated":"2023-01-31 07:45:42.000000000","message":"Done","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"fddf6d33a0c1889e6d89ca1487e3de9cae60d614","unresolved":true,"context_lines":[{"line_number":142,"context_line":""},{"line_number":143,"context_line":"        if self._get_vnfc_zone("},{"line_number":144,"context_line":"                inst_result[vdu_idx], \u0027VDU1\u0027, vdu_idx) !\u003d \u0027nova\u0027:"},{"line_number":145,"context_line":"            while vdu_idx \u003c MAX_SCALE_COUNT:"},{"line_number":146,"context_line":"                vdu_idx +\u003d 1"},{"line_number":147,"context_line":"                expect_vdu_idx_num.add(vdu_idx)"},{"line_number":148,"context_line":""}],"source_content_type":"text/x-python","patch_set":15,"id":"1762e641_94aa5c3a","line":145,"in_reply_to":"f4ee7a87_91707df2","updated":"2023-01-26 04:05:28.000000000","message":"That\u0027s better. I will change it to that.","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"}],"tacker/tests/functional/sol_v2_common/paramgen.py":[{"author":{"_account_id":31668,"deleted":true},"change_message_id":"1cecf6e51e57999f739cf9fd73bced9225b4a7d6","unresolved":true,"context_lines":[{"line_number":1342,"context_line":"    }"},{"line_number":1343,"context_line":""},{"line_number":1344,"context_line":""},{"line_number":1345,"context_line":"# sample6 is a test for retry of AZ selection in StandardUserData"},{"line_number":1346,"context_line":"#"},{"line_number":1347,"context_line":"def sample6_create(vnfd_id):"},{"line_number":1348,"context_line":"    return {"}],"source_content_type":"text/x-python","patch_set":15,"id":"60c6dab8_18efb311","line":1345,"updated":"2023-01-25 09:55:01.000000000","message":"Why don\u0027t you fix it as follows according to the description of sample4 and sample5?\n```\nsample6 is for retry AZ selection test of StandardUserData\n```","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"fddf6d33a0c1889e6d89ca1487e3de9cae60d614","unresolved":true,"context_lines":[{"line_number":1342,"context_line":"    }"},{"line_number":1343,"context_line":""},{"line_number":1344,"context_line":""},{"line_number":1345,"context_line":"# sample6 is a test for retry of AZ selection in StandardUserData"},{"line_number":1346,"context_line":"#"},{"line_number":1347,"context_line":"def sample6_create(vnfd_id):"},{"line_number":1348,"context_line":"    return {"}],"source_content_type":"text/x-python","patch_set":15,"id":"8fa037c5_41400319","line":1345,"in_reply_to":"60c6dab8_18efb311","updated":"2023-01-26 04:05:28.000000000","message":"I will fix.","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"d72660d82a43f079fa7367212d5bfbe1f7d82fa4","unresolved":false,"context_lines":[{"line_number":1342,"context_line":"    }"},{"line_number":1343,"context_line":""},{"line_number":1344,"context_line":""},{"line_number":1345,"context_line":"# sample6 is a test for retry of AZ selection in StandardUserData"},{"line_number":1346,"context_line":"#"},{"line_number":1347,"context_line":"def sample6_create(vnfd_id):"},{"line_number":1348,"context_line":"    return {"}],"source_content_type":"text/x-python","patch_set":15,"id":"bf1458a0_14f7dea7","line":1345,"in_reply_to":"8fa037c5_41400319","updated":"2023-01-31 07:45:42.000000000","message":"Done","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"}],"tacker/tests/functional/sol_v2_common/samples/userdata_standard_az_retry/contents/BaseHOT/simple/nested/VDU1.yaml":[{"author":{"_account_id":31668,"deleted":true},"change_message_id":"1cecf6e51e57999f739cf9fd73bced9225b4a7d6","unresolved":true,"context_lines":[{"line_number":25,"context_line":"      networks:"},{"line_number":26,"context_line":"      - port:"},{"line_number":27,"context_line":"          get_resource: VDU1_CP1"},{"line_number":28,"context_line":"# replace the following line to Port ID when extmanagedVLs\u0027 Ports are"},{"line_number":29,"context_line":"# specified in instantiatevnfrequest"},{"line_number":30,"context_line":"      - port:"},{"line_number":31,"context_line":"          get_resource: VDU1_CP2"}],"source_content_type":"text/x-yaml","patch_set":15,"id":"80446525_3afffc7e","line":28,"updated":"2023-01-25 09:55:01.000000000","message":"\"extManagedVLs\" is easier to read.","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"d72660d82a43f079fa7367212d5bfbe1f7d82fa4","unresolved":false,"context_lines":[{"line_number":25,"context_line":"      networks:"},{"line_number":26,"context_line":"      - port:"},{"line_number":27,"context_line":"          get_resource: VDU1_CP1"},{"line_number":28,"context_line":"# replace the following line to Port ID when extmanagedVLs\u0027 Ports are"},{"line_number":29,"context_line":"# specified in instantiatevnfrequest"},{"line_number":30,"context_line":"      - port:"},{"line_number":31,"context_line":"          get_resource: VDU1_CP2"}],"source_content_type":"text/x-yaml","patch_set":15,"id":"fbec46c7_fa6bbff9","line":28,"in_reply_to":"16b32748_50618a2a","updated":"2023-01-31 07:45:42.000000000","message":"Done","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"fddf6d33a0c1889e6d89ca1487e3de9cae60d614","unresolved":true,"context_lines":[{"line_number":25,"context_line":"      networks:"},{"line_number":26,"context_line":"      - port:"},{"line_number":27,"context_line":"          get_resource: VDU1_CP1"},{"line_number":28,"context_line":"# replace the following line to Port ID when extmanagedVLs\u0027 Ports are"},{"line_number":29,"context_line":"# specified in instantiatevnfrequest"},{"line_number":30,"context_line":"      - port:"},{"line_number":31,"context_line":"          get_resource: VDU1_CP2"}],"source_content_type":"text/x-yaml","patch_set":15,"id":"16b32748_50618a2a","line":28,"in_reply_to":"80446525_3afffc7e","updated":"2023-01-26 04:05:28.000000000","message":"I will fix.","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"1cecf6e51e57999f739cf9fd73bced9225b4a7d6","unresolved":true,"context_lines":[{"line_number":26,"context_line":"      - port:"},{"line_number":27,"context_line":"          get_resource: VDU1_CP1"},{"line_number":28,"context_line":"# replace the following line to Port ID when extmanagedVLs\u0027 Ports are"},{"line_number":29,"context_line":"# specified in instantiatevnfrequest"},{"line_number":30,"context_line":"      - port:"},{"line_number":31,"context_line":"          get_resource: VDU1_CP2"},{"line_number":32,"context_line":"      - port:"}],"source_content_type":"text/x-yaml","patch_set":15,"id":"61cefb54_efcb3994","line":29,"updated":"2023-01-25 09:55:01.000000000","message":"\"InstantiateVnfRequest\" is better according to SOL.","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"fddf6d33a0c1889e6d89ca1487e3de9cae60d614","unresolved":true,"context_lines":[{"line_number":26,"context_line":"      - port:"},{"line_number":27,"context_line":"          get_resource: VDU1_CP1"},{"line_number":28,"context_line":"# replace the following line to Port ID when extmanagedVLs\u0027 Ports are"},{"line_number":29,"context_line":"# specified in instantiatevnfrequest"},{"line_number":30,"context_line":"      - port:"},{"line_number":31,"context_line":"          get_resource: VDU1_CP2"},{"line_number":32,"context_line":"      - port:"}],"source_content_type":"text/x-yaml","patch_set":15,"id":"82af14b6_30e9c383","line":29,"in_reply_to":"61cefb54_efcb3994","updated":"2023-01-26 04:05:28.000000000","message":"I will fix.","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"d72660d82a43f079fa7367212d5bfbe1f7d82fa4","unresolved":false,"context_lines":[{"line_number":26,"context_line":"      - port:"},{"line_number":27,"context_line":"          get_resource: VDU1_CP1"},{"line_number":28,"context_line":"# replace the following line to Port ID when extmanagedVLs\u0027 Ports are"},{"line_number":29,"context_line":"# specified in instantiatevnfrequest"},{"line_number":30,"context_line":"      - port:"},{"line_number":31,"context_line":"          get_resource: VDU1_CP2"},{"line_number":32,"context_line":"      - port:"}],"source_content_type":"text/x-yaml","patch_set":15,"id":"7d82c4c1_46f5e45f","line":29,"in_reply_to":"82af14b6_30e9c383","updated":"2023-01-31 07:45:42.000000000","message":"Done","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"1cecf6e51e57999f739cf9fd73bced9225b4a7d6","unresolved":true,"context_lines":[{"line_number":39,"context_line":"    properties:"},{"line_number":40,"context_line":"      network: { get_param: net1 }"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"# CPs of internal VLs are deleted when extmanagedVLs and port are"},{"line_number":43,"context_line":"# specified in instantiatevnfrequest"},{"line_number":44,"context_line":"  VDU1_CP2:"},{"line_number":45,"context_line":"    type: OS::Neutron::Port"}],"source_content_type":"text/x-yaml","patch_set":15,"id":"69c97ad0_05da333b","line":42,"updated":"2023-01-25 09:55:01.000000000","message":"\"extManagedVLs\" is easier to read.","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"fddf6d33a0c1889e6d89ca1487e3de9cae60d614","unresolved":true,"context_lines":[{"line_number":39,"context_line":"    properties:"},{"line_number":40,"context_line":"      network: { get_param: net1 }"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"# CPs of internal VLs are deleted when extmanagedVLs and port are"},{"line_number":43,"context_line":"# specified in instantiatevnfrequest"},{"line_number":44,"context_line":"  VDU1_CP2:"},{"line_number":45,"context_line":"    type: OS::Neutron::Port"}],"source_content_type":"text/x-yaml","patch_set":15,"id":"7719c89c_dea92df9","line":42,"in_reply_to":"69c97ad0_05da333b","updated":"2023-01-26 04:05:28.000000000","message":"I will fix.","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"d72660d82a43f079fa7367212d5bfbe1f7d82fa4","unresolved":false,"context_lines":[{"line_number":39,"context_line":"    properties:"},{"line_number":40,"context_line":"      network: { get_param: net1 }"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"# CPs of internal VLs are deleted when extmanagedVLs and port are"},{"line_number":43,"context_line":"# specified in instantiatevnfrequest"},{"line_number":44,"context_line":"  VDU1_CP2:"},{"line_number":45,"context_line":"    type: OS::Neutron::Port"}],"source_content_type":"text/x-yaml","patch_set":15,"id":"aef528b7_f97c710a","line":42,"in_reply_to":"7719c89c_dea92df9","updated":"2023-01-31 07:45:42.000000000","message":"Done","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"1cecf6e51e57999f739cf9fd73bced9225b4a7d6","unresolved":true,"context_lines":[{"line_number":40,"context_line":"      network: { get_param: net1 }"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"# CPs of internal VLs are deleted when extmanagedVLs and port are"},{"line_number":43,"context_line":"# specified in instantiatevnfrequest"},{"line_number":44,"context_line":"  VDU1_CP2:"},{"line_number":45,"context_line":"    type: OS::Neutron::Port"},{"line_number":46,"context_line":"    properties:"}],"source_content_type":"text/x-yaml","patch_set":15,"id":"28041710_e90ba49b","line":43,"updated":"2023-01-25 09:55:01.000000000","message":"\"InstantiateVnfRequest\" is better according to SOL.","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"fddf6d33a0c1889e6d89ca1487e3de9cae60d614","unresolved":true,"context_lines":[{"line_number":40,"context_line":"      network: { get_param: net1 }"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"# CPs of internal VLs are deleted when extmanagedVLs and port are"},{"line_number":43,"context_line":"# specified in instantiatevnfrequest"},{"line_number":44,"context_line":"  VDU1_CP2:"},{"line_number":45,"context_line":"    type: OS::Neutron::Port"},{"line_number":46,"context_line":"    properties:"}],"source_content_type":"text/x-yaml","patch_set":15,"id":"2c728e5d_9e6a3f0c","line":43,"in_reply_to":"28041710_e90ba49b","updated":"2023-01-26 04:05:28.000000000","message":"I will fix.","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"d72660d82a43f079fa7367212d5bfbe1f7d82fa4","unresolved":false,"context_lines":[{"line_number":40,"context_line":"      network: { get_param: net1 }"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"# CPs of internal VLs are deleted when extmanagedVLs and port are"},{"line_number":43,"context_line":"# specified in instantiatevnfrequest"},{"line_number":44,"context_line":"  VDU1_CP2:"},{"line_number":45,"context_line":"    type: OS::Neutron::Port"},{"line_number":46,"context_line":"    properties:"}],"source_content_type":"text/x-yaml","patch_set":15,"id":"5d967726_83ff6f2e","line":43,"in_reply_to":"2c728e5d_9e6a3f0c","updated":"2023-01-31 07:45:42.000000000","message":"Done","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"}],"tacker/tests/functional/sol_v2_common/samples/userdata_standard_az_retry/contents/BaseHOT/simple/sample6.yaml":[{"author":{"_account_id":31668,"deleted":true},"change_message_id":"1cecf6e51e57999f739cf9fd73bced9225b4a7d6","unresolved":true,"context_lines":[{"line_number":16,"context_line":"      net2: { get_resource: internalVL1 }"},{"line_number":17,"context_line":"      net3: { get_resource: internalVL2 }"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"# delete the following lines when extmanagedVLs are specified in instantiatevnfrequest"},{"line_number":20,"context_line":"  internalVL1:"},{"line_number":21,"context_line":"    type: OS::Neutron::Net"},{"line_number":22,"context_line":"  internalVL2:"}],"source_content_type":"text/x-yaml","patch_set":15,"id":"88d3fb41_2e5e4500","line":19,"updated":"2023-01-25 09:55:01.000000000","message":"\"extManagedVLs\" is easier to read.\n\"InstantiateVnfRequest\" is better according to SOL.","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"d72660d82a43f079fa7367212d5bfbe1f7d82fa4","unresolved":false,"context_lines":[{"line_number":16,"context_line":"      net2: { get_resource: internalVL1 }"},{"line_number":17,"context_line":"      net3: { get_resource: internalVL2 }"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"# delete the following lines when extmanagedVLs are specified in instantiatevnfrequest"},{"line_number":20,"context_line":"  internalVL1:"},{"line_number":21,"context_line":"    type: OS::Neutron::Net"},{"line_number":22,"context_line":"  internalVL2:"}],"source_content_type":"text/x-yaml","patch_set":15,"id":"d3260698_0619a6a5","line":19,"in_reply_to":"3792ab12_c2ab479d","updated":"2023-01-31 07:45:42.000000000","message":"Done","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"fddf6d33a0c1889e6d89ca1487e3de9cae60d614","unresolved":true,"context_lines":[{"line_number":16,"context_line":"      net2: { get_resource: internalVL1 }"},{"line_number":17,"context_line":"      net3: { get_resource: internalVL2 }"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"# delete the following lines when extmanagedVLs are specified in instantiatevnfrequest"},{"line_number":20,"context_line":"  internalVL1:"},{"line_number":21,"context_line":"    type: OS::Neutron::Net"},{"line_number":22,"context_line":"  internalVL2:"}],"source_content_type":"text/x-yaml","patch_set":15,"id":"3792ab12_c2ab479d","line":19,"in_reply_to":"88d3fb41_2e5e4500","updated":"2023-01-26 04:05:28.000000000","message":"I will fix.","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"}],"tacker/tests/functional/sol_v2_common/samples/userdata_standard_az_retry/contents/Definitions/v2_sample6_df_simple.yaml":[{"author":{"_account_id":31668,"deleted":true},"change_message_id":"d72660d82a43f079fa7367212d5bfbe1f7d82fa4","unresolved":true,"context_lines":[{"line_number":79,"context_line":"        description: VDU1 compute node"},{"line_number":80,"context_line":"        vdu_profile:"},{"line_number":81,"context_line":"          min_number_of_instances: 1"},{"line_number":82,"context_line":"          max_number_of_instances: 5"},{"line_number":83,"context_line":"        sw_image_data:"},{"line_number":84,"context_line":"          name: cirros-0.5.2-x86_64-disk"},{"line_number":85,"context_line":"          version: \u00270.5.2\u0027"}],"source_content_type":"text/x-yaml","patch_set":17,"id":"3f4ae2bb_9a73cc0e","line":82,"updated":"2023-01-31 07:45:42.000000000","message":"Could you write a note about the need to review it along with the flavor if there is a change in the Zuul environment in the future?","commit_id":"cb069e5dbbcb3b6df6ecb35c43bcc732c40d1453"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"c2e6ecf14afcce5535cb39d3c3b40e7da0da8fd6","unresolved":true,"context_lines":[{"line_number":79,"context_line":"        description: VDU1 compute node"},{"line_number":80,"context_line":"        vdu_profile:"},{"line_number":81,"context_line":"          min_number_of_instances: 1"},{"line_number":82,"context_line":"          max_number_of_instances: 5"},{"line_number":83,"context_line":"        sw_image_data:"},{"line_number":84,"context_line":"          name: cirros-0.5.2-x86_64-disk"},{"line_number":85,"context_line":"          version: \u00270.5.2\u0027"}],"source_content_type":"text/x-yaml","patch_set":17,"id":"c52b6969_b3f2f358","line":82,"in_reply_to":"3f4ae2bb_9a73cc0e","updated":"2023-02-01 03:49:46.000000000","message":"I will add.","commit_id":"cb069e5dbbcb3b6df6ecb35c43bcc732c40d1453"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"8cf2ca37f8c4d2293e32b6d180508ff88ddf2c7f","unresolved":true,"context_lines":[{"line_number":79,"context_line":"        description: VDU1 compute node"},{"line_number":80,"context_line":"        vdu_profile:"},{"line_number":81,"context_line":"          min_number_of_instances: 1"},{"line_number":82,"context_line":"          max_number_of_instances: 5"},{"line_number":83,"context_line":"        sw_image_data:"},{"line_number":84,"context_line":"          name: cirros-0.5.2-x86_64-disk"},{"line_number":85,"context_line":"          version: \u00270.5.2\u0027"}],"source_content_type":"text/x-yaml","patch_set":17,"id":"a5266c83_70142825","line":82,"in_reply_to":"4fd8cd13_46109c7b","updated":"2023-02-02 07:51:38.000000000","message":"I will fix.","commit_id":"cb069e5dbbcb3b6df6ecb35c43bcc732c40d1453"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"79e99466068b89b3557a5ef41f1e9b0dd8c70036","unresolved":false,"context_lines":[{"line_number":79,"context_line":"        description: VDU1 compute node"},{"line_number":80,"context_line":"        vdu_profile:"},{"line_number":81,"context_line":"          min_number_of_instances: 1"},{"line_number":82,"context_line":"          max_number_of_instances: 5"},{"line_number":83,"context_line":"        sw_image_data:"},{"line_number":84,"context_line":"          name: cirros-0.5.2-x86_64-disk"},{"line_number":85,"context_line":"          version: \u00270.5.2\u0027"}],"source_content_type":"text/x-yaml","patch_set":17,"id":"f7b5fd3f_a75e959e","line":82,"in_reply_to":"a5266c83_70142825","updated":"2023-02-03 02:12:54.000000000","message":"Done","commit_id":"cb069e5dbbcb3b6df6ecb35c43bcc732c40d1453"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"2e3382b5c6c702ca4f101235799a12c43facc22d","unresolved":true,"context_lines":[{"line_number":79,"context_line":"        description: VDU1 compute node"},{"line_number":80,"context_line":"        vdu_profile:"},{"line_number":81,"context_line":"          min_number_of_instances: 1"},{"line_number":82,"context_line":"          max_number_of_instances: 5"},{"line_number":83,"context_line":"        sw_image_data:"},{"line_number":84,"context_line":"          name: cirros-0.5.2-x86_64-disk"},{"line_number":85,"context_line":"          version: \u00270.5.2\u0027"}],"source_content_type":"text/x-yaml","patch_set":17,"id":"4fd8cd13_46109c7b","line":82,"in_reply_to":"c52b6969_b3f2f358","updated":"2023-02-02 02:17:02.000000000","message":"Thank you for the fix.\nI think one space after \"NOTE:\" is better than two.","commit_id":"cb069e5dbbcb3b6df6ecb35c43bcc732c40d1453"}],"tacker/tests/functional/sol_v2_common/samples/userdata_standard_az_retry/contents/Definitions/v2_sample6_types.yaml":[{"author":{"_account_id":31668,"deleted":true},"change_message_id":"1cecf6e51e57999f739cf9fd73bced9225b4a7d6","unresolved":true,"context_lines":[{"line_number":52,"context_line":"          capability: tosca.capabilities.nfv.VirtualLinkable"},{"line_number":53,"context_line":"    interfaces:"},{"line_number":54,"context_line":"      Vnflcm:"},{"line_number":55,"context_line":"        type: tosca.interfaces.nfv.Vnflcm"}],"source_content_type":"text/x-yaml","patch_set":15,"id":"da3089af_eaf87ca3","line":55,"updated":"2023-01-25 09:55:01.000000000","message":"Could you please add a blank line at the end according to the other files?","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"d72660d82a43f079fa7367212d5bfbe1f7d82fa4","unresolved":false,"context_lines":[{"line_number":52,"context_line":"          capability: tosca.capabilities.nfv.VirtualLinkable"},{"line_number":53,"context_line":"    interfaces:"},{"line_number":54,"context_line":"      Vnflcm:"},{"line_number":55,"context_line":"        type: tosca.interfaces.nfv.Vnflcm"}],"source_content_type":"text/x-yaml","patch_set":15,"id":"97b84ed2_26cc3ec7","line":55,"in_reply_to":"810817a2_d23fd36d","updated":"2023-01-31 07:45:42.000000000","message":"Done","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"fddf6d33a0c1889e6d89ca1487e3de9cae60d614","unresolved":true,"context_lines":[{"line_number":52,"context_line":"          capability: tosca.capabilities.nfv.VirtualLinkable"},{"line_number":53,"context_line":"    interfaces:"},{"line_number":54,"context_line":"      Vnflcm:"},{"line_number":55,"context_line":"        type: tosca.interfaces.nfv.Vnflcm"}],"source_content_type":"text/x-yaml","patch_set":15,"id":"810817a2_d23fd36d","line":55,"in_reply_to":"da3089af_eaf87ca3","updated":"2023-01-26 04:05:28.000000000","message":"I wil fix.","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"}],"tacker/tests/functional/sol_v2_common/samples/userdata_standard_az_retry/pkggen.py":[{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"9927ca63510ab668438d5bb6b256d2371daaf31b","unresolved":true,"context_lines":[{"line_number":29,"context_line":"vnfd_id \u003d uuidutils.generate_uuid()"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"# tacker/tests/etc..."},{"line_number":32,"context_line":"#             /functional/sol_v2_common/samples/smapleX"},{"line_number":33,"context_line":"image_dir \u003d \"../../../../etc/samples/etsi/nfv/common/Files/images/\""},{"line_number":34,"context_line":"image_file \u003d \"cirros-0.5.2-x86_64-disk.img\""},{"line_number":35,"context_line":"image_path \u003d os.path.abspath(image_dir + image_file)"}],"source_content_type":"text/x-python","patch_set":13,"id":"5de56d76_7cd00d9c","line":32,"range":{"start_line":32,"start_character":48,"end_line":32,"end_character":55},"updated":"2022-12-27 05:50:39.000000000","message":"typo: smaple -\u003e sample","commit_id":"166ce18bacb6d1aac7b4335bc219e4911f88f337"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"fa709a98408baa0c86e80d00f9c63fcc0d75f512","unresolved":true,"context_lines":[{"line_number":29,"context_line":"vnfd_id \u003d uuidutils.generate_uuid()"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"# tacker/tests/etc..."},{"line_number":32,"context_line":"#             /functional/sol_v2_common/samples/smapleX"},{"line_number":33,"context_line":"image_dir \u003d \"../../../../etc/samples/etsi/nfv/common/Files/images/\""},{"line_number":34,"context_line":"image_file \u003d \"cirros-0.5.2-x86_64-disk.img\""},{"line_number":35,"context_line":"image_path \u003d os.path.abspath(image_dir + image_file)"}],"source_content_type":"text/x-python","patch_set":13,"id":"a315dea0_8837bdd1","line":32,"range":{"start_line":32,"start_character":48,"end_line":32,"end_character":55},"in_reply_to":"5de56d76_7cd00d9c","updated":"2022-12-28 01:50:58.000000000","message":"I will fix.","commit_id":"166ce18bacb6d1aac7b4335bc219e4911f88f337"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"3a6ed3272d192c9ddb846c1aec7061bae9d4a402","unresolved":false,"context_lines":[{"line_number":29,"context_line":"vnfd_id \u003d uuidutils.generate_uuid()"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"# tacker/tests/etc..."},{"line_number":32,"context_line":"#             /functional/sol_v2_common/samples/smapleX"},{"line_number":33,"context_line":"image_dir \u003d \"../../../../etc/samples/etsi/nfv/common/Files/images/\""},{"line_number":34,"context_line":"image_file \u003d \"cirros-0.5.2-x86_64-disk.img\""},{"line_number":35,"context_line":"image_path \u003d os.path.abspath(image_dir + image_file)"}],"source_content_type":"text/x-python","patch_set":13,"id":"6d2bf844_748afad6","line":32,"range":{"start_line":32,"start_character":48,"end_line":32,"end_character":55},"in_reply_to":"a315dea0_8837bdd1","updated":"2023-01-10 05:32:14.000000000","message":"Done","commit_id":"166ce18bacb6d1aac7b4335bc219e4911f88f337"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"9927ca63510ab668438d5bb6b256d2371daaf31b","unresolved":true,"context_lines":[{"line_number":35,"context_line":"image_path \u003d os.path.abspath(image_dir + image_file)"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"# tacker/sol_refactored/infra_drivers/openstack/userdata_standard.py"},{"line_number":38,"context_line":"#       /tests/functional/sol_v2_common/samples/smapleX"},{"line_number":39,"context_line":"userdata_dir \u003d \"../../../../../sol_refactored/infra_drivers/openstack/\""},{"line_number":40,"context_line":"userdata_file \u003d \"userdata_standard.py\""},{"line_number":41,"context_line":"userdata_path \u003d os.path.abspath(userdata_dir + userdata_file)"}],"source_content_type":"text/x-python","patch_set":13,"id":"79cfc5e1_199807d0","line":38,"range":{"start_line":38,"start_character":48,"end_line":38,"end_character":55},"updated":"2022-12-27 05:50:39.000000000","message":"ditto","commit_id":"166ce18bacb6d1aac7b4335bc219e4911f88f337"},{"author":{"_account_id":35476,"name":"taiki kimura","email":"taiki.kimura@ntt-at.co.jp","username":"tkimura"},"change_message_id":"3a6ed3272d192c9ddb846c1aec7061bae9d4a402","unresolved":false,"context_lines":[{"line_number":35,"context_line":"image_path \u003d os.path.abspath(image_dir + image_file)"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"# tacker/sol_refactored/infra_drivers/openstack/userdata_standard.py"},{"line_number":38,"context_line":"#       /tests/functional/sol_v2_common/samples/smapleX"},{"line_number":39,"context_line":"userdata_dir \u003d \"../../../../../sol_refactored/infra_drivers/openstack/\""},{"line_number":40,"context_line":"userdata_file \u003d \"userdata_standard.py\""},{"line_number":41,"context_line":"userdata_path \u003d os.path.abspath(userdata_dir + userdata_file)"}],"source_content_type":"text/x-python","patch_set":13,"id":"e60db330_9769de55","line":38,"range":{"start_line":38,"start_character":48,"end_line":38,"end_character":55},"in_reply_to":"0ace871e_7e33adf0","updated":"2023-01-10 05:32:14.000000000","message":"Done","commit_id":"166ce18bacb6d1aac7b4335bc219e4911f88f337"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"fa709a98408baa0c86e80d00f9c63fcc0d75f512","unresolved":true,"context_lines":[{"line_number":35,"context_line":"image_path \u003d os.path.abspath(image_dir + image_file)"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"# tacker/sol_refactored/infra_drivers/openstack/userdata_standard.py"},{"line_number":38,"context_line":"#       /tests/functional/sol_v2_common/samples/smapleX"},{"line_number":39,"context_line":"userdata_dir \u003d \"../../../../../sol_refactored/infra_drivers/openstack/\""},{"line_number":40,"context_line":"userdata_file \u003d \"userdata_standard.py\""},{"line_number":41,"context_line":"userdata_path \u003d os.path.abspath(userdata_dir + userdata_file)"}],"source_content_type":"text/x-python","patch_set":13,"id":"0ace871e_7e33adf0","line":38,"range":{"start_line":38,"start_character":48,"end_line":38,"end_character":55},"in_reply_to":"79cfc5e1_199807d0","updated":"2022-12-28 01:50:58.000000000","message":"I will fix.","commit_id":"166ce18bacb6d1aac7b4335bc219e4911f88f337"}],"tacker/tests/unit/sol_refactored/infra_drivers/openstack/test_openstack.py":[{"author":{"_account_id":31668,"deleted":true},"change_message_id":"1cecf6e51e57999f739cf9fd73bced9225b4a7d6","unresolved":true,"context_lines":[{"line_number":4213,"context_line":"            mock_update_stack):"},{"line_number":4214,"context_line":"        # prepare"},{"line_number":4215,"context_line":"        CONF.v2_vnfm.placement_fallback_best_effort \u003d True"},{"line_number":4216,"context_line":"        CONF.v2_vnfm.placement_az_select_retry \u003d 10"},{"line_number":4217,"context_line":""},{"line_number":4218,"context_line":"        vim_info \u003d objects.VimConnectionInfo.from_dict("},{"line_number":4219,"context_line":"            _vim_connection_info_example)"}],"source_content_type":"text/x-python","patch_set":15,"id":"2abf0496_5ed57ca0","line":4216,"updated":"2023-01-25 09:55:01.000000000","message":"There doesn\u0027t seem to be a test for cases where the \"placement_az_select_retry\" limit has been reached, so how about adding it separately from this test?","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"fddf6d33a0c1889e6d89ca1487e3de9cae60d614","unresolved":true,"context_lines":[{"line_number":4213,"context_line":"            mock_update_stack):"},{"line_number":4214,"context_line":"        # prepare"},{"line_number":4215,"context_line":"        CONF.v2_vnfm.placement_fallback_best_effort \u003d True"},{"line_number":4216,"context_line":"        CONF.v2_vnfm.placement_az_select_retry \u003d 10"},{"line_number":4217,"context_line":""},{"line_number":4218,"context_line":"        vim_info \u003d objects.VimConnectionInfo.from_dict("},{"line_number":4219,"context_line":"            _vim_connection_info_example)"}],"source_content_type":"text/x-python","patch_set":15,"id":"5ad39b33_552306b1","line":4216,"in_reply_to":"2abf0496_5ed57ca0","updated":"2023-01-26 04:05:28.000000000","message":"I understand. I will add that test case.","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"d72660d82a43f079fa7367212d5bfbe1f7d82fa4","unresolved":false,"context_lines":[{"line_number":4213,"context_line":"            mock_update_stack):"},{"line_number":4214,"context_line":"        # prepare"},{"line_number":4215,"context_line":"        CONF.v2_vnfm.placement_fallback_best_effort \u003d True"},{"line_number":4216,"context_line":"        CONF.v2_vnfm.placement_az_select_retry \u003d 10"},{"line_number":4217,"context_line":""},{"line_number":4218,"context_line":"        vim_info \u003d objects.VimConnectionInfo.from_dict("},{"line_number":4219,"context_line":"            _vim_connection_info_example)"}],"source_content_type":"text/x-python","patch_set":15,"id":"eaaf4d59_9f75d339","line":4216,"in_reply_to":"5ad39b33_552306b1","updated":"2023-01-31 07:45:42.000000000","message":"Done","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"1cecf6e51e57999f739cf9fd73bced9225b4a7d6","unresolved":true,"context_lines":[{"line_number":4369,"context_line":"        self.driver._update_stack_retry(heat_client, fields_example, inst,"},{"line_number":4370,"context_line":"            STACK_ID, error_ex, vim_info, vdu_ids)"},{"line_number":4371,"context_line":"        self.assertEqual(3, mock_update_stack.call_count)"},{"line_number":4372,"context_line":"        self.assertEqual(expected_zone, use_zone_list[2][\u0027VDU1-1\u0027])"}],"source_content_type":"text/x-python","patch_set":15,"id":"21608c36_942c7175","line":4372,"updated":"2023-01-25 09:55:01.000000000","message":"I think it also is better to check the following.\n```\nself.assertEqual(use_zone_list[2][\u0027VDU1-1\u0027], use_zone_list[2][\u0027VDU1-2\u0027])\n```","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":34226,"name":"Ken Fujimoto","email":"fujimoto@valinux.co.jp","username":"k-fujimoto"},"change_message_id":"fddf6d33a0c1889e6d89ca1487e3de9cae60d614","unresolved":true,"context_lines":[{"line_number":4369,"context_line":"        self.driver._update_stack_retry(heat_client, fields_example, inst,"},{"line_number":4370,"context_line":"            STACK_ID, error_ex, vim_info, vdu_ids)"},{"line_number":4371,"context_line":"        self.assertEqual(3, mock_update_stack.call_count)"},{"line_number":4372,"context_line":"        self.assertEqual(expected_zone, use_zone_list[2][\u0027VDU1-1\u0027])"}],"source_content_type":"text/x-python","patch_set":15,"id":"ce25e231_dab09c9a","line":4372,"in_reply_to":"21608c36_942c7175","updated":"2023-01-26 04:05:28.000000000","message":"I will add that check.","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"},{"author":{"_account_id":31668,"deleted":true},"change_message_id":"d72660d82a43f079fa7367212d5bfbe1f7d82fa4","unresolved":false,"context_lines":[{"line_number":4369,"context_line":"        self.driver._update_stack_retry(heat_client, fields_example, inst,"},{"line_number":4370,"context_line":"            STACK_ID, error_ex, vim_info, vdu_ids)"},{"line_number":4371,"context_line":"        self.assertEqual(3, mock_update_stack.call_count)"},{"line_number":4372,"context_line":"        self.assertEqual(expected_zone, use_zone_list[2][\u0027VDU1-1\u0027])"}],"source_content_type":"text/x-python","patch_set":15,"id":"cfda3037_ab574cb8","line":4372,"in_reply_to":"ce25e231_dab09c9a","updated":"2023-01-31 07:45:42.000000000","message":"Done","commit_id":"2610295d100682dc13e3e90af60ad91f7fb7de59"}]}
