Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 1642270

Summary:	[Deployment] ODL: check for diagstatus instead of netvirt:1 when doing set-manager
Product:	Red Hat OpenStack	Reporter:	Vishal Thapar <vthapar>
Component:	puppet-neutron	Assignee:	Daniel Farrell <dfarrell>
Status:	CLOSED WONTFIX	QA Contact:	Noam Manos <nmanos>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	13.0 (Queens)	CC:	bhaley, jjoyce, jschluet, mkolesni, slinaber, tvignaud
Target Milestone:	z5	Keywords:	Triaged, ZStream
Target Release:	13.0 (Queens)
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:	Deployment
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2019-03-06 16:15:53 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	1605218

Description Vishal Thapar 2018-10-24 05:54:28 UTC

Description of problem:
In ODL deployments, when doing set-manager on OVS we check for netvirt:1. This is not a reliable indicator and should instead use diagstatus to make sure all services are up and can process incoming connections

Refer https://bugzilla.redhat.com/show_bug.cgi?id=1605218 for details.

Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 3 Daniel Farrell 2018-11-29 01:49:50 UTC

First stab at a patch: https://review.openstack.org/#/c/620764/

Janki added for review.

Comment 4 Daniel Farrell 2018-11-30 01:55:21 UTC

Puppet-neutron patch above has good reviews so far, but no +2s.

Corresponding THT patch:

https://review.openstack.org/#/c/621052

Janki and Tim are reviewing.

Comment 5 Daniel Farrell 2018-12-03 15:45:00 UTC

Lots of +1s and one +2, but neither merged yet.

Comment 6 Daniel Farrell 2018-12-05 22:15:44 UTC

621052 is merged. Got a -1 on 620764 due to missing release notes, which I have since added. Also added release notes for 621052 in 623072.

Comment 7 Franck Baudin 2019-03-06 16:15:53 UTC

As per depreciation notice [1], closing this bug. Please reopen if relevant for RHOSP13, as this is the only version shipping ODL.

[1] https://access.redhat.com/documentation/en-us/red_hat_openstack_platform/14/html-single/release_notes/index#deprecated_functionality

Comment 8 Franck Baudin 2019-03-06 16:17:30 UTC

As per depreciation notice [1], closing this bug. Please reopen if relevant for RHOSP13, as this is the only version shipping ODL.

[1] https://access.redhat.com/documentation/en-us/red_hat_openstack_platform/14/html-single/release_notes/index#deprecated_functionality