Exclusive locking should be optional in vendor passthru

Bug #1481665 reported by Dmitry Tantsur
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ironic
Fix Released
Wishlist
Mario Villaplana

Bug Description

Currently all node vendor passthru's always receive a task with exclusive locking. That's not always needed and we should allow driver developers to opt-out from exclusive locking.

Dmitry Tantsur (divius)
tags: added: low-hanging-fruit
Anusha (anusha-iiitm)
Changed in ironic:
assignee: nobody → Anusha (anusha-iiitm)
Anusha (anusha-iiitm)
Changed in ironic:
assignee: Anusha (anusha-iiitm) → nobody
Revision history for this message
Mario Villaplana (mario-villaplana-j) wrote :
Changed in ironic:
assignee: nobody → Mario Villaplana (mario-villaplana-j)
Changed in ironic:
status: Triaged → In Progress
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/226058

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ironic (master)

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

commit ea06393fe853d6b54ab1e2f6d5c1f39e8c61907c
Author: Mario Villaplana <email address hidden>
Date: Thu Nov 19 22:38:58 2015 +0000

    Allow vendor drivers to acquire shared locks

    Previously, all node vendor passthru methods required an exclusive lock
    on the node to validate and start the task. This allows node vendor
    passthru methods to specify require_exclusive_lock=False in their
    passthru decorators to let the conductor acquire a shared lock on a
    node.

    Change-Id: I43cf43bc5c17f44a735e16c3c5cb744cf6911d27
    Closes-Bug: #1481665

Changed in ironic:
status: In Progress → Fix Released
Revision history for this message
Thierry Carrez (ttx) wrote : Fix included in openstack/ironic 6.0.0

This issue was fixed in the openstack/ironic 6.0.0 release.

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.