-
Notifications
You must be signed in to change notification settings - Fork 77
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
migrate glance-simplestreams-sync from Amulet to Zaza #55
Conversation
1861c49
to
2188fc2
Compare
In Amulet, we only run ssl test on xenial_pike. In Zaza, we will run non-ssl and ssl test on each bundle. This is done via `tests` section in tests/tests.yaml. func-test-pr: openstack-charmers/zaza-openstack-tests#55 Partial-Bug: 1828424 Signed-off-by: Joe Guo <guoqiao@gmail.com>
This patch migrate tests from Amulet to Zaza, and Python3 only. Changes about SSL test: In Amulet, we only run SSL test on xenial_pike. In Zaza, we will run non-SSL and SSL test on each bundle. This is done via adding extra test class to `tests` section in tests/tests.yaml. func-test-pr: openstack-charmers/zaza-openstack-tests#55 Partial-Bug: 1828424 Change-Id: I4dacf33d3ab4dedbbe62a36b1b2ce5606e51d234 Signed-off-by: Joe Guo <guoqiao@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add the test_110_local_product_stream test [1]
Regarding the SSL testing. It seems like we are duplicating a lot of code here to generate certificates. And it is also not our prefered deployment method.
Let's consider adding vault to the bundle with the certificates relation instead. This may need to be added to the glance-simplestreams-sync charm.
This patch migrate tests from Amulet to Zaza, and Python3 only. func-test-pr: openstack-charmers/zaza-openstack-tests#55 Partial-Bug: 1828424 Change-Id: I4dacf33d3ab4dedbbe62a36b1b2ce5606e51d234 Signed-off-by: Joe Guo <guoqiao@gmail.com>
5175dd2
to
53c76c2
Compare
Migrate tests from Amulet to Zaza in: charm-glance-simplestreams-sync/tests/basic_deployment.py Signed-off-by: Joe Guo <guoqiao@gmail.com>
53c76c2
to
63e9b4c
Compare
This patch migrate tests from Amulet to Zaza, and Python3 only. func-test-pr: openstack-charmers/zaza-openstack-tests#55 Partial-Bug: 1828424 Change-Id: I4dacf33d3ab4dedbbe62a36b1b2ce5606e51d234 Signed-off-by: Joe Guo <guoqiao@gmail.com>
This patch migrate tests from Amulet to Zaza, and Python3 only. func-test-pr: openstack-charmers/zaza-openstack-tests#55 Partial-Bug: 1828424 Change-Id: I4dacf33d3ab4dedbbe62a36b1b2ce5606e51d234 Signed-off-by: Joe Guo <guoqiao@gmail.com>
This patch migrate tests from Amulet to Zaza, and Python3 only. SSL is enabled for all bundles with charm-vault. func-test-pr: openstack-charmers/zaza-openstack-tests#55 Partial-Bug: 1828424 Change-Id: I4dacf33d3ab4dedbbe62a36b1b2ce5606e51d234 Signed-off-by: Joe Guo <guoqiao@gmail.com>
This patch migrate tests from Amulet to Zaza, and Python3 only. SSL is enabled for all bundles with charm-vault. func-test-pr: openstack-charmers/zaza-openstack-tests#55 Partial-Bug: 1828424 Change-Id: I4dacf33d3ab4dedbbe62a36b1b2ce5606e51d234 Signed-off-by: Joe Guo <guoqiao@gmail.com>
This patch migrate tests from Amulet to Zaza, and Python3 only. SSL is enabled for all bundles with charm-vault. func-test-pr: openstack-charmers/zaza-openstack-tests#55 Partial-Bug: 1828424 Change-Id: I4dacf33d3ab4dedbbe62a36b1b2ce5606e51d234 Signed-off-by: Joe Guo <guoqiao@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great. Thanks!
This patch migrate tests from Amulet to Zaza, and Python3 only. SSL is enabled for all bundles with charm-vault, except trusty, which is not supported by vault. func-test-pr: openstack-charmers/zaza-openstack-tests#55 Partial-Bug: 1828424 Change-Id: I4dacf33d3ab4dedbbe62a36b1b2ce5606e51d234 Signed-off-by: Joe Guo <guoqiao@gmail.com>
* Update charm-glance-simplestreams-sync from branch 'master' - Migrate from Amulet to Zaza This patch migrate tests from Amulet to Zaza, and Python3 only. SSL is enabled for all bundles with charm-vault, except trusty, which is not supported by vault. func-test-pr: openstack-charmers/zaza-openstack-tests#55 Partial-Bug: 1828424 Change-Id: I4dacf33d3ab4dedbbe62a36b1b2ce5606e51d234 Signed-off-by: Joe Guo <guoqiao@gmail.com>
Migrate Amulet tests from charm-glance-simplestreams-sync/tests/, include SSL tests.