don't set roles via metadata in backend api

Bug #938103 reported by Jesse Andrews
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Identity (keystone)
Invalid
Medium
Henry Nash

Bug Description

In the testing code we use update_metadata to configure roles:

        self.metadata_foobar = self.identity_api.update_metadata(
            self.user_foo['id'], self.tenant_bar['id'],
            dict(roles=['keystone_admin'], is_admin='1'))

I think that roles should be a separate (first order concept) from metadata

When implementing a backend, you shouldn't have to examine the metadata to determine if you also need to update metadata.

Tags: blueprint
Joseph Heck (heckj)
Changed in keystone:
status: New → Confirmed
importance: Undecided → Medium
Joseph Heck (heckj)
tags: added: blueprint
Revision history for this message
Pablo Fernando Cargnelutti (pablo-fernando-cargnelutti) wrote :

This is not an issue anymore. Am I right?

Changed in keystone:
assignee: nobody → Pablo Fernando Cargnelutti (pablo-fernando-cargnelutti)
Revision history for this message
Dolph Mathews (dolph) wrote :

Correct! This was fixed in https://review.openstack.org/#/c/71159/

Changed in keystone:
assignee: Pablo Fernando Cargnelutti (pablo-fernando-cargnelutti) → Henry Nash (henry-nash)
status: Confirmed → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.