Issue679

Title evaluate Bitbucket's continuous integration feature
Priority wish Status resolved
Superseder Nosy List florian, jendrik, malte, silvan
Assigned To jendrik Keywords
Optional summary

Created on 2016-10-12.12:36:55 by jendrik, last changed by jendrik.

Messages
msg5769 (view) Author: jendrik Date: 2016-10-24.09:49:12
Merged and pushed.
msg5762 (view) Author: malte Date: 2016-10-21.16:14:18
Fine with me.
msg5759 (view) Author: jendrik Date: 2016-10-21.15:43:49
Any further comments on the code, Malte?
msg5742 (view) Author: jendrik Date: 2016-10-14.19:12:19
Thanks for the comments on bitbucket!

I added this item to the meeting agenda.
msg5739 (view) Author: malte Date: 2016-10-14.18:01:40
Certainly looks interesting, but I think this kind of infrastructure thing needs
to be discussed with everyone. Especially if it means that we have to add a
bitbucket-specific YAML file to the root of the repository. (Yuck!)

IOW, that's the kind of thing I'd like to see discussed at a Fast Downward meeting.
msg5738 (view) Author: jendrik Date: 2016-10-14.15:41:25
After some experimentation, I think the feature works really well and it could be 
a useful addition to our workflow.

I made a pull request here:
https://bitbucket.org/jendrikseipp/downward/pull-requests/59

In the top right on the pull request page you can see whether the issue branch 
passes all tests ("x of 1 passed"). Clicking on the link shows the test output.

Information about passing and failing builds is also available on overview pages, 
e.g. https://bitbucket.org/jendrikseipp/downward/pull-requests/.

If we merge this branch, the only step necessary for enabling continuous 
integration for a different repo is going to "Pipelines" in that repo and 
enabling the pipelines feature.
msg5718 (view) Author: malte Date: 2016-10-12.13:53:18
Interesting! BTW, it's also possible to run buildbot on revisions that are not
yet part of the repository, so we could e.g. have a script that triggers the
buildbot on the current local revision. I'm not sure if this feature is
currently enabled, and if yes, it's only enabled for people who are logged into
the buildbot (because of course this feature allows arbitrary code execution on
all buildslaves). So this could be an alternative that is easier to arrive at
from our current infrastructure. (I'm not saying I prefer either solution, just
wanted to mention the possibilities.)
msg5716 (view) Author: jendrik Date: 2016-10-12.12:36:55
Bitbucket recently added support for continuous integration (under the name 
"pipelines": https://confluence.atlassian.com/bitbucket/bitbucket-pipelines-
792496469.html). In this issue, I'd like to evaluate how useful this feature is 
for our workflow. Ideally, it could allow us to run code and style tests for 
branches before we merge them, simplifying the review process.
History
Date User Action Args
2016-10-24 09:49:13jendriksetstatus: reviewing -> resolved
messages: + msg5769
2016-10-21 16:14:18maltesetmessages: + msg5762
2016-10-21 15:43:49jendriksetmessages: + msg5759
2016-10-14 19:12:19jendriksetmessages: + msg5742
2016-10-14 18:01:40maltesetmessages: + msg5739
2016-10-14 15:41:25jendriksetstatus: unread -> reviewing
messages: + msg5738
2016-10-12 16:29:10silvansetstatus: chatting -> unread
nosy: + silvan
2016-10-12 13:53:18maltesetstatus: unread -> chatting
messages: + msg5718
2016-10-12 13:41:58floriansetnosy: + florian
2016-10-12 12:36:55jendrikcreate