openstack
/
ironic
Code Issues Proposed changes

[Stable only] remove require_exclusive_lock from detect_vendor 

Presumably it was added by a mistake in the Ussuri backport.
Functions that only read data from the BMC must not require
an exclusive lock, and this bug causes a failure when syncing power
state.

Change-Id: I051da583ef6e5131ce36b70b862e91cfdae2b213
(cherry picked from commit e9b16d0261)
This commit is contained in:
Dmitry Tantsur 2021-09-23 14:26:21 +02:00 committed by Julia Kreger
parent 2806b38a37
commit bf156e679e
1 changed files with 0 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
ironic/drivers/modules/ipmitool.py
View File

@ -1231,7 +1231,6 @@ class IPMIManagement(base.ManagementInterface):
response['persistent'] = 'Options apply to all future boots' in out
return response
@task_manager.require_exclusive_lock
@METRICS.timer('IPMIManagement.detect_vendor')
def detect_vendor(self, task):
"""Detects and returns the hardware vendor.