search

juju-solutions / matrix

Automatic testing of big software deployments under various failure conditions
Other
8 stars 9 forks source link

Add compatibility with Jaas #85

Closed pengale closed 7 years ago

pengale commented 7 years ago

I just started up matrix while juju was pointed at the Jaas controller. It failed pretty spectacularly.

We need to handle the situation where an operator is using Jaas. The best way might be to pass in some Jaas arguments as command line arguments to matrix.

adam-stokes commented 7 years ago

We need this for conjure-up as well, would having it in libjuju work? We're migrating our code over to that as well

pengale commented 7 years ago

@battlemidget libjuju is the right place to put the fix (libjuju might already have support, in which case we just need to call .connect with all the right args.)

pengale commented 7 years ago

It looks like the problem here is that matrix never bothers to specify a cloud. Everything else Just Works in python-libjuju.

Working on a patch right now ...

pengale commented 7 years ago

JaaS support mainly automatically works. I have a branch (https://github.com/juju-solutions/matrix/tree/feature/jaas-support) that adds the ability to specific a cloud in matrix, which we need, as the cloud that you're deploying to is ambiguous in JaaS.

Unfortunately, since JaaS is still on Juju 2.0.3 and python-libjuju only supports the latest stable juju 2.1, we are blocked by https://github.com/juju/python-libjuju/issues/49. Without the ability to pick an appropriate facade, we fail with many errors when attempting to connect to the JaaS controller.

pengale commented 7 years ago

Here's a sketch of a proposed fix for python-libjuju: http://paste.ubuntu.com/24320954/

Please leave comments on https://github.com/juju/python-libjuju/issues/49

pengale commented 7 years ago

I unblocked us in python-libjuju. Now I'm working on the next step, which is to add support for selecting a cloud (and preferably auto-selecting a cloud).

pengale commented 7 years ago

PR here: https://github.com/juju-solutions/matrix/pull/127

pengale commented 7 years ago

This is done!