Clean up UUID usage in tests

[Ed Leafe]

Bug #1606647 [0] was created because there were many places in our tests, where the test authors entered values for UUID fields that were not UUID-like, such as 'fake-id'. Since oslo.versionedobjects now enforces UUID validity in any field declared to be a UUIDField, we had to update these tests to use proper UUIDs.

During that process, it was clear that there were many other places where UUIDs were either invalid, or hard-coded to values such as '33333333-aaaa-bbbb-cccc-333333333333', which, while valid, is not a best practice for testing. Instead, tests should use nova.tests.uuidsentinel instead, which will create a named unique UUID for each UUID required for the tests.

As a start, a list of many of these cases is in the reviews for the bug fix patches for #1606647:
https://review.openstack.org/#/c/531479/
https://review.openstack.org/#/c/539254/

As this bug is being filed after the end of Feature Freeze for Queens, and is not critical for Queens, it should be corrected in Rocky at the earliest.

[0] https://launchpad.net/bugs/1606647

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/550238

[Russell Tweed (russell-tweed)]

Going to have to abandon this one; virtually all of the changes from comments in reviews 531479/539254 cause assertion errors in the various tests. May revisit in future but will leave unassigned for now.

[OpenStack Infra (hudson-openstack)]

Change abandoned by Russell Tweed (<email address hidden>) on branch: master
Review: https://review.openstack.org/550238

[Alexey Stupnikov (astupnikov)]

Related change in oslo_utils: https://review.opendev.org/c/openstack/oslo.utils/+/802297

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/nova/+/803226

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/nova/+/803363

[OpenStack Infra (hudson-openstack)]

Change abandoned by "Alexey Stupnikov <email address hidden>" on branch: master
Review: https://review.opendev.org/c/openstack/nova/+/803226

[OpenStack Infra (hudson-openstack)]

Change abandoned by "Alexey Stupnikov <email address hidden>" on branch: master
Review: https://review.opendev.org/c/openstack/nova/+/803363