Skip to content

[release-4.13] OCPBUGS-15128: Dockerfile: bump to ovn23.03-23.03.0-69 for LB templates#1708

Closed
dcbw wants to merge 1 commit intoopenshift:release-4.13from
dcbw:413-ovn-23.03-69-lb-templates
Closed

[release-4.13] OCPBUGS-15128: Dockerfile: bump to ovn23.03-23.03.0-69 for LB templates#1708
dcbw wants to merge 1 commit intoopenshift:release-4.13from
dcbw:413-ovn-23.03-69-lb-templates

Conversation

@dcbw
Copy link
Contributor

@dcbw dcbw commented Jun 14, 2023 •
edited
Loading

We need at >= 23.03-14 for LB templates; bump to latest to get other fixes too.

OVN changes 7 -> 69:
    - Implement MTU Path Discovery for multichassis ports
    - northd: Add logical flow to skip GARP with LLA (#2211240)
    - northd: match only on supported protocols to handle_svc_check (#1913162)
    - northd: Fix address set incremental processing (#2196885)
    - controller: Ignore DNS queries with RRs
    - controller: Handle OpenFlow errors. (#2134880)
    - northd: fix ls_in_hairpin l3dgw flow generation
    - ovn-controller.c: Fix assertion failure during address set update.
    - ovn-controller: fixed port not always set down when unbinding interface (#2150905)
    - ovn-controller: fixed ovn-installed not always properly added or removed.
    - northd: fix use-after-free after lrp destroyed
    - Omit ACLs for nd || nd_ra || nd_rs || mldv1 || mldv2 (#2149731)
    - controller: Prevent race in tunnel cleanup
    - northd.c: Avoid sending ICMP time exceeded for multicast packets.
    - northd.c: TTL discard flow should support for both ipv4 and ipv6.
    - northd: Update the is_stateless helper for router nat
    - northd: revert ct.inv drop flows
    - northd: take into account qos_min_rate in port_has_qos_params
    - northd: prevents sending packet to conntrack for router ports (#2062431)
    - lb: Allow IPv6 template LBs to use explicit backends.
    - controller: lflow: do not use tcp as default IP protocol for ct_snat_to_vip action (#2157846)
    - northd: Drop packets for LBs with no backends (#2177173)
    - northd: Use generic ct.est flows for LR LBs (#2172048 2170885)
    - northd: drop ct.inv packets in post snat and lb_aff_learn stages (#2160685)
    - controller: Add config option per LB to enable/disable CT flush (#2178962)

OVS changes 2 -> 32:
    859071224c ovsdb: monitor: Destroy initial change set when new columns added.
    54e45e3fee ovsdb: Monitor: Keep and maintain the initial change set.
    9529e9aa96 ovsdb: Allow conversion records with no data in a clustered storage.
    7006bb112b ovsdb: Check for ephemeral columns before writing a new schema.
    27678b3a19 ovsdb-tool: Fix cluster-to-standalone for DB conversion records.
    
    61b39d8c47 ofproto-dpif-xlate: Always mask ip proto field. (#2134873)
    037e2d9161 db-ctl-base: Partially revert b8bf410a5. (#2182767)
    5fe322e169 fatal-signal: Don't share signal fds/handles with forked process.
    3fcb817840 cpu: Fix cpuid check for some AMD processors. (#2211747)
    01f0668fdf tc: Fix crash on malformed reply from kernel.
    e913394054 ovs-fields: Modify the width of tpa and spa.
    087439e416 ofproto-dpif-xlate: Fix use-after-free when xlate_actions().
    c6cb828870 tc: Fix cleaning chains.
    615548e532 dpif-netlink: Fix memory leak dpif_netlink_open().
    42edc9a1d5 ofp-parse: Check ranges on string to uint32_t conversion.
    d3a479c4b4 learning-switch: Fix coredump of OpenFlow15 learning-switch.
    694c7b4e0 classifier: Fix missing masks on a final stage with ports trie.
    87249e136 ofproto: Fix re-creation of tunnel backing interfaces on restart.

@dcbw dcbw changed the title Dockerfile: bump to ovn23.03-23.03.0-69 for LB templates [release-4.13] Dockerfile: bump to ovn23.03-23.03.0-69 for LB templates Jun 14, 2023
@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jun 14, 2023
@dcbw dcbw force-pushed the 413-ovn-23.03-69-lb-templates branch from 66ff99e to 327a50e Compare June 15, 2023 12:16
@dcbw dcbw mentioned this pull request Jun 15, 2023
Merged
@dcbw dcbw changed the title [release-4.13] Dockerfile: bump to ovn23.03-23.03.0-69 for LB templates [release-4.13] OCPBUGS-15128: Dockerfile: bump to ovn23.03-23.03.0-69 for LB templates Jun 16, 2023
@dcbw
Copy link
Contributor Author

dcbw commented Jun 16, 2023

/retest-required

Windows jobs failed on deprovision, not during the tests.

@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Jun 16, 2023
@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Jun 16, 2023

@dcbw: This pull request references Jira Issue OCPBUGS-15128, which is invalid:

  • expected the bug to target the "4.13.z" version, but it targets "4.14.0" instead
  • expected dependent Jira Issue OCPBUGS-15127 to be in one of the following states: VERIFIED, RELEASE PENDING, CLOSED (ERRATA), CLOSED (CURRENT RELEASE), CLOSED (DONE), but it is POST instead

Comment /jira refresh to re-evaluate validity if changes to the Jira bug are made, or edit the title of this pull request to link to a different bug.

The bug has been updated to refer to the pull request using the external bug tracker.

Details

In response to this:

We need at >= 23.03-14 for LB templates; bump to latest to get other fixes too.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@dcbw
Copy link
Contributor Author

dcbw commented Jun 16, 2023

/jira refresh

@openshift-ci-robot openshift-ci-robot added the jira/invalid-bug Indicates that a referenced Jira bug is invalid for the branch this PR is targeting. label Jun 16, 2023
@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Jun 16, 2023

@dcbw: This pull request references Jira Issue OCPBUGS-15128, which is invalid:

  • expected dependent Jira Issue OCPBUGS-15127 to be in one of the following states: VERIFIED, RELEASE PENDING, CLOSED (ERRATA), CLOSED (CURRENT RELEASE), CLOSED (DONE), but it is POST instead

Comment /jira refresh to re-evaluate validity if changes to the Jira bug are made, or edit the title of this pull request to link to a different bug.

Details

In response to this:

/jira refresh

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Jun 16, 2023

@dcbw: This pull request references Jira Issue OCPBUGS-15128, which is invalid:

  • expected dependent Jira Issue OCPBUGS-15127 to be in one of the following states: VERIFIED, RELEASE PENDING, CLOSED (ERRATA), CLOSED (CURRENT RELEASE), CLOSED (DONE), but it is POST instead

Comment /jira refresh to re-evaluate validity if changes to the Jira bug are made, or edit the title of this pull request to link to a different bug.

Details

In response to this:

We need at >= 23.03-14 for LB templates; bump to latest to get other fixes too.

   OVN changes 49 -> 69:
   - Implement MTU Path Discovery for multichassis ports
   - northd: Add logical flow to skip GARP with LLA (#2211240)
   - northd: match only on supported protocols to handle_svc_check (#1913162)
   - northd: Fix address set incremental processing (#2196885)
   - controller: Ignore DNS queries with RRs
   - controller: Handle OpenFlow errors. (#2134880)
   - northd: fix ls_in_hairpin l3dgw flow generation
   - ovn-controller.c: Fix assertion failure during address set update.
   - ovn-controller: fixed port not always set down when unbinding interface (#2150905)
   - ovn-controller: fixed ovn-installed not always properly added or removed.
   - northd: fix use-after-free after lrp destroyed
   - Omit ACLs for nd || nd_ra || nd_rs || mldv1 || mldv2 (#2149731)
   - controller: Prevent race in tunnel cleanup
   - northd.c: Avoid sending ICMP time exceeded for multicast packets.
   - northd.c: TTL discard flow should support for both ipv4 and ipv6.
   - northd: Update the is_stateless helper for router nat
   - northd: revert ct.inv drop flows
   - northd: take into account qos_min_rate in port_has_qos_params
   - northd: prevents sending packet to conntrack for router ports (#2062431)
   - lb: Allow IPv6 template LBs to use explicit backends.
   - controller: lflow: do not use tcp as default IP protocol for ct_snat_to_vip action (#2157846)
   - northd: Drop packets for LBs with no backends (#2177173)
   - northd: Use generic ct.est flows for LR LBs (#2172048 2170885)
   - northd: drop ct.inv packets in post snat and lb_aff_learn stages (#2160685)
   - controller: Add config option per LB to enable/disable CT flush (#2178962)

   OVS changes 2 -> 32:
   859071224c ovsdb: monitor: Destroy initial change set when new columns added.
   54e45e3fee ovsdb: Monitor: Keep and maintain the initial change set.
   9529e9aa96 ovsdb: Allow conversion records with no data in a clustered storage.
   7006bb112b ovsdb: Check for ephemeral columns before writing a new schema.
   27678b3a19 ovsdb-tool: Fix cluster-to-standalone for DB conversion records.
   
   61b39d8c47 ofproto-dpif-xlate: Always mask ip proto field. (#2134873)
   037e2d9161 db-ctl-base: Partially revert b8bf410a5. (#2182767)
   5fe322e169 fatal-signal: Don't share signal fds/handles with forked process.
   3fcb817840 cpu: Fix cpuid check for some AMD processors. (#2211747)
   01f0668fdf tc: Fix crash on malformed reply from kernel.
   e913394054 ovs-fields: Modify the width of tpa and spa.
   087439e416 ofproto-dpif-xlate: Fix use-after-free when xlate_actions().
   c6cb828870 tc: Fix cleaning chains.
   615548e532 dpif-netlink: Fix memory leak dpif_netlink_open().
   42edc9a1d5 ofp-parse: Check ranges on string to uint32_t conversion.
   d3a479c4b4 learning-switch: Fix coredump of OpenFlow15 learning-switch.
   694c7b4e0 classifier: Fix missing masks on a final stage with ports trie.
   87249e136 ofproto: Fix re-creation of tunnel backing interfaces on restart.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Jun 16, 2023

@dcbw: This pull request references Jira Issue OCPBUGS-15128, which is invalid:

  • expected dependent Jira Issue OCPBUGS-15127 to be in one of the following states: VERIFIED, RELEASE PENDING, CLOSED (ERRATA), CLOSED (CURRENT RELEASE), CLOSED (DONE), but it is POST instead

Comment /jira refresh to re-evaluate validity if changes to the Jira bug are made, or edit the title of this pull request to link to a different bug.

Details

In response to this:

We need at >= 23.03-14 for LB templates; bump to latest to get other fixes too.

   OVN changes 7 -> 69:
   - Implement MTU Path Discovery for multichassis ports
   - northd: Add logical flow to skip GARP with LLA (#2211240)
   - northd: match only on supported protocols to handle_svc_check (#1913162)
   - northd: Fix address set incremental processing (#2196885)
   - controller: Ignore DNS queries with RRs
   - controller: Handle OpenFlow errors. (#2134880)
   - northd: fix ls_in_hairpin l3dgw flow generation
   - ovn-controller.c: Fix assertion failure during address set update.
   - ovn-controller: fixed port not always set down when unbinding interface (#2150905)
   - ovn-controller: fixed ovn-installed not always properly added or removed.
   - northd: fix use-after-free after lrp destroyed
   - Omit ACLs for nd || nd_ra || nd_rs || mldv1 || mldv2 (#2149731)
   - controller: Prevent race in tunnel cleanup
   - northd.c: Avoid sending ICMP time exceeded for multicast packets.
   - northd.c: TTL discard flow should support for both ipv4 and ipv6.
   - northd: Update the is_stateless helper for router nat
   - northd: revert ct.inv drop flows
   - northd: take into account qos_min_rate in port_has_qos_params
   - northd: prevents sending packet to conntrack for router ports (#2062431)
   - lb: Allow IPv6 template LBs to use explicit backends.
   - controller: lflow: do not use tcp as default IP protocol for ct_snat_to_vip action (#2157846)
   - northd: Drop packets for LBs with no backends (#2177173)
   - northd: Use generic ct.est flows for LR LBs (#2172048 2170885)
   - northd: drop ct.inv packets in post snat and lb_aff_learn stages (#2160685)
   - controller: Add config option per LB to enable/disable CT flush (#2178962)

   OVS changes 2 -> 32:
   859071224c ovsdb: monitor: Destroy initial change set when new columns added.
   54e45e3fee ovsdb: Monitor: Keep and maintain the initial change set.
   9529e9aa96 ovsdb: Allow conversion records with no data in a clustered storage.
   7006bb112b ovsdb: Check for ephemeral columns before writing a new schema.
   27678b3a19 ovsdb-tool: Fix cluster-to-standalone for DB conversion records.
   
   61b39d8c47 ofproto-dpif-xlate: Always mask ip proto field. (#2134873)
   037e2d9161 db-ctl-base: Partially revert b8bf410a5. (#2182767)
   5fe322e169 fatal-signal: Don't share signal fds/handles with forked process.
   3fcb817840 cpu: Fix cpuid check for some AMD processors. (#2211747)
   01f0668fdf tc: Fix crash on malformed reply from kernel.
   e913394054 ovs-fields: Modify the width of tpa and spa.
   087439e416 ofproto-dpif-xlate: Fix use-after-free when xlate_actions().
   c6cb828870 tc: Fix cleaning chains.
   615548e532 dpif-netlink: Fix memory leak dpif_netlink_open().
   42edc9a1d5 ofp-parse: Check ranges on string to uint32_t conversion.
   d3a479c4b4 learning-switch: Fix coredump of OpenFlow15 learning-switch.
   694c7b4e0 classifier: Fix missing masks on a final stage with ports trie.
   87249e136 ofproto: Fix re-creation of tunnel backing interfaces on restart.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Jun 16, 2023

@dcbw: This pull request references Jira Issue OCPBUGS-15128, which is invalid:

  • expected dependent Jira Issue OCPBUGS-15127 to be in one of the following states: VERIFIED, RELEASE PENDING, CLOSED (ERRATA), CLOSED (CURRENT RELEASE), CLOSED (DONE), but it is POST instead

Comment /jira refresh to re-evaluate validity if changes to the Jira bug are made, or edit the title of this pull request to link to a different bug.

Details

In response to this:

We need at >= 23.03-14 for LB templates; bump to latest to get other fixes too.

OVN changes 7 -> 69:
   - Implement MTU Path Discovery for multichassis ports
   - northd: Add logical flow to skip GARP with LLA (#2211240)
   - northd: match only on supported protocols to handle_svc_check (#1913162)
   - northd: Fix address set incremental processing (#2196885)
   - controller: Ignore DNS queries with RRs
   - controller: Handle OpenFlow errors. (#2134880)
   - northd: fix ls_in_hairpin l3dgw flow generation
   - ovn-controller.c: Fix assertion failure during address set update.
   - ovn-controller: fixed port not always set down when unbinding interface (#2150905)
   - ovn-controller: fixed ovn-installed not always properly added or removed.
   - northd: fix use-after-free after lrp destroyed
   - Omit ACLs for nd || nd_ra || nd_rs || mldv1 || mldv2 (#2149731)
   - controller: Prevent race in tunnel cleanup
   - northd.c: Avoid sending ICMP time exceeded for multicast packets.
   - northd.c: TTL discard flow should support for both ipv4 and ipv6.
   - northd: Update the is_stateless helper for router nat
   - northd: revert ct.inv drop flows
   - northd: take into account qos_min_rate in port_has_qos_params
   - northd: prevents sending packet to conntrack for router ports (#2062431)
   - lb: Allow IPv6 template LBs to use explicit backends.
   - controller: lflow: do not use tcp as default IP protocol for ct_snat_to_vip action (#2157846)
   - northd: Drop packets for LBs with no backends (#2177173)
   - northd: Use generic ct.est flows for LR LBs (#2172048 2170885)
   - northd: drop ct.inv packets in post snat and lb_aff_learn stages (#2160685)
   - controller: Add config option per LB to enable/disable CT flush (#2178962)

OVS changes 2 -> 32:
   859071224c ovsdb: monitor: Destroy initial change set when new columns added.
   54e45e3fee ovsdb: Monitor: Keep and maintain the initial change set.
   9529e9aa96 ovsdb: Allow conversion records with no data in a clustered storage.
   7006bb112b ovsdb: Check for ephemeral columns before writing a new schema.
   27678b3a19 ovsdb-tool: Fix cluster-to-standalone for DB conversion records.
   
   61b39d8c47 ofproto-dpif-xlate: Always mask ip proto field. (#2134873)
   037e2d9161 db-ctl-base: Partially revert b8bf410a5. (#2182767)
   5fe322e169 fatal-signal: Don't share signal fds/handles with forked process.
   3fcb817840 cpu: Fix cpuid check for some AMD processors. (#2211747)
   01f0668fdf tc: Fix crash on malformed reply from kernel.
   e913394054 ovs-fields: Modify the width of tpa and spa.
   087439e416 ofproto-dpif-xlate: Fix use-after-free when xlate_actions().
   c6cb828870 tc: Fix cleaning chains.
   615548e532 dpif-netlink: Fix memory leak dpif_netlink_open().
   42edc9a1d5 ofp-parse: Check ranges on string to uint32_t conversion.
   d3a479c4b4 learning-switch: Fix coredump of OpenFlow15 learning-switch.
   694c7b4e0 classifier: Fix missing masks on a final stage with ports trie.
   87249e136 ofproto: Fix re-creation of tunnel backing interfaces on restart.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jun 16, 2023

@dcbw: all tests passed!

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@jcaamano
Copy link
Contributor

jcaamano commented Jun 19, 2023

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jun 19, 2023
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jun 19, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dcbw, jcaamano

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

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@dceara dceara mentioned this pull request Jun 23, 2023
Merged
@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jul 4, 2023
@openshift-merge-robot
Copy link
Contributor

openshift-merge-robot commented Jul 4, 2023

PR needs rebase.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@dcbw dcbw closed this Jul 5, 2023
@openshift-ci-robot
Copy link
Contributor

openshift-ci-robot commented Jul 5, 2023

@dcbw: This pull request references Jira Issue OCPBUGS-15128. The bug has been updated to no longer refer to the pull request using the external bug tracker.

Details

In response to this:

We need at >= 23.03-14 for LB templates; bump to latest to get other fixes too.

OVN changes 7 -> 69:
   - Implement MTU Path Discovery for multichassis ports
   - northd: Add logical flow to skip GARP with LLA (#2211240)
   - northd: match only on supported protocols to handle_svc_check (#1913162)
   - northd: Fix address set incremental processing (#2196885)
   - controller: Ignore DNS queries with RRs
   - controller: Handle OpenFlow errors. (#2134880)
   - northd: fix ls_in_hairpin l3dgw flow generation
   - ovn-controller.c: Fix assertion failure during address set update.
   - ovn-controller: fixed port not always set down when unbinding interface (#2150905)
   - ovn-controller: fixed ovn-installed not always properly added or removed.
   - northd: fix use-after-free after lrp destroyed
   - Omit ACLs for nd || nd_ra || nd_rs || mldv1 || mldv2 (#2149731)
   - controller: Prevent race in tunnel cleanup
   - northd.c: Avoid sending ICMP time exceeded for multicast packets.
   - northd.c: TTL discard flow should support for both ipv4 and ipv6.
   - northd: Update the is_stateless helper for router nat
   - northd: revert ct.inv drop flows
   - northd: take into account qos_min_rate in port_has_qos_params
   - northd: prevents sending packet to conntrack for router ports (#2062431)
   - lb: Allow IPv6 template LBs to use explicit backends.
   - controller: lflow: do not use tcp as default IP protocol for ct_snat_to_vip action (#2157846)
   - northd: Drop packets for LBs with no backends (#2177173)
   - northd: Use generic ct.est flows for LR LBs (#2172048 2170885)
   - northd: drop ct.inv packets in post snat and lb_aff_learn stages (#2160685)
   - controller: Add config option per LB to enable/disable CT flush (#2178962)

OVS changes 2 -> 32:
   859071224c ovsdb: monitor: Destroy initial change set when new columns added.
   54e45e3fee ovsdb: Monitor: Keep and maintain the initial change set.
   9529e9aa96 ovsdb: Allow conversion records with no data in a clustered storage.
   7006bb112b ovsdb: Check for ephemeral columns before writing a new schema.
   27678b3a19 ovsdb-tool: Fix cluster-to-standalone for DB conversion records.
   
   61b39d8c47 ofproto-dpif-xlate: Always mask ip proto field. (#2134873)
   037e2d9161 db-ctl-base: Partially revert b8bf410a5. (#2182767)
   5fe322e169 fatal-signal: Don't share signal fds/handles with forked process.
   3fcb817840 cpu: Fix cpuid check for some AMD processors. (#2211747)
   01f0668fdf tc: Fix crash on malformed reply from kernel.
   e913394054 ovs-fields: Modify the width of tpa and spa.
   087439e416 ofproto-dpif-xlate: Fix use-after-free when xlate_actions().
   c6cb828870 tc: Fix cleaning chains.
   615548e532 dpif-netlink: Fix memory leak dpif_netlink_open().
   42edc9a1d5 ofp-parse: Check ranges on string to uint32_t conversion.
   d3a479c4b4 learning-switch: Fix coredump of OpenFlow15 learning-switch.
   694c7b4e0 classifier: Fix missing masks on a final stage with ports trie.
   87249e136 ofproto: Fix re-creation of tunnel backing interfaces on restart.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@JacobTanenbaum JacobTanenbaum Awaiting requested review from JacobTanenbaum

@jcaamano jcaamano Awaiting requested review from jcaamano

Assignees

@jcaamano jcaamano

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. jira/invalid-bug Indicates that a referenced Jira bug is invalid for the branch this PR is targeting. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. lgtm Indicates that a PR is ready to be merged. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@dcbw @openshift-ci-robot @jcaamano @openshift-merge-robot