diff options
author | Dmitry Tantsur <dtantsur@protonmail.com> | 2021-09-23 14:26:21 +0200 |
---|---|---|
committer | Julia Kreger <juliaashleykreger@gmail.com> | 2021-09-27 13:47:19 +0000 |
commit | bf156e679e88e23e7726d2a728e51c0c5d2f8bb5 (patch) | |
tree | d461a11b82fc80908048f42ad22632e4fb613d43 | |
parent | 2806b38a37ff3dc716514b03c7f04c1e26aaef74 (diff) | |
download | ironic-bf156e679e88e23e7726d2a728e51c0c5d2f8bb5.tar.gz |
[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 e9b16d026171436c66d1651e38c71c9f20acb144)
-rw-r--r-- | ironic/drivers/modules/ipmitool.py | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/ironic/drivers/modules/ipmitool.py b/ironic/drivers/modules/ipmitool.py index 9bead2d79..bbd716b9c 100644 --- a/ironic/drivers/modules/ipmitool.py +++ b/ironic/drivers/modules/ipmitool.py @@ -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. |