Friday, 2019-05-24

openstackgerritIan Wienand proposed zuul/zuul-jobs master: validate-zone-db : add job and make more generic  https://review.opendev.org/66113800:01
openstackgerritTristan Cacqueray proposed zuul/zuul master: trigger: add job filter event  https://review.opendev.org/63990500:20
openstackgerritIan Wienand proposed openstack-infra/zone-zuul-ci.org master: Add zone-check job  https://review.opendev.org/66096700:44
openstackgerritTristan Cacqueray proposed zuul/zuul master: trigger: add job filter event  https://review.opendev.org/63990501:11
openstackgerritTristan Cacqueray proposed zuul/zuul master: webtrigger: add initial driver and event  https://review.opendev.org/55515301:11
*** swest has quit IRC01:33
*** swest has joined #zuul01:49
*** yolanda has quit IRC01:51
*** yolanda has joined #zuul01:54
openstackgerritTristan Cacqueray proposed zuul/zuul master: webtrigger: add web route and rpclistener  https://review.opendev.org/55483901:59
*** altlogbot_0 has quit IRC02:12
*** altlogbot_2 has joined #zuul02:14
openstackgerritTristan Cacqueray proposed zuul/zuul master: web: add build button to trigger job  https://review.opendev.org/63571603:05
*** bhavikdbavishi has joined #zuul03:46
openstackgerritMerged zuul/zuul master: ansible-config: pin ara to <1.0.0  https://review.opendev.org/66085903:48
*** jesusaur has quit IRC03:54
*** jesusaur has joined #zuul03:59
*** bhavikdbavishi1 has joined #zuul04:00
*** bhavikdbavishi has quit IRC04:01
*** bhavikdbavishi1 is now known as bhavikdbavishi04:01
*** bjackman has joined #zuul04:05
*** bhavikdbavishi has quit IRC04:22
*** bhavikdbavishi has joined #zuul04:23
openstackgerritTristan Cacqueray proposed zuul/zuul master: amqp: add basic trigger  https://review.opendev.org/63745804:27
openstackgerritTristan Cacqueray proposed zuul/zuul master: amqp: add message informations to the job variables  https://review.opendev.org/63766604:27
*** bhavikdbavishi has quit IRC04:30
*** bhavikdbavishi has joined #zuul04:31
*** bhavikdbavishi has quit IRC04:31
*** swest has quit IRC04:34
*** raukadah is now known as chandankumar04:59
*** pcaruana has joined #zuul05:08
*** pcaruana has quit IRC05:16
*** pcaruana has joined #zuul05:17
bjackmanDid something unusual happen in the zuul-jobs repo? My scheduler is having a tenant reconfiguration event party05:18
bjackmanSubmitting tenant reconfiguration event for bwt-sw-tenant due to event <GitTriggerEvent ref-updated zuul-jobs ref:refs/changes/05/504605/3 oldrev:0000000000000000000000000000000000000000 newrev:fe6526ac905:19
bjackman97ff71913dbf4f2a3dd36ac6b2fb1b6>05:19
bjackmanSeems to be doing this for every change in zuul-jobs05:19
bjackmanIt's only connected via the git driver05:19
bjackmanbaseurl=https://git.zuul-ci.org/05:20
tristanCbjackman: git driver does emit ref-updated even when new changes land in repo05:23
*** swest has joined #zuul05:25
bjackmanOK, any idea why it would seem to have suddenly spotted every change?05:25
tristanCbjackman: it fetch new changes periodically, every 2 hours iirc05:26
bjackmanI see - it's been unresponsive for a pretty long time processing all the changes05:26
bjackmantristanC, last night, it said " zuul.connection.git.watcher: Walk through projects refs for connection: zuul-ci.org"05:29
bjackmanzuul.connection.git.watcher: Read refs {'refs/heads/master': '9515883f88e1e779ec4c733fa01a1791fda352c3'} for project zuul-jobs05:29
bjackmanThen 2hrs later,05:29
bjackmanzuul.connection.git.watcher: Walk through projects refs for connection: zuul-ci.org05:29
*** swest has quit IRC05:30
bjackman zuul.connection.git.watcher: Read refs {'refs/changes/00/494700/1': '01db060617a375affc93fd1262a2facc1d9e9c72', 'refs/changes/00/494700/2':   ..... EXTREMELY LONG LINE05:30
bjackmanThat was 90 minutes ago and it's still processing all the ref-udpated events05:30
tristanCperhaps gitea is being slow at serving the refs?05:33
bjackmanWell, yesterday there was only 1 ref05:33
bjackmantristanC, so it does seem something changed in the repo05:34
tristanCbjackman: oh indeed, it seems to be because of: https://review.opendev.org/#/c/661008/05:34
tristanCpreviously, gerrit ref wasn't pushed to gitea05:35
bjackmanOK05:35
bjackman"The current version wil now skip performing expensive operations on unknown refs"05:35
bjackmanDoes not seem to be true for my "current version"05:35
bjackmantristanC, Should I update Zuul?05:36
tristanCbjackman: i think this refered to the webservice, not zuul git client :)05:36
bjackmantristanC, Oh I see05:36
tristanCbjackman: let see, perhaps we can add a "--head" to the ls-remote action05:39
openstackgerritTristan Cacqueray proposed zuul/zuul master: git: only list head references  https://review.opendev.org/66118505:42
tristanCbjackman: ^ might reduce the load05:43
bjackmantristanC, ah yeah seems like a good idea.05:44
bjackmanI've just counted the refs and events processed so far, seems at this rate my Zuul will be back to processing jobs in another 45 minutes or so05:44
bjackmanSo no disaster05:44
*** hashar has joined #zuul06:19
AJaegerbjackman, tristanC, see discussion from last night on #openstack-infra: We now mirror refs/changes and refs/notes to gitea. What you see looks like an unwanted site effect ;(06:20
AJaegerBut with the git mirrors before using gitea (1month+ ago), we synced those already...06:21
tristanCAJaeger: perhaps gitea is slower than cgit to serve those refs?06:27
AJaegertristanC: maybe? ;) We just synced those last night (finished 3 hours ago), so there might be backlog from some systems as well...06:28
openstackgerritTristan Cacqueray proposed zuul/zuul master: git: only list head references  https://review.opendev.org/66118506:48
*** yolanda has quit IRC06:49
*** yolanda has joined #zuul06:49
openstackgerritMerged zuul/nodepool master: Use py3 pathlib in DibImageFile  https://review.opendev.org/66019106:56
daniel2SpamapS: FreeBSD isnt like it used to be, it can do a lot of things that used to be Linux only in the past.07:07
SpamapStristanC: nobody should have been pulling from cgit anyway IIRC.07:08
SpamapSdaniel2:I am sure it's possible, but I have doubts that it uses the same low level commands.07:08
daniel2SpamapS: Well I will find out :)07:08
SpamapSIndeed.07:08
SpamapSdaniel2:If you need any insight, I re-wrote a lot of the old diskimage-builder block device handling. It looks like it just got moved in to python.07:09
daniel2SpamapS: Im gonna take it one leg at a time.  First step is to setup a jail and see what happens if I just try and use it normally, and then start hacking away at the problems07:10
daniel2SpamapS: I was talking to the FreeBSD CI Team, and honestly they like the idea of testing this out and think something like this would help the CI infrastructure.07:11
daniel2SpamapS: If you notice my cloak, I've been doing ports for awhile :P07:11
SpamapSdaniel2:werd!07:12
daniel2Honestly though, this channel has been a great help over the months for me, bunch of lovely folks here07:13
SpamapSdaniel2: dib was made to be simple, so I hope it's pretty straight forward.07:13
openstackgerritTristan Cacqueray proposed zuul/zuul master: git: only list heads and tags references  https://review.opendev.org/66118507:31
*** mnaser has quit IRC07:35
*** tima has quit IRC07:36
*** ofosos has quit IRC07:42
*** tima has joined #zuul07:43
*** mnaser has joined #zuul07:43
*** jpena|off is now known as jpena07:50
*** gtema has joined #zuul08:00
*** clayg has quit IRC08:01
*** clayg has joined #zuul08:01
*** mnaser has quit IRC08:02
*** mnaser has joined #zuul08:05
*** tosky has joined #zuul08:10
*** tima has quit IRC08:12
*** tima has joined #zuul08:12
openstackgerritTobias Henkel proposed zuul/zuul master: Support squash merge in Github  https://review.opendev.org/66109608:13
*** zbr has joined #zuul08:19
openstackgerritFelix Schmidt proposed zuul/zuul master: Differentiate between queued and waiting jobs in zuul web UI  https://review.opendev.org/66087808:43
*** saneax has joined #zuul09:13
openstackgerritFelix Schmidt proposed zuul/zuul master: Differentiate between queued and waiting jobs in zuul web UI  https://review.opendev.org/66087809:45
*** electrofelix has joined #zuul10:11
*** panda has quit IRC10:38
*** gtema has quit IRC10:39
*** panda has joined #zuul10:46
*** panda is now known as panda|rover10:48
openstackgerritFabien Boucher proposed zuul/zuul master: A reporter for Elasticsearch  https://review.opendev.org/64492710:53
*** jpena is now known as jpena|lunch11:40
*** rfolco is now known as rfolco|brb11:51
*** bjackman has quit IRC11:54
*** EmilienM is now known as EvilienM11:58
*** sshnaidm is now known as sshnaidm|afk12:06
*** hashar has quit IRC12:09
*** pwhalen has quit IRC12:22
*** hashar has joined #zuul12:23
*** pwhalen has joined #zuul12:24
*** rlandy has joined #zuul12:35
*** jpena|lunch is now known as jpena12:38
*** jamesmcarthur has joined #zuul12:52
openstackgerritFabien Boucher proposed zuul/zuul master: Pagure driver - https://pagure.io/pagure/  https://review.opendev.org/60440412:54
*** flepied has joined #zuul12:56
*** rfolco|brb is now known as rfolco13:18
openstackgerritMarkus Hosch proposed zuul/zuul master: WIP: Fix case sensitivity in codeowners check  https://review.opendev.org/66100313:57
openstackgerritTobias Henkel proposed zuul/zuul master: WIP: Fix case sensitivity in codeowners check  https://review.opendev.org/66100314:14
openstackgerritTobias Henkel proposed zuul/zuul master: Evaluate CODEOWNERS settings during canMerge check  https://review.opendev.org/64455714:18
*** jamesmcarthur has quit IRC14:24
dmsimardtristanC: thanks for the ara pin patch, you beat me to it :)14:28
*** chandankumar is now known as raukadah14:48
*** AshBullock has joined #zuul14:50
openstackgerritFabien Boucher proposed zuul/zuul master: Disable gc in test_scheduler.TestExecutor as done in base assertFinalState  https://review.opendev.org/66131614:52
openstackgerritFabien Boucher proposed zuul/zuul master: Pagure driver - https://pagure.io/pagure/  https://review.opendev.org/60440414:53
*** jamesmcarthur has joined #zuul14:55
*** armstrongs has joined #zuul14:55
*** hashar has quit IRC14:57
armstrongsseem to be hitting    - '  ERROR: Could not find a version that satisfies the requirement dictdiffer (from openshift) (from versions: none)'   - 'ERROR: No matching distribution found for dictdiffer (from openshift)' on ansible 2.7 when installing zuul managed ansible versions14:57
armstrongsseems to have went through now, haha looks like it got tagged14:58
*** jamesmcarthur has quit IRC14:59
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: WIP: registry test job  https://review.opendev.org/66132715:04
*** jamesmcarthur has joined #zuul15:05
corvusmordred, clarkb, fungi: ^ there's a start on a job to test the registry15:05
corvuser, rather, the registry roles15:06
fungiooh!15:09
*** jangutter has quit IRC15:10
corvusi think we need to rethink how we do test jobs for zuul-jobs (we can't really put them in openstack-zuul-jobs anymore if we're going to set up a zuul tenant).  i think we either need to make a new repo (zuul/zuul-jobs-tests) or put them in zuul-jobs itself (but still don't include a project stanza, do that in zuul/project-config)15:10
*** jamesmcarthur has quit IRC15:11
openstackgerritDavid Moreau Simard proposed zuul/zuul master: Pin ARA to <1.0.0  https://review.opendev.org/66133115:15
fungiof the two options, i do prefer the idea of zuul-jobs having its tests of its jobs in its own tree rather than in a separate repo15:18
*** jamesmcarthur has joined #zuul15:19
fungiespecially if it means that it's easier for folks who fork zuul-jobs to test that their alterations don't break a tested contract15:19
mordredyeah - I like the idea of putting them in zuul-jobs - although I think we shoudl be careful to not have them make specific mention of nodesets or whatnot that a zuul-jobs consumer wouldn't necessarily have defined and just rely on the base job to bring things in like that15:20
mordredof course that might make it hard to do multi-node15:20
corvusmordred: yeah, and this is a multinode job (3 nodes!)15:20
corvusi hadn't thought of that... it seems like that's going to push us to the extra-repo model15:21
mordredcorvus: it makes me want a construct that could say "please give me three nodes of whatever nodetype the base job uses"15:22
mordredcorvus: but we'd have to invent that - so extra-repo might be better for now15:22
corvusmordred: yeah, i agree that's an idea worth noodling over.  later.15:23
AshBullockHey all, quick question, I've setup a local executor setup feeding logs to a logging server. I was expecting to see the full output of my task run shown in my job-output.json file, but the tasks are left as empty arrays: http://paste.openstack.org/show/752046/ is there a verbosity setting I'm missing to get this showing?15:25
AshBullockcurrently the testjob.yml has simply two debugs: http://paste.openstack.org/show/752047/  These are what I was expecting to see in the tasks list. Any help would be much appreciated15:27
corvusAshBullock: i notice there's no "end" attribute under duration -- are all of those plays failing?15:27
corvusAshBullock: if it is something like that, you may see more information in the executor log15:29
AshBullockthe plays are passing, but it does look as if those tasks are not being run15:33
corvusAshBullock: oh, it may be due to the delegate_to -- there are special restrictions about delegating to localhost in zuul in untrusted job; i'm not sure if that would be allowed15:34
corvusi think that would cause the play to fail, but that should be visible in the executor log... unless that's an unfortunate situation where it only shows up in debug-level logging15:35
corvusmordred: ^ might know for sure15:36
mordredlooking15:39
AshBullockthanks, I added a fail task in the role and it still passes, so it definitely looks like the tasks are not running15:40
mordredhrm. yeah - I wouldn't expect us to be blocking debug with delegate_to ... but in this case, maybe doing delegate_to: localhost would work better (since localhost is the special inventory value)15:40
mordredoh - weird15:40
*** jamesmcarthur has quit IRC15:46
AshBullockwe've reset the playbook hosts to be: - hosts: localhost   and this has now ran the tasks, however we'll want to use hosts: all in order to target boxes, how do we set hosts within the playbook for zuul?15:48
AshBullockso in a play we are going to need to alternate between local actions and target boxes so how would we do that with zuul as setting localhost won't all us to do that?15:49
AshBullockdoes nodepool available hosts become all and if none are available does it just skip?15:50
clarkbAshBullock: the hosts in our nodeset end up in the zuul job inventory15:52
clarkbs/our/your/15:52
AshBullockok that makes sense, need to set up a nodepool host, should there be error handling in the case there is no hosts in the nodepool, rather than it just passing and not running anything?15:53
AshBullockI guess it's just mimicking what ansible does that point15:53
*** jamesmcarthur has joined #zuul15:55
*** nick_x has joined #zuul15:59
nick_xmorning #zuul, I have a question:15:59
nick_xI've checked out a branch with gerrit,15:59
nick_xand would like to merge/checkin/new branch15:59
nick_xhow do I?15:59
*** jamesmcarthur has quit IRC16:02
*** jamesmcarthur has joined #zuul16:03
clarkbnick_x: commits to branches go through the normal review process in gerrit. If you want to merge say feature/foo into master you can propose a merge commit to the master branch and review and merge it16:04
*** saneax has quit IRC16:06
*** pwhalen has left #zuul16:10
*** flepied has quit IRC16:15
nick_xclarkb, I asked a coworker and he said `checkout master`16:17
nick_xsorry I know my question was a little bit fuzzy lol16:17
nick_xmerge commit to master branch?16:18
clarkbto merge a commit to master branch you would locally make your commit, then run the git review utility to push that to gerrit. Then zuul can test it and merge it for you once it has the appropriate votes16:18
nick_xso ..... just to be clear ..... once all the changes in the branch are merged to master, the branch is effectively == master16:19
clarkbwhat is "the branch" in this context?16:19
nick_xI made a branch "in-which-we-implement-a-new-role"16:20
nick_xI made a branch "in-which-we-implement-a-new-[foo]-role"16:20
nick_xI implemented the new role, committed, reviewed, Zuul said ok buddy16:20
nick_xall is merged16:20
nick_xI'm done with the branch16:20
clarkbya that would be a local working branch that is no longer necessary when the commit is merged to master by zuul16:21
nick_xso now I can do "in-which-we-implement-a-new-[bar]-role" rinse repeat16:21
nick_x:D16:22
clarkbyes16:22
nick_xcool! I'm getting the hang 'o this!16:22
nick_xwhat are you up to clarkb?16:22
clarkbcatching up on changes that happened since I left the computer yesterday16:23
nick_xin opendev? or?16:24
clarkbya16:24
clarkband email and irc messages16:24
nick_xheavy is the crown clarkb16:25
nick_xat least it's friday though16:25
nick_x:)16:29
*** jamesmcarthur has quit IRC16:34
nick_xhmm, I have commits up to date with master, but that haven't been merged  :1  I'm expecting these commits to have merged with my +2 and zuul +216:46
clarkbnick_x: you may also need to do the approval +1 vote16:47
nick_xhmm, let's see16:47
nick_xugh it wants rebased & new patchset x_x grrrr16:49
*** manjeets has quit IRC16:51
*** electrofelix has quit IRC17:02
*** jpena is now known as jpena|off17:04
fungiif there is an explicit parent change (you're using gerrit, right?) and it's been updated more recently, then gerrit won't allow that to merge without a rebase17:05
nick_xyeah fungi that's pretty much what happened17:06
nick_xI +1'd top of stack and was trying to get bottom of stack to merge,17:06
nick_xit's sorted now, I think17:07
*** AshBullock has quit IRC17:09
*** mattw4 has joined #zuul17:09
*** mattw4 has quit IRC17:15
*** electrofelix has joined #zuul17:38
*** electrofelix has quit IRC17:51
*** pcaruana has quit IRC17:53
*** armstrongs has quit IRC18:13
openstackgerritDavid Shrewsbury proposed zuul/zuul master: WIP: Store hold requests in zookeeper  https://review.opendev.org/66111418:36
*** pcaruana has joined #zuul18:48
*** pcaruana has quit IRC18:48
*** tosky has quit IRC19:16
*** panda|rover has quit IRC19:36
*** panda has joined #zuul19:40
corvusSpamapS: we're using paused jobs for our image build/test/deployment pipeline in opendev/system-config if you want to see it in action19:48
*** EvilienM is now known as EmilienM20:33
*** aspiers has quit IRC20:38
clarkbcorvus: mordred do you have that change you asked me to review about gerrit stream events behavior? I am fairly certain I did review it and wanted to check for followups but I cannot find it in gerrit's search20:50
corvusclarkb: https://review.opendev.org/64990020:52
clarkbthanks20:52
mordredcorvus: right there where clarkb pointed to20:57
* mordred is so helpful20:57
clarkbmordred: you got your names mixed up :P20:58
clarkbnow to figure out why that didn't show up in my searches (or I just missed it?)20:58
clarkbheh it shows up now20:58
clarkb(becuse I left a new vote)20:58
clarkber comment20:58
corvusclarkb: hahaha :)20:58
mordredclarkb: and wow - I did get the names backwards .. and when you said I did I looked and thought "no I didn't"20:58
corvusclarkb: i found that extremely amusing given the topic of the change :)20:58
clarkbcorvus: yes exactly20:59
clarkbI am beginning to wonder if maybe they don't use a workflow category20:59
clarkbsince I think that separate category is what solves this for us20:59
*** aspiers has joined #zuul21:00
clarkbI'll have to consider that case if there are sane ways to deal with that they don't require changing gerrit configs21:00
corvusyeah... the only other thing i can think of is a behavior flag which is :/21:01
corvusi'm still noodling on it21:01
*** mattw4 has joined #zuul21:31
evgenylHi everyone, I'm trying to run zuul-scheduler using docker container, but it fails in a crashloop http://paste.openstack.org/show/752058/ , in my config in section `gearman_server`  I have `start=true` what is the best way to debug the problem and to find out why gearman does not get started?21:45
clarkbevgenyl: that message indicates the server named "scheduler" doesn't have dns records or /etc/hosts entries so it doesn't know what to bind to21:46
clarkbevgenyl: you can update the config to bind to :: and it will bind to all addresses21:47
evgenylclarkb: Oops, let me check that, thank you!21:47
evgenylclarkb: Yep, that was the problem, sorry for not reading the logs carefully enough and thanks again.21:52
clarkbno problem21:57
*** jamesmcarthur has joined #zuul22:12
*** rlandy has quit IRC22:16
*** jamesmcarthur has quit IRC22:19
*** jamesmcarthur has joined #zuul22:50
*** jamesmcarthur has quit IRC22:57
*** mattw4 has quit IRC23:09
*** jamesmcarthur has joined #zuul23:33
*** jamesmcarthur has quit IRC23:37

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