yahoo-eng-team team mailing list archive

[Dave Chen <wei.d.chen@xxxxxxxxx>]

Public bug reported:

we use 'region' instead of 'region_id' in EP refer for testing.
https://github.com/openstack/keystone/blob/master/keystone/tests/unit/test_catalog.py#L162

But we are trying to get region_id from the reference data, so this field and any tests depend on region_id shall never take effective.
https://github.com/openstack/keystone/blob/master/keystone/catalog/backends/sql.py#L233

** Affects: keystone
     Importance: Undecided
     Assignee: Dave Chen (wei-d-chen)
         Status: In Progress

** Description changed:

- we use 'region' instead of 'region_id' in EP refer, 
+ we use 'region' instead of 'region_id' in EP refer,
  https://github.com/openstack/keystone/blob/master/keystone/tests/unit/test_catalog.py#L162
  
- But we are trying to get region_id from the reference data, so this field as any tests depend on region_id is never take effective.
+ But we are trying to get region_id from the reference data, so this field and any tests depend on region_id shall never take effective.
  https://github.com/openstack/keystone/blob/master/keystone/catalog/backends/sql.py#L233

** Changed in: keystone
     Assignee: (unassigned) => Dave Chen (wei-d-chen)

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to Keystone.
https://bugs.launchpad.net/bugs/1429576

Title:
  region field in 'new_endpoint_ref' is never effective.

Status in OpenStack Identity (Keystone):
  In Progress

Bug description:
  we use 'region' instead of 'region_id' in EP refer for testing.
  https://github.com/openstack/keystone/blob/master/keystone/tests/unit/test_catalog.py#L162

  But we are trying to get region_id from the reference data, so this field and any tests depend on region_id shall never take effective.
  https://github.com/openstack/keystone/blob/master/keystone/catalog/backends/sql.py#L233

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