Thursday, 2020-07-16

*** jamesmcarthur has joined #zuul00:05
*** jamesmcarthur has quit IRC00:10
*** jamesmcarthur has joined #zuul00:12
*** jamesmcarthur has quit IRC00:16
*** armstrongs has quit IRC00:17
*** jamesmcarthur has joined #zuul00:18
*** jamesmcarthur has quit IRC00:21
*** jamesmcarthur has joined #zuul00:21
*** wuchunyang has quit IRC00:58
*** Goneri has quit IRC01:23
*** rlandy|ruck has quit IRC01:24
*** holser has quit IRC01:28
*** jamesmcarthur has quit IRC02:11
*** jamesmcarthur has joined #zuul02:11
*** rfolco has quit IRC02:15
*** jamesmcarthur has quit IRC02:16
*** bhavikdbavishi has joined #zuul03:25
*** jamesmcarthur has joined #zuul03:29
*** bhavikdbavishi1 has joined #zuul03:32
*** jamesmcarthur has quit IRC03:32
*** jamesmcarthur has joined #zuul03:32
*** bhavikdbavishi has quit IRC03:34
*** bhavikdbavishi1 is now known as bhavikdbavishi03:34
*** jamesmcarthur has quit IRC04:08
*** jamesmcarthur has joined #zuul04:08
*** jamesmcarthur has quit IRC04:09
*** jamesmcarthur_ has joined #zuul04:09
*** jamesmcarthur_ has quit IRC04:11
*** bhavikdbavishi has quit IRC04:27
*** bhavikdbavishi has joined #zuul04:28
*** cloudnull has quit IRC04:52
*** cloudnull has joined #zuul04:53
openstackgerritMerged zuul/zuul master: PF4: Update buildset result page (new layout and styling)  https://review.opendev.org/73801105:31
openstackgerritMerged zuul/zuul master: Consolidate summary, logs and console on a single build page  https://review.opendev.org/74062905:34
*** reiterative has quit IRC05:42
*** saneax has joined #zuul05:47
*** marios has joined #zuul05:52
*** wuchunyang has joined #zuul05:52
*** wuchunyang has quit IRC06:17
*** bhavikdbavishi1 has joined #zuul06:26
*** bhavikdbavishi has quit IRC06:28
*** bhavikdbavishi1 is now known as bhavikdbavishi06:28
*** wuchunyang has joined #zuul06:36
openstackgerritFelix Edel proposed zuul/zuul master: PF4: Update build result page layout  https://review.opendev.org/73997206:48
openstackgerritFelix Edel proposed zuul/zuul master: Fix fetching function on build result page  https://review.opendev.org/74110306:48
openstackgerritFelix Edel proposed zuul/zuul master: WIP: PF4 update builds page and filter toolbar  https://review.opendev.org/74138506:48
*** bhavikdbavishi has quit IRC06:59
*** zbr has quit IRC07:16
*** wuchunyang has quit IRC07:20
*** jcapitao has joined #zuul07:23
*** bolg has joined #zuul07:27
*** zbr has joined #zuul08:02
*** tosky has joined #zuul08:02
*** saneax has quit IRC08:02
*** saneax has joined #zuul08:04
*** bhavikdbavishi has joined #zuul08:07
*** bhavikdbavishi1 has joined #zuul08:10
*** bhavikdbavishi has quit IRC08:12
*** bhavikdbavishi1 is now known as bhavikdbavishi08:12
*** nils has joined #zuul08:14
*** tosky has quit IRC08:16
*** holser has joined #zuul08:30
openstackgerritTobias Henkel proposed zuul/zuul master: Fix multiple prs found when commit is not head  https://review.opendev.org/74140208:30
*** tosky has joined #zuul08:33
*** zbr has quit IRC08:50
openstackgerritTobias Henkel proposed zuul/zuul master: Support per branch change queues  https://review.opendev.org/71853108:53
tobiashzuul-maint: the last ps of per branch change queues fixes a test race. It would be great to get a re-review on this stack ^08:54
openstackgerritTobias Henkel proposed zuul/zuul master: Move queue from pipeline to project  https://review.opendev.org/72018208:54
openstackgerritTobias Henkel proposed zuul/zuul master: Add optional support for circular dependencies  https://review.opendev.org/68535408:55
*** jcapitao has quit IRC08:56
*** jcapitao has joined #zuul08:58
openstackgerritJan Kubovy proposed zuul/zuul master: Scheduler's pause/resume functionality  https://review.opendev.org/70973509:17
openstackgerritJan Kubovy proposed zuul/zuul master: Separate connection registries in tests  https://review.opendev.org/71295809:17
openstackgerritJan Kubovy proposed zuul/zuul master: Prepare Zookeeper for scale-out scheduler  https://review.opendev.org/71726909:17
openstackgerritJan Kubovy proposed zuul/zuul master: Mandatory Zookeeper connection for ZuulWeb in tests  https://review.opendev.org/72125409:17
openstackgerritJan Kubovy proposed zuul/zuul master: Connect merger to Zookeeper  https://review.opendev.org/71622109:17
openstackgerritJan Kubovy proposed zuul/zuul master: Connect fingergw to Zookeeper  https://review.opendev.org/71687509:17
openstackgerritJan Kubovy proposed zuul/zuul master: Connect executor to Zookeeper  https://review.opendev.org/71626209:17
openstackgerritJan Kubovy proposed zuul/zuul master: Driver event ingestion  https://review.opendev.org/71729909:23
openstackgerritJan Kubovy proposed zuul/zuul master: WIP: Store unparsed branch config in Zookeeper  https://review.opendev.org/70571609:23
*** jcapitao has quit IRC09:40
*** jcapitao has joined #zuul09:41
*** bolg has quit IRC09:51
*** bolg has joined #zuul09:56
*** dmellado has quit IRC09:59
avasszuul-jobs-main: https://review.opendev.org/#/c/726830/ should be merged today to remove venv as default tox testenv :)10:00
*** jcapitao has quit IRC10:05
*** dmellado has joined #zuul10:07
*** jcapitao has joined #zuul10:09
*** bhavikdbavishi has quit IRC10:37
tobiashavass: I guess the title is misleading as it's actually Remove, not deprecate ;)10:39
*** bhavikdbavishi has joined #zuul10:40
tobiashcorvus: it would be great if you could re-review 710034 (github auth refactor). It had to be rebased to fix conflicts since your last review.10:47
*** wuchunyang has joined #zuul10:50
*** wuchunyang has quit IRC10:55
*** wuchunyang has joined #zuul10:55
*** wuchunyang has quit IRC10:56
*** jcapitao is now known as jcapitao_lunch11:07
*** bhavikdbavishi has quit IRC11:25
*** bhavikdbavishi has joined #zuul11:43
openstackgerritMerged zuul/zuul master: Don't decrease window size on merge failure  https://review.opendev.org/73062411:51
*** rlandy has joined #zuul11:52
*** rlandy is now known as rlandy|ruck11:52
*** jcapitao_lunch is now known as jcapitao12:03
*** wuchunyang has joined #zuul12:06
openstackgerritTobias Henkel proposed zuul/zuul master: Move fingergw config to fingergw  https://review.opendev.org/66494912:06
openstackgerritTobias Henkel proposed zuul/zuul master: WIP: Route streams to different zones via finger gateway  https://review.opendev.org/66496512:06
openstackgerritTobias Henkel proposed zuul/zuul master: Support ssl encrypted fingergw  https://review.opendev.org/66495012:06
openstackgerritFabien Boucher proposed zuul/zuul master: gitlab: implement the approval reporter  https://review.opendev.org/74143812:10
*** wuchunyang has quit IRC12:10
*** harrymichal has joined #zuul12:15
*** rfolco has joined #zuul12:20
*** bolg has quit IRC12:21
tobiashmordred: where is subunit2html taken from? zuul-jobs has no ensure role for it and it seems to be pre-installed on the opendev nodes13:20
tobiashah found it, os-testr13:31
fungitobiash: yeah, here's how opendev is preinstalling it: https://opendev.org/openstack/project-config/src/branch/master/nodepool/elements/nodepool-base/install.d/91-venv-os-testr13:34
tobiashthanks13:34
fungiseems like it could be a candidate for replacing with another ensure-.* role13:35
tobiashyes, especially since fetch-subunit-output uses it if it finds it13:36
openstackgerritFabien Boucher proposed zuul/zuul master: gitlab: update naming in reference pipeline  https://review.opendev.org/74145013:45
*** bhavikdbavishi has quit IRC13:45
mordredseems like a thing that should live somewhere other than os-testr too13:50
openstackgerritSimon Westphahl proposed zuul/zuul master: Correctly fail cat/fileschanges when update fails  https://review.opendev.org/73976213:54
*** rlandy|ruck is now known as rlandy|ruck|mtg14:02
*** Goneri has joined #zuul14:02
*** ianychoi has joined #zuul14:05
fungimordred: as in split the module out to its own package? looking at https://opendev.org/openstack/os-testr/src/branch/master/os_testr/subunit2html.py it does indeed appear not especially intertwined with ostestr itself14:06
fungiso probably wouldn't be hard to extract14:07
fungiit's basically a standalone script which is set as an entrypoint in that package for lack of a better place, i guess14:07
fungithe subunit2html package name is still available on pypi too14:22
mordredYeah. Like, we put it there for expedience ... But seems like a standalone package might be better. Or we could vendor it into the Zuul role14:22
*** rlandy|ruck|mtg is now known as rlandy|ruck14:34
*** Goneri has quit IRC14:55
*** harrymichal has quit IRC14:55
*** zbr|ruck has quit IRC14:55
*** SotK has quit IRC14:55
*** irclogbot_1 has quit IRC14:55
*** felixedel has quit IRC14:55
*** ericsysmin has quit IRC14:55
*** olaph has quit IRC14:55
*** mhu has quit IRC14:55
*** Goneri has joined #zuul14:57
*** harrymichal has joined #zuul14:57
*** zbr|ruck has joined #zuul14:57
*** SotK has joined #zuul14:57
*** irclogbot_1 has joined #zuul14:57
*** felixedel has joined #zuul14:57
*** ericsysmin has joined #zuul14:57
*** olaph has joined #zuul14:57
*** mhu has joined #zuul14:57
mnasershouldn't this commit be moved to our base python images instead of zuul only? https://opendev.org/zuul/zuul/commit/5a1954eb7bc0c715b4fec3b58ceb11bd8f79d96015:21
*** hamalq has joined #zuul15:22
*** hamalq_ has joined #zuul15:23
*** hamalq has quit IRC15:27
*** marios has quit IRC15:31
openstackgerritFabien Boucher proposed zuul/zuul master: gitlab: support the merge request approval event  https://review.opendev.org/74148315:33
*** kmalloc has joined #zuul15:35
corvusmnaser: i think there were changes to add that, but the zuul image has extra installation steps15:35
corvusmnaser: i think the one before install-from-bindep may be able to be removed now15:36
zbr|ruckneed review on https://review.opendev.org/#/c/740733/ -- improve display of cmd in summary15:36
*** bhavikdbavishi has joined #zuul15:37
zbr|ruckor https://sbarnea.com/ss/Screen-Shot-2020-07-16-16-38-26.png15:41
*** saneax has quit IRC15:59
*** harrymichal has quit IRC16:08
*** harrymichal has joined #zuul16:09
*** sshnaidm is now known as sshnaidm|afk16:28
*** jcapitao has quit IRC16:47
AJaegermordred, corvus, https://review.opendev.org/740560 updates the upload-docker-image job to run in periodic queue, do you want to review that?16:48
*** nils has quit IRC17:04
*** olaph has quit IRC17:11
corvusAJaeger: yes i do! :)17:53
clarkbzbr|ruck: rather than a hover tooltip to explain what the field is I wonder if that should be part of the table?17:55
clarkbit is part of the table in the console log ui right?17:55
clarkbconsistency with that as well as claerly stating what the info is may be good?17:55
*** harrymichal has quit IRC18:01
*** harrymichal has joined #zuul18:01
corvusAJaeger: i think i'm -1ish on the pipeline name thing; i think the rest makes sense (though i think the buildset registry should be used to test images, not dockerhub)18:09
AJaegerthanks, corvus.18:16
*** bhavikdbavishi has quit IRC18:47
openstackgerritMerged zuul/zuul master: Refactor github auth handling into its own class  https://review.opendev.org/71003419:02
*** nils has joined #zuul19:11
*** Goneri has quit IRC19:14
*** Goneri has joined #zuul19:24
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Fix GCS log upload  https://review.opendev.org/74152819:32
corvuszuul-jobs-maint: can we merge that asap?  gerrit's zuul is broken due to that -- https://bugs.chromium.org/p/gerrit/issues/detail?id=1314619:33
*** nils has quit IRC19:37
openstackgerritMerged zuul/zuul-jobs master: Fix GCS log upload  https://review.opendev.org/74152819:49
fungicorvus: ^19:50
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Add a test that exercises the GCS Credentials class  https://review.opendev.org/74153519:54
corvusfungi: agree with your and david's comment ^ :)19:58
openstackgerritAlbin Vass proposed zuul/zuul-jobs master: Remove default tox_envlist: venv  https://review.opendev.org/72683020:00
avasstobiash: just got home, fixed :)20:00
tobiashlgtm20:01
avasscorvus: wanna +3 that since we're supposed to remove the default tox_envlist today?20:02
corvusavass: already did :)20:08
avasscorvus: thanks! :)20:09
fungimnaser: not sure if anyone's mentioned it to you yet, but zuul.vexxhost.dev is serving an ssl cert which expired yesterday20:10
openstackgerritMerged zuul/zuul-jobs master: Remove default tox_envlist: venv  https://review.opendev.org/72683020:17
corvusclarkb, fungi: gerrit's zuul is all fixed now, thanks :)20:18
fungiawesome20:18
fungicorvus: also your test has a problem, it seems20:19
corvus2 problems: the fix is not py27 compat (so yay, test caught a real bug; though one we don't care about since the only current user is all py3); and i forgot a git add20:20
avassfungi, mordred, tobiash: I tried putting subunit2html.py in the fetch-subunu-output role a while ago, but bumped into problems with dependencies for the script https://review.opendev.org/#/c/730930/20:21
corvuslike subunit itself? and testtools?20:22
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Add a test that exercises the GCS Credentials class  https://review.opendev.org/74153520:23
avasscorvus: yeah something like that20:25
clarkbcorvus: comment on your super call in that latest ps20:27
avassit's probably not hard to solve but I didn't put much effort into it since I was just trying to remove a TODO :)20:27
corvuswow that was the wrong super call20:27
openstackgerritGuillaume Chauvel proposed zuul/zuul master: WIP: Scheduler: Reconfiguration ref-updated create/delete  https://review.opendev.org/73919820:29
openstackgerritGuillaume Chauvel proposed zuul/zuul master: WIP: Scheduler: Reconfiguration ref-updated oldrev+newrev  https://review.opendev.org/73907820:29
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: Add a test that exercises the GCS Credentials class  https://review.opendev.org/74153520:30
openstackgerritGuillaume Chauvel proposed zuul/zuul master: WIP: Scheduler: Reconfiguration ref-updated create/delete  https://review.opendev.org/73919820:34
*** wuchunyang has joined #zuul21:01
*** wuchunyang has quit IRC21:06
openstackgerritMerged zuul/zuul-jobs master: Add a test that exercises the GCS Credentials class  https://review.opendev.org/74153521:08
openstackgerritGuillaume Chauvel proposed zuul/zuul master: WIP: Scheduler: Reconfiguration ref-updated oldrev+newrev  https://review.opendev.org/73907821:21
*** rfolco has quit IRC21:45
openstackgerritGuillaume Chauvel proposed zuul/zuul master: WIP: Scheduler: Reconfiguration ref-updated create/delete  https://review.opendev.org/73919821:49
*** rlandy|ruck is now known as rlandy|ruck|afk21:52
*** harrymichal has quit IRC22:19
openstackgerritGuillaume Chauvel proposed zuul/zuul master: tutorial: Rework quick-start and prepare for other tutorials  https://review.opendev.org/73206622:30
openstackgerritGuillaume Chauvel proposed zuul/zuul master: tutorial: Add "gate your first patch"  https://review.opendev.org/73206722:30
openstackgerritGuillaume Chauvel proposed zuul/zuul master: tutorial: Add "Use zuul jobs"  https://review.opendev.org/73206822:30
openstackgerritGuillaume Chauvel proposed zuul/zuul master: tutorial: Add "gate pipeline"  https://review.opendev.org/73206922:30
openstackgerritGuillaume Chauvel proposed zuul/zuul master: tutorial: Add "job secrets"  https://review.opendev.org/73207022:30
openstackgerritGuillaume Chauvel proposed zuul/zuul master: tutorial: Add "job dependencies"  https://review.opendev.org/73207122:30
openstackgerritGuillaume Chauvel proposed zuul/zuul master: Rename quick-start to zuul-tutorial-quick-start  https://review.opendev.org/73765622:30
openstackgerritGuillaume Chauvel proposed zuul/zuul master: [DNM] TEST run zuul tutorials to test stream+callback (+ zuul-jobs change)  https://review.opendev.org/73547722:30
*** rfolco has joined #zuul22:53
*** armstrongs has joined #zuul22:54
*** armstrongs has quit IRC23:03
*** tosky has quit IRC23:04
*** wuchunyang has joined #zuul23:08
*** saneax has joined #zuul23:10
*** wuchunyang has quit IRC23:12
*** zbr|ruck has quit IRC23:16
*** SotK has quit IRC23:16
*** irclogbot_1 has quit IRC23:16
*** felixedel has quit IRC23:16
*** ericsysmin has quit IRC23:16
*** mhu has quit IRC23:16
*** zbr|ruck has joined #zuul23:23
*** SotK has joined #zuul23:23
*** irclogbot_1 has joined #zuul23:23
*** felixedel has joined #zuul23:23
*** ericsysmin has joined #zuul23:23
*** mhu has joined #zuul23:23
openstackgerritGuillaume Chauvel proposed zuul/zuul master: [DNM] Test: run multiple tutorials ('job dependencies' 2 times)  https://review.opendev.org/74155823:26

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