openstack
/
openstack-virtual-baremetal
Code Issues Proposed changes

Add symlinks for old port-security environments 

For ease of migration to OVB 2.0, let's provide a compatibility
interface from the old names. A deprecation warning is also added
to deploy.py to notify users that they are using outdated
environments and should switch.

port-security.yaml is not symlinked because there's no analog in
OVB 2.0. port-security.yaml is just the default now. I suppose that
could be an empty file, but I'm not sure anyone is using it
anyway so for the moment I'm omitting it. That decision can be
revisited if necessary.
This commit is contained in:
Ben Nemec 2018-12-14 16:51:29 +00:00
parent a52aa22125
commit 3a73a83fb1
3 changed files with 10 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
environments/all-networks-port-security.yaml Symbolic link
View File

@ -0,0 +1 @@
all-networks.yaml

1
environments/all-networks-public-bond-port-security.yaml Symbolic link
View File

@ -0,0 +1 @@
all-networks-public-bond.yaml

8
openstack_virtual_baremetal/deploy.py
View File

@ -190,6 +190,14 @@ def _validate_env(args, env_paths):
raise RuntimeError('baremetal_prefix ends with role name. This '
'will break build-nodes-json. Please choose '
'a different baremetal_prefix or role name.')
for path in env_paths:
if 'port-security.yaml' in path:
print('WARNING: port-security environment file detected. '
'port-security is now the default. The existing '
'port-security environment files are deprecated and may be '
'removed in the future. Please use the environment files '
'without "port-security" in their filename instead.'
)
def _get_heat_client():