)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"9e95b19a5a169e3f5735a0d421ca3c1270bab9d8","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Takahiro Miyajima \u003cfj6257jz@fujitsu.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2024-07-31 02:55:03 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix issue about TLS in Tacker NovaClient"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"- Fix so that TLS verification is possible in Tacker NovaClient"},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"372bd03c_7be442a6","line":7,"updated":"2024-09-06 19:00:15.000000000","message":"You should describe the title more specificly.","commit_id":"5ceee98ca0f382d59257087a5b0bee3d8c90e34d"},{"author":{"_account_id":36997,"name":"Takahiro Miyajima","display_name":"f-miyajima","email":"fj6257jz@fujitsu.com","username":"f-miyajima"},"change_message_id":"349250a5723037d888aade5554548ae545c034ab","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Takahiro Miyajima \u003cfj6257jz@fujitsu.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2024-07-31 02:55:03 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix issue about TLS in Tacker NovaClient"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"- Fix so that TLS verification is possible in Tacker NovaClient"},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"44abf298_e4e61859","line":7,"in_reply_to":"372bd03c_7be442a6","updated":"2024-09-10 00:45:01.000000000","message":"I fixed the title more specifically. Please kindly confirm.\n- fixed title: Fix to enable NovaClient to use TLS verification","commit_id":"5ceee98ca0f382d59257087a5b0bee3d8c90e34d"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"9e95b19a5a169e3f5735a0d421ca3c1270bab9d8","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix issue about TLS in Tacker NovaClient"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"- Fix so that TLS verification is possible in Tacker NovaClient"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Closes-Bug: #2071399"},{"line_number":12,"context_line":"Change-Id: Ife5eee7df4326de0e0500da62049f97ad5c20aab"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"e7d40997_bf375113","line":9,"updated":"2024-09-06 19:00:15.000000000","message":"What is done in the update?","commit_id":"5ceee98ca0f382d59257087a5b0bee3d8c90e34d"},{"author":{"_account_id":36997,"name":"Takahiro Miyajima","display_name":"f-miyajima","email":"fj6257jz@fujitsu.com","username":"f-miyajima"},"change_message_id":"349250a5723037d888aade5554548ae545c034ab","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix issue about TLS in Tacker NovaClient"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"- Fix so that TLS verification is possible in Tacker NovaClient"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Closes-Bug: #2071399"},{"line_number":12,"context_line":"Change-Id: Ife5eee7df4326de0e0500da62049f97ad5c20aab"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"29adb534_3d791ba3","line":9,"in_reply_to":"e7d40997_bf375113","updated":"2024-09-10 00:45:01.000000000","message":"I fixed the commit message specifically. Please kindly confirm.\n- fixed message: This patch enables NovaClient to use TLS \n  verification, when specifying `nova_verify_cert` and \n  `nova_ca_cert_file`.","commit_id":"5ceee98ca0f382d59257087a5b0bee3d8c90e34d"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"ae34e4c6761a17536c51b7be4f91925e6f99786d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"6102472a_95196bdb","updated":"2024-08-23 10:34:21.000000000","message":"Thanks for your patch, LGTM.","commit_id":"5ceee98ca0f382d59257087a5b0bee3d8c90e34d"},{"author":{"_account_id":36997,"name":"Takahiro Miyajima","display_name":"f-miyajima","email":"fj6257jz@fujitsu.com","username":"f-miyajima"},"change_message_id":"6ac5ad0c07fa0a413a519bb17fcdf48237969b65","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"f3b5ad3a_bb973a0e","updated":"2024-07-31 07:04:26.000000000","message":"recheck - infra failure in `tacker-functional-devstack-enhanced-policy-sol` job","commit_id":"5ceee98ca0f382d59257087a5b0bee3d8c90e34d"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"49c68eee68f4fdd6c3102b585364cc487e19d779","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"4563d1f6_57394644","updated":"2024-09-11 00:27:12.000000000","message":"LGTM, thanks!","commit_id":"e85355a73335da529091d729ae96eb3219c7722b"}],"tacker/sol_refactored/common/config.py":[{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"5f572fbbabc63250505d65a3a57ded39ef5c2e7b","unresolved":true,"context_lines":[{"line_number":135,"context_line":"             default\u003d\u0027/var/lib/tacker/terraform\u0027,"},{"line_number":136,"context_line":"             help\u003d_(\u0027Temporary directory for Terraform infra-driver to \u0027"},{"line_number":137,"context_line":"                    \u0027store terraform config files\u0027)),"},{"line_number":138,"context_line":"    cfg.BoolOpt(\u0027nova_verify_cert\u0027,"},{"line_number":139,"context_line":"                default\u003dFalse,"},{"line_number":140,"context_line":"                help\u003d_(\u0027Enable certificate verification during SSL/TLS \u0027"},{"line_number":141,"context_line":"                       \u0027communication to nova server.\u0027)),"},{"line_number":142,"context_line":"    cfg.StrOpt(\u0027nova_ca_cert_file\u0027,"},{"line_number":143,"context_line":"               default\u003d\u0027\u0027,"},{"line_number":144,"context_line":"               help\u003d_(\u0027Specifies the root CA certificate to use when the \u0027"},{"line_number":145,"context_line":"                      \u0027nova_verify_cert option is True.\u0027))"},{"line_number":146,"context_line":"]"},{"line_number":147,"context_line":""},{"line_number":148,"context_line":"CONF.register_opts(VNFM_OPTS, \u0027v2_vnfm\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"d3cfe23d_453dce96","line":145,"range":{"start_line":138,"start_character":0,"end_line":145,"end_character":58},"updated":"2024-08-23 10:39:40.000000000","message":"Sorry, just a confirmation,\nIs it possible to set these parameters to `sol-https-v2` job and check it with Zuul Functional test like Heat\u0027s config [1]?\n[1] https://opendev.org/openstack/tacker/src/branch/stable/2024.1/.zuul.yaml#L576-L577","commit_id":"5ceee98ca0f382d59257087a5b0bee3d8c90e34d"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"4f34c3e5427d3569386ae26d4aa2b89e0d189f47","unresolved":false,"context_lines":[{"line_number":135,"context_line":"             default\u003d\u0027/var/lib/tacker/terraform\u0027,"},{"line_number":136,"context_line":"             help\u003d_(\u0027Temporary directory for Terraform infra-driver to \u0027"},{"line_number":137,"context_line":"                    \u0027store terraform config files\u0027)),"},{"line_number":138,"context_line":"    cfg.BoolOpt(\u0027nova_verify_cert\u0027,"},{"line_number":139,"context_line":"                default\u003dFalse,"},{"line_number":140,"context_line":"                help\u003d_(\u0027Enable certificate verification during SSL/TLS \u0027"},{"line_number":141,"context_line":"                       \u0027communication to nova server.\u0027)),"},{"line_number":142,"context_line":"    cfg.StrOpt(\u0027nova_ca_cert_file\u0027,"},{"line_number":143,"context_line":"               default\u003d\u0027\u0027,"},{"line_number":144,"context_line":"               help\u003d_(\u0027Specifies the root CA certificate to use when the \u0027"},{"line_number":145,"context_line":"                      \u0027nova_verify_cert option is True.\u0027))"},{"line_number":146,"context_line":"]"},{"line_number":147,"context_line":""},{"line_number":148,"context_line":"CONF.register_opts(VNFM_OPTS, \u0027v2_vnfm\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"84c278b8_82cdf024","line":145,"range":{"start_line":138,"start_character":0,"end_line":145,"end_character":58},"in_reply_to":"b1213076_1980d3a6","updated":"2024-09-04 09:19:16.000000000","message":"Thanks for your answer, sorry for the late reply.\nI understood as if it\u0027s not a test of AZ reselection, there\u0027s not much point in functional testing.","commit_id":"5ceee98ca0f382d59257087a5b0bee3d8c90e34d"},{"author":{"_account_id":36997,"name":"Takahiro Miyajima","display_name":"f-miyajima","email":"fj6257jz@fujitsu.com","username":"f-miyajima"},"change_message_id":"7755cdda664104e03af47eb7c44984d98adaed15","unresolved":true,"context_lines":[{"line_number":135,"context_line":"             default\u003d\u0027/var/lib/tacker/terraform\u0027,"},{"line_number":136,"context_line":"             help\u003d_(\u0027Temporary directory for Terraform infra-driver to \u0027"},{"line_number":137,"context_line":"                    \u0027store terraform config files\u0027)),"},{"line_number":138,"context_line":"    cfg.BoolOpt(\u0027nova_verify_cert\u0027,"},{"line_number":139,"context_line":"                default\u003dFalse,"},{"line_number":140,"context_line":"                help\u003d_(\u0027Enable certificate verification during SSL/TLS \u0027"},{"line_number":141,"context_line":"                       \u0027communication to nova server.\u0027)),"},{"line_number":142,"context_line":"    cfg.StrOpt(\u0027nova_ca_cert_file\u0027,"},{"line_number":143,"context_line":"               default\u003d\u0027\u0027,"},{"line_number":144,"context_line":"               help\u003d_(\u0027Specifies the root CA certificate to use when the \u0027"},{"line_number":145,"context_line":"                      \u0027nova_verify_cert option is True.\u0027))"},{"line_number":146,"context_line":"]"},{"line_number":147,"context_line":""},{"line_number":148,"context_line":"CONF.register_opts(VNFM_OPTS, \u0027v2_vnfm\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"b1213076_1980d3a6","line":145,"range":{"start_line":138,"start_character":0,"end_line":145,"end_character":58},"in_reply_to":"d3cfe23d_453dce96","updated":"2024-08-28 07:08:47.000000000","message":"It is difficult to do so for the `sol-https-v2` job.\nIn order to confirm the operation of nova this time, it is necessary to prepare the environment so that Availability Zone reselection is performed. If we apply an Availability Zone reselection environment to the `sol-https-v2` job, it is assumed that the existing FT fails to instantiate (they will need to be adjusted to work).","commit_id":"5ceee98ca0f382d59257087a5b0bee3d8c90e34d"}],"tacker/tests/unit/sol_refactored/infra_drivers/openstack/test_openstack.py":[{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"9e95b19a5a169e3f5735a0d421ca3c1270bab9d8","unresolved":true,"context_lines":[{"line_number":4760,"context_line":"        result \u003d self.driver._get_additional_vdu_id(grant_req, inst)"},{"line_number":4761,"context_line":"        self.assertEqual({\u0027VDU1-1\u0027, \u0027VDU1-2\u0027}, result)"},{"line_number":4762,"context_line":""},{"line_number":4763,"context_line":"    def test_nova_utils_init(self):"},{"line_number":4764,"context_line":"        vim_info \u003d objects.VimConnectionInfo.from_dict("},{"line_number":4765,"context_line":"            _vim_connection_info_example)"},{"line_number":4766,"context_line":"        nova_client \u003d nova_utils.NovaClient(vim_info)"}],"source_content_type":"text/x-python","patch_set":2,"id":"0b3c8173_1c5f9116","line":4763,"updated":"2024-09-06 19:00:15.000000000","message":"Please make it more directly to show what is expected in the test.","commit_id":"5ceee98ca0f382d59257087a5b0bee3d8c90e34d"},{"author":{"_account_id":36997,"name":"Takahiro Miyajima","display_name":"f-miyajima","email":"fj6257jz@fujitsu.com","username":"f-miyajima"},"change_message_id":"349250a5723037d888aade5554548ae545c034ab","unresolved":true,"context_lines":[{"line_number":4760,"context_line":"        result \u003d self.driver._get_additional_vdu_id(grant_req, inst)"},{"line_number":4761,"context_line":"        self.assertEqual({\u0027VDU1-1\u0027, \u0027VDU1-2\u0027}, result)"},{"line_number":4762,"context_line":""},{"line_number":4763,"context_line":"    def test_nova_utils_init(self):"},{"line_number":4764,"context_line":"        vim_info \u003d objects.VimConnectionInfo.from_dict("},{"line_number":4765,"context_line":"            _vim_connection_info_example)"},{"line_number":4766,"context_line":"        nova_client \u003d nova_utils.NovaClient(vim_info)"}],"source_content_type":"text/x-python","patch_set":2,"id":"e18b3662_04d1b9ab","line":4763,"in_reply_to":"0b3c8173_1c5f9116","updated":"2024-09-10 00:45:01.000000000","message":"I fixed the method name and added the comment of test content. Please kindly confirm.","commit_id":"5ceee98ca0f382d59257087a5b0bee3d8c90e34d"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"9e95b19a5a169e3f5735a0d421ca3c1270bab9d8","unresolved":true,"context_lines":[{"line_number":4767,"context_line":"        verify \u003d nova_client.client.auth_handle.verify"},{"line_number":4768,"context_line":"        self.assertEqual(False, verify)"},{"line_number":4769,"context_line":""},{"line_number":4770,"context_line":"    def test_nova_utils_init_verify(self):"},{"line_number":4771,"context_line":"        CONF.v2_vnfm.nova_verify_cert \u003d True"},{"line_number":4772,"context_line":"        ca_cert_path \u003d \u0027ca_cert_path\u0027"},{"line_number":4773,"context_line":"        CONF.v2_vnfm.nova_ca_cert_file \u003d ca_cert_path"}],"source_content_type":"text/x-python","patch_set":2,"id":"56178a49_64833ff8","line":4770,"updated":"2024-09-06 19:00:15.000000000","message":"ditto","commit_id":"5ceee98ca0f382d59257087a5b0bee3d8c90e34d"},{"author":{"_account_id":36997,"name":"Takahiro Miyajima","display_name":"f-miyajima","email":"fj6257jz@fujitsu.com","username":"f-miyajima"},"change_message_id":"349250a5723037d888aade5554548ae545c034ab","unresolved":true,"context_lines":[{"line_number":4767,"context_line":"        verify \u003d nova_client.client.auth_handle.verify"},{"line_number":4768,"context_line":"        self.assertEqual(False, verify)"},{"line_number":4769,"context_line":""},{"line_number":4770,"context_line":"    def test_nova_utils_init_verify(self):"},{"line_number":4771,"context_line":"        CONF.v2_vnfm.nova_verify_cert \u003d True"},{"line_number":4772,"context_line":"        ca_cert_path \u003d \u0027ca_cert_path\u0027"},{"line_number":4773,"context_line":"        CONF.v2_vnfm.nova_ca_cert_file \u003d ca_cert_path"}],"source_content_type":"text/x-python","patch_set":2,"id":"8abd4f3e_9d48147f","line":4770,"in_reply_to":"56178a49_64833ff8","updated":"2024-09-10 00:45:01.000000000","message":"I fixed the method name and added the comment of test content. Please kindly confirm.","commit_id":"5ceee98ca0f382d59257087a5b0bee3d8c90e34d"}]}