Upgrade OVN to 22.03 on Focal in charm-guide

Bug #2041900 reported by Pedro Victor Lourenço Fragola
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Charm Guide
Invalid
High
Peter Matulis

Bug Description

- [X ] I have a fix to the document that I can paste below including example: input and output.

Please the steps[0] need to add ovn-central package

Change this:

juju exec -a ovn-central 'apt update && apt -y install \
   --only-upgrade openvswitch-common ovn-common'

to

juju exec -a ovn-central 'apt update && apt -y install \
   --only-upgrade openvswitch-common ovn-common ovn-central'

[0] https://docs.openstack.org/charm-guide/latest/project/procedures/ovn-upgrade-2203.html#ensure-ovn-package-requirements

Tags: charm-guide
Revision history for this message
Peter Matulis (petermatulis) wrote :

Thanks Pedro. We're looking into this.

Revision history for this message
Peter Matulis (petermatulis) wrote :

Pedro, did the missing bits cause a failure of some kind?

Changed in charm-guide:
assignee: nobody → Peter Matulis (petermatulis)
importance: Undecided → High
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to charm-guide (master)
Revision history for this message
Edward Hope-Morley (hopem) wrote :

if this issue is real then presumably the problem also exist for ovn-chassis since we not include ovn-host in the list of packages to upgrade?

Revision history for this message
Trent Lloyd (lathiat) wrote :

Once you upgrade one package from the same source package, the rest will also get upgraded.

The reason is that Debian packages from the same source package will generally depend on each other with an exact version match. For example:

root@master-gopher:~# apt show ovn-central
Package: ovn-central
Version: 20.03.2-0ubuntu0.20.04.4
...
Depends: lsb-base, lsb-release, openvswitch-common (>= 2.13.0~git20200212.15ae9db33-0ubuntu1~), ovn-common (= 20.03.2-0ubuntu0.20.04.4), libc6 (>= 2.29), libcap-ng0 (>= 0.7.9), libssl1.1 (>= 1.1.0), libunbound8 (>= 1.8.0)

We have used this a lot everywhere to upgrade related packages of ceph, neutron, etc.. so I don't see why this change is necessary. This is also noted in the review comments. I think we can drop this?

It would help if @Pedro you have any information about why you opened this bug. Did you see a specific case where you think this didn't work? We'd need some more details.

Revision history for this message
Pedro Victor Lourenço Fragola (pedrovlf) wrote (last edit ):

Hey @all

If the package is going to be upgraded along with its dependencies, particularly the "ovn-central" package... +1 thanks for checking it.

The context we dealt with involved an incomplete upgrade that disrupted OVN. While troubleshooting, we discovered that certain packages, like "ovn-central," weren't updated. Hence, I suggested this to prevent potential issues in the future.

    ovn-central:
      Installed: 20.03.2-0ubuntu0.20.04.4
      Candidate: 22.03.3-0ubuntu0.22.04.1~cloud0
      Version table:
         22.03.3-0ubuntu0.22.04.1~cloud0 500
            500 http://ubuntu-cloud.archive.canonical.com/ubuntu focal-updates/ovn-22.03/main amd64
    ovn-common:
      Installed: 20.03.2-0ubuntu0.20.04.4
      Candidate: 22.03.3-0ubuntu0.22.04.1~cloud0
      Version table:
         22.03.3-0ubuntu0.22.04.1~cloud0 500
            500 http://ubuntu-cloud.archive.canonical.com/ubuntu focal-updates/ovn-22.03/main amd64

Edited: I understand that, with the dependency on the openvswitch-common and ovn-common packages, the ovn-central package has to be updated. Thanks for confirming this.

Revision history for this message
Peter Matulis (petermatulis) wrote :

i'm still unsure what's going on here. packages should be upgraded but there have been reports of them not being upgraded? is there some other issue at play here besides apt dependency management?

Revision history for this message
Pedro Victor Lourenço Fragola (pedrovlf) wrote :

Based on Trent's explanation, I understood that this change is not necessary so this request can be marked as invalid.

Thanks!

Revision history for this message
Peter Matulis (petermatulis) wrote :

alright thank you

Changed in charm-guide:
status: In Progress → Invalid
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on charm-guide (master)

Change abandoned by "Peter Matulis <email address hidden>" on branch: master
Review: https://review.opendev.org/c/openstack/charm-guide/+/899892
Reason: superseded by 4e081814a0df2e11aeb4f61775b549828fdeaf89

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.