x/python-tatuclient
Code Issues Proposed changes

sync and touch Commands Extend the base.DeleteCommand

Browse Source 
To allow the sync-all, sync-domain, sync-record, and touch-domain commands
to display errors correctly, the classes for these commands now extend the
base.DeleteCommand instead of the base.Command.

The base.Command has no formatter.

The base.DeleteCommand has the ShowOne formatter and does not show any
messages when the command is successful, but shows errors when there is
a failure.

Change-Id: I92ca95f167fec9fb36880024e6c84284186126df
Closes-Bug: #1357092
This commit is contained in:
rjrjr 2014-08-14 16:52:01 -07:00
parent edbc00aed7
commit e23e51b5cd
2 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
designateclient/cli/sync.py
View File

@ -20,7 +20,7 @@ from designateclient.cli import base
LOG = logging.getLogger(__name__)
class SyncAllCommand(base.Command):
class SyncAllCommand(base.DeleteCommand):
"""Sync Everything"""
def execute(self, parsed_args):
@ -29,7 +29,7 @@ class SyncAllCommand(base.Command):
LOG.info('Synchronization of all domains scheduled')
class SyncDomainCommand(base.Command):
class SyncDomainCommand(base.DeleteCommand):
"""Sync a single Domain"""
def get_parser(self, prog_name):
@ -45,7 +45,7 @@ class SyncDomainCommand(base.Command):
LOG.info('Synchronization of domain scheduled')
class SyncRecordCommand(base.Command):
class SyncRecordCommand(base.DeleteCommand):
"""Sync a single Record"""
def get_parser(self, prog_name):

2
designateclient/cli/touch.py
View File

@ -21,7 +21,7 @@ from designateclient.cli import base
LOG = logging.getLogger(__name__)
class TouchDomainCommand(base.Command):
class TouchDomainCommand(base.DeleteCommand):
"""Touch a single Domain"""
def get_parser(self, prog_name):