Consistency Group's host entry doesn't contain pool

Bug #1472344 reported by Walt Boring
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
High
Walt Boring

Bug Description

When a driver's create_consistencygroup() is called the group object's host entry doesn't contain pool information.

The volume manager is overwriting the host value with self.host, which doesn't contain the pool. The host value is already part of the db object, so there is no need to overwrite the value.

This is the output of the group_ref in the volume manager.
2015-07-07 10:01:24.826 WARNING cinder.volume.manager [req-69a2870e-9200-4eda-907d-11e2bdc26224 02a09763958544e9a1e911960402cf64 3de539e8ebfc472586e27762e51565b4] VM: Group {'status': u'creating', 'volume_type_id': u'6adb761c-b129-4db2-b
da9-733f5e7a02f5,', '_sa_instance_state': <sqlalchemy.orm.state.InstanceState object at 0x7f6241b1e9d0>, 'user_id': u'02a09763958544e9a1e911960402cf64', 'name': u'walt3', 'availability_zone': u'nova', 'deleted': False, 'created_at': date
time.datetime(2015, 7, 7, 17, 1, 24), 'updated_at': datetime.datetime(2015, 7, 7, 17, 1, 24), 'cgsnapshot_id': None, 'project_id': u'3de539e8ebfc472586e27762e51565b4', 'host': u'alex-devstack@3parfc#orourke-cpg', 'deleted_at': None, 'id'
: u'af8887ef-adf2-407a-9f58-876f5277e378', 'description': None}

This host entry is being saved in the group_ref, which overwrites the existing host value. Notice it's missing the pool.
2015-07-07 10:01:24.827 WARNING cinder.volume.manager [req-69a2870e-9200-4eda-907d-11e2bdc26224 02a09763958544e9a1e911960402cf64 3de539e8ebfc472586e27762e51565b4] VM: self.host alex-devstack@3parfc

This is what the driver sees inside of create_consistencygroup()
2015-07-07 10:01:25.463 WARNING cinder.volume.drivers.san.hp.hp_3par_common [req-69a2870e-9200-4eda-907d-11e2bdc26224 02a09763958544e9a1e911960402cf64 3de539e8ebfc472586e27762e51565b4] GROUP {'status': u'creating', 'volume_type_id': u'6a
db761c-b129-4db2-bda9-733f5e7a02f5,', '_sa_instance_state': <sqlalchemy.orm.state.InstanceState object at 0x7f6241b1e9d0>, 'user_id': u'02a09763958544e9a1e911960402cf64', 'name': u'walt3', 'availability_zone': u'nova', 'deleted': False,
'created_at': datetime.datetime(2015, 7, 7, 17, 1, 24), 'updated_at': datetime.datetime(2015, 7, 7, 17, 1, 24), 'cgsnapshot_id': None, 'project_id': u'3de539e8ebfc472586e27762e51565b4', 'host': 'alex-devstack@3parfc', 'deleted_at': None,
 'id': u'af8887ef-adf2-407a-9f58-876f5277e378', 'description': None}

Changed in cinder:
status: New → Confirmed
Changed in cinder:
status: Confirmed → In Progress
Changed in cinder:
status: In Progress → Fix Committed
Changed in cinder:
importance: Undecided → High
Changed in cinder:
milestone: none → liberty-2
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in cinder:
milestone: liberty-2 → 7.0.0
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.