Bug 1577853 - [Infra] Improve TripleO OpenDaylight healthcheck to use new status URL instead of neutron (TripleO part of work)
Summary: [Infra] Improve TripleO OpenDaylight healthcheck to use new status URL instea...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat OpenStack
Classification: Red Hat
Component: openstack-tripleo-common
Version: 13.0 (Queens)
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: z2
: 13.0 (Queens)
Assignee: Tim Rozet
QA Contact: Tomas Jamrisko
URL:
Whiteboard: Infra
Depends On: 1566080
Blocks:
TreeView+ depends on / blocked
 
Reported: 2018-05-14 09:41 UTC by Michael Vorburger
Modified: 2018-12-24 11:40 UTC (History)
8 users (show)

Fixed In Version: openstack-tripleo-common-8.6.3-3.el7ost
Doc Type: Bug Fix
Doc Text:
The docker healthcheck for OpenDaylight ensured only that the REST interface and neutron NB component was healthy in OpenDaylight. The healthcheck did not include all loaded OpenDaylight components and therefore was not accurate. Use diagstatus URI with docker healthcheck to check all of the loaded OpenDaylight components. OpenDaylight docker container health status is now more accurate.
Clone Of:
Environment:
N/A
Last Closed: 2018-08-29 16:36:37 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Launchpad 1779215 0 None None None 2018-06-28 21:06:53 UTC
OpenStack gerrit 582998 0 None stable/queens: MERGED tripleo-common: Updates OpenDaylight healthcheck URI (I3e51820ec26afb1629e58500c469140a094ce2f7) 2018-07-23 13:21:49 UTC
Red Hat Product Errata RHBA-2018:2574 0 None None None 2018-08-29 16:37:23 UTC

Description Michael Vorburger 2018-05-14 09:41:08 UTC
This issue is to do the TripleO part of the bug 1566080 story (that bug is the ODL paret).

Comment 3 Michael Vorburger 2018-05-23 09:14:43 UTC
Because in an internal documentation draft there was some doubt if this new /diagstatus URL needs any additional Karaf features to be installed, I've just done a quick test, and locally built a current netvirt/karaf from today's master (future Fluorine), and can hereby confirm that http://localhost:8181/diagstatus/ *IS* available after "feature:install odl-netvirt-openstack" (by indirect transitive feature dependency), without requiring odl-infrautils-diagstatus to be explicitly installed anywhere.

Comment 4 Tim Rozet 2018-06-28 21:23:30 UTC
Should we also modify our southbound checks (in OpenDaylightOvs service) for netvirt:1 to also use diagstatus? Is it more accurate?

https://github.com/openstack/tripleo-heat-templates/blob/stable/queens/puppet/services/opendaylight-ovs.yaml#L27

Comment 5 Michael Vorburger 2018-07-02 11:36:12 UTC
> Should we also modify our southbound checks (in OpenDaylightOvs service)
> for netvirt:1 to also use diagstatus? Is it more accurate?

I'm not familiar enough with the exact role of the southbound checks (in OpenDaylightOvs service), but I can state that querying diagstatus is certainly much more accurate than netvirt:1.

Comment 18 Joanne O'Flynn 2018-08-15 07:51:41 UTC
This bug is marked for inclusion in the errata but does not currently contain draft documentation text. To ensure the timely release of this advisory please provide draft documentation text for this bug as soon as possible.

If you do not think this bug requires errata documentation, set the requires_doc_text flag to "-".


To add draft documentation text:

* Select the documentation type from the "Doc Type" drop down field.

* A template will be provided in the "Doc Text" field based on the "Doc Type" value selected. Enter draft text in the "Doc Text" field.

Comment 21 errata-xmlrpc 2018-08-29 16:36:37 UTC
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/RHBA-2018:2574


Note You need to log in before you can comment on or make changes to this bug.