IPMI unit test doesn't cover all platform initilization
Bug #1414903 reported by
Zhai, Edwin
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Ceilometer |
Fix Released
|
Medium
|
Zhai, Edwin |
Bug Description
IPMI/NM unit test was designed to cover initialization on different platforms: IPMI/Non-IPMI, NM/Non-NM. As execution engine is singleton, test for the 2nd platform still use the 1st platform's engine and its initialization doesn't run at all. So only initialization code for one platform are tested.
Changed in ceilometer: | |
assignee: | nobody → Zhai, Edwin (edwin-zhai) |
status: | New → In Progress |
Changed in ceilometer: | |
importance: | Undecided → Medium |
milestone: | none → kilo-2 |
Changed in ceilometer: | |
status: | Fix Committed → Fix Released |
Changed in ceilometer: | |
milestone: | kilo-2 → 2015.1.0 |
To post a comment you must log in.
Reviewed: https:/ /review. openstack. org/150290 /git.openstack. org/cgit/ openstack/ ceilometer/ commit/ ?id=4b37a36fbff 3ad7a93b4afb903 8f793548a6ef04
Committed: https:/
Submitter: Jenkins
Branch: master
commit 4b37a36fbff3ad7 a93b4afb9038f79 3548a6ef04
Author: Edwin Zhai <email address hidden>
Date: Tue Jan 27 14:20:36 2015 +0800
Fix IPMI unit test to cover different platforms
As execution engine is singleton, test for the 2nd platform still use
the 1st platform's engine and its initialization doesn't run at all.
Reset engine init flag after test for each platform to fix it.
Change-Id: I73b69892b8ee42 9ed649741419554 076379c093c
Closes-Bug: #1414903