From e23e51b5cd11f71ce3e0b41f6360b01856fd31eb Mon Sep 17 00:00:00 2001 From: rjrjr Date: Thu, 14 Aug 2014 16:52:01 -0700 Subject: [PATCH] sync and touch Commands Extend the base.DeleteCommand 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 --- designateclient/cli/sync.py | 6 +++--- designateclient/cli/touch.py | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/designateclient/cli/sync.py b/designateclient/cli/sync.py index 96d8a9df..18e2852c 100644 --- a/designateclient/cli/sync.py +++ b/designateclient/cli/sync.py @@ -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): diff --git a/designateclient/cli/touch.py b/designateclient/cli/touch.py index 3f25f032..a9713f02 100644 --- a/designateclient/cli/touch.py +++ b/designateclient/cli/touch.py @@ -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):