This reverts
f20396e051ea4fd623bfc022cc78d9bd52a850e5
f20396e.. was required to fix the failing ovn-sbctl testsuites.
commit
fb496f92ca1eeead8760b5cdfd857165f64deadf addresses these failing
test cases in a proper way.
With commit
f20396e.. its possible that commit to the southbound db is
delayed. When ovnnb_db_run is called, and if ctx->ovnsb_txn is NULL,
ovnnb_db_run returns immediately without generating the logical flows.
ovnnb_db_run is not called again until the northbound db seqno changes.
Signed-off-by: Numan Siddique <nusiddiq@redhat.com>
Signed-off-by: Russell Bryant <russell@ovn.org>
int
main(int argc, char *argv[])
{
int
main(int argc, char *argv[])
{
- unsigned int ovnnb_seqno, ovnsb_seqno;
int res = EXIT_SUCCESS;
struct unixctl_server *unixctl;
int retval;
int res = EXIT_SUCCESS;
struct unixctl_server *unixctl;
int retval;
add_column_noalert(ovnsb_idl_loop.idl, &sbrec_port_binding_col_mac);
ovsdb_idl_add_column(ovnsb_idl_loop.idl, &sbrec_port_binding_col_chassis);
add_column_noalert(ovnsb_idl_loop.idl, &sbrec_port_binding_col_mac);
ovsdb_idl_add_column(ovnsb_idl_loop.idl, &sbrec_port_binding_col_chassis);
- ovnnb_seqno = ovsdb_idl_get_seqno(ovnnb_idl_loop.idl);
- ovnsb_seqno = ovsdb_idl_get_seqno(ovnsb_idl_loop.idl);
-
/* Main loop. */
exiting = false;
while (!exiting) {
/* Main loop. */
exiting = false;
while (!exiting) {
.ovnsb_txn = ovsdb_idl_loop_run(&ovnsb_idl_loop),
};
.ovnsb_txn = ovsdb_idl_loop_run(&ovnsb_idl_loop),
};
- if (ovnnb_seqno != ovsdb_idl_get_seqno(ctx.ovnnb_idl)) {
- ovnnb_seqno = ovsdb_idl_get_seqno(ctx.ovnnb_idl);
- ovnnb_db_run(&ctx);
- }
- if (ovnsb_seqno != ovsdb_idl_get_seqno(ctx.ovnsb_idl)) {
- ovnsb_seqno = ovsdb_idl_get_seqno(ctx.ovnsb_idl);
- ovnsb_db_run(&ctx);
- }
+ ovnnb_db_run(&ctx);
+ ovnsb_db_run(&ctx);
unixctl_server_run(unixctl);
unixctl_server_wait(unixctl);
unixctl_server_run(unixctl);
unixctl_server_wait(unixctl);