yahoo-eng-team team mailing list archive

[Daniel Korn <dkorn@xxxxxxxxxx>]

Public bug reported:

Currently, the way resources names are generated for the integration tests is by concatenating a random integer of a certain interval to a generic string. 
for example: IMAGE_NAME = 'horizonimage' + str(random.randint(0, 1000))

In view of the fact we need unique resource names for the tests, we need
to reduce the possibility of non-trivial failure rates due to
collisions.

One approach, raised in a discussion here: https://review.openstack.org/#/c/121506/10/openstack_dashboard/test/integration_tests/tests/test_image_create_delete.py
suggests to implement a method that will return random strings, and will be used by all tests.

Other offers:
* IMAGE_NAME = 'horizonimage' + str(uuid.uuid4()) 
* expandung the interval

** Affects: horizon
     Importance: Undecided
         Status: New


** Tags: collisions integration-tests random-resource-names

** Tags added: collisions integration-tests

** Tags added: random-resource-names

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Dashboard (Horizon).
https://bugs.launchpad.net/bugs/1399219

Title:
  Collision possibility in random string creation for resources names

Status in OpenStack Dashboard (Horizon):
  New

Bug description:
  Currently, the way resources names are generated for the integration tests is by concatenating a random integer of a certain interval to a generic string. 
  for example: IMAGE_NAME = 'horizonimage' + str(random.randint(0, 1000))

  In view of the fact we need unique resource names for the tests, we
  need to reduce the possibility of non-trivial failure rates due to
  collisions.

  One approach, raised in a discussion here: https://review.openstack.org/#/c/121506/10/openstack_dashboard/test/integration_tests/tests/test_image_create_delete.py
  suggests to implement a method that will return random strings, and will be used by all tests.

  Other offers:
  * IMAGE_NAME = 'horizonimage' + str(uuid.uuid4()) 
  * expandung the interval

To manage notifications about this bug go to:
https://bugs.launchpad.net/horizon/+bug/1399219/+subscriptions