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

conf.c: small fix for args of mount_entry #2050

Merged
merged 1 commit into from Dec 19, 2017

Conversation

3 participants
@tanyifeng
Contributor

tanyifeng commented Dec 19, 2017

Signed-off-by: Yifeng Tan tanyifeng1@huawei.com

@lxc-jenkins

This comment has been minimized.

Show comment
Hide comment
@lxc-jenkins

lxc-jenkins Dec 19, 2017

This pull request didn't trigger Jenkins as its author isn't in the whitelist.

An organization member must perform one of the following:

  • To have this branch tested by Jenkins, use the "ok to test" command.
  • To have a one time test done, use the "test this please" command.

Those commands are simple Github comments of the format: "jenkins: COMMAND"

lxc-jenkins commented Dec 19, 2017

This pull request didn't trigger Jenkins as its author isn't in the whitelist.

An organization member must perform one of the following:

  • To have this branch tested by Jenkins, use the "ok to test" command.
  • To have a one time test done, use the "test this please" command.

Those commands are simple Github comments of the format: "jenkins: COMMAND"

@brauner

This comment has been minimized.

Show comment
Hide comment
@brauner

brauner Dec 19, 2017

Member

jenkins: test this please

Member

brauner commented Dec 19, 2017

jenkins: test this please

@brauner

This comment has been minimized.

Show comment
Hide comment
@brauner
Member

brauner commented Dec 19, 2017

Thanks @tanyifeng!

conf.c: small fix for args of mount_entry
Signed-off-by: Yifeng Tan <tanyifeng1@huawei.com>
@brauner

This comment has been minimized.

Show comment
Hide comment
@brauner

brauner Dec 19, 2017

Member

LGTM the build failures can be ignored. It's network connectivity.

Member

brauner commented Dec 19, 2017

LGTM the build failures can be ignored. It's network connectivity.

@brauner brauner merged commit 92b1770 into lxc:master Dec 19, 2017

3 of 4 checks passed

Testsuite Testsuite failed
Details
Branch target Branch target is correct
Details
Signed-off-by All commits signed-off
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@tanyifeng tanyifeng deleted the tanyifeng:small_fix branch Jan 15, 2018

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