cinder-manage service list shows happy for uninitialized driver
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Cinder |
Fix Released
|
Low
|
Michal Dulko | ||
Kilo |
Won't Fix
|
Undecided
|
Unassigned |
Bug Description
If there is a problem with a cinder driver not initializing correctly, but the c-vol service is up,
cinder-manage shows the cinder-volume service State as ':-)'
For example,
If I try and configure the lefthand driver with an improper setting, such as lefthand_api_url = http://
The service comes up, but the driver is left uninitialized.
2015-04-21 09:30:51.894 ERROR cinder.
2015-04-21 09:30:51.895 DEBUG cinder.
slo.messaging=
tes.middleware=
2015-04-21 09:30:51.896 DEBUG cinder.
cfg.py:2187
2015-04-21 09:30:51.897 DEBUG cinder.
g.py:2187
2015-04-21 09:30:51.895 ERROR cinder.
2015-04-21 09:30:51.895 TRACE cinder.
2015-04-21 09:30:51.895 TRACE cinder.
2015-04-21 09:30:51.895 TRACE cinder.
2015-04-21 09:30:51.895 TRACE cinder.
2015-04-21 09:30:51.895 TRACE cinder.
2015-04-21 09:30:51.895 TRACE cinder.
2015-04-21 09:30:51.895 TRACE cinder.
2015-04-21 09:30:51.895 TRACE cinder.
2015-04-21 09:30:51.895 TRACE cinder.
2015-04-21 09:30:51.895 TRACE cinder.
2015-04-21 09:30:51.895 TRACE cinder.
2015-04-21 09:30:51.895 TRACE cinder.
2015-04-21 09:30:51.895 TRACE cinder.
2015-04-21 09:30:51.895 TRACE cinder.
2015-04-21 09:30:51.895 TRACE cinder.
And later when get_volume_stats() is called:
2015-04-21 09:32:02.021 WARNING cinder.
The c-vol service is up, but the driver can't accept provisioning requests and can't do get_volume_stats().
cinder-manage service list shows it's happy.
cinder-volume trusty-
Shouldn't we say it's somewhere between XXX and :-) ??
Changed in cinder: | |
assignee: | nobody → Vilobh Meshram (vilobhmm) |
Changed in cinder: | |
importance: | Undecided → Low |
Changed in cinder: | |
milestone: | none → liberty-1 |
tags: | added: kilo-backport-potential |
Changed in cinder: | |
status: | Fix Committed → Fix Released |
Changed in cinder: | |
milestone: | liberty-1 → 7.0.0 |
Here along with the checking whether the service_is_up or not we should also check if we were able to successfully initialize the driver.
so if ( initialize driver && service_is_up)
:)
else
XXX