Tuesday, 2018-05-22

pabelangerJust heard of an interesting talk happening in Ottawa this week (local python meet up): https://www.meetup.com/ottawapython/events/rbljkpyxhbgc/00:50
pabelangera new CI system with comparative examples from the Gerrit-based approach (used by OpenStack) and from ApartCI (his product).00:50
pabelangerzxiiro was talking about it00:53
*** swest has quit IRC01:58
*** swest has joined #zuul02:14
*** xinliang has quit IRC02:58
*** xinliang has joined #zuul03:10
*** xinliang has joined #zuul03:10
*** zanzibar_ has joined #zuul03:22
*** threestrands has quit IRC04:02
*** aspiers[m] has joined #zuul04:30
aspiers[m]corvus: here is my tool I mentioned earlier https://pypi.org/project/git-explode/04:32
*** pcaruana has joined #zuul04:44
*** spsurya has joined #zuul04:45
*** pcaruana has quit IRC05:03
Diabelkosee you tomorrow in 301!05:03
*** quiquell|off is now known as quiquell05:38
*** pcaruana has joined #zuul06:46
*** jimi|ansible has quit IRC07:04
*** jimi|ansible has joined #zuul07:11
*** jimi|ansible has joined #zuul07:11
*** gtema has joined #zuul07:47
*** jpena|off is now known as jpena07:47
*** ssbarnea_ has joined #zuul07:49
*** salmannn has joined #zuul08:08
*** threestrands has joined #zuul08:11
*** electrofelix has joined #zuul08:41
*** salmannn has quit IRC08:44
*** threestrands has quit IRC08:46
*** pcaruana has quit IRC10:53
*** xinliang has quit IRC11:00
*** pcaruana has joined #zuul11:09
*** spsurya has quit IRC11:41
*** jpena is now known as jpena|lunch11:52
*** snapiri has joined #zuul12:17
*** dkranz has joined #zuul12:21
*** dkranz has quit IRC12:49
*** jpena|lunch is now known as jpena12:51
*** pcaruana has quit IRC12:54
*** eliqiao has joined #zuul12:58
eliqiaohi , 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
gtemaeliqiao: Depends-on is a possibility for a changed change (change in Prj B depends on change in Prj B), but not the global project dependency13:00
eliqiaookay, 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 #zuul13:13
gtemaeliqiao: 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
gtemaand in commit to which project you specify Depends-on13:13
gtemaeliqiao: 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 Zuul13:20
*** zigo_ is now known as zigo13: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 commit13:22
Diabelkoeliqiao: required projects is used to define a project that you need the code from to run the job13:23
Diabelkoin very simple words13:23
eliqiaoyes, but it doesn't work with new style . see https://docs.openstack.org/infra/zuul/user/gating.html#cross-project-dependencies13:24
eliqiaoit works if I change it to "Depends-On: gerrit commit id"13:25
Diabelkooh13:25
gtemaeliqiao: aah13:25
DiabelkoI've never tried using it that way13:25
Diabelkoin Depends-On: you are supposed to provide review ID13:26
Diabelkonot the full link (at least I thinko so)13:26
Diabelkohttps://review.openstack.org/#/c/513783/13:27
Diabelkolike here for instance13:27
gtemacan you try https://sftests.com/19 ? Not sure what /r/ stand for in your case13:27
eliqiao@gtema, I don't think ^^ will work, because, if I click that, the web returns 404.13:28
*** elyezer has joined #zuul13:29
Diabelkocan you get us a full debug log?13:29
eliqiao@gtema, sorry, s/will/won't/13:29
eliqiao2018-05-22 21:33:23,90213:33
eliqiaohttps://www.irccloud.com/pastebin/7rPfM3lF/13:34
eliqiao@Diablko13: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
gtemathis log is not enough13:36
gtemaa scheduler-debug.log is necessary13:36
clarkbif you look in driver/gerrit/gerritsource.py the change_re regex doesnt appear to allow the r/ prefix13:57
clarkbI think this is likely the issue13:57
clarkbeliqiao: ^13:57
*** quiquell is now known as quiquell|off14: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 #zuul14:19
*** elyezer has quit IRC14:24
*** snapiri has quit IRC14:28
*** spsurya has joined #zuul14:43
*** pcaruana has quit IRC14:58
*** elyezer has joined #zuul15:18
*** rlandy has joined #zuul15:19
*** yolanda has quit IRC15:21
*** electrofelix has quit IRC15:23
*** yolanda has joined #zuul15:25
*** pbrobinson has quit IRC15:45
*** pbrobinson has joined #zuul15:49
*** yolanda has quit IRC15:49
openstackgerritFabien Boucher proposed openstack-infra/zuul master: Fix new depends-on format matching for prefixed gerrit ui  https://review.openstack.org/57000616:21
openstackgerritFabien Boucher proposed openstack-infra/zuul master: Fix new depends-on format matching for prefixed gerrit ui  https://review.openstack.org/57000616:22
clarkbfbo_: lgtm but can we add a test? let me know if that is something I/we can help with16: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
clarkbfbo_: no problem, good night16:57
fbo_clarkb: thanks16:57
*** acozine1 has joined #zuul17:01
*** jpena is now known as jpena|off17:41
*** ssbarnea_ has quit IRC18:05
*** gtema has quit IRC18:26
*** bhavik1 has joined #zuul18:38
*** bhavik1 has quit IRC18:46
*** sshnaidm|rover is now known as sshnaidm|off19:04
openstackgerritMerged openstack-infra/zuul master: github: Do not process comment events for regular issues  https://review.openstack.org/56986819:41
*** zxiiro-away has joined #zuul20:48
*** acozine1 has quit IRC20:58
*** dkranz has quit IRC21:07
*** weshay is now known as weshay_pto21:41
*** threestrands has joined #zuul22:39
*** acozine1 has joined #zuul23:01
openstackgerritAdam Gandelman proposed openstack-infra/zuul master: github: Optimize getPullReviews() a bit  https://review.openstack.org/57007723:10
*** zanzibar_ has quit IRC23:29
*** eliqiao has quit IRC23:56

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!