-
-
Notifications
You must be signed in to change notification settings - Fork 147
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
leapp session start
very slow when SSO is required
#503
Comments
Hello @JacobEvelyn , thanks for letting us kknow! We'll look into this performance issue, in the meanwhile refer to #500 to install the newer version of node as a workaround. |
Hi @pethron, I am the reporter of #500 and I just wanted to inform you that I have since started running into this problem again on a subsequent upgrade. I'm not sure exactly what mechanism causes the delay, but it is interesting to note that the timing is, afaict, always 4 minutes and 1 second for the mfa refresh. my best guess is that this means there is some code that is waiting for a signal and times out when it doesn't receive it after 4 minutes. I'm not sure which piece of the equation that could be in though since it appears to include the leapp cli, the leapp desktop application, node, and the aws cli, all of which are on uncoupled versions. Let me know if there's any further information I could provide to assist with your investigation! |
Noticed similar behaviour when use Credential Process method. Executing commands that make use of profiles using the Taking the Switching back to the credential file method which doesn't require the use of the Leapp CLI seems to avoid this issue - albeit with the disadvantages of using the credentials/config files.
|
Hi @JacobEvelyn, |
Thanks @urz9999. Someone on our team has reported that your NVM workaround does indeed solve the issue for them. Personally it's more annoying for me to install NVM than to just work around the issue so I probably won't bother, but I appreciate you outlining the steps for anyone who needs them. Is it possible to isolate which node function call is responsible for the 4 minute delay? |
Good to add this (semi?) requirement to https://docs.leapp.cloud/latest/cli/ |
I am still observing this behavior as of today, with NodeJS version 20.14.0 (LTS) and Leapp 0.25.2. When I switch to NodeJS version 18.20.3, I get the expected behavior. So it seems like the issue was introduced somewhere between major versions 18 and 20. |
Yeah I think adding to the doc this behaviour could be useful |
Describe the bug
The first time each day that I run
leapp session start
, I'm prompted to log in to AWS via Google SSO, and then the CLI command hangs for ~4 minutes before finally completing successfully. The rest of the day the command will complete in a second or two.Leapp Version
@noovolari/leapp-cli/0.1.49 darwin-arm64 node-v21.3.0
and Leapp 0.22.2I have also experienced this with previous versions of
leapp-cli
(@noovolari/leapp-cli/0.1.26 darwin-arm64 node-v20.5.0
) and Leapp (not sure which versions).To Reproduce
Steps to reproduce the behavior:
leapp session start --sessionId <id>
(Sometimes AWS takes me to a Google SSO page somewhere around steps 3-4, but this is not necessary to reproduce).
5. Observe that CLI command will wait for another 4 minutes before exiting with status code 0.
Expected behavior
Step 5 above to not be there. 😄
Desktop (please complete the following information):
Additional context
One member of our team reported this issue going away (i.e. the command completes right after the browser SSO login is complete) after following these instructions to install a different version of node.
The text was updated successfully, but these errors were encountered: