Skip to content
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

Patch helps to overcome login issue. #802

Merged
merged 1 commit into from
May 24, 2024

Conversation

TasmiyaNalatwad
Copy link
Contributor

Sometimes the console via ssh is not connected and Login attempt fails, This could be due to network issue. Attempting to login for 5 times if the connection is not established in one go.

@abdhaleegit
Copy link
Collaborator

I have added the PR to test with CI jobs, we will merge only if everthing goes fine

@PraveenPenguin
Copy link
Collaborator

@abdhaleegit @TasmiyaNalatwad any update on CI run result?

@TasmiyaNalatwad
Copy link
Contributor Author

@PraveenPenguin @abdhaleegit , This needs more enhancement. I see the function call self.cv_SYSTEM.console.get_console() is not coming out of this function and remaining there forever time as trying to login with credentials. I am trying to give a timeout to this function call and then try logging in for few times like 5 times in a loop.
Atleast it must fail if the connection is not happening and come out with some connection issue message

Sometimes the console via ssh is not connected and Login attempt fails, This could be due to network issue.
Attempting to login for 5 times if the connection is not established in one go.

Signed-off-by: Tasmiya Nalatwad <tasmiya@linux.vnet.ibm.com>
Copy link
Collaborator

@abdhaleegit abdhaleegit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@abdhaleegit abdhaleegit merged commit ff538b4 into open-power:master May 24, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants