)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":32102,"name":"Manpreet Kaur","email":"kaurmanpreet2620@gmail.com","username":"manpreet"},"change_message_id":"792a8e9506acf7c0f05e1b3a0e8149773f99f3b1","unresolved":true,"context_lines":[{"line_number":16,"context_line":"* Enable understanding of primary and dependencies at toscaparser"},{"line_number":17,"context_line":"* Enable understanding of primary and dependencies at schema of tacker"},{"line_number":18,"context_line":"* Extend method that get implementation information"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Change-Id: I7d8e48c7d20e38f119f81803c73480de68708eb9"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"c8990bb8_5e0537dd","line":19,"updated":"2022-05-30 07:11:06.000000000","message":"Missing blueprint information.","commit_id":"d98d9ef1375811c895ba9366f155cbf474f37fe9"},{"author":{"_account_id":32102,"name":"Manpreet Kaur","email":"kaurmanpreet2620@gmail.com","username":"manpreet"},"change_message_id":"c51aa90732f2fc370174e6ffa61d2afd77a94db9","unresolved":false,"context_lines":[{"line_number":16,"context_line":"* Enable understanding of primary and dependencies at toscaparser"},{"line_number":17,"context_line":"* Enable understanding of primary and dependencies at schema of tacker"},{"line_number":18,"context_line":"* Extend method that get implementation information"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Change-Id: I7d8e48c7d20e38f119f81803c73480de68708eb9"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"61b3cdff_10b39ac8","line":19,"in_reply_to":"a0cdcb5b_57ab9dc2","updated":"2022-07-26 14:29:56.000000000","message":"Ack","commit_id":"d98d9ef1375811c895ba9366f155cbf474f37fe9"},{"author":{"_account_id":32736,"name":"Wataru Juso","email":"w-juso@nec.com","username":"w-juso"},"change_message_id":"d342574f9466e5a78af373b6693091cad5093a38","unresolved":true,"context_lines":[{"line_number":16,"context_line":"* Enable understanding of primary and dependencies at toscaparser"},{"line_number":17,"context_line":"* Enable understanding of primary and dependencies at schema of tacker"},{"line_number":18,"context_line":"* Extend method that get implementation information"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Change-Id: I7d8e48c7d20e38f119f81803c73480de68708eb9"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"a0cdcb5b_57ab9dc2","line":19,"in_reply_to":"c8990bb8_5e0537dd","updated":"2022-06-07 04:41:14.000000000","message":"fixed.","commit_id":"d98d9ef1375811c895ba9366f155cbf474f37fe9"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"5427aaa7a79d6097c45f7f3696a8c94b994df690","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"4ae67239_13e206d3","updated":"2022-05-30 08:57:09.000000000","message":"Thanks for your patch, please kindly check my comments.","commit_id":"d98d9ef1375811c895ba9366f155cbf474f37fe9"},{"author":{"_account_id":32102,"name":"Manpreet Kaur","email":"kaurmanpreet2620@gmail.com","username":"manpreet"},"change_message_id":"792a8e9506acf7c0f05e1b3a0e8149773f99f3b1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"0a38a79c_66f4b7bc","updated":"2022-05-30 07:11:06.000000000","message":"Thanks for your patch. Please find review comment.","commit_id":"d98d9ef1375811c895ba9366f155cbf474f37fe9"},{"author":{"_account_id":32736,"name":"Wataru Juso","email":"w-juso@nec.com","username":"w-juso"},"change_message_id":"d342574f9466e5a78af373b6693091cad5093a38","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"c7a1dad6_cc41650d","updated":"2022-06-07 04:41:14.000000000","message":"Thank you for your comments.","commit_id":"73faa25dc69f0fe86fbe320661b72bac61a9a88d"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"613860a9f19a273a309bd228573bff4ae53ae8b8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"6402a305_3f3e22a8","updated":"2022-06-13 06:57:21.000000000","message":"Thanks for your rework, please kindly check additional comments.","commit_id":"73faa25dc69f0fe86fbe320661b72bac61a9a88d"},{"author":{"_account_id":32736,"name":"Wataru Juso","email":"w-juso@nec.com","username":"w-juso"},"change_message_id":"0f449dcba7c32a7925a583c8003e442b68e7c7e0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"4552e2bf_bac76390","updated":"2022-07-22 05:56:28.000000000","message":"Thank you for reviewing.","commit_id":"b24b13c06522b5728c6555654073d02844a6315d"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"58131fd5de7a85408d959f721a395c57d5af96c0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"2cda38fa_6e9c87da","updated":"2022-07-25 05:36:58.000000000","message":"Thank for your rework, please kindly find additional comment.","commit_id":"e1f66b2188cfc90261c1f1aab4960504dea6bfb9"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"1b66d0e5ee9767713d215c93ce3fa858a85b5d1b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"11f24478_5228ff24","updated":"2022-07-25 15:33:10.000000000","message":"Thanks for your rework.\nSorry to bother you many times, but I have some tiny suggestions, please kindly check it.","commit_id":"89bf63bd00c5a4d4ca585479609b9bb9e07f4e5e"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"09bcd25f68c061033e2a626d74a61d3b659e6d12","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"1db3461d_b3492278","updated":"2022-07-26 13:55:59.000000000","message":"Thanks for your rework, LGTM.","commit_id":"18f36b3ee22d35d6789045643b4eb9df68545c8a"},{"author":{"_account_id":32736,"name":"Wataru Juso","email":"w-juso@nec.com","username":"w-juso"},"change_message_id":"17ffb892ef51f1db0ef38023fba1fdc20b61335b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"cbf525d8_ac0b8fc4","updated":"2022-07-26 05:34:07.000000000","message":"thank you for reviewing. I modified them.","commit_id":"18f36b3ee22d35d6789045643b4eb9df68545c8a"}],"specs/zed/support_multi_artifact_of_ansible_driver.rst":[{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"5427aaa7a79d6097c45f7f3696a8c94b994df690","unresolved":true,"context_lines":[{"line_number":5,"context_line":" http://creativecommons.org/licenses/by/3.0/legalcode"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":9,"context_line":"Support multi artifact of ansible driver"},{"line_number":10,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":11,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"122c50bb_7204e0e4","line":8,"range":{"start_line":8,"start_character":40,"end_line":8,"end_character":42},"updated":"2022-05-30 08:57:09.000000000","message":"nit: Could you delete this? It\u0027s better to match the length.","commit_id":"d98d9ef1375811c895ba9366f155cbf474f37fe9"},{"author":{"_account_id":32736,"name":"Wataru Juso","email":"w-juso@nec.com","username":"w-juso"},"change_message_id":"d342574f9466e5a78af373b6693091cad5093a38","unresolved":true,"context_lines":[{"line_number":5,"context_line":" http://creativecommons.org/licenses/by/3.0/legalcode"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":9,"context_line":"Support multi artifact of ansible driver"},{"line_number":10,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":11,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"86016e4f_ceee45a1","line":8,"range":{"start_line":8,"start_character":40,"end_line":8,"end_character":42},"in_reply_to":"122c50bb_7204e0e4","updated":"2022-06-07 04:41:14.000000000","message":"fixed.","commit_id":"d98d9ef1375811c895ba9366f155cbf474f37fe9"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"613860a9f19a273a309bd228573bff4ae53ae8b8","unresolved":false,"context_lines":[{"line_number":5,"context_line":" http://creativecommons.org/licenses/by/3.0/legalcode"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":9,"context_line":"Support multi artifact of ansible driver"},{"line_number":10,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":11,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"8abc5ea3_aba8ab1b","line":8,"range":{"start_line":8,"start_character":40,"end_line":8,"end_character":42},"in_reply_to":"86016e4f_ceee45a1","updated":"2022-06-13 06:57:21.000000000","message":"Ack","commit_id":"d98d9ef1375811c895ba9366f155cbf474f37fe9"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"5427aaa7a79d6097c45f7f3696a8c94b994df690","unresolved":true,"context_lines":[{"line_number":7,"context_line":""},{"line_number":8,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":9,"context_line":"Support multi artifact of ansible driver"},{"line_number":10,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"https://blueprints.launchpad.net/tacker/+spec/add-ansible-mgmt-driver-sample"},{"line_number":13,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"c2c1bc94_5d160c38","line":10,"range":{"start_line":10,"start_character":40,"end_line":10,"end_character":42},"updated":"2022-05-30 08:57:09.000000000","message":"ditto.","commit_id":"d98d9ef1375811c895ba9366f155cbf474f37fe9"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"613860a9f19a273a309bd228573bff4ae53ae8b8","unresolved":false,"context_lines":[{"line_number":7,"context_line":""},{"line_number":8,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":9,"context_line":"Support multi artifact of ansible driver"},{"line_number":10,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"https://blueprints.launchpad.net/tacker/+spec/add-ansible-mgmt-driver-sample"},{"line_number":13,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"29a7dd94_fca20a9c","line":10,"range":{"start_line":10,"start_character":40,"end_line":10,"end_character":42},"in_reply_to":"3b6479c3_ecc72df2","updated":"2022-06-13 06:57:21.000000000","message":"Ack","commit_id":"d98d9ef1375811c895ba9366f155cbf474f37fe9"},{"author":{"_account_id":32736,"name":"Wataru Juso","email":"w-juso@nec.com","username":"w-juso"},"change_message_id":"d342574f9466e5a78af373b6693091cad5093a38","unresolved":true,"context_lines":[{"line_number":7,"context_line":""},{"line_number":8,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":9,"context_line":"Support multi artifact of ansible driver"},{"line_number":10,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"https://blueprints.launchpad.net/tacker/+spec/add-ansible-mgmt-driver-sample"},{"line_number":13,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"3b6479c3_ecc72df2","line":10,"range":{"start_line":10,"start_character":40,"end_line":10,"end_character":42},"in_reply_to":"c2c1bc94_5d160c38","updated":"2022-06-07 04:41:14.000000000","message":"fixed.","commit_id":"d98d9ef1375811c895ba9366f155cbf474f37fe9"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"5427aaa7a79d6097c45f7f3696a8c94b994df690","unresolved":true,"context_lines":[{"line_number":94,"context_line":"          file: /opt/stack/tacker/tacker/vnfm/mgmt_drivers/ansible/ansible.py"},{"line_number":95,"context_line":"        mgmt-drivers-ansible-config-start:"},{"line_number":96,"context_line":"          description: Management driver config_start.yaml"},{"line_number":97,"context_line":"          type: tosca.artifacts.Implementation.nfv.Mistral"},{"line_number":98,"context_line":"          file: ../ScriptAnsible/config_start.yaml"},{"line_number":99,"context_line":"        mgmt-drivers-ansible-config-end:"},{"line_number":100,"context_line":"          description: Management driver config_end.yaml"}],"source_content_type":"text/x-rst","patch_set":2,"id":"c7207a66_46c0de66","line":97,"range":{"start_line":97,"start_character":16,"end_line":97,"end_character":58},"updated":"2022-05-30 08:57:09.000000000","message":"Just confirmation, is this artifact type suitable?\n\n`tosca.artifacts.Implementation.nfv.Mistral` is described in SOL001 as follow:\n```\nThis artifact type represents a Mistral file that contains Mistral language [i.7] constructs that can be executed within a Mistral workbook. Support of this type is optional. \n```\n\n[i.7] refer to below link:\nhttps://docs.openstack.org/mistral/latest/user/wf_lang_v2.html\n\nIs `config_start.yaml` wrriten by Mistral Workflow Language?","commit_id":"d98d9ef1375811c895ba9366f155cbf474f37fe9"},{"author":{"_account_id":32736,"name":"Wataru Juso","email":"w-juso@nec.com","username":"w-juso"},"change_message_id":"0f449dcba7c32a7925a583c8003e442b68e7c7e0","unresolved":true,"context_lines":[{"line_number":94,"context_line":"          file: /opt/stack/tacker/tacker/vnfm/mgmt_drivers/ansible/ansible.py"},{"line_number":95,"context_line":"        mgmt-drivers-ansible-config-start:"},{"line_number":96,"context_line":"          description: Management driver config_start.yaml"},{"line_number":97,"context_line":"          type: tosca.artifacts.Implementation.nfv.Mistral"},{"line_number":98,"context_line":"          file: ../ScriptAnsible/config_start.yaml"},{"line_number":99,"context_line":"        mgmt-drivers-ansible-config-end:"},{"line_number":100,"context_line":"          description: Management driver config_end.yaml"}],"source_content_type":"text/x-rst","patch_set":2,"id":"bc15480e_a5c88420","line":97,"range":{"start_line":97,"start_character":16,"end_line":97,"end_character":58},"in_reply_to":"10fdde2a_eec0cfde","updated":"2022-07-22 05:56:28.000000000","message":"As a policy, I\u0027m thinking that either is fine that modify `artifact_types` or add new define file.\nIn this Spec, I added a new file like `How to Use`.","commit_id":"d98d9ef1375811c895ba9366f155cbf474f37fe9"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"1b66d0e5ee9767713d215c93ce3fa858a85b5d1b","unresolved":false,"context_lines":[{"line_number":94,"context_line":"          file: /opt/stack/tacker/tacker/vnfm/mgmt_drivers/ansible/ansible.py"},{"line_number":95,"context_line":"        mgmt-drivers-ansible-config-start:"},{"line_number":96,"context_line":"          description: Management driver config_start.yaml"},{"line_number":97,"context_line":"          type: tosca.artifacts.Implementation.nfv.Mistral"},{"line_number":98,"context_line":"          file: ../ScriptAnsible/config_start.yaml"},{"line_number":99,"context_line":"        mgmt-drivers-ansible-config-end:"},{"line_number":100,"context_line":"          description: Management driver config_end.yaml"}],"source_content_type":"text/x-rst","patch_set":2,"id":"cbcbc814_2b3ab826","line":97,"range":{"start_line":97,"start_character":16,"end_line":97,"end_character":58},"in_reply_to":"2ee2d0e1_1e20fcc2","updated":"2022-07-25 15:33:10.000000000","message":"Ack","commit_id":"d98d9ef1375811c895ba9366f155cbf474f37fe9"},{"author":{"_account_id":32736,"name":"Wataru Juso","email":"w-juso@nec.com","username":"w-juso"},"change_message_id":"8fcde649464e53cfece25ceffd6efa175ff517e0","unresolved":true,"context_lines":[{"line_number":94,"context_line":"          file: /opt/stack/tacker/tacker/vnfm/mgmt_drivers/ansible/ansible.py"},{"line_number":95,"context_line":"        mgmt-drivers-ansible-config-start:"},{"line_number":96,"context_line":"          description: Management driver config_start.yaml"},{"line_number":97,"context_line":"          type: tosca.artifacts.Implementation.nfv.Mistral"},{"line_number":98,"context_line":"          file: ../ScriptAnsible/config_start.yaml"},{"line_number":99,"context_line":"        mgmt-drivers-ansible-config-end:"},{"line_number":100,"context_line":"          description: Management driver config_end.yaml"}],"source_content_type":"text/x-rst","patch_set":2,"id":"2ee2d0e1_1e20fcc2","line":97,"range":{"start_line":97,"start_character":16,"end_line":97,"end_character":58},"in_reply_to":"46ae3e28_e75b755e","updated":"2022-07-25 10:11:03.000000000","message":"Sorry, I missed the modification. I fixed.","commit_id":"d98d9ef1375811c895ba9366f155cbf474f37fe9"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"613860a9f19a273a309bd228573bff4ae53ae8b8","unresolved":true,"context_lines":[{"line_number":94,"context_line":"          file: /opt/stack/tacker/tacker/vnfm/mgmt_drivers/ansible/ansible.py"},{"line_number":95,"context_line":"        mgmt-drivers-ansible-config-start:"},{"line_number":96,"context_line":"          description: Management driver config_start.yaml"},{"line_number":97,"context_line":"          type: tosca.artifacts.Implementation.nfv.Mistral"},{"line_number":98,"context_line":"          file: ../ScriptAnsible/config_start.yaml"},{"line_number":99,"context_line":"        mgmt-drivers-ansible-config-end:"},{"line_number":100,"context_line":"          description: Management driver config_end.yaml"}],"source_content_type":"text/x-rst","patch_set":2,"id":"10fdde2a_eec0cfde","line":97,"range":{"start_line":97,"start_character":16,"end_line":97,"end_character":58},"in_reply_to":"acbe1dc8_3225cd7b","updated":"2022-06-13 06:57:21.000000000","message":"\u003e So, I think the second way is good.\n\nI think so too.\n\n\u003e * Include the original definition file in the VNF Package, and parse it when LCM.\n\nIs it correct to add `artifact_types` to the following file, not adding a new file?\nhttps://opendev.org/openstack/tacker/src/branch/master/samples/mgmt_driver/ansible/ansible_vnf_package/Definitions/helloworld3_types.yaml","commit_id":"d98d9ef1375811c895ba9366f155cbf474f37fe9"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"58131fd5de7a85408d959f721a395c57d5af96c0","unresolved":true,"context_lines":[{"line_number":94,"context_line":"          file: /opt/stack/tacker/tacker/vnfm/mgmt_drivers/ansible/ansible.py"},{"line_number":95,"context_line":"        mgmt-drivers-ansible-config-start:"},{"line_number":96,"context_line":"          description: Management driver config_start.yaml"},{"line_number":97,"context_line":"          type: tosca.artifacts.Implementation.nfv.Mistral"},{"line_number":98,"context_line":"          file: ../ScriptAnsible/config_start.yaml"},{"line_number":99,"context_line":"        mgmt-drivers-ansible-config-end:"},{"line_number":100,"context_line":"          description: Management driver config_end.yaml"}],"source_content_type":"text/x-rst","patch_set":2,"id":"46ae3e28_e75b755e","line":97,"range":{"start_line":97,"start_character":16,"end_line":97,"end_character":58},"in_reply_to":"bc15480e_a5c88420","updated":"2022-07-25 05:36:58.000000000","message":"Isn\u0027t the type here `tosca.artifacts.Implementation.Yaml`?","commit_id":"d98d9ef1375811c895ba9366f155cbf474f37fe9"},{"author":{"_account_id":32736,"name":"Wataru Juso","email":"w-juso@nec.com","username":"w-juso"},"change_message_id":"d342574f9466e5a78af373b6693091cad5093a38","unresolved":true,"context_lines":[{"line_number":94,"context_line":"          file: /opt/stack/tacker/tacker/vnfm/mgmt_drivers/ansible/ansible.py"},{"line_number":95,"context_line":"        mgmt-drivers-ansible-config-start:"},{"line_number":96,"context_line":"          description: Management driver config_start.yaml"},{"line_number":97,"context_line":"          type: tosca.artifacts.Implementation.nfv.Mistral"},{"line_number":98,"context_line":"          file: ../ScriptAnsible/config_start.yaml"},{"line_number":99,"context_line":"        mgmt-drivers-ansible-config-end:"},{"line_number":100,"context_line":"          description: Management driver config_end.yaml"}],"source_content_type":"text/x-rst","patch_set":2,"id":"acbe1dc8_3225cd7b","line":97,"range":{"start_line":97,"start_character":16,"end_line":97,"end_character":58},"in_reply_to":"c7207a66_46c0de66","updated":"2022-06-07 04:41:14.000000000","message":"No, it works with yaml files of ansible playbook.\nI would like to implement the definition of yaml type.\n\nI think there are two ways.\n・ Modify the toscaparser/elements/TOSCA_definition_1_0.yaml\n・ Include the original definition file in the VNF Package, and parse it when LCM.\n\nThe first way seems like reference from OASIS definition.\nhttp://docs.oasis-open.org/tosca/TOSCA-Simple-Profile-YAML/v1.0/TOSCA-Simple-Profile-YAML-v1.0.html\nSo, I think the second way is good.\nWhat do you think?","commit_id":"d98d9ef1375811c895ba9366f155cbf474f37fe9"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"5427aaa7a79d6097c45f7f3696a8c94b994df690","unresolved":true,"context_lines":[{"line_number":127,"context_line":""},{"line_number":128,"context_line":"Other end user impact"},{"line_number":129,"context_line":"---------------------"},{"line_number":130,"context_line":"None"},{"line_number":131,"context_line":""},{"line_number":132,"context_line":"It also supports the single artifact format currently used by VNFD."},{"line_number":133,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"c07e21a0_cc1c86de","line":130,"range":{"start_line":130,"start_character":0,"end_line":130,"end_character":4},"updated":"2022-05-30 08:57:09.000000000","message":"Could you remove `None` if the following is written as \"Other end user impact\"?","commit_id":"d98d9ef1375811c895ba9366f155cbf474f37fe9"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"613860a9f19a273a309bd228573bff4ae53ae8b8","unresolved":false,"context_lines":[{"line_number":127,"context_line":""},{"line_number":128,"context_line":"Other end user impact"},{"line_number":129,"context_line":"---------------------"},{"line_number":130,"context_line":"None"},{"line_number":131,"context_line":""},{"line_number":132,"context_line":"It also supports the single artifact format currently used by VNFD."},{"line_number":133,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"ea859d68_b6f0ddda","line":130,"range":{"start_line":130,"start_character":0,"end_line":130,"end_character":4},"in_reply_to":"5c801bea_bfadda10","updated":"2022-06-13 06:57:21.000000000","message":"Ack","commit_id":"d98d9ef1375811c895ba9366f155cbf474f37fe9"},{"author":{"_account_id":32736,"name":"Wataru Juso","email":"w-juso@nec.com","username":"w-juso"},"change_message_id":"d342574f9466e5a78af373b6693091cad5093a38","unresolved":true,"context_lines":[{"line_number":127,"context_line":""},{"line_number":128,"context_line":"Other end user impact"},{"line_number":129,"context_line":"---------------------"},{"line_number":130,"context_line":"None"},{"line_number":131,"context_line":""},{"line_number":132,"context_line":"It also supports the single artifact format currently used by VNFD."},{"line_number":133,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"5c801bea_bfadda10","line":130,"range":{"start_line":130,"start_character":0,"end_line":130,"end_character":4},"in_reply_to":"c07e21a0_cc1c86de","updated":"2022-06-07 04:41:14.000000000","message":"It was described as a supplementary explanation of None.\nThis text moved to proposed change.","commit_id":"d98d9ef1375811c895ba9366f155cbf474f37fe9"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"5427aaa7a79d6097c45f7f3696a8c94b994df690","unresolved":true,"context_lines":[{"line_number":145,"context_line":""},{"line_number":146,"context_line":"Implementation"},{"line_number":147,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":148,"context_line":"* Enable understanding of primary and dependencies at toscaparser"},{"line_number":149,"context_line":"* Enable understanding of primary and dependencies at schema of tacker"},{"line_number":150,"context_line":"* Extend method that get implementation information"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"Assignee(s)"},{"line_number":153,"context_line":"-----------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"959cecec_df00a3ef","line":150,"range":{"start_line":148,"start_character":0,"end_line":150,"end_character":51},"updated":"2022-05-30 08:57:09.000000000","message":"How about you write it in Work Items?","commit_id":"d98d9ef1375811c895ba9366f155cbf474f37fe9"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"58131fd5de7a85408d959f721a395c57d5af96c0","unresolved":false,"context_lines":[{"line_number":145,"context_line":""},{"line_number":146,"context_line":"Implementation"},{"line_number":147,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":148,"context_line":"* Enable understanding of primary and dependencies at toscaparser"},{"line_number":149,"context_line":"* Enable understanding of primary and dependencies at schema of tacker"},{"line_number":150,"context_line":"* Extend method that get implementation information"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"Assignee(s)"},{"line_number":153,"context_line":"-----------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"475211a0_8ff58820","line":150,"range":{"start_line":148,"start_character":0,"end_line":150,"end_character":51},"in_reply_to":"243c96be_656ff71f","updated":"2022-07-25 05:36:58.000000000","message":"Ack","commit_id":"d98d9ef1375811c895ba9366f155cbf474f37fe9"},{"author":{"_account_id":32736,"name":"Wataru Juso","email":"w-juso@nec.com","username":"w-juso"},"change_message_id":"0f449dcba7c32a7925a583c8003e442b68e7c7e0","unresolved":true,"context_lines":[{"line_number":145,"context_line":""},{"line_number":146,"context_line":"Implementation"},{"line_number":147,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":148,"context_line":"* Enable understanding of primary and dependencies at toscaparser"},{"line_number":149,"context_line":"* Enable understanding of primary and dependencies at schema of tacker"},{"line_number":150,"context_line":"* Extend method that get implementation information"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"Assignee(s)"},{"line_number":153,"context_line":"-----------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"243c96be_656ff71f","line":150,"range":{"start_line":148,"start_character":0,"end_line":150,"end_character":51},"in_reply_to":"4c1217ba_af2d5e73","updated":"2022-07-22 05:56:28.000000000","message":"I deleted `None`.","commit_id":"d98d9ef1375811c895ba9366f155cbf474f37fe9"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"613860a9f19a273a309bd228573bff4ae53ae8b8","unresolved":true,"context_lines":[{"line_number":145,"context_line":""},{"line_number":146,"context_line":"Implementation"},{"line_number":147,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":148,"context_line":"* Enable understanding of primary and dependencies at toscaparser"},{"line_number":149,"context_line":"* Enable understanding of primary and dependencies at schema of tacker"},{"line_number":150,"context_line":"* Extend method that get implementation information"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"Assignee(s)"},{"line_number":153,"context_line":"-----------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"4c1217ba_af2d5e73","line":150,"range":{"start_line":148,"start_character":0,"end_line":150,"end_character":51},"in_reply_to":"748b5a32_7aa7ad92","updated":"2022-06-13 06:57:21.000000000","message":"`None` is not necessary because `Assignee(s)` and `Work Items` are existed under `Implementation`.","commit_id":"d98d9ef1375811c895ba9366f155cbf474f37fe9"},{"author":{"_account_id":32736,"name":"Wataru Juso","email":"w-juso@nec.com","username":"w-juso"},"change_message_id":"d342574f9466e5a78af373b6693091cad5093a38","unresolved":true,"context_lines":[{"line_number":145,"context_line":""},{"line_number":146,"context_line":"Implementation"},{"line_number":147,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":148,"context_line":"* Enable understanding of primary and dependencies at toscaparser"},{"line_number":149,"context_line":"* Enable understanding of primary and dependencies at schema of tacker"},{"line_number":150,"context_line":"* Extend method that get implementation information"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"Assignee(s)"},{"line_number":153,"context_line":"-----------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"748b5a32_7aa7ad92","line":150,"range":{"start_line":148,"start_character":0,"end_line":150,"end_character":51},"in_reply_to":"959cecec_df00a3ef","updated":"2022-06-07 04:41:14.000000000","message":"There is moved Work Items.","commit_id":"d98d9ef1375811c895ba9366f155cbf474f37fe9"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"291ed4fc3a1217c038c8e9af87d48e2accea5528","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":9,"context_line":"Support multi artifact of ansible driver"},{"line_number":10,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"https://blueprints.launchpad.net/tacker/+spec/add-ansible-mgmt-driver-sample"}],"source_content_type":"text/x-rst","patch_set":3,"id":"a9507fb2_6a8d0827","line":9,"range":{"start_line":9,"start_character":14,"end_line":9,"end_character":22},"updated":"2022-06-27 20:47:41.000000000","message":"nit: artifacts","commit_id":"73faa25dc69f0fe86fbe320661b72bac61a9a88d"},{"author":{"_account_id":32736,"name":"Wataru Juso","email":"w-juso@nec.com","username":"w-juso"},"change_message_id":"0f449dcba7c32a7925a583c8003e442b68e7c7e0","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":9,"context_line":"Support multi artifact of ansible driver"},{"line_number":10,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"https://blueprints.launchpad.net/tacker/+spec/add-ansible-mgmt-driver-sample"}],"source_content_type":"text/x-rst","patch_set":3,"id":"f27a1de6_c854bb3d","line":9,"range":{"start_line":9,"start_character":14,"end_line":9,"end_character":22},"in_reply_to":"a9507fb2_6a8d0827","updated":"2022-07-22 05:56:28.000000000","message":"fixed.","commit_id":"73faa25dc69f0fe86fbe320661b72bac61a9a88d"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"291ed4fc3a1217c038c8e9af87d48e2accea5528","unresolved":true,"context_lines":[{"line_number":11,"context_line":""},{"line_number":12,"context_line":"https://blueprints.launchpad.net/tacker/+spec/add-ansible-mgmt-driver-sample"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"We\u0027ll be able to use the configuration of multi deployment pattern."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Problem description"},{"line_number":17,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"42acce34_f8c2ce64","line":14,"range":{"start_line":14,"start_character":59,"end_line":14,"end_character":66},"updated":"2022-06-27 20:47:41.000000000","message":"nit: patterns","commit_id":"73faa25dc69f0fe86fbe320661b72bac61a9a88d"},{"author":{"_account_id":32736,"name":"Wataru Juso","email":"w-juso@nec.com","username":"w-juso"},"change_message_id":"0f449dcba7c32a7925a583c8003e442b68e7c7e0","unresolved":true,"context_lines":[{"line_number":11,"context_line":""},{"line_number":12,"context_line":"https://blueprints.launchpad.net/tacker/+spec/add-ansible-mgmt-driver-sample"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"We\u0027ll be able to use the configuration of multi deployment pattern."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Problem description"},{"line_number":17,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3bafbece_196c2743","line":14,"range":{"start_line":14,"start_character":59,"end_line":14,"end_character":66},"in_reply_to":"42acce34_f8c2ce64","updated":"2022-07-22 05:56:28.000000000","message":"fixed.","commit_id":"73faa25dc69f0fe86fbe320661b72bac61a9a88d"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"291ed4fc3a1217c038c8e9af87d48e2accea5528","unresolved":true,"context_lines":[{"line_number":20,"context_line":"The configuration information is defined in the implementation."},{"line_number":21,"context_line":"It is executed at the start and end of each VNF lifecycle."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"This current specification does not allow multiple configuration"},{"line_number":24,"context_line":"information to be submitted."},{"line_number":25,"context_line":"For example, user might want to configure different configuration information"},{"line_number":26,"context_line":"for different deployment flavors."}],"source_content_type":"text/x-rst","patch_set":3,"id":"290e7e1f_c060c5d7","line":23,"range":{"start_line":23,"start_character":0,"end_line":23,"end_character":26},"updated":"2022-06-27 20:47:41.000000000","message":"Current implementation correctly? Because it just confusing for this spec for the context.","commit_id":"73faa25dc69f0fe86fbe320661b72bac61a9a88d"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"291ed4fc3a1217c038c8e9af87d48e2accea5528","unresolved":true,"context_lines":[{"line_number":20,"context_line":"The configuration information is defined in the implementation."},{"line_number":21,"context_line":"It is executed at the start and end of each VNF lifecycle."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"This current specification does not allow multiple configuration"},{"line_number":24,"context_line":"information to be submitted."},{"line_number":25,"context_line":"For example, user might want to configure different configuration information"},{"line_number":26,"context_line":"for different deployment flavors."}],"source_content_type":"text/x-rst","patch_set":3,"id":"bac44c68_93262e1d","line":23,"range":{"start_line":23,"start_character":32,"end_line":23,"end_character":41},"updated":"2022-06-27 20:47:41.000000000","message":"`not allow` is not true because we don\u0027t have any restriction for that as a spec. The true is just the current implementation does not work for such a requirement.","commit_id":"73faa25dc69f0fe86fbe320661b72bac61a9a88d"},{"author":{"_account_id":32736,"name":"Wataru Juso","email":"w-juso@nec.com","username":"w-juso"},"change_message_id":"0f449dcba7c32a7925a583c8003e442b68e7c7e0","unresolved":true,"context_lines":[{"line_number":20,"context_line":"The configuration information is defined in the implementation."},{"line_number":21,"context_line":"It is executed at the start and end of each VNF lifecycle."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"This current specification does not allow multiple configuration"},{"line_number":24,"context_line":"information to be submitted."},{"line_number":25,"context_line":"For example, user might want to configure different configuration information"},{"line_number":26,"context_line":"for different deployment flavors."}],"source_content_type":"text/x-rst","patch_set":3,"id":"c9b89f24_2a00026b","line":23,"range":{"start_line":23,"start_character":0,"end_line":23,"end_character":26},"in_reply_to":"290e7e1f_c060c5d7","updated":"2022-07-22 05:56:28.000000000","message":"I couldn\u0027t explain it well.\nI wanted to tell you that only one parameter can be specified.\nI modified the text.","commit_id":"73faa25dc69f0fe86fbe320661b72bac61a9a88d"},{"author":{"_account_id":32736,"name":"Wataru Juso","email":"w-juso@nec.com","username":"w-juso"},"change_message_id":"0f449dcba7c32a7925a583c8003e442b68e7c7e0","unresolved":true,"context_lines":[{"line_number":20,"context_line":"The configuration information is defined in the implementation."},{"line_number":21,"context_line":"It is executed at the start and end of each VNF lifecycle."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"This current specification does not allow multiple configuration"},{"line_number":24,"context_line":"information to be submitted."},{"line_number":25,"context_line":"For example, user might want to configure different configuration information"},{"line_number":26,"context_line":"for different deployment flavors."}],"source_content_type":"text/x-rst","patch_set":3,"id":"08f1658e_3e065e97","line":23,"range":{"start_line":23,"start_character":32,"end_line":23,"end_character":41},"in_reply_to":"bac44c68_93262e1d","updated":"2022-07-22 05:56:28.000000000","message":"same as above.","commit_id":"73faa25dc69f0fe86fbe320661b72bac61a9a88d"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"291ed4fc3a1217c038c8e9af87d48e2accea5528","unresolved":true,"context_lines":[{"line_number":22,"context_line":""},{"line_number":23,"context_line":"This current specification does not allow multiple configuration"},{"line_number":24,"context_line":"information to be submitted."},{"line_number":25,"context_line":"For example, user might want to configure different configuration information"},{"line_number":26,"context_line":"for different deployment flavors."},{"line_number":27,"context_line":""},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"Proposed change"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3e5ea9d8_cd4cbcf5","line":26,"range":{"start_line":25,"start_character":0,"end_line":26,"end_character":33},"updated":"2022-06-27 20:47:41.000000000","message":"I\u0027m not sure why you add this sentence here. What of example?","commit_id":"73faa25dc69f0fe86fbe320661b72bac61a9a88d"},{"author":{"_account_id":32736,"name":"Wataru Juso","email":"w-juso@nec.com","username":"w-juso"},"change_message_id":"0f449dcba7c32a7925a583c8003e442b68e7c7e0","unresolved":true,"context_lines":[{"line_number":22,"context_line":""},{"line_number":23,"context_line":"This current specification does not allow multiple configuration"},{"line_number":24,"context_line":"information to be submitted."},{"line_number":25,"context_line":"For example, user might want to configure different configuration information"},{"line_number":26,"context_line":"for different deployment flavors."},{"line_number":27,"context_line":""},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"Proposed change"}],"source_content_type":"text/x-rst","patch_set":3,"id":"4a5d6932_12ec55e7","line":26,"range":{"start_line":25,"start_character":0,"end_line":26,"end_character":33},"in_reply_to":"3e5ea9d8_cd4cbcf5","updated":"2022-07-22 05:56:28.000000000","message":"We faced a problem that could only implemented branch within the Driver when we wanted to have different configurations for different Deployment Flarvor.\nI wanted to tell you an example of the problem.","commit_id":"73faa25dc69f0fe86fbe320661b72bac61a9a88d"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"291ed4fc3a1217c038c8e9af87d48e2accea5528","unresolved":true,"context_lines":[{"line_number":29,"context_line":"Proposed change"},{"line_number":30,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"Tacker is currently running the Mgmt Driver Configuration with the following"},{"line_number":33,"context_line":"implementation key definition."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"This parameter is defined in TOSCA Simple Profile [#f1]_."},{"line_number":36,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"b831485b_9550c742","line":33,"range":{"start_line":32,"start_character":0,"end_line":33,"end_character":30},"updated":"2022-06-27 20:47:41.000000000","message":"It\u0027s incorrect in grammar, and might no need to be here. Why don\u0027t you move L.59 to here instead?\n\nIn addition, you should make the key `implementation` as a term ``implementation`` in the sentence for better understanding.","commit_id":"73faa25dc69f0fe86fbe320661b72bac61a9a88d"},{"author":{"_account_id":32736,"name":"Wataru Juso","email":"w-juso@nec.com","username":"w-juso"},"change_message_id":"0f449dcba7c32a7925a583c8003e442b68e7c7e0","unresolved":true,"context_lines":[{"line_number":29,"context_line":"Proposed change"},{"line_number":30,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"Tacker is currently running the Mgmt Driver Configuration with the following"},{"line_number":33,"context_line":"implementation key definition."},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"This parameter is defined in TOSCA Simple Profile [#f1]_."},{"line_number":36,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"81f4526b_0b193d21","line":33,"range":{"start_line":32,"start_character":0,"end_line":33,"end_character":30},"in_reply_to":"b831485b_9550c742","updated":"2022-07-22 05:56:28.000000000","message":"I modified this sentence by changing composition.","commit_id":"73faa25dc69f0fe86fbe320661b72bac61a9a88d"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"291ed4fc3a1217c038c8e9af87d48e2accea5528","unresolved":true,"context_lines":[{"line_number":56,"context_line":"          type: tosca.artifacts.Implementation.Python"},{"line_number":57,"context_line":"          file: /opt/stack/tacker/tacker/vnfm/mgmt_drivers/ansible/ansible.py"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"Currently, only one definition can appear in the Implementation key."},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"The proposed change is to implement the primary and dependencies keys defined by TOSCA."},{"line_number":62,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"84a847ff_d85ccf1f","line":59,"updated":"2022-06-27 20:47:41.000000000","message":"Why don\u0027t you move this sentence to the top of the section instead?","commit_id":"73faa25dc69f0fe86fbe320661b72bac61a9a88d"},{"author":{"_account_id":32736,"name":"Wataru Juso","email":"w-juso@nec.com","username":"w-juso"},"change_message_id":"0f449dcba7c32a7925a583c8003e442b68e7c7e0","unresolved":true,"context_lines":[{"line_number":56,"context_line":"          type: tosca.artifacts.Implementation.Python"},{"line_number":57,"context_line":"          file: /opt/stack/tacker/tacker/vnfm/mgmt_drivers/ansible/ansible.py"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"Currently, only one definition can appear in the Implementation key."},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"The proposed change is to implement the primary and dependencies keys defined by TOSCA."},{"line_number":62,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"4e8be763_4d467ee6","line":59,"in_reply_to":"84a847ff_d85ccf1f","updated":"2022-07-22 05:56:28.000000000","message":"Same modifications as above","commit_id":"73faa25dc69f0fe86fbe320661b72bac61a9a88d"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"291ed4fc3a1217c038c8e9af87d48e2accea5528","unresolved":true,"context_lines":[{"line_number":58,"context_line":""},{"line_number":59,"context_line":"Currently, only one definition can appear in the Implementation key."},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"The proposed change is to implement the primary and dependencies keys defined by TOSCA."},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"Dependencies can define a second script."},{"line_number":64,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"c4989dfc_c5706d3f","line":61,"updated":"2022-06-27 20:47:41.000000000","message":"You should better to make `primary` and `dependencies` as terms.","commit_id":"73faa25dc69f0fe86fbe320661b72bac61a9a88d"},{"author":{"_account_id":32736,"name":"Wataru Juso","email":"w-juso@nec.com","username":"w-juso"},"change_message_id":"0f449dcba7c32a7925a583c8003e442b68e7c7e0","unresolved":true,"context_lines":[{"line_number":58,"context_line":""},{"line_number":59,"context_line":"Currently, only one definition can appear in the Implementation key."},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"The proposed change is to implement the primary and dependencies keys defined by TOSCA."},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"Dependencies can define a second script."},{"line_number":64,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"449b2919_eb862195","line":61,"in_reply_to":"c4989dfc_c5706d3f","updated":"2022-07-22 05:56:28.000000000","message":"I modified these difinition to terms.","commit_id":"73faa25dc69f0fe86fbe320661b72bac61a9a88d"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"291ed4fc3a1217c038c8e9af87d48e2accea5528","unresolved":true,"context_lines":[{"line_number":60,"context_line":""},{"line_number":61,"context_line":"The proposed change is to implement the primary and dependencies keys defined by TOSCA."},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"Dependencies can define a second script."},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"If you implemented this dependency, you would define it as follows :"},{"line_number":66,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"ab9be5b4_f5481a6f","line":63,"updated":"2022-06-27 20:47:41.000000000","message":"I\u0027m not sure about `second script`.\n\n* Is it a dependent script for the first script, or independent, or both can be defined?\n\n* It can define only one `second` script, or several ones?","commit_id":"73faa25dc69f0fe86fbe320661b72bac61a9a88d"},{"author":{"_account_id":32736,"name":"Wataru Juso","email":"w-juso@nec.com","username":"w-juso"},"change_message_id":"0f449dcba7c32a7925a583c8003e442b68e7c7e0","unresolved":true,"context_lines":[{"line_number":60,"context_line":""},{"line_number":61,"context_line":"The proposed change is to implement the primary and dependencies keys defined by TOSCA."},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"Dependencies can define a second script."},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"If you implemented this dependency, you would define it as follows :"},{"line_number":66,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"1dd36ee5_044d1725","line":63,"in_reply_to":"ab9be5b4_f5481a6f","updated":"2022-07-22 05:56:28.000000000","message":"`second script` is referenced the first(primary) script.","commit_id":"73faa25dc69f0fe86fbe320661b72bac61a9a88d"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"291ed4fc3a1217c038c8e9af87d48e2accea5528","unresolved":true,"context_lines":[{"line_number":101,"context_line":"          type: tosca.artifacts.Implementation.nfv.Mistral"},{"line_number":102,"context_line":"          file: ../ScriptAnsible/config_end.yaml"},{"line_number":103,"context_line":""},{"line_number":104,"context_line":"Primary defines the script that has run management capabilities,"},{"line_number":105,"context_line":"and dependencies defines the script that performs the actual injection."},{"line_number":106,"context_line":"These definitions allow for flexible configuration by the user."},{"line_number":107,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"c4e5da1a_b28a52e8","line":104,"range":{"start_line":104,"start_character":0,"end_line":104,"end_character":7},"updated":"2022-06-27 20:47:41.000000000","message":"ditto: better to describe as ``primary``","commit_id":"73faa25dc69f0fe86fbe320661b72bac61a9a88d"},{"author":{"_account_id":32736,"name":"Wataru Juso","email":"w-juso@nec.com","username":"w-juso"},"change_message_id":"0f449dcba7c32a7925a583c8003e442b68e7c7e0","unresolved":true,"context_lines":[{"line_number":101,"context_line":"          type: tosca.artifacts.Implementation.nfv.Mistral"},{"line_number":102,"context_line":"          file: ../ScriptAnsible/config_end.yaml"},{"line_number":103,"context_line":""},{"line_number":104,"context_line":"Primary defines the script that has run management capabilities,"},{"line_number":105,"context_line":"and dependencies defines the script that performs the actual injection."},{"line_number":106,"context_line":"These definitions allow for flexible configuration by the user."},{"line_number":107,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"6a251b24_a3382932","line":104,"range":{"start_line":104,"start_character":0,"end_line":104,"end_character":7},"in_reply_to":"c4e5da1a_b28a52e8","updated":"2022-07-22 05:56:28.000000000","message":"fixed.","commit_id":"73faa25dc69f0fe86fbe320661b72bac61a9a88d"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"291ed4fc3a1217c038c8e9af87d48e2accea5528","unresolved":true,"context_lines":[{"line_number":102,"context_line":"          file: ../ScriptAnsible/config_end.yaml"},{"line_number":103,"context_line":""},{"line_number":104,"context_line":"Primary defines the script that has run management capabilities,"},{"line_number":105,"context_line":"and dependencies defines the script that performs the actual injection."},{"line_number":106,"context_line":"These definitions allow for flexible configuration by the user."},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"It also keep supporting the single artifact format currently used by VNFD."}],"source_content_type":"text/x-rst","patch_set":3,"id":"42584c5e_c91a7797","line":105,"range":{"start_line":105,"start_character":4,"end_line":105,"end_character":16},"updated":"2022-06-27 20:47:41.000000000","message":"ditto: better to describe as ``dependencies``","commit_id":"73faa25dc69f0fe86fbe320661b72bac61a9a88d"},{"author":{"_account_id":32736,"name":"Wataru Juso","email":"w-juso@nec.com","username":"w-juso"},"change_message_id":"0f449dcba7c32a7925a583c8003e442b68e7c7e0","unresolved":true,"context_lines":[{"line_number":102,"context_line":"          file: ../ScriptAnsible/config_end.yaml"},{"line_number":103,"context_line":""},{"line_number":104,"context_line":"Primary defines the script that has run management capabilities,"},{"line_number":105,"context_line":"and dependencies defines the script that performs the actual injection."},{"line_number":106,"context_line":"These definitions allow for flexible configuration by the user."},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"It also keep supporting the single artifact format currently used by VNFD."}],"source_content_type":"text/x-rst","patch_set":3,"id":"f8f6f052_f26ef915","line":105,"range":{"start_line":105,"start_character":4,"end_line":105,"end_character":16},"in_reply_to":"42584c5e_c91a7797","updated":"2022-07-22 05:56:28.000000000","message":"fixed.","commit_id":"73faa25dc69f0fe86fbe320661b72bac61a9a88d"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"291ed4fc3a1217c038c8e9af87d48e2accea5528","unresolved":true,"context_lines":[{"line_number":105,"context_line":"and dependencies defines the script that performs the actual injection."},{"line_number":106,"context_line":"These definitions allow for flexible configuration by the user."},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"It also keep supporting the single artifact format currently used by VNFD."},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"Alternatives"},{"line_number":111,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"52140d0d_94a89c7d","line":108,"updated":"2022-06-27 20:47:41.000000000","message":"What do you mean by \"it\"? It just explaining about backward compatibility?","commit_id":"73faa25dc69f0fe86fbe320661b72bac61a9a88d"},{"author":{"_account_id":32736,"name":"Wataru Juso","email":"w-juso@nec.com","username":"w-juso"},"change_message_id":"0f449dcba7c32a7925a583c8003e442b68e7c7e0","unresolved":true,"context_lines":[{"line_number":105,"context_line":"and dependencies defines the script that performs the actual injection."},{"line_number":106,"context_line":"These definitions allow for flexible configuration by the user."},{"line_number":107,"context_line":""},{"line_number":108,"context_line":"It also keep supporting the single artifact format currently used by VNFD."},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"Alternatives"},{"line_number":111,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":3,"id":"a7148fe1_4203575c","line":108,"in_reply_to":"52140d0d_94a89c7d","updated":"2022-07-22 05:56:28.000000000","message":"yes, I say it is backward compatibility.\nI looked again the sentence.","commit_id":"73faa25dc69f0fe86fbe320661b72bac61a9a88d"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"1b66d0e5ee9767713d215c93ce3fa858a85b5d1b","unresolved":true,"context_lines":[{"line_number":50,"context_line":"          file: /opt/stack/tacker/tacker/vnfm/mgmt_drivers/ansible/ansible.py"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"If we want the different configurations for VNFD of different deployment"},{"line_number":53,"context_line":"floavor, we need to implement the conditional branch within a Driver or"},{"line_number":54,"context_line":"provide the another Driver."},{"line_number":55,"context_line":"However, these methods increase the specificity of the Driver."},{"line_number":56,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"610e8b5b_c7517222","line":53,"range":{"start_line":53,"start_character":0,"end_line":53,"end_character":7},"updated":"2022-07-25 15:33:10.000000000","message":"typo: flavor","commit_id":"89bf63bd00c5a4d4ca585479609b9bb9e07f4e5e"},{"author":{"_account_id":32736,"name":"Wataru Juso","email":"w-juso@nec.com","username":"w-juso"},"change_message_id":"17ffb892ef51f1db0ef38023fba1fdc20b61335b","unresolved":true,"context_lines":[{"line_number":50,"context_line":"          file: /opt/stack/tacker/tacker/vnfm/mgmt_drivers/ansible/ansible.py"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"If we want the different configurations for VNFD of different deployment"},{"line_number":53,"context_line":"floavor, we need to implement the conditional branch within a Driver or"},{"line_number":54,"context_line":"provide the another Driver."},{"line_number":55,"context_line":"However, these methods increase the specificity of the Driver."},{"line_number":56,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"89395502_ee6d6fb0","line":53,"range":{"start_line":53,"start_character":0,"end_line":53,"end_character":7},"in_reply_to":"610e8b5b_c7517222","updated":"2022-07-26 05:34:07.000000000","message":"modified.","commit_id":"89bf63bd00c5a4d4ca585479609b9bb9e07f4e5e"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"09bcd25f68c061033e2a626d74a61d3b659e6d12","unresolved":false,"context_lines":[{"line_number":50,"context_line":"          file: /opt/stack/tacker/tacker/vnfm/mgmt_drivers/ansible/ansible.py"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"If we want the different configurations for VNFD of different deployment"},{"line_number":53,"context_line":"floavor, we need to implement the conditional branch within a Driver or"},{"line_number":54,"context_line":"provide the another Driver."},{"line_number":55,"context_line":"However, these methods increase the specificity of the Driver."},{"line_number":56,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"a78c723b_06816f48","line":53,"range":{"start_line":53,"start_character":0,"end_line":53,"end_character":7},"in_reply_to":"89395502_ee6d6fb0","updated":"2022-07-26 13:55:59.000000000","message":"Ack","commit_id":"89bf63bd00c5a4d4ca585479609b9bb9e07f4e5e"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"1b66d0e5ee9767713d215c93ce3fa858a85b5d1b","unresolved":true,"context_lines":[{"line_number":240,"context_line":"  Wataru Juso \u003cw-juso@nec.com\u003e"},{"line_number":241,"context_line":""},{"line_number":242,"context_line":"Other contributors:"},{"line_number":243,"context_line":"  Aldinson C. Esto \u003cesto.aln@nec.com\u003e"},{"line_number":244,"context_line":"  Pooja Singla \u003cpooja.singla@india.nec.com\u003e"},{"line_number":245,"context_line":""},{"line_number":246,"context_line":"Work Items"},{"line_number":247,"context_line":"----------"}],"source_content_type":"text/x-rst","patch_set":6,"id":"6d88ef0d_85501992","line":244,"range":{"start_line":243,"start_character":0,"end_line":244,"end_character":43},"updated":"2022-07-25 15:33:10.000000000","message":"Could you insert a blank line between contributors?","commit_id":"89bf63bd00c5a4d4ca585479609b9bb9e07f4e5e"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"09bcd25f68c061033e2a626d74a61d3b659e6d12","unresolved":false,"context_lines":[{"line_number":240,"context_line":"  Wataru Juso \u003cw-juso@nec.com\u003e"},{"line_number":241,"context_line":""},{"line_number":242,"context_line":"Other contributors:"},{"line_number":243,"context_line":"  Aldinson C. Esto \u003cesto.aln@nec.com\u003e"},{"line_number":244,"context_line":"  Pooja Singla \u003cpooja.singla@india.nec.com\u003e"},{"line_number":245,"context_line":""},{"line_number":246,"context_line":"Work Items"},{"line_number":247,"context_line":"----------"}],"source_content_type":"text/x-rst","patch_set":6,"id":"c5eee8e1_7979da7e","line":244,"range":{"start_line":243,"start_character":0,"end_line":244,"end_character":43},"in_reply_to":"2bb44792_8a15c4db","updated":"2022-07-26 13:55:59.000000000","message":"Ack","commit_id":"89bf63bd00c5a4d4ca585479609b9bb9e07f4e5e"},{"author":{"_account_id":32736,"name":"Wataru Juso","email":"w-juso@nec.com","username":"w-juso"},"change_message_id":"17ffb892ef51f1db0ef38023fba1fdc20b61335b","unresolved":true,"context_lines":[{"line_number":240,"context_line":"  Wataru Juso \u003cw-juso@nec.com\u003e"},{"line_number":241,"context_line":""},{"line_number":242,"context_line":"Other contributors:"},{"line_number":243,"context_line":"  Aldinson C. Esto \u003cesto.aln@nec.com\u003e"},{"line_number":244,"context_line":"  Pooja Singla \u003cpooja.singla@india.nec.com\u003e"},{"line_number":245,"context_line":""},{"line_number":246,"context_line":"Work Items"},{"line_number":247,"context_line":"----------"}],"source_content_type":"text/x-rst","patch_set":6,"id":"2bb44792_8a15c4db","line":244,"range":{"start_line":243,"start_character":0,"end_line":244,"end_character":43},"in_reply_to":"6d88ef0d_85501992","updated":"2022-07-26 05:34:07.000000000","message":"modified.","commit_id":"89bf63bd00c5a4d4ca585479609b9bb9e07f4e5e"}]}