Merge lp:~jsing/juju-deployer/resources-support into lp:juju-deployer
Proposed by
Joel Sing
Status: | Merged | ||||
---|---|---|---|---|---|
Merged at revision: | 209 | ||||
Proposed branch: | lp:~jsing/juju-deployer/resources-support | ||||
Merge into: | lp:juju-deployer | ||||
Diff against target: |
197 lines (+71/-11) 8 files modified
deployer/action/importer.py (+1/-0) deployer/env/base.py (+9/-2) deployer/service.py (+4/-0) deployer/tests/test_data/wiki-resources.yaml (+14/-0) deployer/tests/test_guiserver.py (+3/-3) deployer/tests/test_importer.py (+26/-4) deployer/tests/test_service.py (+2/-0) doc/config.rst (+12/-2) |
||||
To merge this branch: | bzr merge lp:~jsing/juju-deployer/resources-support | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Tim Van Steenburgh (community) | Approve | ||
Review via email: mp+316538@code.launchpad.net |
To post a comment you must log in.
This LGTM but I hit one error in the tests:
======= ======= ======= ======= ======= ======= ======= ======= ======= ======= with_resources (deployer. tests.test_ importer. ImporterTest) ------- ------- ------- ------- ------- ------- ------- ------- ------- tvansteenburgh/ src/juju- deployer/ .tox/py35/ lib/python3. 5/site- packages/ mock/mock. py", line 1305, in patched tvansteenburgh/ src/juju- deployer/ deployer/ tests/test_ importer. py", line 74, in test_importer_ with_resources path.join( self.test_ data_dir, 'wiki-resources .yaml') ]) tvansteenburgh/ src/juju- deployer/ deployer/ config. py", line 34, in __init__ tvansteenburgh/ src/juju- deployer/ deployer/ config. py", line 92, in load included( ): tvansteenburgh/ src/juju- deployer/ deployer/ config. py", line 141, in _resolve_included extend( self._includes( cf)) for cf in self.config_files] tvansteenburgh/ src/juju- deployer/ deployer/ config. py", line 141, in <listcomp> extend( self._includes( cf)) for cf in self.config_files] tvansteenburgh/ src/juju- deployer/ deployer/ config. py", line 123, in _includes load(config_ file) tvansteenburgh/ src/juju- deployer/ deployer/ config. py", line 51, in _yaml_load tvansteenburgh/ src/juju- deployer/ deployer/ tests/test_ data/wiki- resources. yaml'
ERROR: test_importer_
-------
Traceback (most recent call last):
File "/home/
return func(*args, **keywargs)
File "/home/
os.
File "/home/
self.load()
File "/home/
for fp in self._resolve_
File "/home/
[files.
File "/home/
[files.
File "/home/
d = self._yaml_
File "/home/
with open(config_file) as fh:
FileNotFoundError: [Errno 2] No such file or directory: '/home/
Is there a wiki-resources.yaml file that didn't get included in the commit?