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

[BUG] Spinner animation blocks the input prompt from Diagnostic tool #630

Closed
parthosa opened this issue Oct 24, 2023 · 2 comments · Fixed by #631
Closed

[BUG] Spinner animation blocks the input prompt from Diagnostic tool #630

parthosa opened this issue Oct 24, 2023 · 2 comments · Fixed by #631
Assignees
Labels
bug Something isn't working user_tools Scope the wrapper module running CSP, QualX, and reports (python)

Comments

@parthosa
Copy link
Collaborator

Diagnostic tool requires a user prompt to confirm that it will collect sensitive information from the cluster. This prompt interferes with the spinner animation in non-verbose mode and does not show up correctly.

Output:

psarthi@work:~/sample_dir$ spark_rapids_user_tools dataproc diagnostic --cluster test_cluster
Processing...⣷so you want to continue (yes/no):
Processing...⣻
Processing Completed!
psarthi@work:~/sample_dir$
@parthosa parthosa added bug Something isn't working user_tools Scope the wrapper module running CSP, QualX, and reports (python) ? - Needs Triage labels Oct 24, 2023
@mattahrens
Copy link
Collaborator

I've noticed that I lose my cursor on the command line after running a tools command since the Processing... change. I can regain it by running reset, but it would be good to make sure this fix applies for that use case as well.

@parthosa
Copy link
Collaborator Author

Thanks @mattahrens. I will add this fix in this change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working user_tools Scope the wrapper module running CSP, QualX, and reports (python)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants