Use trustee_domain, trustee_domain_admin name instead of trustee_domain_admin_id and trustee_domain_id for trust configuration

Bug #1581372 reported by Michal Adamczyk
18
This bug affects 3 people
Affects Status Importance Assigned to Milestone
Magnum
Fix Released
High
yuanying
puppet-magnum
Fix Released
High
chandan kumar

Bug Description

Dear,

Could you please change using of trustee_domain_id and trustee_domain_admin_id under the config file [trust] section to trustee_domain_admin and trustee_domain name?

It's very problematic to get whole process automated where you need to get ID which is dynamic...

The original discussion here:

http://lists.openstack.org/pipermail/openstack-dev/2016-May/094812.html

Kind regards,
Michal.

hongbin (hongbin034)
Changed in magnum:
status: New → Triaged
importance: Undecided → High
Changed in puppet-magnum:
importance: Undecided → High
status: New → Confirmed
Changed in magnum:
assignee: nobody → yuanying (ootsuka)
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to magnum (master)

Reviewed: https://review.openstack.org/322691
Committed: https://git.openstack.org/cgit/openstack/magnum/commit/?id=1d748491938c377f43fc8baac666be37fa2eaad7
Submitter: Jenkins
Branch: master

commit 1d748491938c377f43fc8baac666be37fa2eaad7
Author: OTSUKA, Yuanying <email address hidden>
Date: Mon May 30 17:18:34 2016 +0900

    Support trustee_domain_name in config file

    To setup magnum easier, name based configuration is important.
    This patch makes config file support trustee_domain_name and
    trustee_domain_admin_name in trust section.
    If name and id of trustee domain are provided by user, both
    values are passed through into keystone.

    Closes-Bug: #1581372
    Change-Id: Ia691aca7c29a471f6ba36a1a371ec1edf830b365

Changed in magnum:
status: In Progress → Fix Released
Changed in puppet-magnum:
assignee: nobody → chandan kumar (chkumar246)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/magnum 3.0.0

This issue was fixed in the openstack/magnum 3.0.0 release.

Revision history for this message
Nate Potter (ntpttr) wrote :

Hi Chandan, are you still working on this for the puppet-magnum module? If not I can take it over.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to puppet-magnum (master)

Fix proposed to branch: master
Review: https://review.openstack.org/382743

Changed in puppet-magnum:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on puppet-magnum (master)

Change abandoned by Chandan Kumar (<email address hidden>) on branch: master
Review: https://review.openstack.org/382743

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to puppet-magnum (master)

Reviewed: https://review.openstack.org/382682
Committed: https://git.openstack.org/cgit/openstack/puppet-magnum/commit/?id=0087b05e09b7e5b33dfca4f25fd0ba0b91b7e31e
Submitter: Jenkins
Branch: master

commit 0087b05e09b7e5b33dfca4f25fd0ba0b91b7e31e
Author: Nate Potter <email address hidden>
Date: Wed Oct 5 15:51:28 2016 -0700

    Add trustee_domain names to config

    This commit adds the trustee_domain_name and trustee_domain_admin_name
    config options to the keystone domain configuration.

    Change-Id: I5a3061a3559b7c8c30997450aae826f381d5eb97
    Closes-bug: #1581372

Changed in puppet-magnum:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/puppet-magnum 10.0.0

This issue was fixed in the openstack/puppet-magnum 10.0.0 release.

Revision history for this message
Marcus Furlong (furlongm) wrote :

Could this be backported to stable/newton? It seems to apply cleanly and works.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to puppet-magnum (stable/newton)

Fix proposed to branch: stable/newton
Review: https://review.openstack.org/483499

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to puppet-magnum (stable/newton)

Reviewed: https://review.openstack.org/483499
Committed: https://git.openstack.org/cgit/openstack/puppet-magnum/commit/?id=87772bd0f1fd70a7fb422b8014b6dd74c9b44900
Submitter: Jenkins
Branch: stable/newton

commit 87772bd0f1fd70a7fb422b8014b6dd74c9b44900
Author: Nate Potter <email address hidden>
Date: Wed Oct 5 15:51:28 2016 -0700

    Add trustee_domain names to config

    This commit adds the trustee_domain_name and trustee_domain_admin_name
    config options to the keystone domain configuration.

    Change-Id: I5a3061a3559b7c8c30997450aae826f381d5eb97
    Depends-On: Ia40914e7669c1e7603b71ef6a6182cca2f552fd2
    Closes-bug: #1581372
    (cherry picked from commit 0087b05e09b7e5b33dfca4f25fd0ba0b91b7e31e)

tags: added: in-stable-newton
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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