pabelanger | Just heard of an interesting talk happening in Ottawa this week (local python meet up): https://www.meetup.com/ottawapython/events/rbljkpyxhbgc/ | 00:50 |
---|---|---|
pabelanger | a new CI system with comparative examples from the Gerrit-based approach (used by OpenStack) and from ApartCI (his product). | 00:50 |
pabelanger | zxiiro was talking about it | 00:53 |
*** swest has quit IRC | 01:58 | |
*** swest has joined #zuul | 02:14 | |
*** xinliang has quit IRC | 02:58 | |
*** xinliang has joined #zuul | 03:10 | |
*** xinliang has joined #zuul | 03:10 | |
*** zanzibar_ has joined #zuul | 03:22 | |
*** threestrands has quit IRC | 04:02 | |
*** aspiers[m] has joined #zuul | 04:30 | |
aspiers[m] | corvus: here is my tool I mentioned earlier https://pypi.org/project/git-explode/ | 04:32 |
*** pcaruana has joined #zuul | 04:44 | |
*** spsurya has joined #zuul | 04:45 | |
*** pcaruana has quit IRC | 05:03 | |
Diabelko | see you tomorrow in 301! | 05:03 |
*** quiquell|off is now known as quiquell | 05:38 | |
*** pcaruana has joined #zuul | 06:46 | |
*** jimi|ansible has quit IRC | 07:04 | |
*** jimi|ansible has joined #zuul | 07:11 | |
*** jimi|ansible has joined #zuul | 07:11 | |
*** gtema has joined #zuul | 07:47 | |
*** jpena|off is now known as jpena | 07:47 | |
*** ssbarnea_ has joined #zuul | 07:49 | |
*** salmannn has joined #zuul | 08:08 | |
*** threestrands has joined #zuul | 08:11 | |
*** electrofelix has joined #zuul | 08:41 | |
*** salmannn has quit IRC | 08:44 | |
*** threestrands has quit IRC | 08:46 | |
*** pcaruana has quit IRC | 10:53 | |
*** xinliang has quit IRC | 11:00 | |
*** pcaruana has joined #zuul | 11:09 | |
*** spsurya has quit IRC | 11:41 | |
*** jpena is now known as jpena|lunch | 11:52 | |
*** snapiri has joined #zuul | 12:17 | |
*** dkranz has joined #zuul | 12:21 | |
*** dkranz has quit IRC | 12:49 | |
*** jpena|lunch is now known as jpena | 12:51 | |
*** pcaruana has quit IRC | 12:54 | |
*** eliqiao has joined #zuul | 12:58 | |
eliqiao | hi , got one question here: Do I need to add a project name (say it B) in required-projects of a job (defined in A ), if I want to A to depend B project ? or only add "Depend-On: B's commit-id/gerrit url" should be fine? | 12:58 |
gtema | eliqiao: Depends-on is a possibility for a changed change (change in Prj B depends on change in Prj B), but not the global project dependency | 13:00 |
eliqiao | okay, gotcha. I meet one bug/issue when using new style "Depend-on:" use, so a little confusing about it. the issue's link's here. | 13:01 |
eliqiao | @gtema https://storyboard.openstack.org/#!/story/2002080 , would you please help to take a look if I use it in wrong way. | 13:02 |
*** pcaruana has joined #zuul | 13:13 | |
gtema | eliqiao: I am likely not the best person for this analysis, but here are my two cents: you need to describe better how you created zuul projects. From story it is not clear what your projects are. | 13:13 |
gtema | and in commit to which project you specify Depends-on | 13:13 |
gtema | eliqiao: if I understand right and you have just one project in gerrit and want to have this change tested - you don't need to specify Depends-On. What you need instead is to configure a project in Zuul | 13:20 |
*** zigo_ is now known as zigo | 13:22 | |
eliqiao | @gtema , sorry for make some confusing to you. not really. I have to project, I have confiured them in Zuul already. what I want to do is when I test project A , I want to depend project B 's specify commit | 13:22 |
Diabelko | eliqiao: required projects is used to define a project that you need the code from to run the job | 13:23 |
Diabelko | in very simple words | 13:23 |
eliqiao | yes, but it doesn't work with new style . see https://docs.openstack.org/infra/zuul/user/gating.html#cross-project-dependencies | 13:24 |
eliqiao | it works if I change it to "Depends-On: gerrit commit id" | 13:25 |
Diabelko | oh | 13:25 |
gtema | eliqiao: aah | 13:25 |
Diabelko | I've never tried using it that way | 13:25 |
Diabelko | in Depends-On: you are supposed to provide review ID | 13:26 |
Diabelko | not the full link (at least I thinko so) | 13:26 |
Diabelko | https://review.openstack.org/#/c/513783/ | 13:27 |
Diabelko | like here for instance | 13:27 |
gtema | can you try https://sftests.com/19 ? Not sure what /r/ stand for in your case | 13:27 |
eliqiao | @gtema, I don't think ^^ will work, because, if I click that, the web returns 404. | 13:28 |
*** elyezer has joined #zuul | 13:29 | |
Diabelko | can you get us a full debug log? | 13:29 |
eliqiao | @gtema, sorry, s/will/won't/ | 13:29 |
eliqiao | 2018-05-22 21:33:23,902 | 13:33 |
eliqiao | https://www.irccloud.com/pastebin/7rPfM3lF/ | 13:34 |
eliqiao | @Diablko | 13:34 |
eliqiao | @Diablko, sorry guys, I should head for home now, late for me, if you found something, please leave messages. thanks in advance. | 13:35 |
gtema | this log is not enough | 13:36 |
gtema | a scheduler-debug.log is necessary | 13:36 |
clarkb | if you look in driver/gerrit/gerritsource.py the change_re regex doesnt appear to allow the r/ prefix | 13:57 |
clarkb | I think this is likely the issue | 13:57 |
clarkb | eliqiao: ^ | 13:57 |
*** quiquell is now known as quiquell|off | 14:06 | |
fbo_ | clarkb: eliqiao oh yes on software factory gerrit is under /r so you're right it won't match. We never tested the new depends-on format. This maybe need to be configurable in Zuul. | 14:07 |
*** dkranz has joined #zuul | 14:19 | |
*** elyezer has quit IRC | 14:24 | |
*** snapiri has quit IRC | 14:28 | |
*** spsurya has joined #zuul | 14:43 | |
*** pcaruana has quit IRC | 14:58 | |
*** elyezer has joined #zuul | 15:18 | |
*** rlandy has joined #zuul | 15:19 | |
*** yolanda has quit IRC | 15:21 | |
*** electrofelix has quit IRC | 15:23 | |
*** yolanda has joined #zuul | 15:25 | |
*** pbrobinson has quit IRC | 15:45 | |
*** pbrobinson has joined #zuul | 15:49 | |
*** yolanda has quit IRC | 15:49 | |
openstackgerrit | Fabien Boucher proposed openstack-infra/zuul master: Fix new depends-on format matching for prefixed gerrit ui https://review.openstack.org/570006 | 16:21 |
openstackgerrit | Fabien Boucher proposed openstack-infra/zuul master: Fix new depends-on format matching for prefixed gerrit ui https://review.openstack.org/570006 | 16:22 |
clarkb | fbo_: lgtm but can we add a test? let me know if that is something I/we can help with | 16:54 |
fbo_ | clarkb: yes sure, I'll have a look to the unittest tomorrow and push a new PS (already a bit late here in france) | 16:57 |
clarkb | fbo_: no problem, good night | 16:57 |
fbo_ | clarkb: thanks | 16:57 |
*** acozine1 has joined #zuul | 17:01 | |
*** jpena is now known as jpena|off | 17:41 | |
*** ssbarnea_ has quit IRC | 18:05 | |
*** gtema has quit IRC | 18:26 | |
*** bhavik1 has joined #zuul | 18:38 | |
*** bhavik1 has quit IRC | 18:46 | |
*** sshnaidm|rover is now known as sshnaidm|off | 19:04 | |
openstackgerrit | Merged openstack-infra/zuul master: github: Do not process comment events for regular issues https://review.openstack.org/569868 | 19:41 |
*** zxiiro-away has joined #zuul | 20:48 | |
*** acozine1 has quit IRC | 20:58 | |
*** dkranz has quit IRC | 21:07 | |
*** weshay is now known as weshay_pto | 21:41 | |
*** threestrands has joined #zuul | 22:39 | |
*** acozine1 has joined #zuul | 23:01 | |
openstackgerrit | Adam Gandelman proposed openstack-infra/zuul master: github: Optimize getPullReviews() a bit https://review.openstack.org/570077 | 23:10 |
*** zanzibar_ has quit IRC | 23:29 | |
*** eliqiao has quit IRC | 23:56 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!