Skip to content

Commit

Permalink
libovsdb: give monitor setup time to process than normal transactions
Browse files Browse the repository at this point in the history
In certain scaling situations, it may take longer than types.OVSDBTimeout for the
monitoring of OVSDB tables to be set up. This change is a follow-up to commit
da9408d, accounting for cases where ovnkube-master
has a lot of startup data. The timeout for monitoring introduced in commit
6ae4edb did not account for that.

Reported-at: https://issues.redhat.com/browse/OCPBUGS-17147
Signed-off-by: Flavio Fernandes <[email protected]>
  • Loading branch information
flavio-fernandes authored and jcaamano committed Aug 4, 2023
1 parent 7c74ebd commit 1cb56c3
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions go-controller/pkg/libovsdb/libovsdb.go
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,7 @@ func NewSBClientWithConfig(cfg config.OvnAuthConfig, promRegistry prometheus.Reg
return nil, err
}

ctx, cancel := context.WithTimeout(context.Background(), types.OVSDBTimeout)
ctx, cancel := context.WithTimeout(context.Background(), types.OVSDBTimeout*2)
go func() {
<-stopCh
cancel()
Expand Down Expand Up @@ -166,7 +166,7 @@ func NewNBClientWithConfig(cfg config.OvnAuthConfig, promRegistry prometheus.Reg
return nil, err
}

ctx, cancel := context.WithTimeout(context.Background(), types.OVSDBTimeout)
ctx, cancel := context.WithTimeout(context.Background(), types.OVSDBTimeout*2)
go func() {
<-stopCh
cancel()
Expand Down

0 comments on commit 1cb56c3

Please sign in to comment.