Bug 2105312 - [OVN] Allow to configure ovsdb probe interval in ml2 conf and ovn metadata conf file
Summary: [OVN] Allow to configure ovsdb probe interval in ml2 conf and ovn metadata co...
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Red Hat OpenStack
Classification: Red Hat
Component: openstack-tripleo-heat-templates
Version: 17.0 (Wallaby)
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: z1
: 17.0
Assignee: Yatin Karel
QA Contact: Joe H. Rahme
URL:
Whiteboard:
Depends On:
Blocks: 2102677 2185914
TreeView+ depends on / blocked
 
Reported: 2022-07-08 14:00 UTC by Yatin Karel
Modified: 2023-04-12 14:46 UTC (History)
9 users (show)

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> ----
Clone Of:
: 2185914 (view as bug list)
Environment:
Last Closed: 2023-04-12 14:46:11 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
OpenStack gerrit 848997 0 None MERGED Remove ineffective ovn_remote_probe_interval 2023-01-23 15:12:52 UTC
OpenStack gerrit 849000 0 None MERGED ml2+ovn: Add more tunable parameters for OVN DB connections 2023-01-23 15:12:52 UTC
OpenStack gerrit 849002 0 None MERGED Remove ineffective ovn_remote_probe_interval 2023-01-23 15:12:51 UTC
OpenStack gerrit 849077 0 None MERGED [ML2 OVN] Add OVNOvsdbProbeInterval parameter 2023-01-23 15:12:50 UTC
OpenStack gerrit 849126 0 None MERGED [ovn metadata] Add more tunable parameters 2023-01-23 15:12:49 UTC
OpenStack gerrit 849462 0 None MERGED [ovn metadata] Add more tunable parameters 2023-01-23 15:12:48 UTC
OpenStack gerrit 849528 0 None MERGED [ML2 OVN] Add OVNOvsdbProbeInterval parameter 2023-01-23 15:12:47 UTC
Red Hat Issue Tracker OSP-16310 0 None None None 2022-07-08 14:10:10 UTC

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.


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