Merge lp:~thumper/python-jujuclient/jes-cache-file into lp:python-jujuclient
Proposed by
Tim Penhey
Status: | Superseded |
---|---|
Proposed branch: | lp:~thumper/python-jujuclient/jes-cache-file |
Merge into: | lp:python-jujuclient |
Diff against target: |
202 lines (+137/-4) 3 files modified
.bzrignore (+1/-0) jujuclient.py (+35/-3) test_jujuclient.py (+101/-1) |
To merge this branch: | bzr merge lp:~thumper/python-jujuclient/jes-cache-file |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
juju-deployers | Pending | ||
Review via email: mp+261586@code.launchpad.net |
This proposal has been superseded by a proposal from 2015-07-17.
Description of the change
With the jes feature flag, we now don't store environment details in .jenv files, but in a single cache.yaml file.
This branch teaches python-jujuclient how to get the required information out of it.
To post a comment you must log in.
I have not been able to run the tests for some reason, and I am unfamiliar with tox:
$ JUJU_TEST_ ENV="mary2" tox -e py27 entry_point( 'tox==1. 6.0', 'console_scripts', 'tox')() python2. 7/dist- packages/ tox/_cmdline. py", line 25, in main python2. 7/dist- packages/ tox/_config. py", line 44, in parseconfig config, inipath) python2. 7/dist- packages/ tox/_config. py", line 236, in __init__ python2. 7/dist- packages/ tox/_config. py", line 292, in _makeenvconfig getdict( section, 'setenv') python2. 7/dist- packages/ tox/_config. py", line 419, in getdict (section, name, None) python2. 7/dist- packages/ tox/_config. py", line 511, in getdefault python2. 7/dist- packages/ tox/_config. py", line 618, in _replace sub(replace_ func, x) python2. 7/dist- packages/ tox/_config. py", line 609, in _replace_ match_no_ quote match(match, quote=False) python2. 7/dist- packages/ tox/_config. py", line 604, in _replace_match python2. 7/dist- packages/ tox/_config. py", line 541, in _replace_env TEST_ENV: "test"' : unkown environment variable 'JUJU_TEST_ ENV:"test" '
Traceback (most recent call last):
File "/usr/bin/tox", line 9, in <module>
load_
File "/usr/lib/
config = parseconfig(args, 'tox')
File "/usr/lib/
parseini(
File "/usr/lib/
config)
File "/usr/lib/
vc.setenv = reader.
File "/usr/lib/
s = self.getdefault
File "/usr/lib/
x = self._replace(x)
File "/usr/lib/
return rexpattern.
File "/usr/lib/
return self._replace_
File "/usr/lib/
return handler(match, quote)
File "/usr/lib/
(envkey, envkey))
tox.ConfigError: ConfigError: substitution env:'JUJU_