Missing option for OSP16.2 OVN migration

Bug #1912948 reported by Roman Safronov
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Low
Unassigned

Bug Description

Attempt to perform OVN migration to OSP16.2 using infrared plugin included in the neutron repo failed due to missing 16.2 option. In order to support OVN migration to this version we need to include this option.

Revision history for this message
Eran Kuris (ekuris) wrote :
Revision history for this message
Bence Romsics (bence-romsics) wrote :

Previously I had the impression everything we had in the neutron repo under tools/ovn_migration/ was meant as an example. Like here:

https://opendev.org/openstack/neutron/src/branch/master/tools/ovn_migration/README.rst

we said this:

"Proof-of-concept ansible script for migrating an OpenStack deployment that uses ML2/OVS to OVN."

I believe it would nice to clear up what exactly we (the Neutron team) want to support from the ovn migration tools. Currently it's not clear to me how much of this could be an upstream responsibility since OSP is a downstream distro.

Also are you sure that the reported version-specific limitation is actually caused by code stored in the neutron repo?

Revision history for this message
Roman Safronov (rsafrono) wrote :

This automation code is used in Red Hat downstream CI and it caused a fail of OVN migration on OSP 16.2, like this:

infrared tripleo-ovn-migration --version 16.2 --registry-mirror rhos-qe-mirror-tlv.usersys.redhat.com:5000 --registry-prefix rhosp16-openstack --external_network nova --image_name cirros-0.4.0-x86_64-disk.img --dvr True

# OUTPUT:
usage: infrared tripleo-ovn-migration [-h]
                                      [--registry-namespace REGISTRY-NAMESPACE]
                                      [--registry-prefix REGISTRY-PREFIX]
                                      [--registry-tag REGISTRY-TAG]
                                      [--registry-mirror REGISTRY-MIRROR]
                                      [--version VERSION]
                                      [--install_from_package INSTALL_FROM_PACKAGE]
                                      [--dvr DVR]
                                      [--create_resources CREATE_RESOURCES]
                                      [--external_network EXTERNAL_NETWORK]
                                      [--image_name IMAGE_NAME] [-v]
                                      [--ansible-args ANSIBLE-ARGS]
                                      [--inventory INVENTORY] [--dry-run]
                                      [-e EXTRA-VARS] [-o OUTPUT]
                                      [--from-file FROM-FILE]
                                      [--generate-answers-file GENERATE-ANSWERS-FILE]
infrared tripleo-ovn-migration: error: argument --version: invalid choice: '16.2' (choose from '7', '8', '9', '10', '11', '12', '13', '14', '15', '16', '16.1', 'kilo', 'liberty', 'mitaka', 'newton', 'ocata', 'pike', 'queens', 'rocky', 'stein', 'train')

So I am just trying to fix a broken thing.

Of course, we in Red Hat can move to using a downstream repo for this ansible tool if it looks like inappropriate here.

Revision history for this message
Bence Romsics (bence-romsics) wrote :

I'm not saying it's inappropriate - it is clearly useful for all ovn users, either as it is, or at least as an example. On the other hand that downstream CI seems to be necessary for the maintenance of it.

Changed in neutron:
status: New → Confirmed
importance: Undecided → Low
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/networking-ovn 7.4.0

This issue was fixed in the openstack/networking-ovn 7.4.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron 18.0.0.0rc1

This issue was fixed in the openstack/neutron 18.0.0.0rc1 release candidate.

Revision history for this message
Brian Haley (brian-haley) wrote :

Since fix seems to be included in later releases will close this bug.

Changed in neutron:
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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