set per request one client for sync protection status

Bug #1577698 reported by yinwei
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Karbor
Won't Fix
High
Unassigned

Bug Description

During integrating hybrid-cloud with smaug, we found that cinder protection plugin uses a global cinder client (one client per protection plugin) to query protection status.
This will be a bug where there're parallel protection requests with multiple tenants.
We need save cinder client as per protection context, and use it to sync status in background.

Revision history for this message
Saggi Mizrahi (ficoos) wrote :

This is unrelated to hybrid cloud. We do need to support having multiple contexts.

Changed in karbor:
milestone: none → ocata
importance: Undecided → High
status: New → Triaged
zengzeng (xuxu)
description: updated
Revision history for this message
zengchen (chenzeng2) wrote :

I think it is not still a bug, if the new protection service framework( http://review.openstack.org/#/c/348163) is merged. Because it will create a set of new individual protection plugin instances for each request(https://review.openstack.org/#/c/348163/24/karbor/services/protection/flows/create_protection.py #58 line). Then the other openstack service clients created in plugin will not be shared between two different requests.

Yuval Brik (jhamhader)
Changed in karbor:
status: Triaged → Won't Fix
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.