Discussion:
[Freeipa-devel] [freeipa PR#627][opened] Add CI helper script invocation to Travis CI
martbab
2017-03-20 11:27:17 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/627
Author: martbab
Title: #627: Add CI helper script invocation to Travis CI
Action: opened

PR body:
"""
This tests whether changes in ipatests do not break any of the helper scripts
used for integration testing. The PR is rebased on
https://github.com/freeipa/freeipa/pull/626 so it should produce green build
with @cheimes's fixes
"""

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/627/head:pr627
git checkout pr627
martbab
2017-03-20 14:52:23 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/627
Author: martbab
Title: #627: Add CI helper script invocation to Travis CI
Action: synchronized

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/627/head:pr627
git checkout pr627
tomaskrizek
2017-03-22 09:26:43 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/627
Title: #627: Add CI helper script invocation to Travis CI

tomaskrizek commented:
"""
Could you please re-rebase this on top of #626 to check if everything works as expected?
"""

See the full comment at https://github.com/freeipa/freeipa/pull/627#issuecomment-288342674
martbab
2017-03-22 11:51:14 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/627
Author: martbab
Title: #627: Add CI helper script invocation to Travis CI
Action: synchronized

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/627/head:pr627
git checkout pr627
martbab
2017-03-22 12:35:56 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/627
Title: #627: Add CI helper script invocation to Travis CI

martbab commented:
"""
@tomaskrizek I guess Green Travis means we can merge #626 and I can then remove those commits from this PR right?
"""

See the full comment at https://github.com/freeipa/freeipa/pull/627#issuecomment-288385253
tomaskrizek
2017-03-22 12:45:39 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/627
Title: #627: Add CI helper script invocation to Travis CI

tomaskrizek commented:
"""
@martbab Exactly, please remove the extra commits.
"""

See the full comment at https://github.com/freeipa/freeipa/pull/627#issuecomment-288387364
martbab
2017-03-22 12:55:02 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/627
Author: martbab
Title: #627: Add CI helper script invocation to Travis CI
Action: synchronized

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/627/head:pr627
git checkout pr627
tomaskrizek
2017-03-22 13:17:03 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/627
Title: #627: Add CI helper script invocation to Travis CI

Label: +ack
tomaskrizek
2017-03-22 13:19:41 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/627
Title: #627: Add CI helper script invocation to Travis CI

Label: +pushed
tomaskrizek
2017-03-22 13:19:45 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/627
Title: #627: Add CI helper script invocation to Travis CI

tomaskrizek commented:
"""
master:

* b6624594bedce75849248469305ae964ce5ea2ef Travis CI: invoke integration test helper scripts before test execution
"""

See the full comment at https://github.com/freeipa/freeipa/pull/627#issuecomment-288395389
tomaskrizek
2017-03-22 13:19:46 UTC
Permalink
URL: https://github.com/freeipa/freeipa/pull/627
Author: martbab
Title: #627: Add CI helper script invocation to Travis CI
Action: closed

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/627/head:pr627
git checkout pr627

Loading...