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

Fixes wrong already running/loaded messages. #301

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

derekmarcotte
Copy link

@derekmarcotte derekmarcotte commented Oct 3, 2018

Fix: This guest appears to be already running/loaded.

There is a race condition where iohyve will fail to start a bhyve
instance when there is a running iohyve with the prefix of the same name
as the iohyve you are trying to start.

e.g. (you have example-other running):

iohyve start example

Will fail with the error message:

This guest appears to be already running/loaded.
Use stop, destroy, or forcekill to remove zombie processes.

This corrects the pgrep regexs to stop matching past the name.

There is a race condition where iohyve will fail to start a bhyve
instance when there is a running iohyve with the prefix of the same name
as the iohyve you are trying to start.

e.g. (you have `example-other` running):

`iohyve start example`

Will fail with the error message:

```
This guest appears to be already running/loaded.
Use stop, destroy, or forcekill to remove zombie processes.
```

This corrects the pgrep regexs to stop matching past the name.
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.

1 participant