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] cleanup error when tray process does not exist #2433

Closed
adrianriobo opened this issue Jun 15, 2021 · 1 comment · Fixed by #2485
Closed

[BUG] cleanup error when tray process does not exist #2433

adrianriobo opened this issue Jun 15, 2021 · 1 comment · Fixed by #2485
Labels

Comments

@adrianriobo
Copy link
Contributor

General information

On windows platform from time to time the tray shows unhandled errors, the only options from there is quit the tray. After quit the tray if the user wants to cleanup crc the log shows an error as it is expected to clean the tray process which does not exist anymore

CRC version

CodeReady Containers version: 1.28.0+08de64bd
OpenShift version: 4.7.13 (not embedded in executable)

Steps to reproduce

  1. Install crc from msi
  2. Exit tray
  3. crc cleanup

Expected

cleanup execution finish successfully

Actual

cleanup execution ends up showing an unhandle error. status code 1

Logs

INFO Stopping tray if running
INFO Uninstalling daemon if installed
INFO Removing hosts file records added by CRC
INFO Uninstalling admin-helper daemon
INFO Will run as admin: uninstall admin-helper daemon
INFO Removing the crc VM if exists
INFO Removing dns server from interface
INFO Will run as admin: Remove dns entry for default switch
INFO Removing pull secret from the keyring
INFO Removing older logs
INFO Removing CRC Machine Instance directory
Failed to kill tray process: exit status 1```

@adrianriobo adrianriobo added kind/bug Something isn't working status/need triage labels Jun 15, 2021
@adrianriobo
Copy link
Contributor Author

Scenario when tray is forced to exist can be checked at #2434

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants