Message ID | 20210517085718.5561-1-dceara@redhat.com |
---|---|
State | Changes Requested |
Headers | show |
Series | [ovs-dev] ovs: Include monitor condition expected seqno fix. | expand |
Hi Dumitru, Can you please explain in the commit message why this change is needed? I don't have anything against bumping the OVS submodule commit, but it's hard to review this change without knowing any context. Thanks, Mark On 5/17/21 4:57 AM, Dumitru Ceara wrote: > Specifically: > b5bb044fbe4c ("ovsdb-cs: Consider all tables when computing expected cond seqno.") > > Signed-off-by: Dumitru Ceara <dceara@redhat.com> > --- > ovs | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/ovs b/ovs > index ac85cdb38..b5bb044fb 160000 > --- a/ovs > +++ b/ovs > @@ -1 +1 @@ > -Subproject commit ac85cdb38c1f33e7952bc4c0347d6c7873fb56a1 > +Subproject commit b5bb044fbe4c1395dcde5cc7d5081ef0099bb8b3 >
On 5/17/21 9:19 PM, Mark Michelson wrote: > Hi Dumitru, > > Can you please explain in the commit message why this change is needed? > I don't have anything against bumping the OVS submodule commit, but it's > hard to review this change without knowing any context. > > Thanks, > Mark Hi Mark, Thanks for the review, I sent a v2: http://patchwork.ozlabs.org/project/ovn/patch/20210518064932.23852-1-dceara@redhat.com/ Thanks, Dumitru
diff --git a/ovs b/ovs index ac85cdb38..b5bb044fb 160000 --- a/ovs +++ b/ovs @@ -1 +1 @@ -Subproject commit ac85cdb38c1f33e7952bc4c0347d6c7873fb56a1 +Subproject commit b5bb044fbe4c1395dcde5cc7d5081ef0099bb8b3
Specifically: b5bb044fbe4c ("ovsdb-cs: Consider all tables when computing expected cond seqno.") Signed-off-by: Dumitru Ceara <dceara@redhat.com> --- ovs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)