1568092: Also check syscall.Errno in getMessageFromErr #5059

Merged
merged 1 commit into from Apr 10, 2016

Conversation

Projects
None yet
3 participants

cherylj commented Apr 9, 2016

This test could've been skipped since lxd is not currently
supported on Centos, but I added the check for the
syscall.Errno to reduce the number of skips that we have.

This was live tested on Ubuntu and CentOS and tests passed
on both.

(Review request: http://reviews.vapour.ws/r/4501/)

1568092: Also check syscall.Errno in getMessageFromErr
This test could've been skipped since lxd is not currently
supported on Centos, but I added the check for the
syscall.Errno to reduce the number of skips that we have.

This was live tested on Ubuntu and CentOS and tests passed
on both.

cherylj commented Apr 10, 2016

$$merge$$

Contributor

jujubot commented Apr 10, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

Contributor

jujubot commented Apr 10, 2016

Build failed: Does not match ['fixes-1568312']
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/7276

cherylj commented Apr 10, 2016

$$JFDI$$

Contributor

jujubot commented Apr 10, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

Contributor

jujubot commented Apr 10, 2016

Build failed: Tests failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/7277

Contributor

jujubot commented Apr 10, 2016

Build failed: Tests failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/7278

@jujubot jujubot merged commit a6246fe into juju:master Apr 10, 2016

Owner

jameinel commented Apr 12, 2016

$$merge$$

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