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

[monitor] wrong statement of break #2048

Merged
merged 1 commit into from Dec 19, 2017
Merged

[monitor] wrong statement of break #2048

merged 1 commit into from Dec 19, 2017

Conversation

duguhaotian
Copy link
Contributor

if lxc_abstract_unix_connect fail and return -1, this code never goto retry.

@lxc-jenkins
Copy link

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
Copy link
Member

brauner commented Dec 18, 2017

@duguhaotian, could you please sign-off your commit with git commit -s?

@brauner brauner added the Incomplete Waiting on more information from reporter label Dec 18, 2017
@duguhaotian
Copy link
Contributor Author

duguhaotian commented Dec 19, 2017

Sorry, I refresh it now.

if lxc_abstract_unix_connect fail and return -1,  this code never goto retry.

Signed-off-by: liuhao <liuhao27@huawei.com>
@brauner
Copy link
Member

brauner commented Dec 19, 2017

@duguhaotian, no worries. :)

@brauner
Copy link
Member

brauner commented Dec 19, 2017

jenkins: test this please

1 similar comment
@brauner
Copy link
Member

brauner commented Dec 19, 2017

jenkins: test this please

@brauner brauner merged commit 90f20db into lxc:master Dec 19, 2017
@brauner
Copy link
Member

brauner commented Dec 19, 2017

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Incomplete Waiting on more information from reporter
Development

Successfully merging this pull request may close these issues.

None yet

3 participants