)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"62a8c3cd800867517a51373b4745344d0f5ac68b","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Masaki Oyama \u003cma-ooyama@kddi.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2022-12-19 04:30:46 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Support onboarding in multiple conductors environment"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This specification describes enhancement of managing VNF packages"},{"line_number":10,"context_line":"by multiple Tacker conductors."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"2b04a949_aaa793c9","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":53},"updated":"2022-12-19 09:36:25.000000000","message":"Could you follow the guideline of commit message?\n\nhttps://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_Git_commit_message_structure\n\u003e The first line should be limited to 50 characters and should not end with a period.","commit_id":"92b693f64d59b394b8f6d9e697cce58cde38f538"},{"author":{"_account_id":34107,"name":"Masaki Oyama","email":"ma-ooyama@kddi.com","username":"oyamamasaki"},"change_message_id":"57879aeef4791129d0ed1de8fd79d08502715735","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Masaki Oyama \u003cma-ooyama@kddi.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2022-12-19 04:30:46 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Support onboarding in multiple conductors environment"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This specification describes enhancement of managing VNF packages"},{"line_number":10,"context_line":"by multiple Tacker conductors."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"6efc27b2_ea963610","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":53},"in_reply_to":"2b04a949_aaa793c9","updated":"2022-12-22 05:36:35.000000000","message":"I fixed the first line to be under 50 characters.","commit_id":"92b693f64d59b394b8f6d9e697cce58cde38f538"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"f5cfa37a000e9189587ff37bcd572707e85eb193","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Masaki Oyama \u003cma-ooyama@kddi.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2022-12-19 04:30:46 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Support onboarding in multiple conductors environment"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This specification describes enhancement of managing VNF packages"},{"line_number":10,"context_line":"by multiple Tacker conductors."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"461eb078_143d8d97","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":53},"in_reply_to":"6efc27b2_ea963610","updated":"2022-12-25 23:52:19.000000000","message":"Ack","commit_id":"92b693f64d59b394b8f6d9e697cce58cde38f538"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"72632a8c4899fb3f95b108af709f60bc482ab7c4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f011811a_3930a949","updated":"2022-12-19 09:28:33.000000000","message":"Thanks for your patch!\nYou may be in the middle of writing it, but I would like to comment. Please kindly find my comments.","commit_id":"92b693f64d59b394b8f6d9e697cce58cde38f538"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"c5eb65e720fd3882ab6f3b5b6e09450694e1b07d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"7efafe36_8431858e","updated":"2022-12-23 01:28:24.000000000","message":"Could you tell us which item in the previous topics corresponds to the spec?","commit_id":"8b81a73ef568b250e9cb669f1f00b311ec15a806"},{"author":{"_account_id":34107,"name":"Masaki Oyama","email":"ma-ooyama@kddi.com","username":"oyamamasaki"},"change_message_id":"a43f6cb573b1cb8b2d902b95ec7e804a2095eebb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"f0b8b60a_b866588a","updated":"2022-12-22 08:47:21.000000000","message":"Please kindly review this spec.","commit_id":"8b81a73ef568b250e9cb669f1f00b311ec15a806"},{"author":{"_account_id":34107,"name":"Masaki Oyama","email":"ma-ooyama@kddi.com","username":"oyamamasaki"},"change_message_id":"82f8b46d6ac7869d3c0d6ee37fe2b7821f13c3cf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"fb86af9c_97b308ef","updated":"2022-12-26 03:45:13.000000000","message":"Thank you for your comment.","commit_id":"8b81a73ef568b250e9cb669f1f00b311ec15a806"},{"author":{"_account_id":34107,"name":"Masaki Oyama","email":"ma-ooyama@kddi.com","username":"oyamamasaki"},"change_message_id":"57879aeef4791129d0ed1de8fd79d08502715735","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"e36a1535_07122e60","updated":"2022-12-22 05:36:35.000000000","message":"Thank you for your comments.","commit_id":"8b81a73ef568b250e9cb669f1f00b311ec15a806"},{"author":{"_account_id":32102,"name":"Manpreet Kaur","email":"kaurmanpreet2620@gmail.com","username":"manpreet"},"change_message_id":"a299b790f55612d090cf6c6eda46b41d9bb542f3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"67fa53db_71159525","updated":"2022-12-26 05:46:34.000000000","message":"Thanks for your patch, please find few comments.","commit_id":"8b81a73ef568b250e9cb669f1f00b311ec15a806"},{"author":{"_account_id":34107,"name":"Masaki Oyama","email":"ma-ooyama@kddi.com","username":"oyamamasaki"},"change_message_id":"82f8b46d6ac7869d3c0d6ee37fe2b7821f13c3cf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"d123bac4_bfd66122","in_reply_to":"7efafe36_8431858e","updated":"2022-12-26 03:45:13.000000000","message":"This spec is related to the topic about supporting object storage.\nAs a result of investigation, I understood tacker already can use object storage for backend storage.\nBut it doesn\u0027t work if multiple tacker conductors are used.\nSo this spec improves this point.","commit_id":"8b81a73ef568b250e9cb669f1f00b311ec15a806"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"4bed19640239a3b34e2fb43305034b5c27685c6f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"01182fe9_c41485cb","updated":"2022-12-28 06:39:16.000000000","message":"Let me confirm some failure cases of downloading CSAR file although I don\u0027t have any objection for the proposal basically.","commit_id":"07679b01919d68f4271a3d390cad1bdf5696fc00"},{"author":{"_account_id":34107,"name":"Masaki Oyama","email":"ma-ooyama@kddi.com","username":"oyamamasaki"},"change_message_id":"34fc0682d1d140c0fdc860ce0c2b285ce1923f68","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"0e3dd51a_4c0b392e","updated":"2022-12-28 07:16:16.000000000","message":"Thank you for your comments.","commit_id":"07679b01919d68f4271a3d390cad1bdf5696fc00"},{"author":{"_account_id":34107,"name":"Masaki Oyama","email":"ma-ooyama@kddi.com","username":"oyamamasaki"},"change_message_id":"a234a362f3bd1d428af22999856082231649aa67","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"3983300d_4d66e316","updated":"2022-12-26 10:16:09.000000000","message":"Thank you for your comments.","commit_id":"07679b01919d68f4271a3d390cad1bdf5696fc00"},{"author":{"_account_id":32102,"name":"Manpreet Kaur","email":"kaurmanpreet2620@gmail.com","username":"manpreet"},"change_message_id":"c5a58d4c0b1d033817b41445c6644606641f4fc2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"9673a120_b6eceabc","updated":"2022-12-28 01:53:53.000000000","message":"Thanks for rework. ","commit_id":"07679b01919d68f4271a3d390cad1bdf5696fc00"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"5b072d22f92df05e2eb0994e9e670bd97e44b905","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"a8439d33_515e74a9","updated":"2022-12-28 17:30:38.000000000","message":"I think we need to have some more discussion about test strategies as replied to your comment, I agree to your proposal. Thanks.","commit_id":"9b5230b69b0095e37a36e512a2700603f88c5a91"}],"specs/2023.1/support-multi-conductors-onboarding.rst":[{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"72632a8c4899fb3f95b108af709f60bc482ab7c4","unresolved":true,"context_lines":[{"line_number":59,"context_line":"    [label \u003d \"\"]"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"    \"Tacker-server\" -\u003e \"Messaging Queue\""},{"line_number":62,"context_line":"    [label \u003d \"4. Send a rpc message of \u0027Upload VNF Package from content\u0027 with fanout\u003dTrue\"]"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"    Client \u003c- \"Tacker-server\""},{"line_number":65,"context_line":"    [label \u003d \"response 202 Accepted\"]"}],"source_content_type":"text/x-rst","patch_set":1,"id":"f31445e8_f0c4b29f","line":62,"range":{"start_line":62,"start_character":85,"end_line":62,"end_character":89},"updated":"2022-12-19 09:28:33.000000000","message":"\"False\" is correct?","commit_id":"92b693f64d59b394b8f6d9e697cce58cde38f538"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"f5cfa37a000e9189587ff37bcd572707e85eb193","unresolved":false,"context_lines":[{"line_number":59,"context_line":"    [label \u003d \"\"]"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"    \"Tacker-server\" -\u003e \"Messaging Queue\""},{"line_number":62,"context_line":"    [label \u003d \"4. Send a rpc message of \u0027Upload VNF Package from content\u0027 with fanout\u003dTrue\"]"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"    Client \u003c- \"Tacker-server\""},{"line_number":65,"context_line":"    [label \u003d \"response 202 Accepted\"]"}],"source_content_type":"text/x-rst","patch_set":1,"id":"0a7a0fb1_5b139765","line":62,"range":{"start_line":62,"start_character":85,"end_line":62,"end_character":89},"in_reply_to":"f11e2d83_ed77a2e8","updated":"2022-12-25 23:52:19.000000000","message":"Ack","commit_id":"92b693f64d59b394b8f6d9e697cce58cde38f538"},{"author":{"_account_id":34107,"name":"Masaki Oyama","email":"ma-ooyama@kddi.com","username":"oyamamasaki"},"change_message_id":"57879aeef4791129d0ed1de8fd79d08502715735","unresolved":true,"context_lines":[{"line_number":59,"context_line":"    [label \u003d \"\"]"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"    \"Tacker-server\" -\u003e \"Messaging Queue\""},{"line_number":62,"context_line":"    [label \u003d \"4. Send a rpc message of \u0027Upload VNF Package from content\u0027 with fanout\u003dTrue\"]"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"    Client \u003c- \"Tacker-server\""},{"line_number":65,"context_line":"    [label \u003d \"response 202 Accepted\"]"}],"source_content_type":"text/x-rst","patch_set":1,"id":"f11e2d83_ed77a2e8","line":62,"range":{"start_line":62,"start_character":85,"end_line":62,"end_character":89},"in_reply_to":"f31445e8_f0c4b29f","updated":"2022-12-22 05:36:35.000000000","message":"You are right. I fixed it to \"False\".","commit_id":"92b693f64d59b394b8f6d9e697cce58cde38f538"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"72632a8c4899fb3f95b108af709f60bc482ab7c4","unresolved":true,"context_lines":[{"line_number":180,"context_line":"Proposed change"},{"line_number":181,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":182,"context_line":""},{"line_number":183,"context_line":"We would enable all conductors in the N-Act cluster"},{"line_number":184,"context_line":"to download a CSAR file at the onboarding process."},{"line_number":185,"context_line":""},{"line_number":186,"context_line":"The process of onboarding VNF package"},{"line_number":187,"context_line":"-------------------------------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"6276f66f_ff78d973","line":184,"range":{"start_line":183,"start_character":0,"end_line":184,"end_character":50},"updated":"2022-12-19 09:28:33.000000000","message":"Just a question,\nWould a non-\"N-Act\" configuration also change the following Onboarding Process?","commit_id":"92b693f64d59b394b8f6d9e697cce58cde38f538"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"f5cfa37a000e9189587ff37bcd572707e85eb193","unresolved":false,"context_lines":[{"line_number":180,"context_line":"Proposed change"},{"line_number":181,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":182,"context_line":""},{"line_number":183,"context_line":"We would enable all conductors in the N-Act cluster"},{"line_number":184,"context_line":"to download a CSAR file at the onboarding process."},{"line_number":185,"context_line":""},{"line_number":186,"context_line":"The process of onboarding VNF package"},{"line_number":187,"context_line":"-------------------------------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"49a1ac1a_9c660f76","line":184,"range":{"start_line":183,"start_character":0,"end_line":184,"end_character":50},"in_reply_to":"2529f21d_aec63bc0","updated":"2022-12-25 23:52:19.000000000","message":"Thanks for your answer, I got it.","commit_id":"92b693f64d59b394b8f6d9e697cce58cde38f538"},{"author":{"_account_id":34107,"name":"Masaki Oyama","email":"ma-ooyama@kddi.com","username":"oyamamasaki"},"change_message_id":"57879aeef4791129d0ed1de8fd79d08502715735","unresolved":true,"context_lines":[{"line_number":180,"context_line":"Proposed change"},{"line_number":181,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":182,"context_line":""},{"line_number":183,"context_line":"We would enable all conductors in the N-Act cluster"},{"line_number":184,"context_line":"to download a CSAR file at the onboarding process."},{"line_number":185,"context_line":""},{"line_number":186,"context_line":"The process of onboarding VNF package"},{"line_number":187,"context_line":"-------------------------------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"2529f21d_aec63bc0","line":184,"range":{"start_line":183,"start_character":0,"end_line":184,"end_character":50},"in_reply_to":"6276f66f_ff78d973","updated":"2022-12-22 05:36:35.000000000","message":"Yes, it follows the same process when a single tacker-conductor is used.","commit_id":"92b693f64d59b394b8f6d9e697cce58cde38f538"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"72632a8c4899fb3f95b108af709f60bc482ab7c4","unresolved":true,"context_lines":[{"line_number":515,"context_line":""},{"line_number":516,"context_line":"#. The Tacker-conductor downloads the VNF packages to its local file systems."},{"line_number":517,"context_line":""},{"line_number":518,"context_line":".. note:: Tacker-conductor must not receive any requests"},{"line_number":519,"context_line":"          until the above process is completed."},{"line_number":520,"context_line":""},{"line_number":521,"context_line":"Data model impact"},{"line_number":522,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"4545f975_523a8375","line":519,"range":{"start_line":518,"start_character":10,"end_line":519,"end_character":47},"updated":"2022-12-19 09:28:33.000000000","message":"Do `any requests` include lcm commands (instantiate, terminate .. etc) that associated with vnf package already onboarded?\nI don\u0027t think there is such a guard in the current implementation, is it wrong?","commit_id":"92b693f64d59b394b8f6d9e697cce58cde38f538"},{"author":{"_account_id":34107,"name":"Masaki Oyama","email":"ma-ooyama@kddi.com","username":"oyamamasaki"},"change_message_id":"57879aeef4791129d0ed1de8fd79d08502715735","unresolved":true,"context_lines":[{"line_number":515,"context_line":""},{"line_number":516,"context_line":"#. The Tacker-conductor downloads the VNF packages to its local file systems."},{"line_number":517,"context_line":""},{"line_number":518,"context_line":".. note:: Tacker-conductor must not receive any requests"},{"line_number":519,"context_line":"          until the above process is completed."},{"line_number":520,"context_line":""},{"line_number":521,"context_line":"Data model impact"},{"line_number":522,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"c5932188_788a98d9","line":519,"range":{"start_line":518,"start_character":10,"end_line":519,"end_character":47},"in_reply_to":"4545f975_523a8375","updated":"2022-12-22 05:36:35.000000000","message":"Yes, those requests are included.\n\nAs you said, there is not such a function now. So I\u0027ll implement that.\nConcretely, this process that the conductor newly added downloads the vnf packages  should be executed before the conductor connects rpc.","commit_id":"92b693f64d59b394b8f6d9e697cce58cde38f538"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"7675cac2b8fc6896dff03d0cf0cb3514dfd8ec6c","unresolved":false,"context_lines":[{"line_number":515,"context_line":""},{"line_number":516,"context_line":"#. The Tacker-conductor downloads the VNF packages to its local file systems."},{"line_number":517,"context_line":""},{"line_number":518,"context_line":".. note:: Tacker-conductor must not receive any requests"},{"line_number":519,"context_line":"          until the above process is completed."},{"line_number":520,"context_line":""},{"line_number":521,"context_line":"Data model impact"},{"line_number":522,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"02fac76e_e0493a27","line":519,"range":{"start_line":518,"start_character":10,"end_line":519,"end_character":47},"in_reply_to":"bc0a881e_aa71c14b","updated":"2022-12-27 23:36:27.000000000","message":"Thanks for your explanation! I understood that this \"Tacker-conductor\" means newly added one.","commit_id":"92b693f64d59b394b8f6d9e697cce58cde38f538"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"f5cfa37a000e9189587ff37bcd572707e85eb193","unresolved":true,"context_lines":[{"line_number":515,"context_line":""},{"line_number":516,"context_line":"#. The Tacker-conductor downloads the VNF packages to its local file systems."},{"line_number":517,"context_line":""},{"line_number":518,"context_line":".. note:: Tacker-conductor must not receive any requests"},{"line_number":519,"context_line":"          until the above process is completed."},{"line_number":520,"context_line":""},{"line_number":521,"context_line":"Data model impact"},{"line_number":522,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"e44e790c_ee6e2c0d","line":519,"range":{"start_line":518,"start_character":10,"end_line":519,"end_character":47},"in_reply_to":"c5932188_788a98d9","updated":"2022-12-25 23:52:19.000000000","message":"Does that mean that while uploading a package containing a large image, I will not be able to perform any operations on other VNF Instances? That would have a serious impact on users.\n\nIs there any way to avoid implementing the guard mentioned here?\n\nTo: Ogawa-san, Takahashi-san\nDo you have any comments on this?","commit_id":"92b693f64d59b394b8f6d9e697cce58cde38f538"},{"author":{"_account_id":34107,"name":"Masaki Oyama","email":"ma-ooyama@kddi.com","username":"oyamamasaki"},"change_message_id":"b60002485ce73802b4807b89bceeb2fe94ccb83e","unresolved":true,"context_lines":[{"line_number":515,"context_line":""},{"line_number":516,"context_line":"#. The Tacker-conductor downloads the VNF packages to its local file systems."},{"line_number":517,"context_line":""},{"line_number":518,"context_line":".. note:: Tacker-conductor must not receive any requests"},{"line_number":519,"context_line":"          until the above process is completed."},{"line_number":520,"context_line":""},{"line_number":521,"context_line":"Data model impact"},{"line_number":522,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"bc0a881e_aa71c14b","line":519,"range":{"start_line":518,"start_character":10,"end_line":519,"end_character":47},"in_reply_to":"e44e790c_ee6e2c0d","updated":"2022-12-27 11:08:18.000000000","message":"Sorry, I mean the tacker-conductor newly added to the cluster should connect rpc after downloading VNF packages.\nSo other existing tacker-conductors could operate LCM while such a process.","commit_id":"92b693f64d59b394b8f6d9e697cce58cde38f538"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"72632a8c4899fb3f95b108af709f60bc482ab7c4","unresolved":true,"context_lines":[{"line_number":568,"context_line":"  Masaki Oyama \u003cma-ooyama@kddi.com\u003e"},{"line_number":569,"context_line":""},{"line_number":570,"context_line":"Other contributors:"},{"line_number":571,"context_line":"  Hitomi Koba \u003chi-koba@kddi.com\u003e"},{"line_number":572,"context_line":"  Yukihiro Kinjo \u003cyu-kinjou@kddi.com\u003e"},{"line_number":573,"context_line":"  Xu Hongjin \u003cho-xu@kddi.com\u003e"},{"line_number":574,"context_line":""},{"line_number":575,"context_line":""},{"line_number":576,"context_line":"Work Items"}],"source_content_type":"text/x-rst","patch_set":1,"id":"0fba9cba_ebfc0435","line":573,"range":{"start_line":571,"start_character":0,"end_line":573,"end_character":29},"updated":"2022-12-19 09:28:33.000000000","message":"Could you insert a blank line between each contributor?","commit_id":"92b693f64d59b394b8f6d9e697cce58cde38f538"},{"author":{"_account_id":34107,"name":"Masaki Oyama","email":"ma-ooyama@kddi.com","username":"oyamamasaki"},"change_message_id":"57879aeef4791129d0ed1de8fd79d08502715735","unresolved":true,"context_lines":[{"line_number":568,"context_line":"  Masaki Oyama \u003cma-ooyama@kddi.com\u003e"},{"line_number":569,"context_line":""},{"line_number":570,"context_line":"Other contributors:"},{"line_number":571,"context_line":"  Hitomi Koba \u003chi-koba@kddi.com\u003e"},{"line_number":572,"context_line":"  Yukihiro Kinjo \u003cyu-kinjou@kddi.com\u003e"},{"line_number":573,"context_line":"  Xu Hongjin \u003cho-xu@kddi.com\u003e"},{"line_number":574,"context_line":""},{"line_number":575,"context_line":""},{"line_number":576,"context_line":"Work Items"}],"source_content_type":"text/x-rst","patch_set":1,"id":"2ad5fe4a_47457852","line":573,"range":{"start_line":571,"start_character":0,"end_line":573,"end_character":29},"in_reply_to":"0fba9cba_ebfc0435","updated":"2022-12-22 05:36:35.000000000","message":"Fixed.","commit_id":"92b693f64d59b394b8f6d9e697cce58cde38f538"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"f5cfa37a000e9189587ff37bcd572707e85eb193","unresolved":false,"context_lines":[{"line_number":568,"context_line":"  Masaki Oyama \u003cma-ooyama@kddi.com\u003e"},{"line_number":569,"context_line":""},{"line_number":570,"context_line":"Other contributors:"},{"line_number":571,"context_line":"  Hitomi Koba \u003chi-koba@kddi.com\u003e"},{"line_number":572,"context_line":"  Yukihiro Kinjo \u003cyu-kinjou@kddi.com\u003e"},{"line_number":573,"context_line":"  Xu Hongjin \u003cho-xu@kddi.com\u003e"},{"line_number":574,"context_line":""},{"line_number":575,"context_line":""},{"line_number":576,"context_line":"Work Items"}],"source_content_type":"text/x-rst","patch_set":1,"id":"c3c6b076_959604c9","line":573,"range":{"start_line":571,"start_character":0,"end_line":573,"end_character":29},"in_reply_to":"2ad5fe4a_47457852","updated":"2022-12-25 23:52:19.000000000","message":"Ack","commit_id":"92b693f64d59b394b8f6d9e697cce58cde38f538"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"72632a8c4899fb3f95b108af709f60bc482ab7c4","unresolved":true,"context_lines":[{"line_number":568,"context_line":"  Masaki Oyama \u003cma-ooyama@kddi.com\u003e"},{"line_number":569,"context_line":""},{"line_number":570,"context_line":"Other contributors:"},{"line_number":571,"context_line":"  Hitomi Koba \u003chi-koba@kddi.com\u003e"},{"line_number":572,"context_line":"  Yukihiro Kinjo \u003cyu-kinjou@kddi.com\u003e"},{"line_number":573,"context_line":"  Xu Hongjin \u003cho-xu@kddi.com\u003e"},{"line_number":574,"context_line":""},{"line_number":575,"context_line":""},{"line_number":576,"context_line":"Work Items"},{"line_number":577,"context_line":"----------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"07e0e233_e1538772","line":574,"range":{"start_line":571,"start_character":0,"end_line":574,"end_character":0},"updated":"2022-12-19 09:28:33.000000000","message":"Could you add between","commit_id":"92b693f64d59b394b8f6d9e697cce58cde38f538"},{"author":{"_account_id":34107,"name":"Masaki Oyama","email":"ma-ooyama@kddi.com","username":"oyamamasaki"},"change_message_id":"57879aeef4791129d0ed1de8fd79d08502715735","unresolved":true,"context_lines":[{"line_number":568,"context_line":"  Masaki Oyama \u003cma-ooyama@kddi.com\u003e"},{"line_number":569,"context_line":""},{"line_number":570,"context_line":"Other contributors:"},{"line_number":571,"context_line":"  Hitomi Koba \u003chi-koba@kddi.com\u003e"},{"line_number":572,"context_line":"  Yukihiro Kinjo \u003cyu-kinjou@kddi.com\u003e"},{"line_number":573,"context_line":"  Xu Hongjin \u003cho-xu@kddi.com\u003e"},{"line_number":574,"context_line":""},{"line_number":575,"context_line":""},{"line_number":576,"context_line":"Work Items"},{"line_number":577,"context_line":"----------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"d67d823e_9e099e55","line":574,"range":{"start_line":571,"start_character":0,"end_line":574,"end_character":0},"in_reply_to":"07e0e233_e1538772","updated":"2022-12-22 05:36:35.000000000","message":"This comment seems same as above.","commit_id":"92b693f64d59b394b8f6d9e697cce58cde38f538"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"f5cfa37a000e9189587ff37bcd572707e85eb193","unresolved":false,"context_lines":[{"line_number":568,"context_line":"  Masaki Oyama \u003cma-ooyama@kddi.com\u003e"},{"line_number":569,"context_line":""},{"line_number":570,"context_line":"Other contributors:"},{"line_number":571,"context_line":"  Hitomi Koba \u003chi-koba@kddi.com\u003e"},{"line_number":572,"context_line":"  Yukihiro Kinjo \u003cyu-kinjou@kddi.com\u003e"},{"line_number":573,"context_line":"  Xu Hongjin \u003cho-xu@kddi.com\u003e"},{"line_number":574,"context_line":""},{"line_number":575,"context_line":""},{"line_number":576,"context_line":"Work Items"},{"line_number":577,"context_line":"----------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"0f4f7615_6d4ee62a","line":574,"range":{"start_line":571,"start_character":0,"end_line":574,"end_character":0},"in_reply_to":"d67d823e_9e099e55","updated":"2022-12-25 23:52:19.000000000","message":"Sorry, wrong comment.","commit_id":"92b693f64d59b394b8f6d9e697cce58cde38f538"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"72632a8c4899fb3f95b108af709f60bc482ab7c4","unresolved":true,"context_lines":[{"line_number":577,"context_line":"----------"},{"line_number":578,"context_line":"* Implement onboarding function"},{"line_number":579,"context_line":"* Add unit test"},{"line_number":580,"context_line":"* Add functional test"},{"line_number":581,"context_line":""},{"line_number":582,"context_line":"Dependencies"},{"line_number":583,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":1,"id":"98287dc4_e96de3e9","line":580,"range":{"start_line":580,"start_character":2,"end_line":580,"end_character":21},"updated":"2022-12-19 09:28:33.000000000","message":"Just a comment,\nIf you want to add automated tests in Zuul, I think you need to change the host setting in current Zuul environment.","commit_id":"92b693f64d59b394b8f6d9e697cce58cde38f538"},{"author":{"_account_id":34107,"name":"Masaki Oyama","email":"ma-ooyama@kddi.com","username":"oyamamasaki"},"change_message_id":"57879aeef4791129d0ed1de8fd79d08502715735","unresolved":true,"context_lines":[{"line_number":577,"context_line":"----------"},{"line_number":578,"context_line":"* Implement onboarding function"},{"line_number":579,"context_line":"* Add unit test"},{"line_number":580,"context_line":"* Add functional test"},{"line_number":581,"context_line":""},{"line_number":582,"context_line":"Dependencies"},{"line_number":583,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":1,"id":"d68a0e7f_45c5044f","line":580,"range":{"start_line":580,"start_character":2,"end_line":580,"end_character":21},"in_reply_to":"98287dc4_e96de3e9","updated":"2022-12-22 05:36:35.000000000","message":"Yes, the automated tests for multiple tacker-conductors are absolutely needed.\nI\u0027ll implement the tests but haven\u0027t yet considered details of how to implement that.","commit_id":"92b693f64d59b394b8f6d9e697cce58cde38f538"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"f5cfa37a000e9189587ff37bcd572707e85eb193","unresolved":false,"context_lines":[{"line_number":577,"context_line":"----------"},{"line_number":578,"context_line":"* Implement onboarding function"},{"line_number":579,"context_line":"* Add unit test"},{"line_number":580,"context_line":"* Add functional test"},{"line_number":581,"context_line":""},{"line_number":582,"context_line":"Dependencies"},{"line_number":583,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":1,"id":"f62571ae_80cf4963","line":580,"range":{"start_line":580,"start_character":2,"end_line":580,"end_character":21},"in_reply_to":"d68a0e7f_45c5044f","updated":"2022-12-25 23:52:19.000000000","message":"Thanks for your answer.\nSince we have experienced difficulties in creating a new Zuul environment, it is a better to create a test patch that creates a Zuul job in advance. This is just a comment.","commit_id":"92b693f64d59b394b8f6d9e697cce58cde38f538"},{"author":{"_account_id":32102,"name":"Manpreet Kaur","email":"kaurmanpreet2620@gmail.com","username":"manpreet"},"change_message_id":"a299b790f55612d090cf6c6eda46b41d9bb542f3","unresolved":true,"context_lines":[{"line_number":70,"context_line":"    \"Tacker-conductor 1\" -\u003e \"Backend Storage\""},{"line_number":71,"context_line":"    [label \u003d \"5. Download the CSAR file to local file system\"]"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"    \"Tacker-conductor 1\" \u003c- \"Backend Storage\""},{"line_number":74,"context_line":"    [label \u003d \"\"]"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"    \"Tacker-conductor 1\" -\u003e \"DB\""}],"source_content_type":"text/x-rst","patch_set":2,"id":"c0a1aa32_09f989a4","line":73,"range":{"start_line":73,"start_character":0,"end_line":73,"end_character":45},"updated":"2022-12-26 05:46:34.000000000","message":"Just a suggestion. \nIMO, the step#5 description should be label here as \"Tacker-conductor 1\" downloaded CSAR file from Backend Storage to its local file system. Whereas the previous step is moreover a request raised by \"Tacker-conductor 1\" to the download CSAR file.","commit_id":"8b81a73ef568b250e9cb669f1f00b311ec15a806"},{"author":{"_account_id":32102,"name":"Manpreet Kaur","email":"kaurmanpreet2620@gmail.com","username":"manpreet"},"change_message_id":"c5a58d4c0b1d033817b41445c6644606641f4fc2","unresolved":false,"context_lines":[{"line_number":70,"context_line":"    \"Tacker-conductor 1\" -\u003e \"Backend Storage\""},{"line_number":71,"context_line":"    [label \u003d \"5. Download the CSAR file to local file system\"]"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"    \"Tacker-conductor 1\" \u003c- \"Backend Storage\""},{"line_number":74,"context_line":"    [label \u003d \"\"]"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"    \"Tacker-conductor 1\" -\u003e \"DB\""}],"source_content_type":"text/x-rst","patch_set":2,"id":"4d3c5766_0d2c9841","line":73,"range":{"start_line":73,"start_character":0,"end_line":73,"end_character":45},"in_reply_to":"8941c52e_9bd0ba47","updated":"2022-12-28 01:53:53.000000000","message":"Ack","commit_id":"8b81a73ef568b250e9cb669f1f00b311ec15a806"},{"author":{"_account_id":34107,"name":"Masaki Oyama","email":"ma-ooyama@kddi.com","username":"oyamamasaki"},"change_message_id":"a234a362f3bd1d428af22999856082231649aa67","unresolved":true,"context_lines":[{"line_number":70,"context_line":"    \"Tacker-conductor 1\" -\u003e \"Backend Storage\""},{"line_number":71,"context_line":"    [label \u003d \"5. Download the CSAR file to local file system\"]"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"    \"Tacker-conductor 1\" \u003c- \"Backend Storage\""},{"line_number":74,"context_line":"    [label \u003d \"\"]"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"    \"Tacker-conductor 1\" -\u003e \"DB\""}],"source_content_type":"text/x-rst","patch_set":2,"id":"8941c52e_9bd0ba47","line":73,"range":{"start_line":73,"start_character":0,"end_line":73,"end_character":45},"in_reply_to":"c0a1aa32_09f989a4","updated":"2022-12-26 10:16:09.000000000","message":"Changed the label here as \"5. Send a request to download the CSAR file\".\nAlso changed the label next step as \"Download the CSAR file from Backend Storage to the local file system\".","commit_id":"8b81a73ef568b250e9cb669f1f00b311ec15a806"},{"author":{"_account_id":32102,"name":"Manpreet Kaur","email":"kaurmanpreet2620@gmail.com","username":"manpreet"},"change_message_id":"a299b790f55612d090cf6c6eda46b41d9bb542f3","unresolved":true,"context_lines":[{"line_number":138,"context_line":"    [label \u003d \"\"]"},{"line_number":139,"context_line":""},{"line_number":140,"context_line":"    \"Tacker-conductor 1\" -\u003e \"Backend Storage\""},{"line_number":141,"context_line":"    [label \u003d \"5. Download the CSAR file to local file system\"]"},{"line_number":142,"context_line":""},{"line_number":143,"context_line":"    \"Tacker-conductor 1\" -\u003e \"DB\""},{"line_number":144,"context_line":"    [label \u003d \"6. Update onboardingState to ONBOARDED\"]"}],"source_content_type":"text/x-rst","patch_set":2,"id":"948a93de_e3fdf6bd","line":141,"range":{"start_line":141,"start_character":0,"end_line":141,"end_character":62},"updated":"2022-12-26 05:46:34.000000000","message":"Ditto","commit_id":"8b81a73ef568b250e9cb669f1f00b311ec15a806"},{"author":{"_account_id":32102,"name":"Manpreet Kaur","email":"kaurmanpreet2620@gmail.com","username":"manpreet"},"change_message_id":"c5a58d4c0b1d033817b41445c6644606641f4fc2","unresolved":false,"context_lines":[{"line_number":138,"context_line":"    [label \u003d \"\"]"},{"line_number":139,"context_line":""},{"line_number":140,"context_line":"    \"Tacker-conductor 1\" -\u003e \"Backend Storage\""},{"line_number":141,"context_line":"    [label \u003d \"5. Download the CSAR file to local file system\"]"},{"line_number":142,"context_line":""},{"line_number":143,"context_line":"    \"Tacker-conductor 1\" -\u003e \"DB\""},{"line_number":144,"context_line":"    [label \u003d \"6. Update onboardingState to ONBOARDED\"]"}],"source_content_type":"text/x-rst","patch_set":2,"id":"0d38f8fd_c3148ae5","line":141,"range":{"start_line":141,"start_character":0,"end_line":141,"end_character":62},"in_reply_to":"03bccc36_e396acc0","updated":"2022-12-28 01:53:53.000000000","message":"Ack","commit_id":"8b81a73ef568b250e9cb669f1f00b311ec15a806"},{"author":{"_account_id":34107,"name":"Masaki Oyama","email":"ma-ooyama@kddi.com","username":"oyamamasaki"},"change_message_id":"a234a362f3bd1d428af22999856082231649aa67","unresolved":true,"context_lines":[{"line_number":138,"context_line":"    [label \u003d \"\"]"},{"line_number":139,"context_line":""},{"line_number":140,"context_line":"    \"Tacker-conductor 1\" -\u003e \"Backend Storage\""},{"line_number":141,"context_line":"    [label \u003d \"5. Download the CSAR file to local file system\"]"},{"line_number":142,"context_line":""},{"line_number":143,"context_line":"    \"Tacker-conductor 1\" -\u003e \"DB\""},{"line_number":144,"context_line":"    [label \u003d \"6. Update onboardingState to ONBOARDED\"]"}],"source_content_type":"text/x-rst","patch_set":2,"id":"03bccc36_e396acc0","line":141,"range":{"start_line":141,"start_character":0,"end_line":141,"end_character":62},"in_reply_to":"948a93de_e3fdf6bd","updated":"2022-12-26 10:16:09.000000000","message":"Changed as same as above.\nAlso I noticed the location of the sentence about downloading the CSAR file is wrong. So I moved it to right location.","commit_id":"8b81a73ef568b250e9cb669f1f00b311ec15a806"},{"author":{"_account_id":32102,"name":"Manpreet Kaur","email":"kaurmanpreet2620@gmail.com","username":"manpreet"},"change_message_id":"a299b790f55612d090cf6c6eda46b41d9bb542f3","unresolved":true,"context_lines":[{"line_number":158,"context_line":"          ``[glance_store]`` section in ``tacker.conf``."},{"line_number":159,"context_line":"          For example, the type of a backend storage is configured by"},{"line_number":160,"context_line":"          ``stores`` in ``[glance_store]``."},{"line_number":161,"context_line":"          Configurations of a local file system for vnf packages are defined"},{"line_number":162,"context_line":"          in  ``[vnf_package]`` section in ``tacker.conf``."},{"line_number":163,"context_line":"          For example, the path where vnf packages are downloaded"},{"line_number":164,"context_line":"          is configured by ``vnf_package_csar_path`` in ``[vnf_package]``."}],"source_content_type":"text/x-rst","patch_set":2,"id":"1007f91b_4b01b102","line":161,"range":{"start_line":161,"start_character":52,"end_line":161,"end_character":55},"updated":"2022-12-26 05:46:34.000000000","message":"Please change to \"VNF\".","commit_id":"8b81a73ef568b250e9cb669f1f00b311ec15a806"},{"author":{"_account_id":34107,"name":"Masaki Oyama","email":"ma-ooyama@kddi.com","username":"oyamamasaki"},"change_message_id":"a234a362f3bd1d428af22999856082231649aa67","unresolved":true,"context_lines":[{"line_number":158,"context_line":"          ``[glance_store]`` section in ``tacker.conf``."},{"line_number":159,"context_line":"          For example, the type of a backend storage is configured by"},{"line_number":160,"context_line":"          ``stores`` in ``[glance_store]``."},{"line_number":161,"context_line":"          Configurations of a local file system for vnf packages are defined"},{"line_number":162,"context_line":"          in  ``[vnf_package]`` section in ``tacker.conf``."},{"line_number":163,"context_line":"          For example, the path where vnf packages are downloaded"},{"line_number":164,"context_line":"          is configured by ``vnf_package_csar_path`` in ``[vnf_package]``."}],"source_content_type":"text/x-rst","patch_set":2,"id":"18af50ad_1c866d78","line":161,"range":{"start_line":161,"start_character":52,"end_line":161,"end_character":55},"in_reply_to":"1007f91b_4b01b102","updated":"2022-12-26 10:16:09.000000000","message":"I fixed it.","commit_id":"8b81a73ef568b250e9cb669f1f00b311ec15a806"},{"author":{"_account_id":32102,"name":"Manpreet Kaur","email":"kaurmanpreet2620@gmail.com","username":"manpreet"},"change_message_id":"c5a58d4c0b1d033817b41445c6644606641f4fc2","unresolved":false,"context_lines":[{"line_number":158,"context_line":"          ``[glance_store]`` section in ``tacker.conf``."},{"line_number":159,"context_line":"          For example, the type of a backend storage is configured by"},{"line_number":160,"context_line":"          ``stores`` in ``[glance_store]``."},{"line_number":161,"context_line":"          Configurations of a local file system for vnf packages are defined"},{"line_number":162,"context_line":"          in  ``[vnf_package]`` section in ``tacker.conf``."},{"line_number":163,"context_line":"          For example, the path where vnf packages are downloaded"},{"line_number":164,"context_line":"          is configured by ``vnf_package_csar_path`` in ``[vnf_package]``."}],"source_content_type":"text/x-rst","patch_set":2,"id":"12942fea_535c6c32","line":161,"range":{"start_line":161,"start_character":52,"end_line":161,"end_character":55},"in_reply_to":"18af50ad_1c866d78","updated":"2022-12-28 01:53:53.000000000","message":"Ack","commit_id":"8b81a73ef568b250e9cb669f1f00b311ec15a806"},{"author":{"_account_id":32102,"name":"Manpreet Kaur","email":"kaurmanpreet2620@gmail.com","username":"manpreet"},"change_message_id":"a299b790f55612d090cf6c6eda46b41d9bb542f3","unresolved":true,"context_lines":[{"line_number":168,"context_line":"can download a CSAR file to its local file system"},{"line_number":169,"context_line":"following the process above."},{"line_number":170,"context_line":"But other conductors that don\u0027t receive the request"},{"line_number":171,"context_line":"can\u0027t download a CSAR file at the timing,"},{"line_number":172,"context_line":"and so these conductors can\u0027t operate instantiation"},{"line_number":173,"context_line":"because they can\u0027t refer the CSAR file."},{"line_number":174,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"c95c477a_9503be77","line":171,"range":{"start_line":171,"start_character":34,"end_line":171,"end_character":40},"updated":"2022-12-26 05:46:34.000000000","message":"Please change to \"time\".","commit_id":"8b81a73ef568b250e9cb669f1f00b311ec15a806"},{"author":{"_account_id":32102,"name":"Manpreet Kaur","email":"kaurmanpreet2620@gmail.com","username":"manpreet"},"change_message_id":"c5a58d4c0b1d033817b41445c6644606641f4fc2","unresolved":false,"context_lines":[{"line_number":168,"context_line":"can download a CSAR file to its local file system"},{"line_number":169,"context_line":"following the process above."},{"line_number":170,"context_line":"But other conductors that don\u0027t receive the request"},{"line_number":171,"context_line":"can\u0027t download a CSAR file at the timing,"},{"line_number":172,"context_line":"and so these conductors can\u0027t operate instantiation"},{"line_number":173,"context_line":"because they can\u0027t refer the CSAR file."},{"line_number":174,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"88505791_39d47e5c","line":171,"range":{"start_line":171,"start_character":34,"end_line":171,"end_character":40},"in_reply_to":"67a1314a_16a77813","updated":"2022-12-28 01:53:53.000000000","message":"Ack","commit_id":"8b81a73ef568b250e9cb669f1f00b311ec15a806"},{"author":{"_account_id":34107,"name":"Masaki Oyama","email":"ma-ooyama@kddi.com","username":"oyamamasaki"},"change_message_id":"a234a362f3bd1d428af22999856082231649aa67","unresolved":true,"context_lines":[{"line_number":168,"context_line":"can download a CSAR file to its local file system"},{"line_number":169,"context_line":"following the process above."},{"line_number":170,"context_line":"But other conductors that don\u0027t receive the request"},{"line_number":171,"context_line":"can\u0027t download a CSAR file at the timing,"},{"line_number":172,"context_line":"and so these conductors can\u0027t operate instantiation"},{"line_number":173,"context_line":"because they can\u0027t refer the CSAR file."},{"line_number":174,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"67a1314a_16a77813","line":171,"range":{"start_line":171,"start_character":34,"end_line":171,"end_character":40},"in_reply_to":"c95c477a_9503be77","updated":"2022-12-26 10:16:09.000000000","message":"I fixed it.","commit_id":"8b81a73ef568b250e9cb669f1f00b311ec15a806"},{"author":{"_account_id":32102,"name":"Manpreet Kaur","email":"kaurmanpreet2620@gmail.com","username":"manpreet"},"change_message_id":"a299b790f55612d090cf6c6eda46b41d9bb542f3","unresolved":true,"context_lines":[{"line_number":170,"context_line":"But other conductors that don\u0027t receive the request"},{"line_number":171,"context_line":"can\u0027t download a CSAR file at the timing,"},{"line_number":172,"context_line":"and so these conductors can\u0027t operate instantiation"},{"line_number":173,"context_line":"because they can\u0027t refer the CSAR file."},{"line_number":174,"context_line":""},{"line_number":175,"context_line":"Consider a user who wants to use external storage"},{"line_number":176,"context_line":"such as Object Storage to store VNF packages"}],"source_content_type":"text/x-rst","patch_set":2,"id":"2c2f92fb_231c24ae","line":173,"range":{"start_line":173,"start_character":25,"end_line":173,"end_character":28},"updated":"2022-12-26 05:46:34.000000000","message":"Please add \"to\".","commit_id":"8b81a73ef568b250e9cb669f1f00b311ec15a806"},{"author":{"_account_id":34107,"name":"Masaki Oyama","email":"ma-ooyama@kddi.com","username":"oyamamasaki"},"change_message_id":"a234a362f3bd1d428af22999856082231649aa67","unresolved":true,"context_lines":[{"line_number":170,"context_line":"But other conductors that don\u0027t receive the request"},{"line_number":171,"context_line":"can\u0027t download a CSAR file at the timing,"},{"line_number":172,"context_line":"and so these conductors can\u0027t operate instantiation"},{"line_number":173,"context_line":"because they can\u0027t refer the CSAR file."},{"line_number":174,"context_line":""},{"line_number":175,"context_line":"Consider a user who wants to use external storage"},{"line_number":176,"context_line":"such as Object Storage to store VNF packages"}],"source_content_type":"text/x-rst","patch_set":2,"id":"c823d5a9_e6637193","line":173,"range":{"start_line":173,"start_character":25,"end_line":173,"end_character":28},"in_reply_to":"2c2f92fb_231c24ae","updated":"2022-12-26 10:16:09.000000000","message":"Fixed it.","commit_id":"8b81a73ef568b250e9cb669f1f00b311ec15a806"},{"author":{"_account_id":32102,"name":"Manpreet Kaur","email":"kaurmanpreet2620@gmail.com","username":"manpreet"},"change_message_id":"c5a58d4c0b1d033817b41445c6644606641f4fc2","unresolved":false,"context_lines":[{"line_number":170,"context_line":"But other conductors that don\u0027t receive the request"},{"line_number":171,"context_line":"can\u0027t download a CSAR file at the timing,"},{"line_number":172,"context_line":"and so these conductors can\u0027t operate instantiation"},{"line_number":173,"context_line":"because they can\u0027t refer the CSAR file."},{"line_number":174,"context_line":""},{"line_number":175,"context_line":"Consider a user who wants to use external storage"},{"line_number":176,"context_line":"such as Object Storage to store VNF packages"}],"source_content_type":"text/x-rst","patch_set":2,"id":"ca9c7d3a_095d8ae8","line":173,"range":{"start_line":173,"start_character":25,"end_line":173,"end_character":28},"in_reply_to":"c823d5a9_e6637193","updated":"2022-12-28 01:53:53.000000000","message":"Ack","commit_id":"8b81a73ef568b250e9cb669f1f00b311ec15a806"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"4bed19640239a3b34e2fb43305034b5c27685c6f","unresolved":true,"context_lines":[{"line_number":176,"context_line":"such as Object Storage to store VNF packages"},{"line_number":177,"context_line":"and make it shareable by multiple conductors,"},{"line_number":178,"context_line":"it is necessary to enable all conductors to download CSAR files."},{"line_number":179,"context_line":""},{"line_number":180,"context_line":"Proposed change"},{"line_number":181,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":182,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"2c375ebc_e742dab4","line":179,"updated":"2022-12-28 06:39:16.000000000","message":"Cloud I confirm your background of the proposal? Multiple nodes in this case will be deployed on different locations? I wonder downloading CSAR file might be failed if connection isn\u0027t good or file size isn\u0027t small. How the procedure is recovered or discarded for such a failure case if my concern is true?","commit_id":"07679b01919d68f4271a3d390cad1bdf5696fc00"},{"author":{"_account_id":34107,"name":"Masaki Oyama","email":"ma-ooyama@kddi.com","username":"oyamamasaki"},"change_message_id":"34fc0682d1d140c0fdc860ce0c2b285ce1923f68","unresolved":true,"context_lines":[{"line_number":176,"context_line":"such as Object Storage to store VNF packages"},{"line_number":177,"context_line":"and make it shareable by multiple conductors,"},{"line_number":178,"context_line":"it is necessary to enable all conductors to download CSAR files."},{"line_number":179,"context_line":""},{"line_number":180,"context_line":"Proposed change"},{"line_number":181,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":182,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"5687ab62_7ca33f53","line":179,"in_reply_to":"2c375ebc_e742dab4","updated":"2022-12-28 07:16:16.000000000","message":"The case assumed in this spec is multiple tacker-conductors are running on different nodes.\nIf some tacker-conductor failed to download CSAR file, the onboarding operation must be failed and OnboardingState should be still `CREATED`.\nIn this case, user should retry the operation.","commit_id":"07679b01919d68f4271a3d390cad1bdf5696fc00"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"e760c9b21b048bfc0582ab58aaf4560b22cd0ffe","unresolved":true,"context_lines":[{"line_number":176,"context_line":"such as Object Storage to store VNF packages"},{"line_number":177,"context_line":"and make it shareable by multiple conductors,"},{"line_number":178,"context_line":"it is necessary to enable all conductors to download CSAR files."},{"line_number":179,"context_line":""},{"line_number":180,"context_line":"Proposed change"},{"line_number":181,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":182,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"fd2849c7_5f0161cf","line":179,"in_reply_to":"5687ab62_7ca33f53","updated":"2022-12-28 09:20:37.000000000","message":"Could you add a description (sequence diagram is preferable) for the case and make it clear how user know the failure? Thanks.","commit_id":"07679b01919d68f4271a3d390cad1bdf5696fc00"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"5b072d22f92df05e2eb0994e9e670bd97e44b905","unresolved":true,"context_lines":[{"line_number":176,"context_line":"such as Object Storage to store VNF packages"},{"line_number":177,"context_line":"and make it shareable by multiple conductors,"},{"line_number":178,"context_line":"it is necessary to enable all conductors to download CSAR files."},{"line_number":179,"context_line":""},{"line_number":180,"context_line":"Proposed change"},{"line_number":181,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":182,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"fd73e781_ad64ee43","line":179,"in_reply_to":"9e864373_fd0cae7c","updated":"2022-12-28 17:30:38.000000000","message":"For finding a failure, I\u0027m concerning not only a daily usage of a command, but also functional test for which how a client find the failure without useless several retries or timeout because we\u0027ve agreed to reduce total amount of time for functional tests as a project goal. We still need to discuss about the issue when you propose a patch for the change. I agree to the proposal for now.","commit_id":"07679b01919d68f4271a3d390cad1bdf5696fc00"},{"author":{"_account_id":34107,"name":"Masaki Oyama","email":"ma-ooyama@kddi.com","username":"oyamamasaki"},"change_message_id":"d23ae5e820bc7103b2bf32d9fac5b6b4f01f12f3","unresolved":true,"context_lines":[{"line_number":176,"context_line":"such as Object Storage to store VNF packages"},{"line_number":177,"context_line":"and make it shareable by multiple conductors,"},{"line_number":178,"context_line":"it is necessary to enable all conductors to download CSAR files."},{"line_number":179,"context_line":""},{"line_number":180,"context_line":"Proposed change"},{"line_number":181,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":182,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"9e864373_fd0cae7c","line":179,"in_reply_to":"fd2849c7_5f0161cf","updated":"2022-12-28 09:40:58.000000000","message":"I added the description that onboardingState is not updated to ONBOARDED when even one of the conductor fails to download CSAR file.\nUsers can know the failure by executing such a show VNF pakcage command as same as current implementation.","commit_id":"07679b01919d68f4271a3d390cad1bdf5696fc00"}]}