Canonical Juju

Controller bootstrap fails with Openstack Devstack

Bug #1751287 reported by Miika Kankare
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Canonical Juju
Fix Released
Critical
John A Meinel
Canonical Juju 2.4-beta1
2.3
Fix Released
Critical
John A Meinel
Canonical Juju 2.3.5

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

Revision history for this message
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
Revision history for this message
John A Meinel (jameinel) wrote :

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

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
Revision history for this message
David Douglas (ddouglas-austin) wrote :

adding field-critical due to blocked deployment.

John A Meinel (jameinel)
Changed in juju:
importance: Medium → High
John A Meinel (jameinel)
Changed in juju:
importance: High → Critical
Revision history for this message
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.

Revision history for this message
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.

Revision history for this message
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.

Revision history for this message
John A Meinel (jameinel) wrote :

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

Revision history for this message
John A Meinel (jameinel) wrote :

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

Revision history for this message
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  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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