)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":32102,"name":"Manpreet Kaur","email":"kaurmanpreet2620@gmail.com","username":"manpreet"},"change_message_id":"dfbd4cf8a7b28c5a18357fee19c528412eed3b4e","unresolved":true,"context_lines":[{"line_number":9,"context_line":"This specification describes Terraform Infra-driver in Tacker. The"},{"line_number":10,"context_line":"scope of the present document includes VNF instantiation and"},{"line_number":11,"context_line":"termination with this new infra-driver, using AWS as an example NFVI."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I4df1f8f241742b07784751b507466509693c7a8a"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"4ae994b8_2ad470ed","line":12,"updated":"2023-04-11 15:56:36.000000000","message":"Please add blueprint tag.","commit_id":"b21bec33ff461ed7f4d46a8dc7e9a89e9a899e63"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"b6b3f3354bd11a66f25cbf1a3e13e1c93af6a8d3","unresolved":true,"context_lines":[{"line_number":9,"context_line":"This specification describes Terraform Infra-driver in Tacker. The"},{"line_number":10,"context_line":"scope of the present document includes VNF instantiation and"},{"line_number":11,"context_line":"termination with this new infra-driver, using AWS as an example NFVI."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I4df1f8f241742b07784751b507466509693c7a8a"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"a596cac9_c7f1e451","line":12,"in_reply_to":"4ae994b8_2ad470ed","updated":"2023-07-09 14:18:18.000000000","message":"I added the blueprint tag.","commit_id":"b21bec33ff461ed7f4d46a8dc7e9a89e9a899e63"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"e5ca55571f3f59419bf94ee0b2791ba62b67384b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"78844450_14348b6c","updated":"2023-05-16 13:23:31.000000000","message":"Thanks for your patch, please kindly find my comments.","commit_id":"b21bec33ff461ed7f4d46a8dc7e9a89e9a899e63"},{"author":{"_account_id":32102,"name":"Manpreet Kaur","email":"kaurmanpreet2620@gmail.com","username":"manpreet"},"change_message_id":"dfbd4cf8a7b28c5a18357fee19c528412eed3b4e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"787c3044_ee9d692f","updated":"2023-04-11 15:56:36.000000000","message":"Thanks for your patch. Please find few comments.","commit_id":"b21bec33ff461ed7f4d46a8dc7e9a89e9a899e63"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"82ba4c484ba8961bf85f9088f829908b5d7600e4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7aed8369_0365ae46","updated":"2023-06-26 01:43:32.000000000","message":"ping","commit_id":"b21bec33ff461ed7f4d46a8dc7e9a89e9a899e63"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"b6b3f3354bd11a66f25cbf1a3e13e1c93af6a8d3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"93ea62ce_b626c4b8","updated":"2023-07-09 14:18:18.000000000","message":"Sorry for the late reply.\n\nI\u0027ve fixed all comments.\nPlease kindly review this patch again.","commit_id":"41561331d457a963b474dfc538e4163a456b7cb9"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"b068c0f19147d7a2ca7f50a45236ee060e806e42","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"aca7dffd_5b0ad085","updated":"2023-07-10 08:29:37.000000000","message":"recheck, no results notified","commit_id":"a0c51f7018b037396f4fcbf03074530e125c8e23"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"3a8790235fc673d8fd337eeefcd399886f6e0353","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"a40183e3_12be16fa","updated":"2023-07-11 09:28:21.000000000","message":"Thanks for your updating.\nRevised content is LGTM, but I have a additional question, please kindly find it.","commit_id":"69b9f99f96ea4d8a2a3a0b561cff7217c0bdc63d"}],"specs/2023.2/terraform-infra-driver.rst":[{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"e5ca55571f3f59419bf94ee0b2791ba62b67384b","unresolved":true,"context_lines":[{"line_number":145,"context_line":"    Terraform infra-driver executes terraform command to apply configuration"},{"line_number":146,"context_line":"    files to Terraform."},{"line_number":147,"context_line":"#. Call target service API"},{"line_number":148,"context_line":"    Terraform calls target service APIs according to the configuration file"},{"line_number":149,"context_line":"#. Create VM(s)"},{"line_number":150,"context_line":"    Target service (e.g., OpenStack Nova, AWS EC2, etc) creates VM(s)."},{"line_number":151,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"b7327877_36182e3a","line":148,"updated":"2023-05-16 13:23:31.000000000","message":"nits: Could you add a period to end of line.","commit_id":"b21bec33ff461ed7f4d46a8dc7e9a89e9a899e63"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"b6b3f3354bd11a66f25cbf1a3e13e1c93af6a8d3","unresolved":true,"context_lines":[{"line_number":145,"context_line":"    Terraform infra-driver executes terraform command to apply configuration"},{"line_number":146,"context_line":"    files to Terraform."},{"line_number":147,"context_line":"#. Call target service API"},{"line_number":148,"context_line":"    Terraform calls target service APIs according to the configuration file"},{"line_number":149,"context_line":"#. Create VM(s)"},{"line_number":150,"context_line":"    Target service (e.g., OpenStack Nova, AWS EC2, etc) creates VM(s)."},{"line_number":151,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"ecd607da_c976bf56","line":148,"in_reply_to":"b7327877_36182e3a","updated":"2023-07-09 14:18:18.000000000","message":"Will fix in the next PS.","commit_id":"b21bec33ff461ed7f4d46a8dc7e9a89e9a899e63"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"3a8790235fc673d8fd337eeefcd399886f6e0353","unresolved":false,"context_lines":[{"line_number":145,"context_line":"    Terraform infra-driver executes terraform command to apply configuration"},{"line_number":146,"context_line":"    files to Terraform."},{"line_number":147,"context_line":"#. Call target service API"},{"line_number":148,"context_line":"    Terraform calls target service APIs according to the configuration file"},{"line_number":149,"context_line":"#. Create VM(s)"},{"line_number":150,"context_line":"    Target service (e.g., OpenStack Nova, AWS EC2, etc) creates VM(s)."},{"line_number":151,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"550bbfe2_7921495b","line":148,"in_reply_to":"ecd607da_c976bf56","updated":"2023-07-11 09:28:21.000000000","message":"Ack","commit_id":"b21bec33ff461ed7f4d46a8dc7e9a89e9a899e63"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"e5ca55571f3f59419bf94ee0b2791ba62b67384b","unresolved":true,"context_lines":[{"line_number":203,"context_line":"Given that terraform config file is located in VNF packages, state files are"},{"line_number":204,"context_line":"created and managed for each VNF package. Terraform provides several options"},{"line_number":205,"context_line":"(i.e., backend) to store the state file [#tf_state_backend]_. Based on the"},{"line_number":206,"context_line":"available backend, the available options for Tacker is the following:"},{"line_number":207,"context_line":""},{"line_number":208,"context_line":"#. Store state files as a local file"},{"line_number":209,"context_line":"#. Store state files in InstantiatedVnfInfo"}],"source_content_type":"text/x-rst","patch_set":1,"id":"0771e660_2808a528","line":206,"range":{"start_line":206,"start_character":52,"end_line":206,"end_character":68},"updated":"2023-05-16 13:23:31.000000000","message":"are the followings?","commit_id":"b21bec33ff461ed7f4d46a8dc7e9a89e9a899e63"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"b6b3f3354bd11a66f25cbf1a3e13e1c93af6a8d3","unresolved":true,"context_lines":[{"line_number":203,"context_line":"Given that terraform config file is located in VNF packages, state files are"},{"line_number":204,"context_line":"created and managed for each VNF package. Terraform provides several options"},{"line_number":205,"context_line":"(i.e., backend) to store the state file [#tf_state_backend]_. Based on the"},{"line_number":206,"context_line":"available backend, the available options for Tacker is the following:"},{"line_number":207,"context_line":""},{"line_number":208,"context_line":"#. Store state files as a local file"},{"line_number":209,"context_line":"#. Store state files in InstantiatedVnfInfo"}],"source_content_type":"text/x-rst","patch_set":1,"id":"ae828439_03d487db","line":206,"range":{"start_line":206,"start_character":52,"end_line":206,"end_character":68},"in_reply_to":"0771e660_2808a528","updated":"2023-07-09 14:18:18.000000000","message":"Will fix in the next PS.","commit_id":"b21bec33ff461ed7f4d46a8dc7e9a89e9a899e63"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"3a8790235fc673d8fd337eeefcd399886f6e0353","unresolved":false,"context_lines":[{"line_number":203,"context_line":"Given that terraform config file is located in VNF packages, state files are"},{"line_number":204,"context_line":"created and managed for each VNF package. Terraform provides several options"},{"line_number":205,"context_line":"(i.e., backend) to store the state file [#tf_state_backend]_. Based on the"},{"line_number":206,"context_line":"available backend, the available options for Tacker is the following:"},{"line_number":207,"context_line":""},{"line_number":208,"context_line":"#. Store state files as a local file"},{"line_number":209,"context_line":"#. Store state files in InstantiatedVnfInfo"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9103cd4b_c0885bdb","line":206,"range":{"start_line":206,"start_character":52,"end_line":206,"end_character":68},"in_reply_to":"ae828439_03d487db","updated":"2023-07-11 09:28:21.000000000","message":"Ack","commit_id":"b21bec33ff461ed7f4d46a8dc7e9a89e9a899e63"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"e5ca55571f3f59419bf94ee0b2791ba62b67384b","unresolved":true,"context_lines":[{"line_number":211,"context_line":"#. Store state files in Kubernetes Secret"},{"line_number":212,"context_line":"#. Store state files in PostgresDB"},{"line_number":213,"context_line":""},{"line_number":214,"context_line":"The first option is the easiest way and most feasible. As Tacker extracts a VNF"},{"line_number":215,"context_line":"package into a local directory, we can place a state file in that local"},{"line_number":216,"context_line":"directory."},{"line_number":217,"context_line":""},{"line_number":218,"context_line":"If the first option is not possible, for example such local directory is not"}],"source_content_type":"text/x-rst","patch_set":1,"id":"c8e6edc1_3f32497c","line":215,"range":{"start_line":214,"start_character":55,"end_line":215,"end_character":30},"updated":"2023-05-16 13:23:31.000000000","message":"Just a question, I\u0027m sorry if I\u0027m wrong, but I think VNF Package is extracted per vnfdId.\nDoes option 1 mean that only one VNF Instance can be created from one VNF Package (\u003d vnfdId)?","commit_id":"b21bec33ff461ed7f4d46a8dc7e9a89e9a899e63"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"436ed4667c8095414324b632ef6e6edee2773290","unresolved":false,"context_lines":[{"line_number":211,"context_line":"#. Store state files in Kubernetes Secret"},{"line_number":212,"context_line":"#. Store state files in PostgresDB"},{"line_number":213,"context_line":""},{"line_number":214,"context_line":"The first option is the easiest way and most feasible. As Tacker extracts a VNF"},{"line_number":215,"context_line":"package into a local directory, we can place a state file in that local"},{"line_number":216,"context_line":"directory."},{"line_number":217,"context_line":""},{"line_number":218,"context_line":"If the first option is not possible, for example such local directory is not"}],"source_content_type":"text/x-rst","patch_set":1,"id":"fa3c1de7_1b456d75","line":215,"range":{"start_line":214,"start_character":55,"end_line":215,"end_character":30},"in_reply_to":"03e3956f_0935a769","updated":"2023-07-12 00:44:56.000000000","message":"Thank you, I understand that this feature is currently under development with option 1 (create temp directory for each VNF instance to store the \"state file\" and \"state lock file\".)","commit_id":"b21bec33ff461ed7f4d46a8dc7e9a89e9a899e63"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"3a8790235fc673d8fd337eeefcd399886f6e0353","unresolved":true,"context_lines":[{"line_number":211,"context_line":"#. Store state files in Kubernetes Secret"},{"line_number":212,"context_line":"#. Store state files in PostgresDB"},{"line_number":213,"context_line":""},{"line_number":214,"context_line":"The first option is the easiest way and most feasible. As Tacker extracts a VNF"},{"line_number":215,"context_line":"package into a local directory, we can place a state file in that local"},{"line_number":216,"context_line":"directory."},{"line_number":217,"context_line":""},{"line_number":218,"context_line":"If the first option is not possible, for example such local directory is not"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9b8979c9_95cbdccb","line":215,"range":{"start_line":214,"start_character":55,"end_line":215,"end_character":30},"in_reply_to":"1b2137bf_33364d85","updated":"2023-07-11 09:28:21.000000000","message":"Thanks for your answer. Sorry, I have question again.\n\nIsn\u0027t a temporary directory created for each VNF Package (each vnfdId)?\nand I think its directory path is configured by `vnf_package_csar_path` option.\n\nIf I\u0027m missing, where does the current code create a temporary directory for each VNF instance?","commit_id":"b21bec33ff461ed7f4d46a8dc7e9a89e9a899e63"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"23adff8e0835fe4a1c9a774b349aa0fa95f7eb0d","unresolved":true,"context_lines":[{"line_number":211,"context_line":"#. Store state files in Kubernetes Secret"},{"line_number":212,"context_line":"#. Store state files in PostgresDB"},{"line_number":213,"context_line":""},{"line_number":214,"context_line":"The first option is the easiest way and most feasible. As Tacker extracts a VNF"},{"line_number":215,"context_line":"package into a local directory, we can place a state file in that local"},{"line_number":216,"context_line":"directory."},{"line_number":217,"context_line":""},{"line_number":218,"context_line":"If the first option is not possible, for example such local directory is not"}],"source_content_type":"text/x-rst","patch_set":1,"id":"03e3956f_0935a769","line":215,"range":{"start_line":214,"start_character":55,"end_line":215,"end_character":30},"in_reply_to":"9b8979c9_95cbdccb","updated":"2023-07-11 11:50:42.000000000","message":"Sorry, it\u0027s completely misleading.\n\n```\nBy the way, currently, in our codes THAT IS UNDER DEVELOPMENT AND NOT YET PUSHED, temporal directory is created for each VNF instance.\n```\n\nLike you said. The Tacker creates a temporary directory for each VNF Package.\n\nDoes this make sense for you?","commit_id":"b21bec33ff461ed7f4d46a8dc7e9a89e9a899e63"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"b6b3f3354bd11a66f25cbf1a3e13e1c93af6a8d3","unresolved":true,"context_lines":[{"line_number":211,"context_line":"#. Store state files in Kubernetes Secret"},{"line_number":212,"context_line":"#. Store state files in PostgresDB"},{"line_number":213,"context_line":""},{"line_number":214,"context_line":"The first option is the easiest way and most feasible. As Tacker extracts a VNF"},{"line_number":215,"context_line":"package into a local directory, we can place a state file in that local"},{"line_number":216,"context_line":"directory."},{"line_number":217,"context_line":""},{"line_number":218,"context_line":"If the first option is not possible, for example such local directory is not"}],"source_content_type":"text/x-rst","patch_set":1,"id":"1b2137bf_33364d85","line":215,"range":{"start_line":214,"start_character":55,"end_line":215,"end_character":30},"in_reply_to":"c8e6edc1_3f32497c","updated":"2023-07-09 14:18:18.000000000","message":"You\u0027re right. I fixed up the description. By the way, currently, in our actual codes, temporal directory is created for each VNF instance.","commit_id":"b21bec33ff461ed7f4d46a8dc7e9a89e9a899e63"},{"author":{"_account_id":32102,"name":"Manpreet Kaur","email":"kaurmanpreet2620@gmail.com","username":"manpreet"},"change_message_id":"dfbd4cf8a7b28c5a18357fee19c528412eed3b4e","unresolved":true,"context_lines":[{"line_number":380,"context_line":""},{"line_number":381,"context_line":"* Terraform v1.4.0 or later"},{"line_number":382,"context_line":""},{"line_number":383,"context_line":"Testing"},{"line_number":384,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":385,"context_line":""},{"line_number":386,"context_line":"Terraform supports several providers including OpenStack [#tf_os]_, Kubernetes"}],"source_content_type":"text/x-rst","patch_set":1,"id":"edfb1ebf_c1f98efa","line":383,"updated":"2023-04-11 15:56:36.000000000","message":"Are we planning to document testing process for Terraform drivers unlike others?","commit_id":"b21bec33ff461ed7f4d46a8dc7e9a89e9a899e63"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"b6b3f3354bd11a66f25cbf1a3e13e1c93af6a8d3","unresolved":true,"context_lines":[{"line_number":380,"context_line":""},{"line_number":381,"context_line":"* Terraform v1.4.0 or later"},{"line_number":382,"context_line":""},{"line_number":383,"context_line":"Testing"},{"line_number":384,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":385,"context_line":""},{"line_number":386,"context_line":"Terraform supports several providers including OpenStack [#tf_os]_, Kubernetes"}],"source_content_type":"text/x-rst","patch_set":1,"id":"40892432_59317a07","line":383,"in_reply_to":"edfb1ebf_c1f98efa","updated":"2023-07-09 14:18:18.000000000","message":"No, we won\u0027t create documents for tests. Could you tell me why did you think so?","commit_id":"b21bec33ff461ed7f4d46a8dc7e9a89e9a899e63"},{"author":{"_account_id":32102,"name":"Manpreet Kaur","email":"kaurmanpreet2620@gmail.com","username":"manpreet"},"change_message_id":"dfbd4cf8a7b28c5a18357fee19c528412eed3b4e","unresolved":true,"context_lines":[{"line_number":387,"context_line":"[#tf_k8s]_, Docker [#tf_local]_ and local files [#tf_docker]_. The easiest way"},{"line_number":388,"context_line":"is to use OpenStack in the functional tests. As terraform infra-driver is"},{"line_number":389,"context_line":"transparent for VNF packages, its normality must not be affected by the"},{"line_number":390,"context_line":"difference of used providers. In this sense, we can tests its normality with"},{"line_number":391,"context_line":"other available providers, such as Kubernetes, docker or local provider."},{"line_number":392,"context_line":""},{"line_number":393,"context_line":"Documentation Impact"}],"source_content_type":"text/x-rst","patch_set":1,"id":"f2dddb8f_4aec464d","line":390,"range":{"start_line":390,"start_character":52,"end_line":390,"end_character":57},"updated":"2023-04-11 15:56:36.000000000","message":"Please change to \u0027test\u0027.","commit_id":"b21bec33ff461ed7f4d46a8dc7e9a89e9a899e63"},{"author":{"_account_id":33455,"name":"Hiromu Asahina","email":"hiromu.a5a@gmail.com","username":"h_asahina"},"change_message_id":"b6b3f3354bd11a66f25cbf1a3e13e1c93af6a8d3","unresolved":true,"context_lines":[{"line_number":387,"context_line":"[#tf_k8s]_, Docker [#tf_local]_ and local files [#tf_docker]_. The easiest way"},{"line_number":388,"context_line":"is to use OpenStack in the functional tests. As terraform infra-driver is"},{"line_number":389,"context_line":"transparent for VNF packages, its normality must not be affected by the"},{"line_number":390,"context_line":"difference of used providers. In this sense, we can tests its normality with"},{"line_number":391,"context_line":"other available providers, such as Kubernetes, docker or local provider."},{"line_number":392,"context_line":""},{"line_number":393,"context_line":"Documentation Impact"}],"source_content_type":"text/x-rst","patch_set":1,"id":"362265a8_9b9f97fe","line":390,"range":{"start_line":390,"start_character":52,"end_line":390,"end_character":57},"in_reply_to":"f2dddb8f_4aec464d","updated":"2023-07-09 14:18:18.000000000","message":"Will fix in the next PS.","commit_id":"b21bec33ff461ed7f4d46a8dc7e9a89e9a899e63"}]}