tripleo

Docker healthcheck fails for OpenDaylight due to missing features loaded in ODL

Bug #1751857 reported by Tim Rozet
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tripleo
Fix Released
High
Tim Rozet
tripleo rocky-1

Bug Description

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) of netvirt-ui feature for the heat parameter in the opendaylight service.

Tim Rozet (trozet)
Changed in tripleo:
assignee: nobody → Tim Rozet (trozet)
milestone: none → queens-rc1
importance: Undecided → Critical
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tripleo-heat-templates (master)

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

Emilien Macchi (emilienm)
Changed in tripleo:
importance: Critical → High
Alex Schultz (alex-schultz)
Changed in tripleo:
milestone: queens-rc1 → rocky-1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tripleo-heat-templates (master)

Reviewed: https://review.openstack.org/548078
Committed: https://git.openstack.org/cgit/openstack/tripleo-heat-templates/commit/?id=e581c27e32202606e4580b6ba8b42c3bb9ae097e
Submitter: Zuul
Branch: master

commit e581c27e32202606e4580b6ba8b42c3bb9ae097e
Author: Tim Rozet <email address hidden>
Date: Mon Feb 26 13:27:39 2018 -0500

    Fixes OpenDaylight healthcheck/GUI feature

    Deployments with OpenDaylight fail because the Docker healthcheck fails.
    This is due to the fact that the URL used for querying with the
    healthcheck is not available, due to a feature mismatch. This patch
    fixes the GUI feature being installed to address the issue.

    Closes-Bug: 1751857

    Change-Id: Ia62aebb9c03e09c1986c1592a38ab455e68fd103
    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 : Fix proposed to tripleo-heat-templates (stable/queens)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tripleo-heat-templates (stable/queens)

Reviewed: https://review.openstack.org/550481
Committed: https://git.openstack.org/cgit/openstack/tripleo-heat-templates/commit/?id=5871c9f79fbd80d98b2a57c2cbc1bd72e56100e2
Submitter: Zuul
Branch: stable/queens

commit 5871c9f79fbd80d98b2a57c2cbc1bd72e56100e2
Author: Tim Rozet <email address hidden>
Date: Mon Feb 26 13:27:39 2018 -0500

    Fixes OpenDaylight healthcheck/GUI feature

    Deployments with OpenDaylight fail because the Docker healthcheck fails.
    This is due to the fact that the URL used for querying with the
    healthcheck is not available, due to a feature mismatch. This patch
    fixes the GUI feature being installed to address the issue.

    Closes-Bug: 1751857

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

tags: added: in-stable-queens
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/tripleo-heat-templates 8.0.0

This issue was fixed in the openstack/tripleo-heat-templates 8.0.0 release.

Revision history for this message
Tim Rozet (trozet) wrote :

According to discussion with ODL folks, dlux GUI is also no longer supported. Therefore index.html will not exist for healthcheck to work. We need to use a different URI ('/controller/nb/v2/neutron') and remove dlux from the default features loaded.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to tripleo-common (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/560598

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to tripleo-heat-templates (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/560624

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to tripleo-common (master)

Reviewed: https://review.openstack.org/560598
Committed: https://git.openstack.org/cgit/openstack/tripleo-common/commit/?id=9e72a429d1ae1d6be2e747a973be6b6c072cd630
Submitter: Zuul
Branch: master

commit 9e72a429d1ae1d6be2e747a973be6b6c072cd630
Author: Tim Rozet <email address hidden>
Date: Wed Apr 11 15:47:18 2018 -0400

    Fixes ODL healthcheck URI

    In the past we used index.html to ensure ODL container was healthy.
    This page was provided by the GUI plugins into ODL. Now that those
    plugins are no longer supported, we need to move to a more reliable URI
    path.

    Related-Bug: 1751857

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to tripleo-heat-templates (master)

Reviewed: https://review.openstack.org/560624
Committed: https://git.openstack.org/cgit/openstack/tripleo-heat-templates/commit/?id=f51f5336798751d9a23b69c4c301f748251a6064
Submitter: Zuul
Branch: master

commit f51f5336798751d9a23b69c4c301f748251a6064
Author: Tim Rozet <email address hidden>
Date: Wed Apr 11 15:52:47 2018 -0400

    Removes odl-dlux-gui feature for ODL

    The GUI feature is no longer supported with ODL and needs to be removed.
    We relied on the URL provided by this feature in order to run our docker
    healtcheck, which is modified in the depends-on patch to a new URI.

    Depends-On: I2f33d2cf6a96005ef1d18468a8d2fcc71b17b6f8

    Related-Bug: 1751857

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to tripleo-common (stable/queens)

Related fix proposed to branch: stable/queens
Review: https://review.openstack.org/561252

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to tripleo-heat-templates (stable/queens)

Related fix proposed to branch: stable/queens
Review: https://review.openstack.org/561253

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to tripleo-common (stable/queens)

Reviewed: https://review.openstack.org/561252
Committed: https://git.openstack.org/cgit/openstack/tripleo-common/commit/?id=372fde4d7db3b8c5a90f1a8eebd302e0eb63d71d
Submitter: Zuul
Branch: stable/queens

commit 372fde4d7db3b8c5a90f1a8eebd302e0eb63d71d
Author: Tim Rozet <email address hidden>
Date: Wed Apr 11 15:47:18 2018 -0400

    Fixes ODL healthcheck URI

    In the past we used index.html to ensure ODL container was healthy.
    This page was provided by the GUI plugins into ODL. Now that those
    plugins are no longer supported, we need to move to a more reliable URI
    path.

    Related-Bug: 1751857

    Change-Id: I2f33d2cf6a96005ef1d18468a8d2fcc71b17b6f8
    Signed-off-by: Tim Rozet <email address hidden>
    (cherry picked from commit 9e72a429d1ae1d6be2e747a973be6b6c072cd630)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to tripleo-heat-templates (stable/queens)

Reviewed: https://review.openstack.org/561253
Committed: https://git.openstack.org/cgit/openstack/tripleo-heat-templates/commit/?id=a51fa30dce1b972c251e9e8aea50d653e2fa6bdf
Submitter: Zuul
Branch: stable/queens

commit a51fa30dce1b972c251e9e8aea50d653e2fa6bdf
Author: Tim Rozet <email address hidden>
Date: Wed Apr 11 15:52:47 2018 -0400

    Removes odl-dlux-gui feature for ODL

    The GUI feature is no longer supported with ODL and needs to be removed.
    We relied on the URL provided by this feature in order to run our docker
    healtcheck, which is modified in the depends-on patch to a new URI.

    Depends-On: I2f33d2cf6a96005ef1d18468a8d2fcc71b17b6f8

    Related-Bug: 1751857

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/tripleo-heat-templates 9.0.0.0b2

This issue was fixed in the openstack/tripleo-heat-templates 9.0.0.0b2 development milestone.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.