Red Hat Bugzilla – Bug 1549218
[Deployment] Docker healthcheck fails for OpenDaylight due to missing features loaded in ODL
Last modified: 2018-10-18 03:18:04 EDT
Description of problem: Docker healthcheck works by checking for index.html in ODL. However, netvirt-ui feature which offers this URL was removed from Oxygen build. In order to fix the problem, we can load the dlux gui feature instead (odl-dlux-core).
(In reply to Tim Rozet from comment #0) > Description of problem: > Docker healthcheck works by checking for index.html in ODL. However, > netvirt-ui feature which offers this URL was removed from Oxygen build. In > order to fix the problem, we can load the dlux gui feature instead > (odl-dlux-core). Can we change how the docker healthcheck works? If not, is there another way we can get the index.html to show up without installing dlux? that project is pretty stale and maybe slated for removal. It'd be nice not to have to load it.
Patch merged into upstream queens
There is a REST endpoint we can use that seems like a better option than using the presence of that index.html file. we can hit this: curl -u "admin:admin" http://${ODL_IP}:8081/jolokia/read/org.opendaylight.infrautils:type=ready,name=SystemState The response is something like this: {"request":{"mbean":"org.opendaylight.infrautils:name=SystemState,type=ready","type":"read"},"value":{"SystemState":"ACTIVE"},"timestamp":1521496065,"status":200} notice the SystemState:ACTIVE. This would require the odl-infrautils-ready feature to be installed.
Lowering severity on this as unhealthy containers do not fail deployments. See https://bugs.launchpad.net/tripleo/+bug/1752925
Created attachment 1419132 [details] Feature list
Checked with openstack-tripleo-heat-templates-8.0.2-0.20180327213843.f25e2d8.el7ost.noarch I don't see the feature odl-dlux-core (I attached the features list). Also the container in unhealthy.
I am seeing unhealthy status too. I suspect it has something to do with jolokia or some config somewhere. $ curl http://10.10.125.12:8081/index.html <html> <head> <meta http-equiv="Content-Type" content="text/html;charset=utf-8"/> <title>Error 401 Unauthorized</title> </head> <body><h2>HTTP ERROR 401</h2> <p>Problem accessing /index.html. Reason: <pre> Unauthorized</pre></p> </body> </html> "Unauthorised" reason is weird too. $ curl -u admin:admin http://10.10.125.12:8081/index.html <html> <head> <meta http-equiv="Content-Type" content="text/html;charset=utf-8"/> <title>Error 404 Not Found</title> </head> <body><h2>HTTP ERROR 404</h2> <p>Problem accessing /index.html. Reason: <pre> Not Found</pre></p> </body> </html>
(In reply to jamo luhrsen from comment #9) > There is a REST endpoint we can use that seems like a better option than > using the presence of that index.html file. > > we can hit this: > > curl -u "admin:admin" > http://${ODL_IP}:8081/jolokia/read/org.opendaylight.infrautils:type=ready, > name=SystemState > > The response is something like this: > > {"request":{"mbean":"org.opendaylight.infrautils:name=SystemState, > type=ready","type":"read"},"value":{"SystemState":"ACTIVE"},"timestamp": > 1521496065,"status":200} > > notice the SystemState:ACTIVE. > > This would require the odl-infrautils-ready feature to be installed. We moved to using index.html because it doesnot need (atleast when it was implemented) credentails. TripleO can provide custom credentials and no proper way was found to get credentials from any file in ODL.
Agreed that re-packaging DLUX again is not the way we want to go IMHO... Let us instead first clarify what we really want to check for, and then change the health check to probe something better than that /index.html? FYI that always was fairly useless, anyway (the fact that HTTP GET /index.html returns HTTP never really proved anything; just Karaf is running, but netvirt et al can be completely broken).
Let's decide what should be the way to do this check. For now I propose to curl the /controller/nb/v2/neutron path and see that it returns a non-error code (currently from what I see you get a 302 Found code). We can consider better alternatives for future versions but I suggest we track it in a different BZ.
Agreed, lets use /controller/nb/v2/neutron for the short term, and improve it later in bug 1566080.
All patches are merged upstream
*** Bug 1549298 has been marked as a duplicate of this bug. ***
dropping master external trackers as they have all landed on stable/queens
Checked with: openstack-tripleo-heat-templates-8.0.2-0.20180416194362.29a5ad5.el7ost.noarch
*** Bug 1570927 has been marked as a duplicate of this bug. ***
Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. https://access.redhat.com/errata/RHEA-2018:2086