-
Notifications
You must be signed in to change notification settings - Fork 197
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: background start for connector plugins #1315
Conversation
Signed-off-by: Enrique Lacal <enrique.lacal@kaleido.io>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great @EnriqueL8!
Couple of minor suggestions from patterns we've found helpful in other code, and a query on goroutines.
Signed-off-by: Enrique Lacal <enrique.lacal@kaleido.io>
Signed-off-by: Enrique Lacal <enrique.lacal@kaleido.io>
There seems to be flaky test I introduced - will fix that |
Signed-off-by: Enrique Lacal <enrique.lacal@kaleido.io>
Signed-off-by: Enrique Lacal <enrique.lacal@kaleido.io>
Codecov Report
@@ Coverage Diff @@
## main #1315 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 310 310
Lines 20960 21040 +80
=========================================
+ Hits 20960 21040 +80
|
Signed-off-by: Enrique Lacal <enrique.lacal@kaleido.io>
e.backgroundStart = e.ethconnectConf.GetBool(EthconnectBackgroundStart) | ||
if e.backgroundStart { | ||
e.backgroundRetry = &retry.Retry{ | ||
InitialDelay: fftypes.ParseToDuration(defaultBackgroundInitialDelay), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think these should be config settings @EnriqueL8
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay, I did think about it but wasn't 100% sure :) - will do it!
Signed-off-by: Enrique Lacal <enrique.lacal@kaleido.io>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
No description provided.