openstack/neutron-vpnaas
Code Issues Proposed changes

Fix usage of method ensure_dir

Browse Source 
This patch is for stopping using ensure_dir in device_driver, as
the neutron.common.utils.ensure_dir is deprecated for removal, also
ensure_tree(path, 0o755) from oslo_utils.fileutils is suggested to be
used to create a directory.

Co-Authored-By: Cao Xuan Hoang <hoangcx@vn.fujitsu.com>
Change-Id: Ic9b4b86e65052c06c33c8628bb0c6700cdfd875b
This commit is contained in:
rajat29 2017-10-13 14:54:52 +05:30 committed by Cao Xuan Hoang
parent e3da5c10c4
commit 5fc1079da3
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
neutron_vpnaas/services/vpn/device_drivers/ipsec.py
View File

@ -26,7 +26,6 @@ import netaddr
from neutron.agent.linux import ip_lib
from neutron.agent.linux import utils as agent_utils
from neutron.common import rpc as n_rpc
from neutron.common import utils as n_utils
from neutron.plugins.common import utils as plugin_utils
from neutron_lib.api import validators
from neutron_lib import constants
@ -38,6 +37,7 @@ from oslo_log import helpers as log_helpers
from oslo_log import log as logging
import oslo_messaging
from oslo_service import loopingcall
from oslo_utils import fileutils
import six
from neutron_vpnaas._i18n import _
@ -238,10 +238,10 @@ class BaseSwanProcess(object):
def ensure_config_dir(self, vpnservice):
"""Create config directory if it does not exist."""
n_utils.ensure_dir(self.config_dir)
fileutils.ensure_tree(self.config_dir, 0o755)
for subdir in self.CONFIG_DIRS:
dir_path = os.path.join(self.config_dir, subdir)
n_utils.ensure_dir(dir_path)
fileutils.ensure_tree(dir_path, 0o755)
def _gen_config_content(self, template_file, vpnservice):
template = _get_template(template_file)