hard-coded charm path to keystone cause tox test failure
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
OpenStack Keystone Charm |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
I try to execute tox test for charm-keystone, however some will fail if the repo name is not keystone:
step to reproduce:
$ git clone https:/
$ cd charm-keystone
$ tox -e func-smoke
The test will try to deploy a charm bundle using yaml from test/bundles and fail with error message similar to below
2021-10-19 14:30:23 [INFO] Deploying bundle '/home/
2021-10-19 14:30:23 [INFO] Rendered template '<Template 'local-
2021-10-19 14:30:23 [INFO] Deploying overlay '/tmp/tmp0qf6tt
2021-10-19 14:30:24 [INFO] ERROR cannot deploy bundle: the provided bundle has the following errors:
2021-10-19 14:30:24 [INFO] charm path in application "keystone" does not exist: /home/ubuntu/
Traceback (most recent call last):
File "/home/
utils.
File "/home/
raise subprocess.
subprocess.
verlay', '/tmp/tmp0qf6tt
the reason for such failure is because bundle yaml in test/bundles hardcode charm path to application keystone with "../../
# test/bundles/
applications:
keystone:
charm: ../../../keystone
num_units: 3
deploy with this will cause path not found error if repo name is something other than "keystone"
eventhough we can get around by renaming the repo name to "keystone" , a better approach will be to change the charm path simply to ../../
ubuntu@
diff --git a/tests/
index 7c94373..f77afb2 100644
--- a/tests/
+++ b/tests/
@@ -35,7 +35,7 @@ applications:
- '2'
keystone:
- charm: ../../../keystone
+ charm: ../../
Changed in charm-keystone: | |
status: | New → In Progress |
Changed in charm-keystone: | |
milestone: | none → 22.04 |
Changed in charm-keystone: | |
status: | Fix Committed → Fix Released |
Reviewed: https:/ /review. opendev. org/c/openstack /charm- keystone/ +/815131 /opendev. org/openstack/ charm-keystone/ commit/ 5d19a1fab336857 ff82f7809a14182 267b860d8f
Committed: https:/
Submitter: "Zuul (22348)"
Branch: master
commit 5d19a1fab336857 ff82f7809a14182 267b860d8f
Author: Andy Wu <email address hidden>
Date: Fri Oct 22 16:03:11 2021 +0000
Fix keystone charm path setting in test bundles
Set keystone charm path in test/bundle/*.yaml to "../../"
Closes-Bug: #1947793
Change-Id: I4d32c78ddf83f6 deb86c5b84d3599 29d23b94ba5