we should be able to construct auth_url from auth_host, auth_protocol, auth_port and version fields

Bug #1727005 reported by Senthilnathan Murugappan
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
R3.2
Fix Committed
High
Ignatious Johnson Christopher
R4.0
Fix Committed
High
Ignatious Johnson Christopher
R4.1
Fix Committed
High
Ignatious Johnson Christopher
Trunk
Fix Committed
High
Ignatious Johnson Christopher

Bug Description

Before the latest R3.2 changes for KeystoneV3 we were able to construct auth_url from auth_host and other keystone params.
Now if the auth_url is not specified we default to https://127.0.0.1:35357/v3 as auth_url, believe it would be better to construct the auth_url from auth_host etal if not specified which will also be backward compatible with pre-R3.2.6 releases.

Tags: config
Revision history for this message
Ignatious Johnson Christopher (ijohnson-x) wrote :

We are constructing auth_url from the auth_* params provisioned. https://github.com/Juniper/contrail-controller/blame/R3.2/src/config/api-server/vnc_auth_keystone.py#L158

Prior to the fix for bug https://bugs.launchpad.net/juniperopenstack/+bug/1716430, we defaulted to v2 always.

Fix will be,

1. If auth_url specified and v3 in auth_url do v3 auth
2. If auth_url specified and v2.0 in auth_url do v2 auth
3. If auth_url specified and v3/v2.0 not in auth_url do v3 auth
4. If auth_url is not specified default to v2 auth (backward compatible)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/36802
Submitter: Ignatious Johnson Christopher (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R4.1

Review in progress for https://review.opencontrail.org/36803
Submitter: Ignatious Johnson Christopher (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R4.0

Review in progress for https://review.opencontrail.org/36804
Submitter: Ignatious Johnson Christopher (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R3.2

Review in progress for https://review.opencontrail.org/36805
Submitter: Ignatious Johnson Christopher (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/36802
Submitter: Ignatious Johnson Christopher (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R4.1

Review in progress for https://review.opencontrail.org/36803
Submitter: Ignatious Johnson Christopher (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R4.0

Review in progress for https://review.opencontrail.org/36804
Submitter: Ignatious Johnson Christopher (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] R3.2

Review in progress for https://review.opencontrail.org/36805
Submitter: Ignatious Johnson Christopher (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/36803
Committed: http://github.com/Juniper/contrail-controller/commit/e1cbacd9173795551a3aea6a27d7549a4b5d1264
Submitter: Zuul (<email address hidden>)
Branch: R4.1

commit e1cbacd9173795551a3aea6a27d7549a4b5d1264
Author: Ignatious Johnson Christopher <email address hidden>
Date: Tue Oct 24 22:54:53 2017 -0700

Default to keystone v2 auth if auth_url is not

specified in contrail keystone auth config file,
to make it backward compatible to pre 3.2.6 builds.

Change-Id: I49886e5403ee051cbe5200d3da0dc4184b507ffc
Closes-Bug: 1727005

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/36805
Committed: http://github.com/Juniper/contrail-controller/commit/88d4721fa51a49985a47bc3f13dfd1f20d37f8f6
Submitter: Zuul (<email address hidden>)
Branch: R3.2

commit 88d4721fa51a49985a47bc3f13dfd1f20d37f8f6
Author: Ignatious Johnson Christopher <email address hidden>
Date: Tue Oct 24 22:54:53 2017 -0700

Default to keystone v2 auth if auth_url is not

specified in contrail keystone auth config file,
to make it backward compatible to pre 3.2.6 builds.

Change-Id: I49886e5403ee051cbe5200d3da0dc4184b507ffc
Closes-Bug: 1727005

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/36802
Committed: http://github.com/Juniper/contrail-controller/commit/767f9e74b5f5b31227aa938dcd8f2b7d1507e56a
Submitter: Zuul (<email address hidden>)
Branch: master

commit 767f9e74b5f5b31227aa938dcd8f2b7d1507e56a
Author: Ignatious Johnson Christopher <email address hidden>
Date: Tue Oct 24 22:54:53 2017 -0700

Default to keystone v2 auth if auth_url is not

specified in contrail keystone auth config file,
to make it backward compatible to pre 3.2.6 builds.

Change-Id: I49886e5403ee051cbe5200d3da0dc4184b507ffc
Closes-Bug: 1727005

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/36804
Committed: http://github.com/Juniper/contrail-controller/commit/391e2c0785f25c17b810963c841336fd64145827
Submitter: Zuul (<email address hidden>)
Branch: R4.0

commit 391e2c0785f25c17b810963c841336fd64145827
Author: Ignatious Johnson Christopher <email address hidden>
Date: Tue Oct 24 22:54:53 2017 -0700

Default to keystone v2 auth if auth_url is not

specified in contrail keystone auth config file,
to make it backward compatible to pre 3.2.6 builds.

Change-Id: I49886e5403ee051cbe5200d3da0dc4184b507ffc
Closes-Bug: 1727005

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.