-
-
Notifications
You must be signed in to change notification settings - Fork 215
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
tests: Add incus-user test #159
Conversation
c665b3e
to
d5b9582
Compare
test/suites/incus_user.sh
Outdated
|
||
incus-user --group nogroup & | ||
USER_PID="$!" | ||
sleep 3 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems fragile - given how slow github runners can sometimes be, it would seem worth checking availability over the socket in a loop, if possible.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving anyway, as the comment was just a suggestion. Leaving it to you to merge though, if you want to skip the suggestion.
Signed-off-by: Stéphane Graber <stgraber@stgraber.org>
Signed-off-by: Stéphane Graber <stgraber@stgraber.org>
Closes lxc#142 Signed-off-by: Stéphane Graber <stgraber@stgraber.org>
Closes #142