tripleo

OpenDaylight healthcheck is broken due to jetty config changes

Bug #1751928 reported by Tim Rozet on 2018-02-26

This bug affects 1 person

Affects		Status	Importance	Assigned to	Milestone
	tripleo	Fix Released	High	Tim Rozet	tripleo rocky-1

Bug Description

ODL healthcheck parses the jetty config file for host IP and port. Recently jetty config was changed in ODL and this no longer works. We should use the pax config file anyway as it presents the data in an INI key=value format, rather than in an XML file. Also, there is no support for TLS healthcheck.

Tags:

Tim Rozet (trozet) on 2018-02-26

Changed in tripleo:
importance:	Undecided → Critical
assignee:	nobody → Tim Rozet (trozet)
milestone:	none → queens-rc1
status:	New → In Progress

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2018-02-26: Fix proposed to tripleo-common (master)

Fix proposed to branch: master
Review: https://review.openstack.org/548097

Emilien Macchi (emilienm) on 2018-02-27

Changed in tripleo:
importance:	Critical → High

Alex Schultz (alex-schultz) on 2018-03-02

Changed in tripleo:
milestone:	queens-rc1 → rocky-1

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2018-03-05: Fix merged to tripleo-common (master)

Reviewed: https://review.openstack.org/548097
Committed: https://git.openstack.org/cgit/openstack/tripleo-common/commit/?id=737439aab51089c27e71137fc30388a87474dc25
Submitter: Zuul
Branch: master

commit 737439aab51089c27e71137fc30388a87474dc25
Author: Tim Rozet <email address hidden>
Date: Mon Feb 26 17:42:33 2018 -0500

Fixes OpenDaylight healthcheck

    Due to Jetty XML config changes for OpenDaylight, healtcheck no longer
    works. This patch moves the healtcheck to use the pax web config file,
    which should be a more consistent method of finding the values for host
    IP/port (INI settings). Also adds logic to support healthcheck with TLS
    based deployments.

Closes-Bug: 1751928

Change-Id: I94f351ee3c71ddefe0b5c2bfdcd784d303ccbb24
Signed-off-by: Tim Rozet <email address hidden>

Changed in tripleo:
status:	In Progress → Fix Released

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2018-03-07: Fix proposed to tripleo-common (stable/queens)

Fix proposed to branch: stable/queens
Review: https://review.openstack.org/550482

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2018-03-08: Fix merged to tripleo-common (stable/queens)

Reviewed: https://review.openstack.org/550482
Committed: https://git.openstack.org/cgit/openstack/tripleo-common/commit/?id=0166826b0d0d3e858a3d89e2b5217254c8624361
Submitter: Zuul
Branch: stable/queens

commit 0166826b0d0d3e858a3d89e2b5217254c8624361
Author: Tim Rozet <email address hidden>
Date: Mon Feb 26 17:42:33 2018 -0500

Fixes OpenDaylight healthcheck

Closes-Bug: 1751928

    Change-Id: I94f351ee3c71ddefe0b5c2bfdcd784d303ccbb24
    Signed-off-by: Tim Rozet <email address hidden>
    (cherry picked from commit 737439aab51089c27e71137fc30388a87474dc25)