This repository has been archived by the owner. It is now read-only.

Switched get_mozmill-environments from a matrix job to a standard job. #178

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
2 participants
@davehunt
Member

davehunt commented Oct 23, 2012

Workaround for issue #151

@davehunt

This comment has been minimized.

Show comment Hide comment
@davehunt

davehunt Oct 24, 2012

Member

Updated to avoid copying superfluous files.

Member

davehunt commented Oct 24, 2012

Updated to avoid copying superfluous files.

@whimboo

This comment has been minimized.

Show comment Hide comment
@whimboo

whimboo Oct 26, 2012

Contributor

It looks fine now. I'm going to land this on staging. Once proven we will port this to production too.

Contributor

whimboo commented Oct 26, 2012

It looks fine now. I'm going to land this on staging. Once proven we will port this to production too.

@whimboo

This comment has been minimized.

Show comment Hide comment
@whimboo

whimboo Oct 29, 2012

Contributor

Looks like a bug in the xshell plugin stopped us from delivering it to production. As Dave pointed out it has been fixed by jenkinsci/xshell-plugin@a13e799 but we cannot upgrade to it because it requires a higher Jenkins version. Not sure why we haven't seen this problem earlier.

Contributor

whimboo commented Oct 29, 2012

Looks like a bug in the xshell plugin stopped us from delivering it to production. As Dave pointed out it has been fixed by jenkinsci/xshell-plugin@a13e799 but we cannot upgrade to it because it requires a higher Jenkins version. Not sure why we haven't seen this problem earlier.

@whimboo whimboo reopened this Oct 29, 2012

@whimboo

This comment has been minimized.

Show comment Hide comment
@whimboo

whimboo Oct 29, 2012

Contributor

I should have reopened the issue and not the pull request.

Contributor

whimboo commented Oct 29, 2012

I should have reopened the issue and not the pull request.

@whimboo whimboo closed this Oct 29, 2012

davehunt added a commit to davehunt/mozmill-ci that referenced this pull request Feb 1, 2013

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