VolumeRetypeWithoutMigration fails when cinder-volume is clustered

[Giulio Fidente]

When cinder-volume is clustered, cinder-volume tasks *can* be executed by any node within the same cluser, not only by the node which initially created the volume (setting the os-vol-host-attr:host value accordingly).

The VolumeRetypeWithoutMigration test is assuming the :host value won't change, while it could in case instances were clustered and, in that scenario, cluser_name instead won't change.

[OpenStack Infra (hudson-openstack)]

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

[Gorka Eguileor (gorka)]

The solution should be simple, just check the cluster_name field and see if it's in a cluster, and if it is ignore the host field and just use the cluster_name, but when I added the cluster functionality for A/A y forgot to add the field to the volume information.

A solution would be to check if Cinder is running microversion 3.7 or higher, and if it is, after we create the volume we ask Cinder for the service information on that volume's host and see if it is in a cluster.

If it is in a cluster we have to compare the cluster for the service running the host value from before and the one from after the retype instead of the host field itself.

[Martin Kopec (mkopec)]

What is the status here? The related review is -w by its owner. Is anyone working on this?

[Giulio Fidente (gfidente)]

Martin, I think the necessary "cluster_name" property of a volume wasn't exposed in previous versions of the API, but it might be now and we might able to reprise the existing submission then.

Gorka, can you help us understand if https://review.opendev.org/#/c/708332/1/tempest/api/volume/admin/test_volume_retype.py is supposed to work with recent stable branches?

[Martin Kopec (mkopec)]

how is it looking now? anything I can help with?

[Giulio Fidente (gfidente)]

hi Martin, thanks for jumping in... help would be much appreciated here indeed.

I think the best way forward is what Gorka suggested here https://bugs.launchpad.net/tempest/+bug/1863729/comments/2 but I couldn't find myself the time to work on it

Gorka is the cluster_name property exposed in recent microversions?

[Lukas Piwowarski (lukas-piwowarski)]

Hi, it looks like nobody is working on this. I'm moving this back to confirmed as the test is still not fixed [1].

[1] https://github.com/openstack/tempest/blame/master/tempest/api/volume/admin/test_volume_retype.py