Monday, 2021-04-12

*** tosky has quit IRC00:00
*** ajitha has joined #zuul00:20
*** iurygregory has quit IRC01:05
*** sanjayu__ has joined #zuul01:46
*** evrardjp has quit IRC02:33
*** evrardjp has joined #zuul02:33
*** ajitha has quit IRC02:40
*** cloudnull8 has quit IRC03:02
*** cloudnull8 has joined #zuul03:02
*** bhavikdbavishi has joined #zuul03:14
*** bhavikdbavishi1 has joined #zuul03:19
*** bhavikdbavishi has quit IRC03:20
*** bhavikdbavishi1 is now known as bhavikdbavishi03:20
*** bhavikdbavishi has quit IRC04:07
*** bhavikdbavishi has joined #zuul04:09
*** tristanC_ has joined #zuul04:15
*** tristanC has quit IRC04:20
*** jfoufas1 has joined #zuul04:28
*** bhavikdbavishi has quit IRC04:37
*** vishalmanchanda has joined #zuul04:41
*** ricolin has joined #zuul04:41
*** ykarel has joined #zuul04:56
*** bhavikdbavishi has joined #zuul05:05
*** bhavikdbavishi1 has joined #zuul05:08
*** bhavikdbavishi has quit IRC05:09
*** bhavikdbavishi1 is now known as bhavikdbavishi05:09
*** ajitha has joined #zuul06:13
*** fsvsbs has joined #zuul06:41
openstackgerritIan Wienand proposed zuul/nodepool master: [dnm] ZK TLS documentation update  https://review.opendev.org/c/zuul/nodepool/+/78582006:46
*** dmsimard has quit IRC06:48
openstackgerritIan Wienand proposed zuul/nodepool master: Account for resource usage of leaked nodes  https://review.opendev.org/c/zuul/nodepool/+/78582106:50
*** dmsimard has joined #zuul06:51
*** bhavikdbavishi has quit IRC06:56
*** bhavikdbavishi has joined #zuul07:31
*** dmsimard has quit IRC07:32
*** dmsimard has joined #zuul07:33
*** bhavikdbavishi1 has joined #zuul07:34
*** tosky has joined #zuul07:35
*** bhavikdbavishi has quit IRC07:35
*** bhavikdbavishi1 is now known as bhavikdbavishi07:35
*** jpena|off is now known as jpena07:54
*** rpittau|afk is now known as rpittau08:04
iceyis there any way to access the comment that triggers a pipeline?08:25
icey(from within a job)08:25
avassicey: it's not part of the inventory so the only way I know is to query gerrit from the job08:36
*** holser has joined #zuul09:02
*** nils has joined #zuul09:32
iceyavass: do you think a proposal adding it to the inventory would be accepted? or possibly, adding a section that is build from named captures in the comment regex, if present?09:57
avassicey: I wouldn't be opposed it. but for zuul to be able to gate a project effectively the configuration for the check/gate pipelines needs to be part of a repository10:18
avasswe have a similar need where we have secondary pipelines building specific projects in a monorepo triggered by a comment in gerrit. Currently those are one pipeline per project that needs to be built which is a bit ugly, so reading the comment in the job instead would be useful.10:20
iceyavass: the desire is to have a pipeline (or something) that can be used to selectively retry a single job, for example, one jbo in a check pipeline fails intermittently and could be retried to give a reviewer confidence that the failure was transient (still bad and should be resolved)10:20
iceyavass: a bunch of additional pipelines is the closest today it seems10:20
avassyeah10:20
avassicey: you could have an initial dispatching job that sits in a config project and has access to a secret so it can query gerrit for that comment to evaluate it and set child_jobs accordingly:10:24
avasshttps://zuul-ci.org/docs/zuul/reference/jobs.html#skipping-dependent-jobs10:24
iceyI love the concept, but I'd really love to not have to query gerrit and try to get the correct comment :-/10:25
iceywhat if two rechecks are requested in two comments, my code just probably got it wrong :-P10:26
avassI agree :)10:26
*** bhavikdbavishi has quit IRC10:48
*** iurygregory has joined #zuul11:11
*** jpena is now known as jpena|lunch11:32
*** rlandy has joined #zuul11:42
*** rlandy is now known as rlandy|rover11:43
*** ashbullock has joined #zuul12:00
*** jcapitao has joined #zuul12:00
*** sduthil has joined #zuul12:14
*** jpena|lunch is now known as jpena12:30
ashbullockHey all, I'm trying to deploy the zuul-web component, having trouble with the yarn build and install, I'm looking at this doc: https://zuul-ci.org/docs/zuul/reference/developer/javascript.html, if I run yarn build before the pip install, i receive: "error: [Errno 17] File exists: '../zuul/web/static' -> 'web/build'" I assume because pip is trying12:40
ashbullockto run the build itself, but when I only run the pip install of zuul, I don't see any static content folder, does anyone know what I'm missing? Thanks in advance12:40
*** bhavikdbavishi has joined #zuul12:47
*** cloudnull8 is now known as cloudnull12:48
*** bhavikdbavishi1 has joined #zuul12:56
*** bhavikdbavishi has quit IRC12:58
*** bhavikdbavishi1 is now known as bhavikdbavishi12:58
*** ashbullock has quit IRC13:12
*** ashbullock has joined #zuul13:14
openstackgerritThierry Carrez proposed zuul/zuul-website master: Clean up supporter logos  https://review.opendev.org/c/zuul/zuul-website/+/78587413:24
*** bhavikdbavishi has quit IRC13:52
corvusi somehow managed to get a vaccine appointment for this morning, so my plan is to make releases of zuul and zuul-operator later today14:10
fungisounds good. hopefully side effects don't impact your plans14:11
corvus++14:13
mordredI did not have any side effects with shot 1 - sandy's arm was sore and she was a bit tired for a couple of days - fwiw14:14
corvusyeah, this is pfizer first shot, so i'm hoping/expecting to be useful afterwords.  i will plan to be useless after the 2nd shot.14:15
*** ykarel has quit IRC14:21
*** ykarel has joined #zuul14:24
*** josefwells has joined #zuul14:27
*** josefwells has quit IRC14:42
clarkbLarissa took a long nap a few hours after the first pfizer.  I ended up doing dadops much longer than expected that day14:45
fungiyeah, my first shot of the moderna vaccine resulted in no noticeable side effects14:49
clarkbI'll be eligible in a week then I get to play the refresh game14:54
*** dpawlik has quit IRC14:58
*** ykarel is now known as ykarel|away15:19
clarkbtobiash: corvus: should I abandon https://review.opendev.org/c/zuul/zuul/+/784142 as it will be sorted out in the work for consistent buildset refs?15:30
tobiashclarkb: yes, makes sense15:30
*** ykarel|away has quit IRC15:33
openstackgerritKenny Ho proposed zuul/zuul master: Make stream log file read and writable by all  https://review.opendev.org/c/zuul/zuul/+/78591015:51
*** y2kenny has joined #zuul15:54
*** bhavikdbavishi has joined #zuul16:01
*** vishalmanchanda has quit IRC16:10
*** hamalq has joined #zuul16:47
*** hamalq has quit IRC16:47
*** hamalq has joined #zuul16:48
*** rpittau is now known as rpittau|afk16:59
corvuszuul-maint: how does this look for tagging zuul-operator? commit 0366b867bf98d357c02e835573a862ddd9491d2e (HEAD -> master, tag: 0.1.0, origin/master, origin/HEAD, gerrit/master, refs/changes/39/785739/2)17:07
mordredcorvus: how do I know you're not now being mind-controlled?17:08
tobiashcorvus: lgtm17:10
*** jcapitao has quit IRC17:10
corvusmordred: have my mental faculties ever appeared to be under the influence of a sentient intelligence?17:10
fungithe cia controls me through radio signals i pick up with my fillings17:11
mordredcorvus: excellent point17:11
fungii confirm 0366b86 does look like the branch tip, 0.1.0 as the first tag seems fine, sort of irrelevant what number we pick for an initial release i guess. don't want it to be 1.0.0 yet?17:13
mordredthe api spec number in the CRD is a pre-release already - so no, I think 0.1.0 is more appropriate17:14
corvusfungi: plus, 1.0.0 seems awkward for a release we're making because the very next commit is likely to have incompatabilities17:15
*** bhavikdbavishi has quit IRC17:15
corvuswhile 1.0 -> 2.0 would encapsulate that fine, 1.0 still sort of sends a "this is ready" message which i don't think is right17:15
fungi2.0.0! ;)17:15
fungiyeah, 0.1.0 seems reasonable in that case17:15
avasslooks good to me17:19
avassand 0.1.0 sounds like a good idea. I don't think it would look good if the operator ends up with version 21.x.y before it's ready unless we want to switch over to CalVer :)17:23
corvuszuul-maint: how does this look for zuul?  commit 9c3fce2820fb46aa39dbf89984386420fd7a7f70 (HEAD -> master, tag: 4.2.0, origin/master, gerrit/master)17:24
corvus(we restarted opendev on that on friday, and it's looking good so far)17:25
tobiashlgtm17:26
clarkb9c3fce2820fb46aa39dbf89984386420fd7a7f70 as 4.2.0 lgtm17:26
fungilooks like master branch tip, and yeah the log between that and 4.1.0 looks like more than just bug fixes so i agree 4.2.0 seems righty17:27
jfoufas1mordred you lucky americans got your shot of [x], here we have to wait...17:32
*** ajitha has quit IRC17:32
corvusjfoufas1: i was expecting to wait several more weeks, but my local government surprised everyone by opening eligibility early right when they were saying they would have shortages for 2 weeks.  i really don't know what the plan is, but i didn't ask twice i just booked an appointment.  :)17:33
mordredjfoufas1: well, we handled things poorly up until now ...17:33
fungiyeah, like my sister in law was required to go back to teaching small school children even though the state hadn't authorized vaccinations for school teachers17:34
fungithat was a weird decision17:35
jfoufas1yes, I guess with these politicians, that Dante Allegri put in the lower parts of hell, it's bound to go wrong17:35
jfoufas1at least some of the time17:36
clarkbya I'm not elligible for another week and expect it to be difficult to get an appointment at that time.17:47
clarkbwill have to wait and see17:47
*** jpena is now known as jpena|off17:48
corvushttps://hub.docker.com/r/zuul/zuul-operator/tags?page=1&ordering=last_updated17:56
corvuslol there's a :0 tag :)17:56
avasshaha17:58
*** jfoufas1 has quit IRC18:05
openstackgerritMerged zuul/zuul-website master: Clean up supporter logos  https://review.opendev.org/c/zuul/zuul-website/+/78587418:19
*** josefwells has joined #zuul18:21
*** josefwells has quit IRC18:35
corvusokay, releases done and emails sent18:36
fungiawesome18:54
fungijust in time for the openinfra foundation newsletter this week ;)18:55
*** rlandy|rover is now known as rlandy|rover|bia19:06
*** rlandy|rover|bia is now known as rlandy|rover19:37
*** nils has quit IRC20:00
*** ashbullock has quit IRC20:25
openstackgerritJames E. Blair proposed zuul/zuul master: Move key_store_password to keystore section in zuul.conf  https://review.opendev.org/c/zuul/zuul/+/78597220:57
corvusclarkb: 785091 should be interesting/easy review for you :)21:00
openstackgerritAlbin Vass proposed zuul/nodepool master: WIP: Digitalocean driver  https://review.opendev.org/c/zuul/nodepool/+/75955921:07
avasscorvus: now using the statemachine framework ^ :)21:08
corvus\o/21:08
*** hamalq has quit IRC21:11
*** hamalq has joined #zuul21:11
corvustobiash, swest, felixedel: the next batch of sos changes lgtm, except i think we may want to send an announcement out about the key store password before merging that; it's a shame we didn't think of that before 4.0.21:41
corvusif folks agree with 785972 then we can send that out asap and merge in a few days i guess?  unless we think we need to wait a week or 221:44
*** sshnaidm|pto has quit IRC21:44
corvus22:05 < openstackgerrit> James E. Blair proposed opendev/system-config master: Add zuul keystore password  https://review.opendev.org/c/opendev/system-config/+/78598022:05
corvusis to prep for that ^22:06
openstackgerritTristan Cacqueray proposed zuul/zuul master: Add a tool to decrypt a secret  https://review.opendev.org/c/zuul/zuul/+/63977122:19
openstackgerritJames E. Blair proposed zuul/zuul-operator master: Move some config to configmaps  https://review.opendev.org/c/zuul/zuul-operator/+/78598522:54
clarkbcorvus: agreed, I've approved it :)22:55
openstackgerritKenny Ho proposed zuul/zuul master: [WIP] Add failing remote_zuul_stream test  https://review.opendev.org/c/zuul/zuul/+/78591023:05
*** tosky has quit IRC23:24
y2kennyI think contributing a test case is the best I can do with the stream log error.  (My python skill is just not quite up to the task.)  This feels like a multithreading related issue but I am not 100%.23:25
clarkby2kenny: I haven't kept up, is the issue that it fails when using become?23:27
y2kennyclarkb: it happens when I alternate between become: True and become: False23:27
y2kennyusing with_items23:28
y2kennyI think the test case I am adding in 785910 reproduces  what I am seeing locally23:28
y2kenny(I am assuming zuul-tox-remote runs the test case pre-submit.)23:29
clarkbyes it should fail in the check pipeline23:30
y2kennyclarkb: ok, in that case I will leave the test case as is, unless there are other place you guys want me to keep track of this.  (I forgot where zuul's bug tracker is.)23:32
y2kennyoh nvm... storyboard... found it23:33
clarkby2kenny: writing a story and putting it in your change would probably be a good way to keep track of it23:34
y2kennyclarkb: will do23:35
openstackgerritJames E. Blair proposed zuul/zuul-operator master: Pass through environment to scheduler, web and launcher  https://review.opendev.org/c/zuul/zuul-operator/+/78598823:43
openstackgerritJames E. Blair proposed zuul/zuul-operator master: Allow terminationGracePeriodSeconds to be configurable  https://review.opendev.org/c/zuul/zuul-operator/+/78598923:48
*** rlandy|rover has quit IRC23:49
openstackgerritMerged zuul/zuul master: Switch to ZooKeeperSimpleBase where possible  https://review.opendev.org/c/zuul/zuul/+/78509123:52

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!