Controller bootstrap fails with Openstack Devstack

Bug #1751287 reported by Miika Kankare on 2018-02-23
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
juju
Critical
John A Meinel
2.3
Critical
John A Meinel

Bug Description

Bootstrapping the controller fails with a Devstack environment, as the Identity URL is a bit different.

The error is:

ERROR cannot set config: cannot create a client: version part of identity url https://devstack.example.com/identity/v3 not valid

And comes from this check: https://github.com/juju/juju/blob/juju-2.3.3/provider/openstack/provider.go#L847

This is due to the /identity/ part in the URL. So it'll fail with any URL that has other things in addition to the /vX.Y.

The configuration for the Identity URL seems to be here: https://git.openstack.org/cgit/openstack-dev/devstack/tree/files/apache-keystone.template#n43

John A Meinel (jameinel) wrote :

It seems like it would be reasonably to just look at the end of the path, rather than assuming the entire path is only the version URL.

It seems that we only strip of a trailing '/', so probably we could do:
path := strings.TrimSuffix(strings.ToLower(url.Path), "/")
_, versionNumStr := path.Split(path)
versionNumStr = strings.TrimPrefix("v")

Changed in juju:
importance: Undecided → Medium
status: New → Triaged
tags: added: bitesize openstack-provider
John A Meinel (jameinel) wrote :
Changed in juju:
assignee: nobody → John A Meinel (jameinel)
status: Triaged → In Progress
milestone: none → 2.3.5
milestone: 2.3.5 → 2.4-beta1
David Douglas (ddouglas-austin) wrote :

adding field-critical due to blocked deployment.

John A Meinel (jameinel) on 2018-03-15
Changed in juju:
importance: Medium → High
John A Meinel (jameinel) on 2018-03-15
Changed in juju:
importance: High → Critical
John A Meinel (jameinel) wrote :

Given its devstack, is this a blocked Production deployment, or a blocked prototype deployment? I wouldn't think anyone would run Devstack for real workloads.

We'll try to get a snap built today so you can test if this fix is sufficient, which could make it fairly easy to get the fix into the next 2.3 release which should be happening soon.

Ryan Beisner (1chb1n) wrote :

Be aware that devstack is not for production use. It is a developer solution, which is intended for iteration/burn-down scenarios.

Miika Kankare (kuula) wrote :

Yes, I'm aware of that. I encountered this in a testing scenario.

I actually made a similar fix myself earlier, built Juju and tried with that. I got past the parsing of the version, but ran into more trouble with the underlying Goose library.

John A Meinel (jameinel) wrote :

Another instance of this (private because it contains sensitive logs) https://bugs.launchpad.net/juju/+bug/1756135

John A Meinel (jameinel) wrote :

marking Released in 2.4 because there hasn't been a 2.4 release without this fix, and it is fixed in the branch.

Changed in juju:
assignee: John A Meinel (jameinel) → Heather Lanigan (hmlanigan)
assignee: Heather Lanigan (hmlanigan) → John A Meinel (jameinel)
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers