ovs/ovn source(with master branch) deployments broken

Bug #2015728 reported by yatin
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
neutron
Fix Released
High
yatin

Bug Description

With [1] ovn/ovs jobs running with OVS_BRANCH=master,OVN_BRANCH=main are broken, fails as below:-

utilities/ovn-dbctl.c: In function ‘server_loop’:
utilities/ovn-dbctl.c:1105:5: error: too few arguments to function ‘daemonize_start’
 1105 | daemonize_start(false);
      | ^~~~~~~~~~~~~~~
In file included from utilities/ovn-dbctl.c:22:
/opt/stack/ovs/lib/daemon.h:170:6: note: declared here
  170 | void daemonize_start(bool access_datapath, bool access_hardware_ports);
      | ^~~~~~~~~~~~~~~
make[1]: *** [Makefile:2374: utilities/ovn-dbctl.o] Error 1
make[1]: *** Waiting for unfinished jobs....

Example failure:- https://zuul.openstack.org/build/b7b1700e2e5941f7a52b57ca411db722

Builds:-
- https://zuul.openstack.org/builds?job_name=neutron-ovn-tempest-ipv6-only-ovs-master
- https://zuul.openstack.org/builds?job_name=neutron-ovn-tempest-full-multinode-ovs-master
- https://zuul.openstack.org/builds?job_name=neutron-ovn-tempest-ovs-master
- https://zuul.openstack.org/builds?job_name=neutron-ovn-tempest-ovs-master-centos-9-stream
- https://zuul.openstack.org/builds?job_name=ovn-octavia-provider-functional-master
- https://zuul.openstack.org/builds?job_name=ovn-octavia-provider-tempest-master

Until ovn main branch is adapted to this change we need to pin ovs_branch to working commit or better stable branch(as done with [2])

Also i noticed some of these jobs running in neutron/ovn-octavia stable branches, that likely not needed to be run, so should be checked and cleaned up.

[1] https://github.com/openvswitch/ovs/commit/07cf5810de
[2] https://github.com/ovn-org/ovn/commit/b61e819bf9673

Tags: gate-failure
yatin (yatinkarel)
Changed in neutron:
status: New → Confirmed
importance: Undecided → Critical
assignee: nobody → yatin (yatinkarel)
tags: added: gat
Changed in neutron:
importance: Critical → High
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to neutron (master)

Related fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/neutron/+/879938

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to ovn-octavia-provider (master)

Related fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/ovn-octavia-provider/+/879939

yatin (yatinkarel)
tags: removed: gat
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to ovn-octavia-provider (master)

Reviewed: https://review.opendev.org/c/openstack/ovn-octavia-provider/+/879939
Committed: https://opendev.org/openstack/ovn-octavia-provider/commit/f9319a185b957b98f18225a0c15d65df4049bc6c
Submitter: "Zuul (22348)"
Branch: master

commit f9319a185b957b98f18225a0c15d65df4049bc6c
Author: yatinkarel <email address hidden>
Date: Mon Apr 10 12:42:17 2023 +0530

    Pin OVS_BRANCH to working commit

    With [1] OVN main branch compilation fails, until
    main branch is fixed to work with ovs master, let's
    pin OVS_BRANCH to working commit.

    [1] https://github.com/openvswitch/ovs/commit/07cf5810de

    Related-Bug: #2015728
    Change-Id: Icdd1affc944de6c1e00da9539e13a8d698cfc0e6

Revision history for this message
yatin (yatinkarel) wrote :

<< Also i noticed some of these jobs running in neutron/ovn-octavia stable branches, that likely not needed to be run, so should be checked and cleaned up.

These jobs need not run on stable branches and removing these is part of release preparation, pushed https://review.opendev.org/q/topic:drop-master-jobs

tags: added: gate-failure
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to neutron (master)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/879938
Committed: https://opendev.org/openstack/neutron/commit/3de8ebebd8d974c12b5a969f40190082173ffe3e
Submitter: "Zuul (22348)"
Branch: master

commit 3de8ebebd8d974c12b5a969f40190082173ffe3e
Author: yatinkarel <email address hidden>
Date: Mon Apr 10 12:34:50 2023 +0530

    Pin OVS_BRANCH to working commit

    With [1] OVN main branch compilation fails, until
    main branch is fixed to work with ovs master, let's
    pin OVS_BRANCH to working commit.

    [1] https://github.com/openvswitch/ovs/commit/07cf5810de

    Related-Bug: #2015728
    Change-Id: I15b57ff65d861d52ce8e1c2f12d2e084952df8f9

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

Duplicates of this bug

Other bug subscribers

Remote bug watches

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