You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I didn't observe this yet, but based on my recent work in #349, I realize there may be another bug in the plugin refresh. It may be nothing, this issue is more of a reminder to investigate and verify if it is an issue or not, and if so to fix it.
My thought is that I don't recall there being anywhere in the refresh logic that checks whether a plugin is active or not on refresh, and if it was inactive but succeeded refresh grpc requests, change its state back to active. It may be there and I missed it, but worth investigating since now reads/scans are gated on active state to reduce overhead wait time on grpc timeout.
The text was updated successfully, but these errors were encountered:
possible bug, related to #349
I didn't observe this yet, but based on my recent work in #349, I realize there may be another bug in the plugin refresh. It may be nothing, this issue is more of a reminder to investigate and verify if it is an issue or not, and if so to fix it.
My thought is that I don't recall there being anywhere in the refresh logic that checks whether a plugin is active or not on refresh, and if it was inactive but succeeded refresh grpc requests, change its state back to active. It may be there and I missed it, but worth investigating since now reads/scans are gated on active state to reduce overhead wait time on grpc timeout.
The text was updated successfully, but these errors were encountered: