cinder/detach-notification-31ae15dafdef36c1.yaml at e9c18c7f94357de32e08f9745de20fe0684a013b - cinder - OpenDev: Free Software Needs Free Tools

openstack/cinder

Gorka Eguileor 68d4944577 Fix detach notification

Our current `attachment_delete` methods in the volume API and the
manager are using DB methods directly, which makes the OVOs present in
those methods get out of sync with the latest data, which leads to
notifications having the wrong data when we send them on volume detach.

This patch replaces DB method calls with OVO calls and moves the
notification call to the end of the method, where we have the final
status on the volume.

It also adds the missing detach.start notification when deleting an
attachment in the reserved state.

Closes-Bug: #1916980
Closes-Bug: #1935011
Change-Id: Ie48cf55deacd08e7716201dac00ede8d57e6632f

2021-09-15 16:55:50 +02:00

10 lines

324 B

YAML

Raw Blame History

 ---
 fixes:
   - |
     `Bug #1916980 <https://bugs.launchpad.net/cinder/+bug/1916980>`_: Fixed
     stale volume notification information on volume detach.
   - |
     `Bug #1935011 <https://bugs.launchpad.net/cinder/+bug/1935011>`_: Fixed
     missing detach.start notification when deleting an attachment in reserved
     state.