Controller bootstrap fails with Openstack Devstack

[Miika Kankare]

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)]

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")

[John A Meinel (jameinel)]

https://github.com/juju/juju/pull/8496

[David Douglas (ddouglas-austin)]

adding field-critical due to blocked deployment.

[John A Meinel (jameinel)]

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)]

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)]

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)]

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

[John A Meinel (jameinel)]

https://github.com/juju/juju/pull/8522

[John A Meinel (jameinel)]

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