)]}'
{"devstack/lib/tacker":[{"author":{"_account_id":26588,"name":"Hiroyuki Jo","email":"joxyuki@gmail.com","username":"joxyuki"},"change_message_id":"62da717ee9a95d25656a7787a6bc2ae289abd420","unresolved":false,"context_lines":[{"line_number":179,"context_line":""},{"line_number":180,"context_line":"function create_vnf_packages_csar_dirs {"},{"line_number":181,"context_line":"    # Create vnf package csar extract path dir."},{"line_number":182,"context_line":"    sudo install -d -o $STACK_USER ${VNF_PACKAGE_CSAR_PATH}"},{"line_number":183,"context_line":""},{"line_number":184,"context_line":"    # Create dir for storing csar files in the glance store."},{"line_number":185,"context_line":"    sudo install -d -o $STACK_USER ${FILESYSTEM_STORE_DATA_DIR}"}],"source_content_type":"application/x-shellscript","patch_set":8,"id":"5faad753_28f12cce","line":182,"range":{"start_line":182,"start_character":3,"end_line":182,"end_character":59},"updated":"2019-09-12 01:43:16.000000000","message":"In my VM (Ubuntu 18.04), this command results $DATA_DIR/tacker directory owned by root. csar_files, vnfpackage directory is owned by $STACK_USER. Therefore, Line412 is failed in `permission denied`.\nI think we need to make one by one (tacker and tacker/*).","commit_id":"a4b67f0a7d3d9e07b6737a65fae03e086a0159a9"},{"author":{"_account_id":26222,"name":"Cong Phuoc Hoang","email":"hoangphuocbk2.07@gmail.com","username":"hoangphuoc"},"change_message_id":"c31c33c0dc1d9861d6d403a26eff48966ea7a436","unresolved":false,"context_lines":[{"line_number":204,"context_line":""},{"line_number":205,"context_line":"    iniset $TACKER_CONF vnf_package vnf_package_csar_path \"$VNF_PACKAGE_CSAR_PATH\""},{"line_number":206,"context_line":"    iniset $TACKER_CONF glance_store filesystem_store_datadir \"$FILESYSTEM_STORE_DATA_DIR\""},{"line_number":207,"context_line":"    iniset $TACKER_CONF glance_store default_backend \"file\""},{"line_number":208,"context_line":"    create_vnf_packages_csar_dirs"},{"line_number":209,"context_line":""},{"line_number":210,"context_line":"    # Format logging"}],"source_content_type":"application/x-shellscript","patch_set":8,"id":"5faad753_0d6a79b2","line":207,"range":{"start_line":207,"start_character":54,"end_line":207,"end_character":58},"updated":"2019-09-10 02:33:12.000000000","message":"I think it should be \"GLANCE_DEFAULT_BACKEND\"","commit_id":"a4b67f0a7d3d9e07b6737a65fae03e086a0159a9"},{"author":{"_account_id":26588,"name":"Hiroyuki Jo","email":"joxyuki@gmail.com","username":"joxyuki"},"change_message_id":"42a4f5329d534289d03ee313151ea2cd8519fef8","unresolved":false,"context_lines":[{"line_number":412,"context_line":"    # get_or_add_user_project_role advsvc $DEFAULT_VIM_USER $DEFAULT_VIM_PROJECT_NAME"},{"line_number":413,"context_line":""},{"line_number":414,"context_line":"    echo \"Set register default VIM sh which is called by ansible role...\""},{"line_number":415,"context_line":"    _create_tacker_data_dirs"},{"line_number":416,"context_line":"    cp $TACKER_DIR/devstack/vim_config.yaml $TACKER_DATA_DIR"},{"line_number":417,"context_line":"    VIM_CONFIG_FILE\u003d\"$TACKER_DATA_DIR/vim_config.yaml\""},{"line_number":418,"context_line":"    sed -e \"s|^auth_url:.*$|auth_url: \\\u0027${KEYSTONE_SERVICE_URI}\\\u0027|\" -i $VIM_CONFIG_FILE"}],"source_content_type":"application/x-shellscript","patch_set":9,"id":"5faad753_af5e620f","line":415,"range":{"start_line":415,"start_character":3,"end_line":415,"end_character":28},"updated":"2019-09-13 07:21:55.000000000","message":"$TACKER_DATA_DIR/vnfpackagesand $TACKER_DATA_DIR/csar_files need to be created before starting tacker.conductor at Line153. So we should remain _create_tacker_data_dirs in configure_tacker like patchset 8.","commit_id":"dc9aa8e8b97d4ae38e3f4c9e407821350a2f1841"},{"author":{"_account_id":1011,"name":"Tushar Patil","email":"tushar.vitthal.patil@gmail.com","username":"tpatil"},"change_message_id":"6117035528cf648a87fb0510fea5729f54a0d03f","unresolved":false,"context_lines":[{"line_number":412,"context_line":"    # get_or_add_user_project_role advsvc $DEFAULT_VIM_USER $DEFAULT_VIM_PROJECT_NAME"},{"line_number":413,"context_line":""},{"line_number":414,"context_line":"    echo \"Set register default VIM sh which is called by ansible role...\""},{"line_number":415,"context_line":"    _create_tacker_data_dirs"},{"line_number":416,"context_line":"    cp $TACKER_DIR/devstack/vim_config.yaml $TACKER_DATA_DIR"},{"line_number":417,"context_line":"    VIM_CONFIG_FILE\u003d\"$TACKER_DATA_DIR/vim_config.yaml\""},{"line_number":418,"context_line":"    sed -e \"s|^auth_url:.*$|auth_url: \\\u0027${KEYSTONE_SERVICE_URI}\\\u0027|\" -i $VIM_CONFIG_FILE"}],"source_content_type":"application/x-shellscript","patch_set":9,"id":"5faad753_6f694a1f","line":415,"range":{"start_line":415,"start_character":3,"end_line":415,"end_character":28},"in_reply_to":"5faad753_af5e620f","updated":"2019-09-13 07:58:25.000000000","message":"I think it\u0027s better to create the required directories in post-config phase [1] which occurs before the services are started. I will push a new PS soon. Also, I will add code to remove the directories during stack clean phase.\n\n[1] : https://docs.openstack.org/devstack/latest/configuration.html","commit_id":"dc9aa8e8b97d4ae38e3f4c9e407821350a2f1841"}]}
