Allow nova to delete volume with cinder auth

Bug #1850279 reported by YU CHENGDE
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
kolla-ansible
Fix Released
Medium
YU CHENGDE
Rocky
Fix Committed
Medium
Mark Goddard
Stein
Fix Committed
Medium
Mark Goddard
Train
Fix Released
Medium
YU CHENGDE

Bug Description

Brief Description
-----------------
  To launch an instance and select "delete volume while deleting instance".
  After the instance deleted, the attached volume still remained.
  please refer to attached file as nova_compute.log

Reference
-----------------
  https://bugs.launchpad.net/nova/+bug/1849481

Steps to Reproduce
------------------
  1. modify "reclaim_instance_interval" value from 0 -> 180 in nova/conf/compute.py on docker nova_api and nova compute
  2. docker restart nova_api & nova_compute
  3. login to Openstack from the browser, and launch an instance with "delete volume while deleting instance"
  4. Delete instance
  5. Check attached volume still be there, and status is "running"

Expected Behavior
------------------
  Delete instance and attached volume should be deleted also.

Environment
------------------
  Broswer: Chrome 77.0.3865.120
  OS: macOS Mojave 10.14.6
  kolla-ansible version: stable/stein
  Openstack version: stein
  Openstack deployed system: Centos Linux version 3.10.0-1062.1.2.el7.x86_64

Revision history for this message
YU CHENGDE (chant) wrote :
YU CHENGDE (chant)
summary: - Allow nova to do volume control with cinder auth
+ Allow nova to delete volume with cinder auth
Changed in kolla-ansible:
assignee: nobody → YU CHENGDE (chant)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to kolla-ansible (stable/stein)

Fix proposed to branch: stable/stein
Review: https://review.opendev.org/691920

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on kolla-ansible (stable/stein)

Change abandoned by YU CHENGDE (yu.chengde@99cloud.net) on branch: stable/stein
Review: https://review.opendev.org/691920
Reason: propose to master

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to kolla-ansible (master)

Fix proposed to branch: master
Review: https://review.opendev.org/692036

Changed in kolla-ansible:
status: New → In Progress
Changed in kolla-ansible:
assignee: YU CHENGDE (chant) → Radosław Piliszek (yoctozepto)
Mark Goddard (mgoddard)
Changed in kolla-ansible:
importance: Undecided → Medium
Changed in kolla-ansible:
assignee: Radosław Piliszek (yoctozepto) → Mark Goddard (mgoddard)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to kolla-ansible (master)

Reviewed: https://review.opendev.org/692036
Committed: https://git.openstack.org/cgit/openstack/kolla-ansible/commit/?id=44a302d7d239bfbe1950e36df533e1ccedd4233d
Submitter: Zuul
Branch: master

commit 44a302d7d239bfbe1950e36df533e1ccedd4233d
Author: yuchengde <yu.chengde@99cloud.net>
Date: Wed Oct 30 13:57:34 2019 +0800

    Allow nova to delete volume with cinder auth

    If "reclaim_instance_interval" has been set in nova conf,
    attched volume may not be delete while instacne deleted.
    Adding cinder auth in nova conf can solve the problem.

    Change-Id: I9eb3a74c2f6976043cc35a94915f1fcecb9ef601
    Closes-Bug: 1850279

Changed in kolla-ansible:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to kolla-ansible (stable/stein)

Fix proposed to branch: stable/stein
Review: https://review.opendev.org/692780

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to kolla-ansible (stable/rocky)

Fix proposed to branch: stable/rocky
Review: https://review.opendev.org/692781

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to kolla-ansible (stable/stein)

Reviewed: https://review.opendev.org/692780
Committed: https://git.openstack.org/cgit/openstack/kolla-ansible/commit/?id=a7ccc2db2e86abdbc821db68b26a830bfb3e6355
Submitter: Zuul
Branch: stable/stein

commit a7ccc2db2e86abdbc821db68b26a830bfb3e6355
Author: yuchengde <yu.chengde@99cloud.net>
Date: Wed Oct 30 13:57:34 2019 +0800

    Allow nova to delete volume with cinder auth

    If "reclaim_instance_interval" has been set in nova conf,
    attched volume may not be delete while instacne deleted.
    Adding cinder auth in nova conf can solve the problem.

    Change-Id: I9eb3a74c2f6976043cc35a94915f1fcecb9ef601
    Closes-Bug: 1850279
    (cherry picked from commit 44a302d7d239bfbe1950e36df533e1ccedd4233d)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to kolla-ansible (stable/rocky)

Reviewed: https://review.opendev.org/692781
Committed: https://git.openstack.org/cgit/openstack/kolla-ansible/commit/?id=756cd45f00b72e78a1a6dfbf48b5295714cfb883
Submitter: Zuul
Branch: stable/rocky

commit 756cd45f00b72e78a1a6dfbf48b5295714cfb883
Author: yuchengde <yu.chengde@99cloud.net>
Date: Wed Oct 30 13:57:34 2019 +0800

    Allow nova to delete volume with cinder auth

    If "reclaim_instance_interval" has been set in nova conf,
    attched volume may not be delete while instacne deleted.
    Adding cinder auth in nova conf can solve the problem.

    Change-Id: I9eb3a74c2f6976043cc35a94915f1fcecb9ef601
    Closes-Bug: 1850279
    (cherry picked from commit 44a302d7d239bfbe1950e36df533e1ccedd4233d)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/kolla-ansible 9.0.0.0rc1

This issue was fixed in the openstack/kolla-ansible 9.0.0.0rc1 release candidate.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/kolla-ansible 7.2.0

This issue was fixed in the openstack/kolla-ansible 7.2.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/kolla-ansible 8.1.0

This issue was fixed in the openstack/kolla-ansible 8.1.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to kolla-ansible (master)

Related fix proposed to branch: master
Review: https://review.opendev.org/746039

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to kolla-ansible (master)

Reviewed: https://review.opendev.org/746039
Committed: https://git.openstack.org/cgit/openstack/kolla-ansible/commit/?id=de16013bd630019c4febdb618a3e0c5bf93f7e6f
Submitter: Zuul
Branch: master

commit de16013bd630019c4febdb618a3e0c5bf93f7e6f
Author: Jegor van Opdorp <email address hidden>
Date: Thu Aug 13 11:45:26 2020 +0200

    Add cinder auth config to nova-cell nova.conf.j2

    Fixes an issue during deleting evacuated instances with encrypted block
    devices.

    Change-Id: I9b9b689ef7e1e41b597e2c5f6b96f3ed011193c5
    Closes-Bug: 1891462
    Related-Bug: 1850279

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to kolla-ansible (stable/ussuri)

Related fix proposed to branch: stable/ussuri
Review: https://review.opendev.org/747438

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to kolla-ansible (stable/train)

Related fix proposed to branch: stable/train
Review: https://review.opendev.org/747439

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to kolla-ansible (stable/train)

Reviewed: https://review.opendev.org/747439
Committed: https://git.openstack.org/cgit/openstack/kolla-ansible/commit/?id=ad42c70f97bf29779b06f565bfdac12f8bc6ec61
Submitter: Zuul
Branch: stable/train

commit ad42c70f97bf29779b06f565bfdac12f8bc6ec61
Author: Jegor van Opdorp <email address hidden>
Date: Thu Aug 13 11:45:26 2020 +0200

    Add cinder auth config to nova-cell nova.conf.j2

    Fixes an issue during deleting evacuated instances with encrypted block
    devices.

    Change-Id: I9b9b689ef7e1e41b597e2c5f6b96f3ed011193c5
    Closes-Bug: 1891462
    Related-Bug: 1850279
    (cherry picked from commit de16013bd630019c4febdb618a3e0c5bf93f7e6f)

tags: added: in-stable-train
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to kolla-ansible (stable/ussuri)

Reviewed: https://review.opendev.org/747438
Committed: https://git.openstack.org/cgit/openstack/kolla-ansible/commit/?id=652d3eeb5fd2aad1a7951405df2b63847167140b
Submitter: Zuul
Branch: stable/ussuri

commit 652d3eeb5fd2aad1a7951405df2b63847167140b
Author: Jegor van Opdorp <email address hidden>
Date: Thu Aug 13 11:45:26 2020 +0200

    Add cinder auth config to nova-cell nova.conf.j2

    Fixes an issue during deleting evacuated instances with encrypted block
    devices.

    Change-Id: I9b9b689ef7e1e41b597e2c5f6b96f3ed011193c5
    Closes-Bug: 1891462
    Related-Bug: 1850279
    (cherry picked from commit de16013bd630019c4febdb618a3e0c5bf93f7e6f)

tags: added: in-stable-ussuri
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.