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
Add a ceilometer test case and fix existing ones for trusty/mitaka #185
Conversation
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.
Apart from one nit, the test code changes look good to me, do you have a Gerrit Review up where a result of a test run with this code in effect can be viewed?
assert(ceil.samples.list() == []) | ||
assert(ceil.meters.list() == []) | ||
|
||
# NOTE(beisner): need to add more functional tests |
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.
Probably safe to remove this comment now
|
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.
LGTM, thanks!
Zaza tests can be found here: openstack-charmers/zaza-openstack-tests#59 openstack-charmers/zaza-openstack-tests#185 Support Python 3.8 by pulling a version of charm-helpers containing juju/charm-helpers#423 Func-Test-PR: openstack-charmers/zaza-openstack-tests#185 Closes-Bug: #1828424 Change-Id: I844836b0c2f7556da0b73a7917b8ec18052ba841
* Update charm-ceilometer from branch 'master' - Porting tests from Amulet to Zaza and support for Python 3.8 Zaza tests can be found here: openstack-charmers/zaza-openstack-tests#59 openstack-charmers/zaza-openstack-tests#185 Support Python 3.8 by pulling a version of charm-helpers containing juju/charm-helpers#423 Func-Test-PR: openstack-charmers/zaza-openstack-tests#185 Closes-Bug: #1828424 Change-Id: I844836b0c2f7556da0b73a7917b8ec18052ba841
https://bugs.launchpad.net/charm-ceilometer/+bug/1828424