[ovs-build] Passed: dceara/ovs#101 (fix-idl-expected-cond-seqno-branch-2.14 - ec9bcfb)
Travis CI
builds at travis-ci.org
Mon May 17 09:17:10 UTC 2021
Build Update for dceara/ovs
-------------------------------------
Build: #101
Status: Passed
Duration: 15 mins and 26 secs
Commit: ec9bcfb (fix-idl-expected-cond-seqno-branch-2.14)
Author: Dumitru Ceara
Message: ovsdb-idl: Consider all tables when computing expected cond seqno.
In ovsdb_idl_db_set_condition(), take into account all pending
condition changes for all tables when computing the db->cond_seqno at
which the monitor is expected to be updated.
In the following scenario, with two tables, A and B, the old code
performed the following steps:
1. Initial db->cond_seqno = X.
2. Client changes condition for table A:
- A->new_cond gets set
- expected cond seqno returned to the client: X + 1
3. ovsdb-idl sends the monitor_cond_change for table A
- A->req_cond <- A->new_cond
4. Client changes condition for table B:
- B->new_cond gets set
- expected cond seqno returned to the client: X + 1
- however, because the condition change at step 3 is still not replied
to, table B's monitor_cond_change request is not sent yet.
5. ovsdb-idl receives the reply for the condition change at step 3:
- db->cond_seqno <- X + 1
6. ovsdb-idl sends the monitor_cond_change for table B
7. ovsdb-idl receives the reply for the condition change at step 6:
- db->cond_seqno <- X + 2
The client was incorrectly informed that it will have all relevant
updates for table B at seqno X + 1 while actually that happens later, at
seqno X + 2.
Fixes: 46437c5232bd ("ovsdb-idl: Enhance conditional monitoring API")
Acked-by: Ben Pfaff <blp at ovn.org>
Signed-off-by: Dumitru Ceara <dceara at redhat.com>
(cherry picked from commit b5bb044fbe4c1395dcde5cc7d5081ef0099bb8b3)
View the changeset: https://github.com/dceara/ovs/compare/224f4ba6a814^...ec9bcfb1b7de
View the full build log and details: https://travis-ci.org/github/dceara/ovs/builds/771402034?utm_medium=notification&utm_source=email
--
You can unsubscribe from build emails from the dceara/ovs repository going to https://travis-ci.org/account/preferences/unsubscribe?repository=25057358&utm_medium=notification&utm_source=email.
Or unsubscribe from *all* email updating your settings at https://travis-ci.org/account/preferences/unsubscribe?utm_medium=notification&utm_source=email.
Or configure specific recipients for build notifications in your .travis.yml file. See https://docs.travis-ci.com/user/notifications.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openvswitch.org/pipermail/ovs-build/attachments/20210517/518a0555/attachment.html>
More information about the build
mailing list