API and Conductor raise different exceptions when rejecting a request due to maintenance mode

Bug #1286410 reported by aeva black
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ironic
Fix Released
Medium
Rohan

Bug Description

The API service and the Conductor service both check if a node is in maintenance mode before starting to provision it, but they raise different exceptions.

API raises NodeInMaintenance

Conductor raises InstanceDeployFailure

aeva black (tenbrae)
Changed in ironic:
status: New → Triaged
importance: Undecided → Medium
Rohan (kanaderohan)
Changed in ironic:
assignee: nobody → Rohan (kanaderohan)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ironic (master)

Fix proposed to branch: master
Review: https://review.openstack.org/77426

Changed in ironic:
status: Triaged → In Progress
aeva black (tenbrae)
Changed in ironic:
milestone: none → icehouse-rc1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ironic (master)

Reviewed: https://review.openstack.org/77426
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=dc92f2f5a38e4f7af0b0e3a74cfce9091bc51c82
Submitter: Jenkins
Branch: master

commit dc92f2f5a38e4f7af0b0e3a74cfce9091bc51c82
Author: Rohan Kanade <email address hidden>
Date: Sun Mar 2 13:58:01 2014 +0100

    Fix wrong exception raised by conductor for node

    * Conductor should raise NodeInMaintenance exception while provisioning
      node which is under maintainence, same as the exception raised by the
      REST API

    Change-Id: Ieab302fb96c7b1c11eaefc79621f0c635a16fefc
    Closes-Bug: #1286410

Changed in ironic:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in ironic:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in ironic:
milestone: icehouse-rc1 → 2014.1
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.