[openstack-dev] [nova] nova-manage cell_v2 map_instances uses invalid UUID as marker in the db

Stephen Finucane sfinucan at redhat.com
Thu May 10 14:07:20 UTC 2018

On Tue, 2018-05-08 at 11:49 +0200, Bal√°zs Gibizer wrote:
> Hi,
> The oslo UUIDField emits a warning if the string used as a field value 
> does not pass the validation of the uuid.UUID(str(value)) call [3]. All 
> the offending places are fixed in nova except the nova-manage cell_v2 
> map_instances call [1][2]. That call uses markers in the DB that are 
> not valid UUIDs. If we could fix this last offender then we could merge 
> the patch [4] that changes the this warning to an exception in the nova 
> tests to avoid such future rule violations.
> However I'm not sure it is easy to fix. Replacing 
> '00000000-0000-0000-0000-00000000' might work but I don't know what to 
> do with instance_uuid.replace(' ', '-') [2] to make it a valid uuid. 
> Also I think that if there is an unfinished mapping in the deployment 
> and then the marker is changed in the code that leads to 
> inconsistencies.
> I'm open to any suggestions.
> Cheers,
> gibi

This is a somewhat complicated issue. I haven't got any ideas to solve
this (edleafe tried and failed) but I have submitted a patch to explain
why we do this, pending a real resolution.



> [1] 
> https://github.com/openstack/nova/blob/09af976016a83288df22ac6ed1cce1676c2294cc/nova/cmd/manage.py#L1168
> [2] 
> https://github.com/openstack/nova/blob/09af976016a83288df22ac6ed1cce1676c2294cc/nova/cmd/manage.py#L1180
> [3] 
> https://github.com/openstack/oslo.versionedobjects/blob/29e643e4a93333866b33965b68fc8dfb8acf30fa/oslo_versionedobjects/fields.py#L359
> [4] https://review.openstack.org/#/c/540386
> __________________________________________________________________________
> OpenStack Development Mailing List (not for usage questions)
> Unsubscribe: OpenStack-dev-request at lists.openstack.org?subject:unsubscribe
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

More information about the OpenStack-dev mailing list