Implement ref-relax-for-delete

Bug #1803185 reported by Witaut Bajaryn
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
Trunk
Fix Committed
Undecided
Unassigned

Bug Description

Services like kube-manager use it to be able to delete a NetworkPolicy while having VirtualNetworks that refer to it.

The commit that added ref-relax-for-delete in Python API server: https://github.com/Juniper/contrail/commit/610ce99ad52e9598fa02c1cebf6ad6d098b525a9 (Note: the link does not work for some reason. Clone the repo and look at the commit instead).

Witaut Bajaryn (vozhyk-)
information type: Proprietary → Public
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/47646
Submitter: Witaut Bajaryn (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/47789
Submitter: Witaut Bajaryn (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/47790
Submitter: Witaut Bajaryn (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/47646
Submitter: Witaut Bajaryn (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/47789
Submitter: Witaut Bajaryn (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/47790
Submitter: Witaut Bajaryn (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/47646
Submitter: Witaut Bajaryn (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/47790
Submitter: Witaut Bajaryn (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/47646
Submitter: Witaut Bajaryn (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/47790
Submitter: Witaut Bajaryn (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/47646
Submitter: Witaut Bajaryn (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/47789
Submitter: Witaut Bajaryn (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/47646
Submitter: Witaut Bajaryn (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/47789
Submitter: Witaut Bajaryn (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/47646
Submitter: Witaut Bajaryn (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/47789
Submitter: Witaut Bajaryn (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/47646
Submitter: Witaut Bajaryn (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/47789
Submitter: Witaut Bajaryn (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/47646
Submitter: Witaut Bajaryn (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/47646
Submitter: Pawel Zadrozny (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/47789
Submitter: Pawel Zadrozny (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/47789
Committed: http://github.com/Juniper/contrail/commit/e4ab82f186403bbfa7d0d1e7fdea7031efc18b15
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master

commit e4ab82f186403bbfa7d0d1e7fdea7031efc18b15
Author: Witaut Bajaryn <email address hidden>
Date: Wed Nov 21 20:10:18 2018 +0100

Add client.GRPC

ContrailServiceClient does not implement Service,
because its methods have additional `opts ...grpc.CallOption` arguments.

Make a shim so that it can be used where a Service is expected.

Partial-Bug: 1803185
Change-Id: Ic35e68523de093b5ef173f71dfb3adb1b0961de6

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

Review in progress for https://review.opencontrail.org/47646
Submitter: Witaut Bajaryn (<email address hidden>)

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

Reviewed: https://review.opencontrail.org/47646
Committed: http://github.com/Juniper/contrail/commit/fe1bf290104ef368deaafc5e35127b4db5d61d0f
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master

commit fe1bf290104ef368deaafc5e35127b4db5d61d0f
Author: Witaut Bajaryn <email address hidden>
Date: Thu Nov 8 20:52:53 2018 +0100

Implement ref-relax-for-delete endpoint

Add a `relaxed` boolean column to all reference tables.
When relaxing a reference, set relaxed = true in the ref table.
When deleting a resource, delete all backrefs with relaxed = true.

Add basemodels.KindToSchemaID.

Closes-Bug: 1803185
Change-Id: Ieed6cc45eba28f5ae8a5f4796b033f7038ba9abe

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.