[ovs-git] [openvswitch/ovs] 05ba45: ovsdb-idlc: Make set and map update operations tak...

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/openvswitch/ovs
  Commit: 05ba459ff81e54743d72c2e056804cdbc8efbea7
      https://github.com/openvswitch/ovs/commit/05ba459ff81e54743d72c2e056804cdbc8efbea7
  Author: Ben Pfaff <blp at ovn.org>
  Date:   2016-08-31 (Wed, 31 Aug 2016)

  Changed paths:
    M ovsdb/ovsdb-idlc.in
    M python/ovs/db/types.py

  Log Message:
  -----------
  ovsdb-idlc: Make set and map update operations take const arguments.

In a call like "ovsrec_bridge_update_ports_delvalue(bridge, port)", there's
no reason for the port argument to be nonconst, because the call doesn't
do anything to the port at all--it only searches the list of ports in the
bridge for that particular port and, if it finds it, removes it.

Signed-off-by: Ben Pfaff <blp at ovn.org>
Acked-by: Russell Bryant <russell at ovn.org>


  Commit: 19472b7d8e030d440d60d03bcc38b17c2810ab01
      https://github.com/openvswitch/ovs/commit/19472b7d8e030d440d60d03bcc38b17c2810ab01
  Author: Ben Pfaff <blp at ovn.org>
  Date:   2016-08-31 (Wed, 31 Aug 2016)

  Changed paths:
    M ovn/controller/encaps.c

  Log Message:
  -----------
  ovn-controller: Drop incremental processing from encapsulation code.

This commit reverts encaps.c to its content just before commit 1d45d5a9666d
(ovn-controller: Change encaps_run to work incrementally.).  I then
reintroduced the UDP checksum support originallly added in commit
36283d7884f3 (ovn-controller: Use UDP checksums when creating Geneve
tunnels.)  I also read the other commits following the incremental
processing commit to verify that this change didn't lose any bug fixes.

This commit takes advantage of the "addvalue" and "delvalue" functions
now available in the IDL to simplify some code.

Signed-off-by: Ben Pfaff <blp at ovn.org>
Acked-by: Russell Bryant <russell at ovn.org>


Compare: https://github.com/openvswitch/ovs/compare/23c16b512473...19472b7d8e03