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

Fix LXC detection on lxc 1+ by also checking for lxc-start binary #1070

Merged
merged 1 commit into from Oct 7, 2017

Conversation

Projects
None yet
2 participants
@jeunito
Contributor

jeunito commented Oct 7, 2017

Signed-off-by: Jose Asuncion jeunito@gmail.com

Description

Adds checking of lxc-start to lxc_version_exists?

Issues Resolved

#488

Check List

also check for lxc-start
Signed-off-by: Jose Asuncion <jeunito@gmail.com>

@jeunito jeunito requested a review from chef/client-maintainers as a code owner Oct 7, 2017

@tas50 tas50 merged commit 986ab7f into chef:master Oct 7, 2017

3 checks passed

DCO This commit has a DCO Signed-off-by
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@tas50

This comment has been minimized.

Show comment
Hide comment
@tas50

tas50 Oct 7, 2017

Member

Nice fix. Thanks

Member

tas50 commented Oct 7, 2017

Nice fix. Thanks

@tas50 tas50 changed the title from also check for lxc-start to Fix LXC detection on lxc 1+ by also checking for lxc-start binary Oct 7, 2017

@jeunito

This comment has been minimized.

Show comment
Hide comment
@jeunito

jeunito Oct 7, 2017

Contributor
Contributor

jeunito commented Oct 7, 2017

@chef chef locked and limited conversation to collaborators Feb 14, 2018

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.