← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1429723] [NEW] Column "role_id" of table assignment should be properly reference with table role

 

Public bug reported:

'role_id' should be referenced with 'id' of 'role' table, but this is not specified in current code, see
https://github.com/openstack/keystone/blob/master/keystone/assignment/backends/sql.py#L404

It seems the upgrade script is okay.
https://github.com/openstack/keystone/blob/master/keystone/common/sql/migrate_repo/versions/038_add_assignment_table.py#L39

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

** Description changed:

- 'role_id' should be referenced with 'id' of 'role', but this is not specified in current code, see
+ 'role_id' should be referenced with 'id' of 'role' table, but this is not specified in current code, see
  https://github.com/openstack/keystone/blob/master/keystone/assignment/backends/sql.py#L404
- 
  
  It seems the upgrade script is okay.
  https://github.com/openstack/keystone/blob/master/keystone/common/sql/migrate_repo/versions/038_add_assignment_table.py#L39

** 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/1429723

Title:
  Column "role_id" of table assignment should be properly reference with
  table role

Status in OpenStack Identity (Keystone):
  New

Bug description:
  'role_id' should be referenced with 'id' of 'role' table, but this is not specified in current code, see
  https://github.com/openstack/keystone/blob/master/keystone/assignment/backends/sql.py#L404

  It seems the upgrade script is okay.
  https://github.com/openstack/keystone/blob/master/keystone/common/sql/migrate_repo/versions/038_add_assignment_table.py#L39

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


Follow ups

References