Set ovn_match_northd_version to false by default

[booxter]

true value is needed when executing "fail safe upgrades" only [0]. (These are upgrades where an LTS release is skipped; or when ovn-controllers are updated after northd.) This is not the upgrade scenario that we follow.

This was fixed in puppet modules in [1] but sadly was missed when the code was imported to this repo (several months before the change in puppet modules.)

[0] https://docs.ovn.org/en/latest/intro/install/ovn-upgrades.html#fail-safe-upgrade [1] https://opendev.org/openstack/puppet-tripleo/commit/68b4a7c87d3af1200d18149365844d40683cb530

Depends-On: https://github.com/openstack-k8s-operators/openstack-operator/pull/792

[openshift-ci[bot]]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: booxter

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[roles/edpm_ovn/OWNERS](https://github.com/openstack-k8s-operators/edpm-ansible/blob/main/roles/edpm_ovn/OWNERS)~~ [booxter] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment

[karelyatin]

/hold

Discussing this over slack

[booxter]

@karelyatin considering the slack discussion, are we to move forward with this (post beta obviously) or are we going to wait for staged ovn update work from Dan to land?

[booxter]

recheck

[booxter]

@karelyatin https://github.com/openstack-k8s-operators/openstack-operator/pull/792 landed. Can we now unhold this PR?

[karelyatin]

/unhold

[karelyatin]

/lgtm