Juniper Openstack

contrail-schema transformer not deleting VM instances once service instance is deleted

  1. Series trunk
  2. Bug #1499125
Bug #1499125 reported by Ranjini Rajendran
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R2.20
Fix Committed
Critical
Sachin Bansal
Juniper Openstack r2.22
Trunk
Fix Committed
Critical
Sachin Bansal
Juniper Openstack r2.21

Bug Description

Steps to reproduce:

1. Create a service instance.
2. Create a service policy using this SI
3. Attach the service policy to appropriate VNs.
4. Delete the SI from Contrail WebUI.

Schema transformer introspect will still show the SI as not deleted.
Restarting the schema transformer also doesnt fix this.

Then when same SI is created again, service chain becomes broken
This caused basic service chain to not work in R2.21 build 99

Tags: config
Nischal Sheth (nsheth)
tags: added: config
information type: Proprietary → Public
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R2.20

Review in progress for https://review.opencontrail.org/14026
Submitter: Sachin Bansal (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/14026
Committed: http://github.org/Juniper/contrail-controller/commit/b5fe2fa3a6a2648e3e7a8d0759cd53dfce9ada93
Submitter: Zuul
Branch: R2.20

commit b5fe2fa3a6a2648e3e7a8d0759cd53dfce9ada93
Author: Sachin Bansal <email address hidden>
Date: Wed Sep 23 19:58:10 2015 -0700

Use vm_name to delete the VM object

Schema transformer was incorrecty using si_name to delete VM object. It should
use vm_name. Because of this, stale VM objects remain in schema transformer
local memory and so, if SI is created again, it thinks the VM exists but no VMIs
exist and hence service chain is not created.

This is partial fix because we still do not break service chain when SI is deleted.

Change-Id: I4458352e3af0feeb1f68c57920b6875991cfac81
Partial-Bug: 1499125

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/14095
Submitter: Sachin Bansal (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/14095
Committed: http://github.org/Juniper/contrail-controller/commit/b8d047ecab31f8f6a7da498aad71f4bc6730bad3
Submitter: Zuul
Branch: master

commit b8d047ecab31f8f6a7da498aad71f4bc6730bad3
Author: Sachin Bansal <email address hidden>
Date: Mon Sep 28 14:29:19 2015 -0700

Destroy service chain if service instance is deleted

Currently, once a service chain is created, we do not check again until the service instance is removed from policy or policy is detached from VN. But it is possible that either the VM, VMI or service instance has been deleted or has some problem. In those cases, we should check the validity of the service chain again and destroy it if everything is no longer valid.

Change-Id: Ic16d4af50057f6e842d1d4e2356d72728681a182
Closes-Bug: 1499125

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R2.20.x

Review in progress for https://review.opencontrail.org/14213
Submitter: Hampapur Ajay (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/14213
Committed: http://github.org/Juniper/contrail-controller/commit/5558248eff5c014d6949365ffe4091331b25760b
Submitter: Zuul
Branch: R2.20.x

commit 5558248eff5c014d6949365ffe4091331b25760b
Author: Sachin Bansal <email address hidden>
Date: Wed Sep 23 19:58:10 2015 -0700

Use vm_name to delete the VM object

Schema transformer was incorrecty using si_name to delete VM object. It should
use vm_name. Because of this, stale VM objects remain in schema transformer
local memory and so, if SI is created again, it thinks the VM exists but no VMIs
exist and hence service chain is not created.

This is partial fix because we still do not break service chain when SI is deleted.

Change-Id: I4458352e3af0feeb1f68c57920b6875991cfac81
Partial-Bug: 1499125
(cherry picked from commit b5fe2fa3a6a2648e3e7a8d0759cd53dfce9ada93)

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.