This code does some checking to validate the existing encaps for a
chassis to see if they need to be updated. This typo resulted in
ovn-controller re-creating its encap(s) every time this code ran, making
ovn-controller and ovsdb-server eat up a CPU in my testing.
Signed-off-by: Russell Bryant <rbryant@redhat.com>
Acked-by: Ben Pfaff <blp@nicira.com>
bool same = true;
for (int i = 0; i < chassis_rec->n_encaps; i++) {
cur_tunnels |= get_tunnel_type(chassis_rec->encaps[i]->type);
- same = same && strcmp(chassis_rec->encaps[i]->ip, encap_ip);
+ same = same && !strcmp(chassis_rec->encaps[i]->ip, encap_ip);
}
same = same && req_tunnels == cur_tunnels;