openstack/heat
Code Issues Proposed changes

Fix stack deleting problem in magnum stacks

Browse Source 
In magnumclient/exceptions.py exceptions are imported
from magnumclient.common.apiclient so exception class
is not the same as expected in heat/engine/clients/os/magnum.py

Closes-Bug: #1570781
Change-Id: Ie60a735470b9adc38675b613950e6480e28136ab
This commit is contained in:
Dmitriy Uvarenkov 2016-04-15 13:09:15 +03:00
parent c5cff92978
commit 7f07e9788e
2 changed files with 12 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
heat/engine/clients/os/magnum.py
View File

@ -11,13 +11,18 @@
# License for the specific language governing permissions and limitations
# under the License.
from magnumclient.openstack.common.apiclient import exceptions as mc_exc
from magnumclient.v1 import client as magnum_client
from heat.common import exception
from heat.engine.clients import client_plugin
from heat.engine import constraints
try:
from magnumclient.common.apiclient import exceptions as mc_exc
except ImportError:
# NOTE (duvarenkov): For magnumclient versions before 2.0.0.
from magnumclient.openstack.common.apiclient import exceptions as mc_exc
CLIENT_NAME = 'magnum'

8
heat/tests/openstack/magnum/test_bay.py
View File

@ -16,8 +16,6 @@ import mock
from oslo_config import cfg
import six
from magnumclient.openstack.common.apiclient import exceptions as mc_exc
from heat.common import exception
from heat.common import template_format
from heat.engine.clients.os import magnum as mc
@ -28,6 +26,12 @@ from heat.engine import template
from heat.tests import common
from heat.tests import utils
try:
from magnumclient.common.apiclient import exceptions as mc_exc
except ImportError:
# NOTE (duvarenkov): For magnumclient versions before 2.0.0.
from magnumclient.openstack.common.apiclient import exceptions as mc_exc
magnum_template = '''
heat_template_version: 2015-04-30