yahoo-eng-team team mailing list archive

[Balazs Gibizer <balazs.gibizer@xxxxxxxxxxxx>]

Public bug reported:

Placement allows setting the parent_rp_uuid of an RP to itself. This
leads to a trivial loop in the RP tree.


Version, current master:
stack@ubuntu:~/nova$ git log --oneline | head -n1
4ea64ca Merge "manage: Remove dead code"

To reproduce in devstack I used a not yet merged osc-placement patch
adding support to api version 1.14.

Steps:

stack@ubuntu:~/osc-placement$ openstack --debug resource provider list --os-placement-api-version 1.14
+--------------------------------------+--------+------------+--------------------------------------+----------------------+
| uuid                                 | name   | generation | root_provider_uuid                   | parent_provider_uuid |
+--------------------------------------+--------+------------+--------------------------------------+----------------------+
| f4d95373-b15f-4dd9-94ed-f7908fe10dd1 | ubuntu |          1 | f4d95373-b15f-4dd9-94ed-f7908fe10dd1 | None                 |
+--------------------------------------+--------+------------+--------------------------------------+----------------------+

stack@ubuntu:~/osc-placement$ openstack --debug resource provider  --os-
placement-api-version 1.14 set  f4d95373-b15f-4dd9-94ed-f7908fe10dd1
--name ubuntu --parent-provider-uuid f4d95373-b15f-4dd9-94ed-
f7908fe10dd1

+----------------------+--------------------------------------+
| Field                | Value                                |
+----------------------+--------------------------------------+
| uuid                 | f4d95373-b15f-4dd9-94ed-f7908fe10dd1 |
| name                 | ubuntu                               |
| generation           | 1                                    |
| root_provider_uuid   | f4d95373-b15f-4dd9-94ed-f7908fe10dd1 |
| parent_provider_uuid | f4d95373-b15f-4dd9-94ed-f7908fe10dd1 |
+----------------------+--------------------------------------+

stack@ubuntu:~/osc-placement$ openstack --debug resource provider list --os-placement-api-version 1.14
+--------------------------------------+--------+------------+--------------------------------------+--------------------------------------+
| uuid                                 | name   | generation | root_provider_uuid                   | parent_provider_uuid                 |
+--------------------------------------+--------+------------+--------------------------------------+--------------------------------------+
| f4d95373-b15f-4dd9-94ed-f7908fe10dd1 | ubuntu |          1 | f4d95373-b15f-4dd9-94ed-f7908fe10dd1 | f4d95373-b15f-4dd9-94ed-f7908fe10dd1 |
+--------------------------------------+--------+------------+--------------------------------------+--------------------------------------+

Full debug output with has been attached.

** Affects: nova
     Importance: Undecided
         Status: New


** Tags: placement

** Tags added: placement

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

Title:
  placement allows RP parent loop in PUT resource_providers/{uuid}

Status in OpenStack Compute (nova):
  New

Bug description:
  Placement allows setting the parent_rp_uuid of an RP to itself. This
  leads to a trivial loop in the RP tree.

  
  Version, current master:
  stack@ubuntu:~/nova$ git log --oneline | head -n1
  4ea64ca Merge "manage: Remove dead code"

  To reproduce in devstack I used a not yet merged osc-placement patch
  adding support to api version 1.14.

  Steps:

  stack@ubuntu:~/osc-placement$ openstack --debug resource provider list --os-placement-api-version 1.14
  +--------------------------------------+--------+------------+--------------------------------------+----------------------+
  | uuid                                 | name   | generation | root_provider_uuid                   | parent_provider_uuid |
  +--------------------------------------+--------+------------+--------------------------------------+----------------------+
  | f4d95373-b15f-4dd9-94ed-f7908fe10dd1 | ubuntu |          1 | f4d95373-b15f-4dd9-94ed-f7908fe10dd1 | None                 |
  +--------------------------------------+--------+------------+--------------------------------------+----------------------+

  stack@ubuntu:~/osc-placement$ openstack --debug resource provider
  --os-placement-api-version 1.14 set  f4d95373-b15f-4dd9-94ed-
  f7908fe10dd1  --name ubuntu --parent-provider-uuid f4d95373-b15f-4dd9
  -94ed-f7908fe10dd1

  +----------------------+--------------------------------------+
  | Field                | Value                                |
  +----------------------+--------------------------------------+
  | uuid                 | f4d95373-b15f-4dd9-94ed-f7908fe10dd1 |
  | name                 | ubuntu                               |
  | generation           | 1                                    |
  | root_provider_uuid   | f4d95373-b15f-4dd9-94ed-f7908fe10dd1 |
  | parent_provider_uuid | f4d95373-b15f-4dd9-94ed-f7908fe10dd1 |
  +----------------------+--------------------------------------+

  stack@ubuntu:~/osc-placement$ openstack --debug resource provider list --os-placement-api-version 1.14
  +--------------------------------------+--------+------------+--------------------------------------+--------------------------------------+
  | uuid                                 | name   | generation | root_provider_uuid                   | parent_provider_uuid                 |
  +--------------------------------------+--------+------------+--------------------------------------+--------------------------------------+
  | f4d95373-b15f-4dd9-94ed-f7908fe10dd1 | ubuntu |          1 | f4d95373-b15f-4dd9-94ed-f7908fe10dd1 | f4d95373-b15f-4dd9-94ed-f7908fe10dd1 |
  +--------------------------------------+--------+------------+--------------------------------------+--------------------------------------+

  Full debug output with has been attached.

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