openstack/deb-sahara
Code Issues Proposed changes

Removing service.engine._log_operation_exception

Browse Source 
Method service.engine._log_operation_exception has not
been used for a long time. It should be removed.

Change-Id: If7931c949a13dfdb5c15010dfdfbb9bdfcd9d15a
Closes-bug: #1418499
This commit is contained in:
Andrey Pavlov 2015-02-05 14:37:39 +03:00
parent 0cf5ac1cb3
commit 35dd8bc578
1 changed files with 0 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
sahara/service/engine.py
View File

@ -25,7 +25,6 @@ from sahara import conductor as c
from sahara import context from sahara import context
from sahara.i18n import _ from sahara.i18n import _
from sahara.i18n import _LI from sahara.i18n import _LI
from sahara.i18n import _LW
from sahara.service import networks from sahara.service import networks
from sahara.utils import cluster_progress_ops as cpo from sahara.utils import cluster_progress_ops as cpo
from sahara.utils import edp from sahara.utils import edp
@ -183,12 +182,3 @@ sed '/^Defaults requiretty*/ s/^/#/' -i /etc/sudoers\n
update.update({"info": info, update.update({"info": info,
"end_time": datetime.datetime.now()}) "end_time": datetime.datetime.now()})
conductor.job_execution_update(ctx, je, update) conductor.job_execution_update(ctx, je, update)
def _log_operation_exception(self, message, cluster, ex):
# we want to log the initial exception even if cluster was deleted
cluster_name = cluster.name if cluster is not None else '_unknown_'
LOG.warn(message, {'cluster': cluster_name, 'reason': ex})
if cluster is None:
LOG.warn(
_LW("Presumably the operation failed because the cluster was "
"deleted by a user during the process."))