diff options
| author | Daniel Watkins <daniel.watkins@canonical.com> | 2019-03-04 22:36:07 +0000 |
|---|---|---|
| committer | Server Team CI Bot <josh.powers+server-team-bot@canonical.com> | 2019-03-04 22:36:07 +0000 |
| commit | 109772c2e9066f5ae53aa2806c4bb4a2ab6f4bff (patch) | |
| tree | 838b2ab736e92a1ea9003ad5b73b9e5c284e29b0 /cloudinit/cmd/tests/test_clean.py | |
| parent | 5352dd99eb2937b4eaaaf596b40ad7ca69d87f64 (diff) | |
| download | vyos-cloud-init-109772c2e9066f5ae53aa2806c4bb4a2ab6f4bff.tar.gz vyos-cloud-init-109772c2e9066f5ae53aa2806c4bb4a2ab6f4bff.zip | |
clean: correctly determine the path for excluding seed directory
Previously, init.paths.cloud_dir has a trailing slash, which meant that
"/var/lib/cloud//seed" was being compared to "/var/lib/cloud/seed" and
(of course), never matching.
In this commit, switch to using os.path.join to avoid this case (and
update the tests to catch it in future).
LP: #1818571
Diffstat (limited to 'cloudinit/cmd/tests/test_clean.py')
| -rw-r--r-- | cloudinit/cmd/tests/test_clean.py | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/cloudinit/cmd/tests/test_clean.py b/cloudinit/cmd/tests/test_clean.py index 15c3294e..f092ab3d 100644 --- a/cloudinit/cmd/tests/test_clean.py +++ b/cloudinit/cmd/tests/test_clean.py @@ -22,7 +22,8 @@ class TestClean(CiTestCase): class FakeInit(object): cfg = {'def_log_file': self.log1, 'output': {'all': '|tee -a {0}'.format(self.log2)}} - paths = mypaths(cloud_dir=self.artifact_dir) + # Ensure cloud_dir has a trailing slash, to match real behaviour + paths = mypaths(cloud_dir='{}/'.format(self.artifact_dir)) def __init__(self, ds_deps): pass |