Need to branch jujuclient to a tempdir for deploy.py

Bug #1287117 reported by Evan
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ubuntu CI Engine
Fix Released
Low
Evan
Ubuntu CI Services
Fix Released
Low
Evan

Bug Description

10:26 AM <vila> ev: deployment failed: ImportError: No module named jujuclient , known issue ?
10:27 AM <ev> vila: yeah, you need to install python-jujuclient. We should get that in a tempdir and PYTHONPATH updated in deploy.py, but hack around it for now by manually installing

Tags: airline

Related branches

Revision history for this message
Chris Johnston (cjohnston) wrote :

IIRC we were only branching things that weren't packaged, or the packaged versions weren't helpful (bugs, too old, etc).. If we are going to branch jujuclient, where do we stop? juju-core? Other packages? Having a base set of required packages shouldn't be a problem.

Revision history for this message
Evan (ev) wrote :

We branch and pin juju-deployer. It logically follows that we branch and pin its core library.

As the bug points out, you can easily find yourself in the situation where the deploy fails with a confusing error because there's nothing codifying the dependency on jujuclient.

Revision history for this message
Andy Doan (doanac) wrote :

This usually happens when someone runs our deploy script from a venv that doesn't have Juju installed into it. I don't know what the best course of action is. I always deploy outside of a venv.

Revision history for this message
Vincent Ladeuil (vila) wrote :

@Andy: I was deploying outside of a venv and I had python-jujuclient installed on the host.

Revision history for this message
Andy Doan (doanac) wrote : Re: [Bug 1287117] Re: Need to branch jujuclient to a tempdir for deploy.py

On 03/03/2014 11:13 AM, Vincent Ladeuil wrote:
> @Andy: I was deploying outside of a venv and I had python-jujuclient
> installed on the host.
n/m - i'm not sure how that would happen then.

Changed in ubuntu-ci-services-itself:
status: New → Fix Committed
Changed in ubuntu-ci-services-itself:
status: Fix Committed → Fix Released
assignee: nobody → Evan Dandrea (ev)
Ursula Junque (ursinha)
Changed in uci-engine:
assignee: nobody → Evan Dandrea (ev)
importance: Undecided → Low
milestone: none → phase-0
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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