Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[DownstreamMerge] 8-25-2022 #1253

Merged

Commits on Aug 10, 2022

  1. Bump libovsdb to include ovn-org/libovsdb#321

    Signed-off-by: Zenghui Shi <zshi@redhat.com>
    zshi-redhat committed Aug 10, 2022
    Configuration menu
    Copy the full SHA
    77d6f0f View commit details
    Browse the repository at this point in the history
  2. Remove stale chassis for hosts that run ovnkube-node on DPU

    In the DPU 2-clusters design, a host with DPU will join the cluster
    with ovn-controller running in the host first, then it will be moved
    to the DPU. Therefore, we will have 2 chassis with the same hostname
    in the ovn sbdb.
    
    With this patch, the master will try to remove the stale chassis when
    the Chassis ID is inconsistent with the value of node annotation
    'k8s.ovn.org/node-chassis-id'.
    
    Signed-off-by: Peng Liu <pliu@redhat.com>
    pliurh committed Aug 10, 2022
    Configuration menu
    Copy the full SHA
    dc4c190 View commit details
    Browse the repository at this point in the history

Commits on Aug 15, 2022

  1. Configuration menu
    Copy the full SHA
    031842e View commit details
    Browse the repository at this point in the history

Commits on Aug 16, 2022

  1. fedora Dockerfile : Switch to f36.

    Signed-off-by: Numan Siddique <numans@ovn.org>
    numansiddique committed Aug 16, 2022
    Configuration menu
    Copy the full SHA
    db2b713 View commit details
    Browse the repository at this point in the history
  2. OVN Metrics: Ensure stopwatch metrics are reported in seconds

    Currently, they are reported in milliseconds.
    Prometheus mandates time values are reported in seconds.
    
    Signed-off-by: Martin Kennelly <mkennell@redhat.com>
    martinkennelly committed Aug 16, 2022
    Configuration menu
    Copy the full SHA
    4baab3d View commit details
    Browse the repository at this point in the history
  3. Merge pull request #3118 from martinkennelly/fix_metric_time_scale

    OVN-K Metrics: Ensure stopwatch metrics are reported in seconds
    jcaamano committed Aug 16, 2022
    Configuration menu
    Copy the full SHA
    d4b5f1a View commit details
    Browse the repository at this point in the history

Commits on Aug 17, 2022

  1. Add OVN controller southbound database connection metric

    Report OVN controller connection status via a metric from each
    OVN kube node.
    
    The metric is updated every 2 minutes in-order not to incur
    polling overhead which is not insignificant.
    
    Prometheus scrape interval is usually every 30 seconds, and is
    this value on OCP.
    
    The reason we report this from every node instead of grabbing it
    from SB DB via (nb_cfg trigger), is to save SB DB from each
    OVN controller having to write to it.
    This maybe an issue when scaling node.
    
    Signed-off-by: Martin Kennelly <mkennell@redhat.com>
    martinkennelly committed Aug 17, 2022
    Configuration menu
    Copy the full SHA
    25e5700 View commit details
    Browse the repository at this point in the history
  2. OVN-K Metric: update metric docs to include ovn-c sbdb connection metric

    Signed-off-by: Martin Kennelly <mkennell@redhat.com>
    martinkennelly committed Aug 17, 2022
    Configuration menu
    Copy the full SHA
    64a5aea View commit details
    Browse the repository at this point in the history
  3. egressip: add metrics

    Add metrics to observe core events related to EgressIP:
    
    - name: egress_ips_assign_latency_seconds -- Histogram **
      desc: The latency of egress IP assignment to ovn nb database
    
    - name: egress_ips_unassign_latency_seconds -- Histogram **
      desc: The latency of egress IP unassignment from ovn nb database
    
    - name: egress_ips_node_unreachable_total -- Counter
      desc: The total number of times assigned egress IP(s) were unreachable
    
    - name: egress_ips_rebalance_total -- Counter
      desc: The total number of times assigned egress IP(s) needed to be moved to a different node
    
    Add flags to explicitly enable the histogram metrics, since we only see value
    in having them when scale testing egress ips. The flag introduced here is:
        --metrics-enable-eip-scale
    
    Signed-off-by: Flavio Fernandes <flaviof@redhat.com>
    flavio-fernandes committed Aug 17, 2022
    Configuration menu
    Copy the full SHA
    31beab4 View commit details
    Browse the repository at this point in the history

Commits on Aug 18, 2022

  1. Revert "Followup to EndpointSlices PR for ovn-k node"

    This reverts commit 503de2f.
    
    Signed-off-by: Riccardo Ravaioli <rravaiol@redhat.com>
    ricky-rav committed Aug 18, 2022
    Configuration menu
    Copy the full SHA
    fa09172 View commit details
    Browse the repository at this point in the history
  2. Revert "Remove endpoint support & add endpointSlices support for ovnk…

    …ube-node"
    
    This reverts commit 1a225ea.
    
    Signed-off-by: Riccardo Ravaioli <rravaiol@redhat.com>
    ricky-rav committed Aug 18, 2022
    Configuration menu
    Copy the full SHA
    a9b5ad9 View commit details
    Browse the repository at this point in the history
  3. Merge pull request #3117 from martinkennelly/add_ovn_controller_sbdb_…

    …metric
    
    OVN-K metrics: Add OVN controller southbound database connection
    jcaamano committed Aug 18, 2022
    Configuration menu
    Copy the full SHA
    6b8acf0 View commit details
    Browse the repository at this point in the history
  4. Bump @actions/core in /.github/actions/cleanup-action

    Bumps [@actions/core](https://github.com/actions/toolkit/tree/HEAD/packages/core) from 1.2.6 to 1.9.1.
    - [Release notes](https://github.com/actions/toolkit/releases)
    - [Changelog](https://github.com/actions/toolkit/blob/main/packages/core/RELEASES.md)
    - [Commits](https://github.com/actions/toolkit/commits/HEAD/packages/core)
    
    ---
    updated-dependencies:
    - dependency-name: "@actions/core"
      dependency-type: direct:production
    ...
    
    Signed-off-by: dependabot[bot] <support@github.com>
    dependabot[bot] committed Aug 18, 2022
    Configuration menu
    Copy the full SHA
    072fbb3 View commit details
    Browse the repository at this point in the history
  5. Merge pull request #3098 from pliurh/remove_stale_node

    Remove stale chassis for hosts that run ovnkube-node on DPU
    trozet committed Aug 18, 2022
    Configuration menu
    Copy the full SHA
    f78409c View commit details
    Browse the repository at this point in the history
  6. Merge pull request #3122 from ricky-rav/revert_epslice

    Revert EndpointSlice commits
    trozet committed Aug 18, 2022
    Configuration menu
    Copy the full SHA
    5360fae View commit details
    Browse the repository at this point in the history

Commits on Aug 19, 2022

  1. Delete stale egress ip before assigning new ip

    This deletes any stale ip from node allocations map before assignment
    so that update on same egress ip object would assign new ip address
    on a particular assignable node in any timing scenarios.
    
    Signed-off-by: Periyasamy Palanisamy <pepalani@redhat.com>
    pperiyasamy committed Aug 19, 2022
    Configuration menu
    Copy the full SHA
    0a55854 View commit details
    Browse the repository at this point in the history
  2. Bump libovsdb to include ovn-org/libovsdb#325

    Signed-off-by: Nadia Pinaeva <npinaeva@redhat.com>
    npinaeva committed Aug 19, 2022
    Configuration menu
    Copy the full SHA
    4c26b7c View commit details
    Browse the repository at this point in the history
  3. Merge pull request #3075 from numansiddique/f36

    fedora Dockerfile : Switch to f36 and OVN 22.06.
    trozet committed Aug 19, 2022
    Configuration menu
    Copy the full SHA
    000511d View commit details
    Browse the repository at this point in the history
  4. Merge pull request #3124 from ovn-org/dependabot/npm_and_yarn/dot-git…

    …hub/actions/cleanup-action/actions/core-1.9.1
    
    Bump @actions/core from 1.2.6 to 1.9.1 in /.github/actions/cleanup-action
    trozet committed Aug 19, 2022
    Configuration menu
    Copy the full SHA
    8d83b81 View commit details
    Browse the repository at this point in the history
  5. Merge pull request #3058 from flavio-fernandes/egressip_metrics

    egressip: add metrics
    trozet committed Aug 19, 2022
    Configuration menu
    Copy the full SHA
    550a7cb View commit details
    Browse the repository at this point in the history

Commits on Aug 22, 2022

  1. Remove default ACL severity (was used before as a workaround for

    libovsdb not being able to reset ACl severity to nil), use nil to reset.
    Unify ACL logging setup (ovn/acl.go) to make ACL severity properly
    set and reset (since severity is *string, can be error-prone)
    Add unified handling for ACL logging that is always disabled (
    like arpAllowACL)
    
    Signed-off-by: Nadia Pinaeva <npinaeva@redhat.com>
    npinaeva committed Aug 22, 2022
    Configuration menu
    Copy the full SHA
    44ad754 View commit details
    Browse the repository at this point in the history
  2. Use nbdb constants instead of strings.

    Add egress firewall external ID constant.
    
    Signed-off-by: Nadia Pinaeva <npinaeva@redhat.com>
    npinaeva committed Aug 22, 2022
    Configuration menu
    Copy the full SHA
    d574bb2 View commit details
    Browse the repository at this point in the history
  3. Fix updating network policy ACL loglevel for default ACLs:

    namespace update handler will only update them if
    len(nsInfo.networkPolicies) > 0, which is not the case during the first
    ns policy creation - it can lead to default ACLs loglevel not being
    updated.
    
    Signed-off-by: Nadia Pinaeva <npinaeva@redhat.com>
    npinaeva committed Aug 22, 2022
    Configuration menu
    Copy the full SHA
    57b0303 View commit details
    Browse the repository at this point in the history

Commits on Aug 23, 2022

  1. Remove endpoint support & add endpointSlices support for ovnkube-node

    Signed-off-by: Pardhakeswar Pacha <ppacha@nvidia.com>
    (cherry picked from commit 1a225ea)
    Pardhakeswar authored and ricky-rav committed Aug 23, 2022
    Configuration menu
    Copy the full SHA
    d7e76e3 View commit details
    Browse the repository at this point in the history
  2. Followup to EndpointSlices PR for ovn-k node

    Signed-off-by: Riccardo Ravaioli <rravaiol@redhat.com>
    (cherry picked from commit 503de2f)
    ricky-rav committed Aug 23, 2022
    Configuration menu
    Copy the full SHA
    135df9e View commit details
    Browse the repository at this point in the history
  3. Fix informer selector for endpointslices

    The selector in the endpointslice informer now correctly chooses only endpointslices with a non-empty "kubernetes.io/service-name" label and without "service.kubernetes.io/headless" label.
    
    Signed-off-by: Riccardo Ravaioli <rravaiol@redhat.com>
    ricky-rav committed Aug 23, 2022
    Configuration menu
    Copy the full SHA
    2990fc3 View commit details
    Browse the repository at this point in the history
  4. Merge pull request #3093 from npinaeva/acl-loglevel-nodefault

    Remove default ACL severity
    jcaamano committed Aug 23, 2022
    Configuration menu
    Copy the full SHA
    dbf5873 View commit details
    Browse the repository at this point in the history
  5. Merge pull request #3126 from ricky-rav/epslice_fix_upstream

    OCPBUGS-417: Fix informer selector for endpointslices
    trozet committed Aug 23, 2022
    Configuration menu
    Copy the full SHA
    d20913f View commit details
    Browse the repository at this point in the history

Commits on Aug 24, 2022

  1. pods: deleteLogicalPort should not fail when port is already gone

    When the logical port is already deleted, deleteLogicalPort should
    not return an error. Otherwise, delete will be retried up for
    maxFailedAttempts (currently set to 15).
    
    This issue introduced in commit be8786a
    
    Signed-off-by: Flavio Fernandes <flaviof@redhat.com>
    Co-authored-by: Tim Rozet <trozet@redhat.com>
    Reported-at: https://issues.redhat.com/browse/OCPBUGS-469
    flavio-fernandes and trozet committed Aug 24, 2022
    Configuration menu
    Copy the full SHA
    f7fe67d View commit details
    Browse the repository at this point in the history

Commits on Aug 25, 2022

  1. Merge pull request #3129 from flavio-fernandes/deleteLogicalPort_fix

    pods: deleteLogicalPort should not fail when port is already gone
    trozet committed Aug 25, 2022
    Configuration menu
    Copy the full SHA
    0403e1a View commit details
    Browse the repository at this point in the history
  2. Merge pull request #3105 from pperiyasamy/egress-ip-update-assignment

    Delete stale egress ip before assigning new ip
    trozet committed Aug 25, 2022
    Configuration menu
    Copy the full SHA
    a86eaf4 View commit details
    Browse the repository at this point in the history

Commits on Aug 26, 2022

  1. Revert "Revert "Remove endpoint support & add endpointSlices support …

    …for ovnkube-node""
    
    This reverts commit 69942d0.
    ricky-rav committed Aug 26, 2022
    Configuration menu
    Copy the full SHA
    f6a610b View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    51dd9c1 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    826f36f View commit details
    Browse the repository at this point in the history