)]}'
{"specs/victoria/container-network-function.rst":[{"author":{"_account_id":22290,"name":"hyunsik Yang","email":"yangun@dcn.ssu.ac.kr","username":"hyunsikYang"},"change_message_id":"cf4af144b9481dd701cefaf17efb097e654a4aaa","unresolved":false,"context_lines":[{"line_number":75,"context_line":"                                    +-------+--------------+"},{"line_number":76,"context_line":"    +--------------------------+    |       v         VIM  |"},{"line_number":77,"context_line":"    |+---------+   +---------+ |    |  +----------+        |"},{"line_number":78,"context_line":"    ||  CISI   |   |  CISI   |\u003c+----+--+  CISM    |        |"},{"line_number":79,"context_line":"    |+---------+   +---------+ |    |  +----------+        |"},{"line_number":80,"context_line":"    |                          |    +----------------------+"},{"line_number":81,"context_line":"    |     Pre-installed        |"}],"source_content_type":"text/x-rst","patch_set":3,"id":"bf51134e_171639a9","line":78,"range":{"start_line":78,"start_character":21,"end_line":78,"end_character":47},"updated":"2020-06-22 23:55:38.000000000","message":"I think we should write down full meaning of CISI and CISM.","commit_id":"bb7b885fd4e8b24a00c5802914016a5d933ee679"},{"author":{"_account_id":27880,"name":"Yoshito Ito","email":"yoshito.itou.dr@hco.ntt.co.jp","username":"yoshito-ito"},"change_message_id":"f1ee932b9288a2a2e20e0fc1b3263e26ab316ed9","unresolved":false,"context_lines":[{"line_number":75,"context_line":"                                    +-------+--------------+"},{"line_number":76,"context_line":"    +--------------------------+    |       v         VIM  |"},{"line_number":77,"context_line":"    |+---------+   +---------+ |    |  +----------+        |"},{"line_number":78,"context_line":"    ||  CISI   |   |  CISI   |\u003c+----+--+  CISM    |        |"},{"line_number":79,"context_line":"    |+---------+   +---------+ |    |  +----------+        |"},{"line_number":80,"context_line":"    |                          |    +----------------------+"},{"line_number":81,"context_line":"    |     Pre-installed        |"}],"source_content_type":"text/x-rst","patch_set":3,"id":"bf51134e_5990d1ed","line":78,"range":{"start_line":78,"start_character":21,"end_line":78,"end_character":47},"in_reply_to":"bf51134e_171639a9","updated":"2020-06-24 01:38:38.000000000","message":"I added full meaning of CIS and CISM. I changed CISI to CIS according to the definition in IFA029.","commit_id":"bb7b885fd4e8b24a00c5802914016a5d933ee679"},{"author":{"_account_id":17255,"name":"Toshiaki Takahashi","email":"ts-takahashi@nec.com","username":"takahashi_ts"},"change_message_id":"adfced62382cbc0f768938df68fa9b5b082ed245","unresolved":false,"context_lines":[{"line_number":121,"context_line":"  +---------------------------+                 +-----------------------------------+"},{"line_number":122,"context_line":""},{"line_number":123,"context_line":""},{"line_number":124,"context_line":"The diagram shows that Kubernetes driver will use CNF definition in YAML format"},{"line_number":125,"context_line":"to instantiate CNF on pre-installed cluster. VNFD will not contain any"},{"line_number":126,"context_line":"resource information such as VDU, Connection points, Virtual links because"},{"line_number":127,"context_line":"all required components of CNF will be specified in CNF definition file. VNFD"}],"source_content_type":"text/x-rst","patch_set":3,"id":"bf51134e_90788fc4","line":124,"range":{"start_line":124,"start_character":50,"end_line":124,"end_character":64},"updated":"2020-06-23 06:46:35.000000000","message":"In this spec, is \"CNF definition\" Kubernetes manifest file? If so, is it better to explain that?\nOnly \"CNF definition\" seems more general terms.","commit_id":"bb7b885fd4e8b24a00c5802914016a5d933ee679"},{"author":{"_account_id":27880,"name":"Yoshito Ito","email":"yoshito.itou.dr@hco.ntt.co.jp","username":"yoshito-ito"},"change_message_id":"f1ee932b9288a2a2e20e0fc1b3263e26ab316ed9","unresolved":false,"context_lines":[{"line_number":121,"context_line":"  +---------------------------+                 +-----------------------------------+"},{"line_number":122,"context_line":""},{"line_number":123,"context_line":""},{"line_number":124,"context_line":"The diagram shows that Kubernetes driver will use CNF definition in YAML format"},{"line_number":125,"context_line":"to instantiate CNF on pre-installed cluster. VNFD will not contain any"},{"line_number":126,"context_line":"resource information such as VDU, Connection points, Virtual links because"},{"line_number":127,"context_line":"all required components of CNF will be specified in CNF definition file. VNFD"}],"source_content_type":"text/x-rst","patch_set":3,"id":"bf51134e_592c51ab","line":124,"range":{"start_line":124,"start_character":50,"end_line":124,"end_character":64},"in_reply_to":"bf51134e_90788fc4","updated":"2020-06-24 01:38:38.000000000","message":"I changed it to \"Kubernetes object file\" because the name of \"manifest\" is a little confusing to the one in VNF Package. According to Kubernetes official document, the file name seems not clearly defined so I use \"Kubernetes object file\" in this spec.","commit_id":"bb7b885fd4e8b24a00c5802914016a5d933ee679"},{"author":{"_account_id":22290,"name":"hyunsik Yang","email":"yangun@dcn.ssu.ac.kr","username":"hyunsikYang"},"change_message_id":"cf4af144b9481dd701cefaf17efb097e654a4aaa","unresolved":false,"context_lines":[{"line_number":124,"context_line":"The diagram shows that Kubernetes driver will use CNF definition in YAML format"},{"line_number":125,"context_line":"to instantiate CNF on pre-installed cluster. VNFD will not contain any"},{"line_number":126,"context_line":"resource information such as VDU, Connection points, Virtual links because"},{"line_number":127,"context_line":"all required components of CNF will be specified in CNF definition file. VNFD"},{"line_number":128,"context_line":"will be used only to identify the flavour of CNF."},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"Sample VNFD file:"},{"line_number":131,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"bf51134e_37389d3b","line":128,"range":{"start_line":127,"start_character":73,"end_line":128,"end_character":49},"updated":"2020-06-22 23:55:38.000000000","message":"Hi. For this, I think the best way is that when we create CNF, \nwe make a system that it doesn\u0027t check VNFD. \n\nOr, it process internally, first and changed it. \n\nWhat do you think?","commit_id":"bb7b885fd4e8b24a00c5802914016a5d933ee679"},{"author":{"_account_id":27880,"name":"Yoshito Ito","email":"yoshito.itou.dr@hco.ntt.co.jp","username":"yoshito-ito"},"change_message_id":"f1ee932b9288a2a2e20e0fc1b3263e26ab316ed9","unresolved":false,"context_lines":[{"line_number":124,"context_line":"The diagram shows that Kubernetes driver will use CNF definition in YAML format"},{"line_number":125,"context_line":"to instantiate CNF on pre-installed cluster. VNFD will not contain any"},{"line_number":126,"context_line":"resource information such as VDU, Connection points, Virtual links because"},{"line_number":127,"context_line":"all required components of CNF will be specified in CNF definition file. VNFD"},{"line_number":128,"context_line":"will be used only to identify the flavour of CNF."},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"Sample VNFD file:"},{"line_number":131,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"bf51134e_dc6defcc","line":128,"range":{"start_line":127,"start_character":73,"end_line":128,"end_character":49},"in_reply_to":"bf51134e_37389d3b","updated":"2020-06-24 01:38:38.000000000","message":"IMO, it\u0027s better to use VNFD for two reasons.\n1. CNF with VM-based App\n  CNF can be deployed with VM based applications. In the case,\n  it\u0027s better to allow operators to define CNF and VNF in a\n  single VNFD.\n\n2. Single VNF Package for k8s of pre-installed and on VM\n  It\u0027s simple for operators to deploy CNF with single VNF\n  Package for both case of instantiation on pre-installed k8s\n  and on k8s inside VM. In this case, we need top-level VNF \n  and lower-level VNFD. For pre-installed k8s, we use an empty \n  VNFD as described in this spec. For CNF inside VM, we can \n  use VNFD as usual specified in my the other spec of \"CNF on \n  Kubernetes cluster inside VM\".","commit_id":"bb7b885fd4e8b24a00c5802914016a5d933ee679"},{"author":{"_account_id":22290,"name":"hyunsik Yang","email":"yangun@dcn.ssu.ac.kr","username":"hyunsikYang"},"change_message_id":"8aa05722093e592006017dffcade3ca3f45b307c","unresolved":false,"context_lines":[{"line_number":124,"context_line":"The diagram shows that Kubernetes driver will use CNF definition in YAML format"},{"line_number":125,"context_line":"to instantiate CNF on pre-installed cluster. VNFD will not contain any"},{"line_number":126,"context_line":"resource information such as VDU, Connection points, Virtual links because"},{"line_number":127,"context_line":"all required components of CNF will be specified in CNF definition file. VNFD"},{"line_number":128,"context_line":"will be used only to identify the flavour of CNF."},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"Sample VNFD file:"},{"line_number":131,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"bf51134e_976fbb29","line":128,"range":{"start_line":127,"start_character":73,"end_line":128,"end_character":49},"in_reply_to":"bf51134e_dc6defcc","updated":"2020-06-25 06:34:43.000000000","message":"I meant that when we only create CNF, we don\u0027t need to use VNFD checking procedure. That is, if VNFD is not necessary when we create CNF with kubernetes object file , we can change it to pass without  checking VNFD. \n\nIn the future , in the tacker, we can create CNF using VNFD or kubernetes manifest file. \n\nSo, if VNFD is not necessary in your case, using fake VNFD is not good way.  \n\nI\u0027m not sure I understood your intention correctly.","commit_id":"bb7b885fd4e8b24a00c5802914016a5d933ee679"},{"author":{"_account_id":22290,"name":"hyunsik Yang","email":"yangun@dcn.ssu.ac.kr","username":"hyunsikYang"},"change_message_id":"cf4af144b9481dd701cefaf17efb097e654a4aaa","unresolved":false,"context_lines":[{"line_number":162,"context_line":"          type: string"},{"line_number":163,"context_line":""},{"line_number":164,"context_line":"      substitution_mappings:"},{"line_number":165,"context_line":"        node_type: NTT.Tacker.KubernetesCluster"},{"line_number":166,"context_line":"        properties:"},{"line_number":167,"context_line":"          flavour_id: pre_installed"},{"line_number":168,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"bf51134e_3711bdad","line":165,"range":{"start_line":165,"start_character":19,"end_line":165,"end_character":22},"updated":"2020-06-22 23:55:38.000000000","message":"Can we make it more general?","commit_id":"bb7b885fd4e8b24a00c5802914016a5d933ee679"},{"author":{"_account_id":27880,"name":"Yoshito Ito","email":"yoshito.itou.dr@hco.ntt.co.jp","username":"yoshito-ito"},"change_message_id":"f1ee932b9288a2a2e20e0fc1b3263e26ab316ed9","unresolved":false,"context_lines":[{"line_number":162,"context_line":"          type: string"},{"line_number":163,"context_line":""},{"line_number":164,"context_line":"      substitution_mappings:"},{"line_number":165,"context_line":"        node_type: NTT.Tacker.KubernetesCluster"},{"line_number":166,"context_line":"        properties:"},{"line_number":167,"context_line":"          flavour_id: pre_installed"},{"line_number":168,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"bf51134e_5c61dfd2","line":165,"range":{"start_line":165,"start_character":19,"end_line":165,"end_character":22},"in_reply_to":"bf51134e_3711bdad","updated":"2020-06-24 01:38:38.000000000","message":"I changed it to \"Company.Tacker.KubernetesCluster\".","commit_id":"bb7b885fd4e8b24a00c5802914016a5d933ee679"},{"author":{"_account_id":22290,"name":"hyunsik Yang","email":"yangun@dcn.ssu.ac.kr","username":"hyunsikYang"},"change_message_id":"cf4af144b9481dd701cefaf17efb097e654a4aaa","unresolved":false,"context_lines":[{"line_number":326,"context_line":"    VnfLcmDriver \u003c-- KubernetesDriver;"},{"line_number":327,"context_line":""},{"line_number":328,"context_line":"    VnfLcmDriver -\u003e KubernetesDriver [label \u003d \"post_vnf_instantiation()\"];"},{"line_number":329,"context_line":"    KubernetesDriver --\u003e\u003e KubernetesDriver[label \u003d"},{"line_number":330,"context_line":"        \"2. Update DB for VNFC resources[TBD]\"];"},{"line_number":331,"context_line":"    VnfLcmDriver \u003c-- KubernetesDriver;"},{"line_number":332,"context_line":"  }"},{"line_number":333,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"bf51134e_57e8b17a","line":330,"range":{"start_line":329,"start_character":25,"end_line":330,"end_character":48},"updated":"2020-06-22 23:55:38.000000000","message":"If it is not essential procedure, we doesn\u0027t need to mention it here. What do you think?","commit_id":"bb7b885fd4e8b24a00c5802914016a5d933ee679"},{"author":{"_account_id":27880,"name":"Yoshito Ito","email":"yoshito.itou.dr@hco.ntt.co.jp","username":"yoshito-ito"},"change_message_id":"f1ee932b9288a2a2e20e0fc1b3263e26ab316ed9","unresolved":false,"context_lines":[{"line_number":326,"context_line":"    VnfLcmDriver \u003c-- KubernetesDriver;"},{"line_number":327,"context_line":""},{"line_number":328,"context_line":"    VnfLcmDriver -\u003e KubernetesDriver [label \u003d \"post_vnf_instantiation()\"];"},{"line_number":329,"context_line":"    KubernetesDriver --\u003e\u003e KubernetesDriver[label \u003d"},{"line_number":330,"context_line":"        \"2. Update DB for VNFC resources[TBD]\"];"},{"line_number":331,"context_line":"    VnfLcmDriver \u003c-- KubernetesDriver;"},{"line_number":332,"context_line":"  }"},{"line_number":333,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"bf51134e_7c8ae369","line":330,"range":{"start_line":329,"start_character":25,"end_line":330,"end_character":48},"in_reply_to":"bf51134e_57e8b17a","updated":"2020-06-24 01:38:38.000000000","message":"I put this procedure because we may need to update InstantiatedVnfInfo with k8s objects here. I welcome your feedback. I\u0027m not sure what information should be stored in Tacker DB. I assume to update VnfcResourceInfo in InstantiatedVnfInfo with k8s object files and not to store status of Pods.","commit_id":"bb7b885fd4e8b24a00c5802914016a5d933ee679"},{"author":{"_account_id":22290,"name":"hyunsik Yang","email":"yangun@dcn.ssu.ac.kr","username":"hyunsikYang"},"change_message_id":"8aa05722093e592006017dffcade3ca3f45b307c","unresolved":false,"context_lines":[{"line_number":326,"context_line":"    VnfLcmDriver \u003c-- KubernetesDriver;"},{"line_number":327,"context_line":""},{"line_number":328,"context_line":"    VnfLcmDriver -\u003e KubernetesDriver [label \u003d \"post_vnf_instantiation()\"];"},{"line_number":329,"context_line":"    KubernetesDriver --\u003e\u003e KubernetesDriver[label \u003d"},{"line_number":330,"context_line":"        \"2. Update DB for VNFC resources[TBD]\"];"},{"line_number":331,"context_line":"    VnfLcmDriver \u003c-- KubernetesDriver;"},{"line_number":332,"context_line":"  }"},{"line_number":333,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"bf51134e_776ae719","line":330,"range":{"start_line":329,"start_character":25,"end_line":330,"end_character":48},"in_reply_to":"bf51134e_7c8ae369","updated":"2020-06-25 06:34:43.000000000","message":"Can i get the example of this?","commit_id":"bb7b885fd4e8b24a00c5802914016a5d933ee679"},{"author":{"_account_id":17255,"name":"Toshiaki Takahashi","email":"ts-takahashi@nec.com","username":"takahashi_ts"},"change_message_id":"adfced62382cbc0f768938df68fa9b5b082ed245","unresolved":false,"context_lines":[{"line_number":488,"context_line":"--------------------------------"},{"line_number":489,"context_line":""},{"line_number":490,"context_line":"In this spec we will support Kubernetes v1.5.0 and Kubernetes python"},{"line_number":491,"context_line":"client v11.0. Following Kubernetes APIs will be supported."},{"line_number":492,"context_line":""},{"line_number":493,"context_line":"* API Group ``core`` (CoreV1Api)"},{"line_number":494,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"bf51134e_b029539c","line":491,"range":{"start_line":491,"start_character":14,"end_line":491,"end_character":58},"updated":"2020-06-23 06:46:35.000000000","message":"Just question.\nThis spec\u0027s feature support all APIs of above API Group?\ni.e. \n* Do we need API-aware implementation?\n* Are there any APIs which you drop for some reason?","commit_id":"bb7b885fd4e8b24a00c5802914016a5d933ee679"},{"author":{"_account_id":27880,"name":"Yoshito Ito","email":"yoshito.itou.dr@hco.ntt.co.jp","username":"yoshito-ito"},"change_message_id":"f1ee932b9288a2a2e20e0fc1b3263e26ab316ed9","unresolved":false,"context_lines":[{"line_number":488,"context_line":"--------------------------------"},{"line_number":489,"context_line":""},{"line_number":490,"context_line":"In this spec we will support Kubernetes v1.5.0 and Kubernetes python"},{"line_number":491,"context_line":"client v11.0. Following Kubernetes APIs will be supported."},{"line_number":492,"context_line":""},{"line_number":493,"context_line":"* API Group ``core`` (CoreV1Api)"},{"line_number":494,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"bf51134e_dc57afc9","line":491,"range":{"start_line":491,"start_character":14,"end_line":491,"end_character":58},"in_reply_to":"bf51134e_b029539c","updated":"2020-06-24 01:38:38.000000000","message":"\u003e Do we need API-aware implementation?\n\nYes. As we use heat-client for OpenStack, we have multiple\nclient per API Groups. Let me note that it\u0027s not so \ncomplicated because the API Group is described in k8s object\nfiles so easy to find appropriate API client.\n\n\u003e Are there any APIs which you drop for some reasen?\n\nI dropped alpha and beta APIs. They might be unstable and \nchanged in newer versions. Also, some minor k8s object \nresource are dropped because they are rarely used.","commit_id":"bb7b885fd4e8b24a00c5802914016a5d933ee679"}]}