Multi-backend domain code/tests could use a bit of tidy up

Bug #1340815 reported by Henry Nash
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Identity (keystone)
Fix Released
Low
Unassigned

Bug Description

The multi-domain backend code has a number of tidy-up items that were deferred from the review:

- Re-factoring _set_domain_id_and_mapping() in identity/core.py
- Potential relaxation of the constraint that user/group membership cannot cross a backend boundary
- Corner case testing for exceptions
- Potentially add anything multi-backend test that is in-between the simple and complex tests that have already been defined (e.g. add one that as SQL as the default identity backend, with one LDAP specific domain)
- Potentially add a test that puts the SQL driver as a specific backend driver for a domain (since we only support one SQL driver, this is an odd configuration, but still probably worth while)

tags: added: test-improvement
Dolph Mathews (dolph)
Changed in keystone:
importance: Undecided → Low
status: New → Triaged
Revision history for this message
Henry Nash (henry-nash) wrote :

The last item listed:

- Potentially add a test that puts the SQL driver as a specific backend driver for a domain (since we only support one SQL driver, this is an odd configuration, but still probably worth while)

has been fixed as part of: https://review.openstack.org/#/c/121246/

Revision history for this message
Steve Martinelli (stevemar) wrote :

wondering if this is still an issue?

Revision history for this message
Steve Martinelli (stevemar) wrote :

AFAICT this has been fixed

Changed in keystone:
status: Triaged → Fix Released
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.