[ovs-git] [ovn-org/ovn] d4d6ac: lb: Remove hairpin_snat_ips from northd load balan...

Dumitru Ceara noreply at github.com
Tue Jun 15 17:40:21 UTC 2021


  Branch: refs/heads/master
  Home:   https://github.com/ovn-org/ovn
  Commit: d4d6acae708f41274d3e36eec5d425f1b03093ca
      https://github.com/ovn-org/ovn/commit/d4d6acae708f41274d3e36eec5d425f1b03093ca
  Author: Dumitru Ceara <dceara at redhat.com>
  Date:   2021-06-15 (Tue, 15 Jun 2021)

  Changed paths:
    M lib/lb.c
    M lib/lb.h

  Log Message:
  -----------
  lb: Remove hairpin_snat_ips from northd load balancers.

The parsed values are not used anywhere so we might as well avoid
parsing them.

Fixes: cc4d5520064f ("Support configuring Load Balancer hairpin source IP.")
Signed-off-by: Dumitru Ceara <dceara at redhat.com>
Signed-off-by: Numan Siddique <numans at ovn.org>


  Commit: c92cef33e02d8f39e6107b16c07eaa040105532b
      https://github.com/ovn-org/ovn/commit/c92cef33e02d8f39e6107b16c07eaa040105532b
  Author: Dumitru Ceara <dceara at redhat.com>
  Date:   2021-06-15 (Tue, 15 Jun 2021)

  Changed paths:
    M lib/lb.c
    M lib/lb.h
    M northd/ovn-northd.c

  Log Message:
  -----------
  northd: Consolidate load balancer healthcheck/svc code.

Part of the load balancer healthcheck/svc configuration parsing was
done in ovn-northd.c and part in lib/lb.c.  Consolidate this in
ovn-northd.c.  As a side effect this will remove dependency on the
'ports' hmap in lib/lb.c.  This enables future commits to optimize
load balancer configuration parsing.

Signed-off-by: Dumitru Ceara <dceara at redhat.com>
Signed-off-by: Numan Siddique <numans at ovn.org>


  Commit: d27509de940a82ca96f2596ffc0bc2ae2da8b193
      https://github.com/ovn-org/ovn/commit/d27509de940a82ca96f2596ffc0bc2ae2da8b193
  Author: Dumitru Ceara <dceara at redhat.com>
  Date:   2021-06-15 (Tue, 15 Jun 2021)

  Changed paths:
    M lib/lb.c
    M lib/lb.h
    M northd/ovn-northd.c

  Log Message:
  -----------
  northd: Precompute load balancer IP sets.

There's no need to parse the IP sets every time we iterate through them.
It's enough to parse them once for every main loop iteration.

Reported-at: https://bugzilla.redhat.com/1962338
Signed-off-by: Dumitru Ceara <dceara at redhat.com>
Signed-off-by: Numan Siddique <numans at ovn.org>


Compare: https://github.com/ovn-org/ovn/compare/d299a255823a...d27509de940a


More information about the git mailing list