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

Bug 2105312

Summary:	[OVN] Allow to configure ovsdb probe interval in ml2 conf and ovn metadata conf file
Product:	Red Hat OpenStack	Reporter:	Yatin Karel <ykarel>
Component:	openstack-tripleo-heat-templates	Assignee:	Yatin Karel <ykarel>
Status:	CLOSED NEXTRELEASE	QA Contact:	Joe H. Rahme <jhakimra>
Severity:	high	Docs Contact:
Priority:	high
Version:	17.0 (Wallaby)	CC:	erpeters, gthiemon, jamsmith, jlibosva, jschluet, mburns, rhos-maint, tkajinam, twilson
Target Milestone:	z1	Keywords:	Regression, Triaged
Target Release:	17.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	openstack-tripleo-heat-templates-14.3.1-0.20220719171727.feca772.el9ost puppet-tripleo-14.2.3-0.20220718160753.41752a3.el9ost puppet-neutron-18.5.1-0.20220714150330.3bdf311.el9ost	Doc Type:	Known Issue
Doc Text:	There is currently a known issue where the `ovn/ovsdb_probe_interval` value is not configured in the file `ml2_conf.ini` with the value specified by `OVNOvsdbProbeInterval` because a patch required to configure the neutron server based on `OVNOvsdbProbeInterval` is not included in 17.0.1. + Workaround: Deployments that use `OVNOvsdbProbeInterval` must use `ExtraConfig` hooks in the following manner to configure the neutron server: + ---- parameter_defaults: OVNOvsdbProbeInterval: <probe interval in milliseconds> ControllerExtraConfig: neutron::config::plugin_ml2_config: ovn/ovsdb_probe_interval: value: <probe interval in milliseconds> ----	Story Points:	---
Clone Of:
Clones:	2185914 (view as bug list)		Environment:
Last Closed:	2023-04-12 14:46:11 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:	2102677, 2185914

Description Yatin Karel 2022-07-08 14:00:38 UTC

Description of problem:
In case of scale env the default value of [ovn]/ovsdb_probe_interval which is 60 seconds may not be sufficient, So add a parameter in tripleo-heat-templates to configure in ml2_conf.ini and ovn meta data agent config file(neutron_ovn_metadata_agent.ini)
Originally detected the need of it in https://bugzilla.redhat.com/show_bug.cgi?id=2102677

With https://review.opendev.org/c/openstack/puppet-neutron/+/817198 patch in master it already configures ml2_conf.ini which is used by neutron server.
It also needs to be added to ovn meta data agent config file which is used by ovn metadata container. And backport to stable/wallaby and rhos 17.0.

Also there is a wrong parameter in ovn metadata conf for the same purpose which is confusing, so also that needs to be cleaned up, that's already cleaned in master https://review.opendev.org/q/ce2357d422dfe5c24861caaa6219eae27642c1e4

Comment 1 Takashi Kajinami 2022-07-08 14:25:00 UTC

> Also there is a wrong parameter in ovn metadata conf for the same purpose which is confusing, so also that needs to be cleaned up, that's already cleaned in master https://review.opendev.org/q/ce2357d422dfe5c24861caaa6219eae27642c1e4

Please note that in Puppet OpenStack we should NOT remove any parameter in stable branches
as we regard it as breacking change in the class interface.

I expect this change is limited within tripleo.(just remove the puppet parameter definition)

Comment 2 Takashi Kajinami 2022-07-08 14:27:10 UTC

(In reply to Takashi Kajinami from comment #1)
...
> I expect this change is limited within tripleo.(just remove the puppet
> parameter definition)

Maybe backporting https://review.opendev.org/c/openstack/tripleo-heat-templates/+/839157 would be enough.

Comment 3 Takashi Kajinami 2022-07-08 14:30:13 UTC

So what we'd need are

1. Backport  https://review.opendev.org/c/openstack/puppet-neutron/+/817198 back to wallaby
   (note: make sure we backport it to yoga and xena first)

2. Create a similar commit to add these tunables to ovn_metadata_agent.ini and backport that back to wallaby

3. Bacport https://review.opendev.org/c/openstack/tripleo-heat-templates/+/839157 to wallaby

Comment 4 Terry Wilson 2022-07-08 15:14:40 UTC

It should be noted that the parameter doesn't actually do anything and never has--it's misnamed. The parameter is ovn/ovsdb_probe_interval

Comment 5 Takashi Kajinami 2022-07-08 15:36:37 UTC

> 3. Bacport https://review.opendev.org/c/openstack/tripleo-heat-templates/+/839157 to wallaby

I noticed that we also need to backport https://review.opendev.org/c/openstack/puppet-tripleo/+/839156 to get rid of the wrong parameter used.