)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"efd58139bc95a8198348adbe9eee09d38ad41df8","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Aldinson C. Esto \u003cesto.aln@nec.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2020-08-27 23:49:47 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Support for VNF Package Modify"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Supported modify for VNF Package"},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"9f560f44_9f829cff","line":7,"updated":"2020-09-03 09:55:34.000000000","message":"The commit message is not clear.\nI understand that you will implement the \"Individual VNF package (GET)\" required for UpdateVNF.\nIs it right?","commit_id":"d075e1ed733a9d83f3bbb4c7730fca5fbe978896"},{"author":{"_account_id":31813,"name":"Koichi Edagawa","email":"edagawa.kc@nec.com","username":"k-edagawa"},"change_message_id":"b08236d864898a7738122f194ee3febad5efe44f","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Aldinson C. Esto \u003cesto.aln@nec.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2020-08-27 23:49:47 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Support for VNF Package Modify"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Supported modify for VNF Package"},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"9f560f44_1085cc07","line":7,"in_reply_to":"9f560f44_52ea03d8","updated":"2020-09-19 11:20:06.000000000","message":"Sorry for the delay. Fixed.\nAs for your understanding, it is correct.","commit_id":"d075e1ed733a9d83f3bbb4c7730fca5fbe978896"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"57ae8e4fd7a52ba6b1cd4d99528ba8a7f23ef0aa","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Aldinson C. Esto \u003cesto.aln@nec.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2020-08-27 23:49:47 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Support for VNF Package Modify"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Supported modify for VNF Package"},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"9f560f44_52ea03d8","line":7,"in_reply_to":"9f560f44_9f829cff","updated":"2020-09-15 07:07:15.000000000","message":"Did you check this comment?","commit_id":"d075e1ed733a9d83f3bbb4c7730fca5fbe978896"}],"tacker/api/vnflcm/v1/__init__.py":[{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"87e42aebcc79ad9d0c6c98faf59afdc97544f94d","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# Copyright (C) 2020 NTT DATA"},{"line_number":2,"context_line":"# All Rights Reserved."},{"line_number":3,"context_line":"#"},{"line_number":4,"context_line":"#    Licensed under the Apache License, Version 2.0 (the \"License\"); you may"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_cb42b00f","line":1,"updated":"2020-09-02 08:22:30.000000000","message":"I think the Copyright is incorrect.","commit_id":"d075e1ed733a9d83f3bbb4c7730fca5fbe978896"},{"author":{"_account_id":32219,"name":"Aldinson C. Esto","email":"esto.aln@nec.com","username":"esto-aln"},"change_message_id":"04cb1fae46934adde038ff005e0a1fb8c976992f","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# Copyright (C) 2020 NTT DATA"},{"line_number":2,"context_line":"# All Rights Reserved."},{"line_number":3,"context_line":"#"},{"line_number":4,"context_line":"#    Licensed under the Apache License, Version 2.0 (the \"License\"); you may"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_cc37abe4","line":1,"in_reply_to":"9f560f44_cb42b00f","updated":"2020-09-14 12:19:52.000000000","message":"It is removed.","commit_id":"d075e1ed733a9d83f3bbb4c7730fca5fbe978896"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"efd58139bc95a8198348adbe9eee09d38ad41df8","unresolved":false,"context_lines":[{"line_number":16,"context_line":"from oslo_config import cfg"},{"line_number":17,"context_line":"from oslo_log import log as logging"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"import six"},{"line_number":20,"context_line":"import webob"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"from tacker.common import csar_utils"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_7f01886a","line":19,"updated":"2020-09-03 09:55:34.000000000","message":"You don\u0027t need to use six because Python2 support is not required.","commit_id":"d075e1ed733a9d83f3bbb4c7730fca5fbe978896"},{"author":{"_account_id":32219,"name":"Aldinson C. Esto","email":"esto.aln@nec.com","username":"esto-aln"},"change_message_id":"70dd7c88f0630feab153d156f8b63d64643f63ae","unresolved":false,"context_lines":[{"line_number":16,"context_line":"from oslo_config import cfg"},{"line_number":17,"context_line":"from oslo_log import log as logging"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"import six"},{"line_number":20,"context_line":"import webob"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"from tacker.common import csar_utils"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_d899927a","line":19,"in_reply_to":"9f560f44_32914f55","updated":"2020-09-22 15:52:28.000000000","message":"It is fixed in 747678.","commit_id":"d075e1ed733a9d83f3bbb4c7730fca5fbe978896"},{"author":{"_account_id":32219,"name":"Aldinson C. Esto","email":"esto.aln@nec.com","username":"esto-aln"},"change_message_id":"27f4c2948b5243e39a3c4207345f44371b437110","unresolved":false,"context_lines":[{"line_number":16,"context_line":"from oslo_config import cfg"},{"line_number":17,"context_line":"from oslo_log import log as logging"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"import six"},{"line_number":20,"context_line":"import webob"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"from tacker.common import csar_utils"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_e3c1dd83","line":19,"in_reply_to":"9f560f44_7f01886a","updated":"2020-09-14 13:48:52.000000000","message":"\"Changed not to use six.\n→ Parent has been modified (747678), and it is supported there. \"","commit_id":"d075e1ed733a9d83f3bbb4c7730fca5fbe978896"},{"author":{"_account_id":32219,"name":"Aldinson C. Esto","email":"esto.aln@nec.com","username":"esto-aln"},"change_message_id":"68c1a7f98d0aa5577287ee41495dfe44aff09d9e","unresolved":false,"context_lines":[{"line_number":16,"context_line":"from oslo_config import cfg"},{"line_number":17,"context_line":"from oslo_log import log as logging"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"import six"},{"line_number":20,"context_line":"import webob"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"from tacker.common import csar_utils"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_270dd834","line":19,"in_reply_to":"9f560f44_89e189d2","updated":"2020-09-24 18:39:25.000000000","message":"Please check because it has been resolved by rebasing","commit_id":"d075e1ed733a9d83f3bbb4c7730fca5fbe978896"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"d15f95cd31c8d00f2098c11e716ca496cf426847","unresolved":false,"context_lines":[{"line_number":16,"context_line":"from oslo_config import cfg"},{"line_number":17,"context_line":"from oslo_log import log as logging"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"import six"},{"line_number":20,"context_line":"import webob"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"from tacker.common import csar_utils"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_89e189d2","line":19,"in_reply_to":"9f560f44_d899927a","updated":"2020-09-23 00:43:25.000000000","message":"This patch has not been rebased to the latest 747678 patchset.\nTherefore, the fix does not seem to be complete.\nPlease rebase immediately.","commit_id":"d075e1ed733a9d83f3bbb4c7730fca5fbe978896"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"57ae8e4fd7a52ba6b1cd4d99528ba8a7f23ef0aa","unresolved":false,"context_lines":[{"line_number":16,"context_line":"from oslo_config import cfg"},{"line_number":17,"context_line":"from oslo_log import log as logging"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"import six"},{"line_number":20,"context_line":"import webob"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"from tacker.common import csar_utils"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_32914f55","line":19,"in_reply_to":"9f560f44_e3c1dd83","updated":"2020-09-15 07:07:15.000000000","message":"Parent (747678) doesn\u0027t seem to be fixed, is it right?","commit_id":"d075e1ed733a9d83f3bbb4c7730fca5fbe978896"},{"author":{"_account_id":17255,"name":"Toshiaki Takahashi","email":"ts-takahashi@nec.com","username":"takahashi_ts"},"change_message_id":"dfdce95b6cb8429992449b55b1b179ac03429e26","unresolved":false,"context_lines":[{"line_number":24,"context_line":"from tacker.glance_store import store as glance_store"},{"line_number":25,"context_line":"from tacker import objects"},{"line_number":26,"context_line":"from tacker.objects import fields"},{"line_number":27,"context_line":"from tacker.vnfm.nfvo_client import FaliedDownloadContentException"},{"line_number":28,"context_line":"from tacker.vnfm.nfvo_client import UndefinedExternalSettingException"},{"line_number":29,"context_line":"from tacker.vnfm.nfvo_client import VnfPackageRequest"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"CONF \u003d cfg.CONF"},{"line_number":32,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_da249d62","line":29,"range":{"start_line":27,"start_character":0,"end_line":29,"end_character":53},"updated":"2020-09-11 11:04:23.000000000","message":"Only modules can be imported, objects cannot be imported.","commit_id":"d075e1ed733a9d83f3bbb4c7730fca5fbe978896"},{"author":{"_account_id":32219,"name":"Aldinson C. Esto","email":"esto.aln@nec.com","username":"esto-aln"},"change_message_id":"27f4c2948b5243e39a3c4207345f44371b437110","unresolved":false,"context_lines":[{"line_number":24,"context_line":"from tacker.glance_store import store as glance_store"},{"line_number":25,"context_line":"from tacker import objects"},{"line_number":26,"context_line":"from tacker.objects import fields"},{"line_number":27,"context_line":"from tacker.vnfm.nfvo_client import FaliedDownloadContentException"},{"line_number":28,"context_line":"from tacker.vnfm.nfvo_client import UndefinedExternalSettingException"},{"line_number":29,"context_line":"from tacker.vnfm.nfvo_client import VnfPackageRequest"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"CONF \u003d cfg.CONF"},{"line_number":32,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_c3be59ff","line":29,"range":{"start_line":27,"start_character":0,"end_line":29,"end_character":53},"in_reply_to":"9f560f44_da249d62","updated":"2020-09-14 13:48:52.000000000","message":"It is fixed.","commit_id":"d075e1ed733a9d83f3bbb4c7730fca5fbe978896"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"87e42aebcc79ad9d0c6c98faf59afdc97544f94d","unresolved":false,"context_lines":[{"line_number":33,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"class SyncVnfPackage:"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"    @classmethod"},{"line_number":39,"context_line":"    def create_package(cls, context, vnf_package_info):"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_0b1b081c","line":36,"updated":"2020-09-02 08:22:30.000000000","message":"I think it isn\u0027t the processing content that should be described in __init__.py, so I think it\u0027s better to describe it in another file.","commit_id":"d075e1ed733a9d83f3bbb4c7730fca5fbe978896"},{"author":{"_account_id":32219,"name":"Aldinson C. Esto","email":"esto.aln@nec.com","username":"esto-aln"},"change_message_id":"27f4c2948b5243e39a3c4207345f44371b437110","unresolved":false,"context_lines":[{"line_number":33,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"class SyncVnfPackage:"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"    @classmethod"},{"line_number":39,"context_line":"    def create_package(cls, context, vnf_package_info):"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_c36c999f","line":36,"in_reply_to":"9f560f44_0b1b081c","updated":"2020-09-14 13:48:52.000000000","message":"Modified the file.\nParent has been changed (747678), and it is supported there.","commit_id":"d075e1ed733a9d83f3bbb4c7730fca5fbe978896"},{"author":{"_account_id":32219,"name":"Aldinson C. Esto","email":"esto.aln@nec.com","username":"esto-aln"},"change_message_id":"70dd7c88f0630feab153d156f8b63d64643f63ae","unresolved":false,"context_lines":[{"line_number":33,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"class SyncVnfPackage:"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"    @classmethod"},{"line_number":39,"context_line":"    def create_package(cls, context, vnf_package_info):"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_18894a25","line":36,"in_reply_to":"9f560f44_128ecb31","updated":"2020-09-22 15:52:28.000000000","message":"It is fixed in 747678.","commit_id":"d075e1ed733a9d83f3bbb4c7730fca5fbe978896"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"57ae8e4fd7a52ba6b1cd4d99528ba8a7f23ef0aa","unresolved":false,"context_lines":[{"line_number":33,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"class SyncVnfPackage:"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"    @classmethod"},{"line_number":39,"context_line":"    def create_package(cls, context, vnf_package_info):"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_128ecb31","line":36,"in_reply_to":"9f560f44_c36c999f","updated":"2020-09-15 07:07:15.000000000","message":"Parent (747678) doesn\u0027t seem to be fixed, is it right?","commit_id":"d075e1ed733a9d83f3bbb4c7730fca5fbe978896"},{"author":{"_account_id":17255,"name":"Toshiaki Takahashi","email":"ts-takahashi@nec.com","username":"takahashi_ts"},"change_message_id":"dfdce95b6cb8429992449b55b1b179ac03429e26","unresolved":false,"context_lines":[{"line_number":50,"context_line":""},{"line_number":51,"context_line":"        try:"},{"line_number":52,"context_line":"            download_vnf_packages_resp \u003d \\"},{"line_number":53,"context_line":"                VnfPackageRequest.download_vnf_packages(vnf_package.id)"},{"line_number":54,"context_line":"        except UndefinedExternalSettingException as exc:"},{"line_number":55,"context_line":"            raise webob.exc.HTTPNotFound(explanation\u003dsix.text_type(exc))"},{"line_number":56,"context_line":"        except (FaliedDownloadContentException, Exception) as exc:"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_5affeda9","line":53,"updated":"2020-09-11 11:04:23.000000000","message":"The artifactPath is required as the second argument of download_vnf_packages(), but since it is not passed, it is possible that the pkg information has not been downloaded correctly.\nSince artifactPath exists in additionalArtifacts, it is necessary to get it from there and pass the artifactPath.","commit_id":"d075e1ed733a9d83f3bbb4c7730fca5fbe978896"},{"author":{"_account_id":32219,"name":"Aldinson C. Esto","email":"esto.aln@nec.com","username":"esto-aln"},"change_message_id":"27f4c2948b5243e39a3c4207345f44371b437110","unresolved":false,"context_lines":[{"line_number":50,"context_line":""},{"line_number":51,"context_line":"        try:"},{"line_number":52,"context_line":"            download_vnf_packages_resp \u003d \\"},{"line_number":53,"context_line":"                VnfPackageRequest.download_vnf_packages(vnf_package.id)"},{"line_number":54,"context_line":"        except UndefinedExternalSettingException as exc:"},{"line_number":55,"context_line":"            raise webob.exc.HTTPNotFound(explanation\u003dsix.text_type(exc))"},{"line_number":56,"context_line":"        except (FaliedDownloadContentException, Exception) as exc:"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_632aed2f","line":53,"in_reply_to":"9f560f44_5affeda9","updated":"2020-09-14 13:48:52.000000000","message":"Since artifactPath exists in additionalArtifacts,\nModified it so that it gets from there and passes the artifactPath.","commit_id":"d075e1ed733a9d83f3bbb4c7730fca5fbe978896"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"efd58139bc95a8198348adbe9eee09d38ad41df8","unresolved":false,"context_lines":[{"line_number":58,"context_line":"                explanation\u003dsix.text_type(exc))"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"        try:"},{"line_number":61,"context_line":"            (location, size, checksum, multihash, loc_meta) \u003d cls.__store_csar("},{"line_number":62,"context_line":"                context, vnf_package.id, download_vnf_packages_resp)"},{"line_number":63,"context_line":"            cls.__update_vnf_package(vnf_package, multihash, location)"},{"line_number":64,"context_line":"            vnf_data, flavours \u003d cls.__load_csar(context, vnf_package)"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_bfd320d3","line":61,"updated":"2020-09-03 09:55:34.000000000","message":"Size, checksum, loc_meta are unused, are they unnecessary?","commit_id":"d075e1ed733a9d83f3bbb4c7730fca5fbe978896"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"57ae8e4fd7a52ba6b1cd4d99528ba8a7f23ef0aa","unresolved":false,"context_lines":[{"line_number":58,"context_line":"                explanation\u003dsix.text_type(exc))"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"        try:"},{"line_number":61,"context_line":"            (location, size, checksum, multihash, loc_meta) \u003d cls.__store_csar("},{"line_number":62,"context_line":"                context, vnf_package.id, download_vnf_packages_resp)"},{"line_number":63,"context_line":"            cls.__update_vnf_package(vnf_package, multihash, location)"},{"line_number":64,"context_line":"            vnf_data, flavours \u003d cls.__load_csar(context, vnf_package)"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_b29d5f6e","line":61,"in_reply_to":"9f560f44_2318f51c","updated":"2020-09-15 07:07:15.000000000","message":"Parent (747678) doesn\u0027t seem to be fixed, is it right?","commit_id":"d075e1ed733a9d83f3bbb4c7730fca5fbe978896"},{"author":{"_account_id":32219,"name":"Aldinson C. Esto","email":"esto.aln@nec.com","username":"esto-aln"},"change_message_id":"70dd7c88f0630feab153d156f8b63d64643f63ae","unresolved":false,"context_lines":[{"line_number":58,"context_line":"                explanation\u003dsix.text_type(exc))"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"        try:"},{"line_number":61,"context_line":"            (location, size, checksum, multihash, loc_meta) \u003d cls.__store_csar("},{"line_number":62,"context_line":"                context, vnf_package.id, download_vnf_packages_resp)"},{"line_number":63,"context_line":"            cls.__update_vnf_package(vnf_package, multihash, location)"},{"line_number":64,"context_line":"            vnf_data, flavours \u003d cls.__load_csar(context, vnf_package)"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_f88d7631","line":61,"in_reply_to":"9f560f44_b29d5f6e","updated":"2020-09-22 15:52:28.000000000","message":"It is fixed in 747678.","commit_id":"d075e1ed733a9d83f3bbb4c7730fca5fbe978896"},{"author":{"_account_id":32219,"name":"Aldinson C. Esto","email":"esto.aln@nec.com","username":"esto-aln"},"change_message_id":"27f4c2948b5243e39a3c4207345f44371b437110","unresolved":false,"context_lines":[{"line_number":58,"context_line":"                explanation\u003dsix.text_type(exc))"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"        try:"},{"line_number":61,"context_line":"            (location, size, checksum, multihash, loc_meta) \u003d cls.__store_csar("},{"line_number":62,"context_line":"                context, vnf_package.id, download_vnf_packages_resp)"},{"line_number":63,"context_line":"            cls.__update_vnf_package(vnf_package, multihash, location)"},{"line_number":64,"context_line":"            vnf_data, flavours \u003d cls.__load_csar(context, vnf_package)"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_2318f51c","line":61,"in_reply_to":"9f560f44_bfd320d3","updated":"2020-09-14 13:48:52.000000000","message":"Changed unused variables to \"_\" notation.\n→ Parent has been modified (747678), and it is supported there. \"","commit_id":"d075e1ed733a9d83f3bbb4c7730fca5fbe978896"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"efd58139bc95a8198348adbe9eee09d38ad41df8","unresolved":false,"context_lines":[{"line_number":61,"context_line":"            (location, size, checksum, multihash, loc_meta) \u003d cls.__store_csar("},{"line_number":62,"context_line":"                context, vnf_package.id, download_vnf_packages_resp)"},{"line_number":63,"context_line":"            cls.__update_vnf_package(vnf_package, multihash, location)"},{"line_number":64,"context_line":"            vnf_data, flavours \u003d cls.__load_csar(context, vnf_package)"},{"line_number":65,"context_line":"            vnf_package_vnfd \u003d cls.__create_vnf_package_vnfd("},{"line_number":66,"context_line":"                context, vnf_data, vnf_package.id)"},{"line_number":67,"context_line":"        except Exception as exc:"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_df01b43b","line":64,"updated":"2020-09-03 09:55:34.000000000","message":"flavours is unused, is it unnecessary?","commit_id":"d075e1ed733a9d83f3bbb4c7730fca5fbe978896"},{"author":{"_account_id":31072,"name":"Hiroo Kitamura","email":"hiroo.kitamura@ntt-at.co.jp","username":"Hiroo.Kitamura"},"change_message_id":"57ae8e4fd7a52ba6b1cd4d99528ba8a7f23ef0aa","unresolved":false,"context_lines":[{"line_number":61,"context_line":"            (location, size, checksum, multihash, loc_meta) \u003d cls.__store_csar("},{"line_number":62,"context_line":"                context, vnf_package.id, download_vnf_packages_resp)"},{"line_number":63,"context_line":"            cls.__update_vnf_package(vnf_package, multihash, location)"},{"line_number":64,"context_line":"            vnf_data, flavours \u003d cls.__load_csar(context, vnf_package)"},{"line_number":65,"context_line":"            vnf_package_vnfd \u003d cls.__create_vnf_package_vnfd("},{"line_number":66,"context_line":"                context, vnf_data, vnf_package.id)"},{"line_number":67,"context_line":"        except Exception as exc:"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_f2a7d7bb","line":64,"in_reply_to":"9f560f44_0315f1e5","updated":"2020-09-15 07:07:15.000000000","message":"Parent (747678) doesn\u0027t seem to be fixed, is it right?","commit_id":"d075e1ed733a9d83f3bbb4c7730fca5fbe978896"},{"author":{"_account_id":32219,"name":"Aldinson C. Esto","email":"esto.aln@nec.com","username":"esto-aln"},"change_message_id":"27f4c2948b5243e39a3c4207345f44371b437110","unresolved":false,"context_lines":[{"line_number":61,"context_line":"            (location, size, checksum, multihash, loc_meta) \u003d cls.__store_csar("},{"line_number":62,"context_line":"                context, vnf_package.id, download_vnf_packages_resp)"},{"line_number":63,"context_line":"            cls.__update_vnf_package(vnf_package, multihash, location)"},{"line_number":64,"context_line":"            vnf_data, flavours \u003d cls.__load_csar(context, vnf_package)"},{"line_number":65,"context_line":"            vnf_package_vnfd \u003d cls.__create_vnf_package_vnfd("},{"line_number":66,"context_line":"                context, vnf_data, vnf_package.id)"},{"line_number":67,"context_line":"        except Exception as exc:"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_0315f1e5","line":64,"in_reply_to":"9f560f44_df01b43b","updated":"2020-09-14 13:48:52.000000000","message":"Changed unused variables to \"_\" notation.\n→ Parent has been modified (747678), and it is supported there. \"","commit_id":"d075e1ed733a9d83f3bbb4c7730fca5fbe978896"},{"author":{"_account_id":32219,"name":"Aldinson C. Esto","email":"esto.aln@nec.com","username":"esto-aln"},"change_message_id":"70dd7c88f0630feab153d156f8b63d64643f63ae","unresolved":false,"context_lines":[{"line_number":61,"context_line":"            (location, size, checksum, multihash, loc_meta) \u003d cls.__store_csar("},{"line_number":62,"context_line":"                context, vnf_package.id, download_vnf_packages_resp)"},{"line_number":63,"context_line":"            cls.__update_vnf_package(vnf_package, multihash, location)"},{"line_number":64,"context_line":"            vnf_data, flavours \u003d cls.__load_csar(context, vnf_package)"},{"line_number":65,"context_line":"            vnf_package_vnfd \u003d cls.__create_vnf_package_vnfd("},{"line_number":66,"context_line":"                context, vnf_data, vnf_package.id)"},{"line_number":67,"context_line":"        except Exception as exc:"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_38840e0b","line":64,"in_reply_to":"9f560f44_f2a7d7bb","updated":"2020-09-22 15:52:28.000000000","message":"It is fixed in 747678.","commit_id":"d075e1ed733a9d83f3bbb4c7730fca5fbe978896"}],"tacker/api/vnflcm/v1/controller.py":[{"author":{"_account_id":17255,"name":"Toshiaki Takahashi","email":"ts-takahashi@nec.com","username":"takahashi_ts"},"change_message_id":"dfdce95b6cb8429992449b55b1b179ac03429e26","unresolved":false,"context_lines":[{"line_number":609,"context_line":"                                vnf package info: %s\u003d[%s]\") %"},{"line_number":610,"context_line":"                            (input_id, body_data[input_id]))"},{"line_number":611,"context_line":"                        return self._make_problem_detail("},{"line_number":612,"context_line":"                            msg, 404, title\u003d\u0027Not Found\u0027)"},{"line_number":613,"context_line":""},{"line_number":614,"context_line":"                    vnfd_pkg \u003d SyncVnfPackage.create_package(context,"},{"line_number":615,"context_line":"                        vnf_package_info)"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_5a740d36","line":612,"range":{"start_line":612,"start_character":33,"end_line":612,"end_character":36},"updated":"2020-09-11 11:04:23.000000000","message":"I think \"400 Bad Request\" is better here.","commit_id":"d075e1ed733a9d83f3bbb4c7730fca5fbe978896"},{"author":{"_account_id":31813,"name":"Koichi Edagawa","email":"edagawa.kc@nec.com","username":"k-edagawa"},"change_message_id":"1699e1fbfaf67475d6ac3cb27561f38555033a07","unresolved":false,"context_lines":[{"line_number":609,"context_line":"                                vnf package info: %s\u003d[%s]\") %"},{"line_number":610,"context_line":"                            (input_id, body_data[input_id]))"},{"line_number":611,"context_line":"                        return self._make_problem_detail("},{"line_number":612,"context_line":"                            msg, 404, title\u003d\u0027Not Found\u0027)"},{"line_number":613,"context_line":""},{"line_number":614,"context_line":"                    vnfd_pkg \u003d SyncVnfPackage.create_package(context,"},{"line_number":615,"context_line":"                        vnf_package_info)"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_a0423b67","line":612,"range":{"start_line":612,"start_character":33,"end_line":612,"end_character":36},"in_reply_to":"9f560f44_5a740d36","updated":"2020-09-14 13:28:57.000000000","message":"Fixed.","commit_id":"d075e1ed733a9d83f3bbb4c7730fca5fbe978896"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"b61925a6a2d293d45e9ad4dbef8ec02e11f51935","unresolved":false,"context_lines":[{"line_number":650,"context_line":""},{"line_number":651,"context_line":"                    if not vnfd_pkg:"},{"line_number":652,"context_line":"                        msg \u003d ("},{"line_number":653,"context_line":"                            _(\"Can not find requested to NFVO,\\"},{"line_number":654,"context_line":"                            vnf package vnfd: %s\") %"},{"line_number":655,"context_line":"                            body_data[input_id])"},{"line_number":656,"context_line":"                        return self._make_problem_detail("}],"source_content_type":"text/x-python","patch_set":7,"id":"9f560f44_471fc35a","line":653,"range":{"start_line":653,"start_character":62,"end_line":653,"end_character":63},"updated":"2020-09-16 18:07:03.000000000","message":"Is it better to terminate with double quote, then start with another string line, instead of continuing with back slash?","commit_id":"a6217d8d50c3151b3ab355522b50fef9e18522a4"},{"author":{"_account_id":31813,"name":"Koichi Edagawa","email":"edagawa.kc@nec.com","username":"k-edagawa"},"change_message_id":"b08236d864898a7738122f194ee3febad5efe44f","unresolved":false,"context_lines":[{"line_number":650,"context_line":""},{"line_number":651,"context_line":"                    if not vnfd_pkg:"},{"line_number":652,"context_line":"                        msg \u003d ("},{"line_number":653,"context_line":"                            _(\"Can not find requested to NFVO,\\"},{"line_number":654,"context_line":"                            vnf package vnfd: %s\") %"},{"line_number":655,"context_line":"                            body_data[input_id])"},{"line_number":656,"context_line":"                        return self._make_problem_detail("}],"source_content_type":"text/x-python","patch_set":7,"id":"9f560f44_707b6828","line":653,"range":{"start_line":653,"start_character":62,"end_line":653,"end_character":63},"in_reply_to":"9f560f44_471fc35a","updated":"2020-09-19 11:20:06.000000000","message":"Because it becomes an error in the character limit (78) even if it starts with another string line,\nI want to correspond with the current correction.","commit_id":"a6217d8d50c3151b3ab355522b50fef9e18522a4"}]}