juju-solutions / matrix

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

matrix doesn't use lxd cred in 2.1 #93

Closed kwmonroe closed 7 years ago

kwmonroe commented 7 years ago

Juju 2.1 now requires credentials for lxd. In cwr-ci, we support this with the set-credentials action. This works for cwr models, but fails when matrix tries to create a model:

matrix:124:load_suites: Parsing /usr/local/lib/python3.5/dist-packages/matrix/matrix.yaml
matrix:461:add_model: Creating model matrix-happy-pig
matrix:380:run: Error adding model: no credential specified
------------------------------------------------------------------------------
matrix:431:cleanup: Error while running crashdump.
pengale commented 7 years ago

Oof. I think that we need to be more specific about our controller connection in matrix. The refactor shouldn't be too bad ...

johnsca commented 7 years ago

I think we just need to remove the localhost / lxd exemption in the code to get the credentials from the files.