-
Notifications
You must be signed in to change notification settings - Fork 2
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
chore(loopbackRpcIssuer): reduced random wait #161
Conversation
Coverage Report
File Coverage
|
38c1e07
to
f05e98d
Compare
5e9f2c2
to
0580186
Compare
Maybe I do not understand this correctly, as I haven't worked with the admin panel code so far, but (if that was your intention) you did not change the duration of the random wait, but the return value of the get_rpc_timeout function. To change the duration, you'd have to edit |
@nzbr Wow. Good catch. What I did there didn't make any sense at all. |
f05e98d
to
94c95fe
Compare
In most cases the random wait doesn't help, as users just want to use the admin panel, not test how it reacts to bad network condition. For now I just reduced it, maybe later we can add a toggle Signed-off-by: Lukas Mertens <[email protected]> commit-id:fdcfca23
94c95fe
to
3b1912f
Compare
0580186
to
699820c
Compare
✓ Commit merged in pull request #164 |
In most cases the random wait doesn't help, as users just want to use
the admin panel, not test how it reacts to bad network condition. For
now I just reduced it, maybe later we can add a toggle
Signed-off-by: Lukas Mertens [email protected]
Stack: