openstack
/
designate
Code Issues Proposed changes

Merge "Validate NS records on a pool during an update"

This commit is contained in:
Jenkins 2017-02-01 11:53:32 +00:00 committed by Gerrit Code Review
parent ca615479a3 c135dc7e95
commit 2c0ecad121
1 changed files with 16 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
designate/manage/pool.py
View File

@ -142,6 +142,16 @@ class PoolCommands(base.Commands):
pool = DesignateAdapter.parse('YAML', xpool, pool)
# TODO(graham): We should be doing a full validation, but right
# now there is quirks validating through nested objects.
for ns_record in pool.ns_records:
try:
ns_record.validate()
except exceptions.InvalidObject as e:
LOG.error(e.errors.to_list()[0]['message'])
sys.exit(1)
if dry_run:
output_msg.append("Update Pool: %s" % pool)
else:
@ -149,7 +159,12 @@ class PoolCommands(base.Commands):
except exceptions.PoolNotFound:
pool = DesignateAdapter.parse('YAML', xpool, objects.Pool())
# pool = objects.Pool.from_dict(xpool)
for ns_record in pool.ns_records:
try:
ns_record.validate()
except exceptions.InvalidObject as e:
LOG.error(e.errors.to_list()[0]['message'])
sys.exit(1)
if dry_run:
output_msg.append("Create Pool: %s" % pool)
else: