Friday, 2020-08-21

*** PrinzElvis has joined #zuul00:00
*** dcastellani has joined #zuul00:00
*** aprice has joined #zuul00:01
*** rlandy has quit IRC00:02
*** evrardjp has quit IRC00:05
*** evrardjp has joined #zuul00:06
*** iamweswilson has quit IRC00:09
*** dcastellani has quit IRC00:12
*** aprice has quit IRC00:12
*** sgw1 has quit IRC00:13
*** dcastellani has joined #zuul00:14
*** aprice has joined #zuul00:15
*** iamweswilson has joined #zuul00:15
*** _erlon_ has quit IRC00:21
*** rfolco has quit IRC00:24
*** sgw has quit IRC00:27
*** Goneri has joined #zuul01:01
*** Goneri has quit IRC01:14
openstackgerritMerged zuul/zuul-jobs master: Revert "Ensure git is installed in prepare-workspace-git role"  https://review.opendev.org/74723801:44
openstackgerritIan Wienand proposed zuul/zuul-jobs master: tox: include command output in log/error  https://review.opendev.org/74732502:57
*** evrardjp_ has joined #zuul03:12
*** evrardjp has quit IRC03:15
*** bhavikdbavishi has joined #zuul03:43
*** bhavikdbavishi has quit IRC04:24
*** evrardjp_ has quit IRC04:33
*** evrardjp has joined #zuul04:35
*** raukadah is now known as chandankumar04:41
*** bhavikdbavishi has joined #zuul04:56
*** vishalmanchanda has joined #zuul05:08
*** sgw has joined #zuul05:15
*** bhavikdbavishi has quit IRC06:20
*** sgw has quit IRC06:38
*** yolanda has quit IRC06:38
*** weshay|ruck has quit IRC06:38
*** aluria has quit IRC06:38
*** danpawlik has quit IRC06:38
*** fbo|pto has quit IRC06:38
*** SotK has quit IRC06:38
*** noonedeadpunk has quit IRC06:38
*** sgw has joined #zuul06:43
*** yolanda has joined #zuul06:43
*** weshay|ruck has joined #zuul06:43
*** aluria has joined #zuul06:43
*** danpawlik has joined #zuul06:43
*** fbo|pto has joined #zuul06:43
*** SotK has joined #zuul06:43
*** noonedeadpunk has joined #zuul06:43
*** sanjayu_ has quit IRC07:03
openstackgerritSimon Westphahl proposed zuul/zuul master: Prevent Git GC issue between merger and executor  https://review.opendev.org/74716007:09
openstackgerritSimon Westphahl proposed zuul/zuul master: Prevent Git GC issue between merger and executor  https://review.opendev.org/74716007:11
*** saneax has joined #zuul07:13
*** hashar has joined #zuul07:17
*** jpena|off is now known as jpena07:31
*** bhavikdbavishi has joined #zuul07:39
*** felixedel has quit IRC07:50
*** mhu has quit IRC07:50
openstackgerritJan Kubovy proposed zuul/zuul master: WIP: Switch to using zookeeper instead of gearman for jobs (keep gearman for mergers)  https://review.opendev.org/74441608:09
*** nils has joined #zuul08:16
*** tosky has joined #zuul08:22
*** felixedel has joined #zuul08:22
*** mhu has joined #zuul08:22
*** hashar has quit IRC08:33
*** hashar has joined #zuul08:37
*** danpawlik has quit IRC08:45
*** tdasilva has quit IRC08:45
*** tdasilva has joined #zuul08:46
*** danpawlik has joined #zuul08:47
*** hashar has quit IRC08:47
*** hashar has joined #zuul08:53
*** openstackgerrit has quit IRC08:54
*** openstackgerrit has joined #zuul08:56
openstackgerritFelix Edel proposed zuul/zuul master: UI: Show Ref on build(set) result page if no branch is set  https://review.opendev.org/74736308:56
openstackgerritSimon Westphahl proposed zuul/zuul master: Prevent Git GC issue between merger and executor  https://review.opendev.org/74716009:08
openstackgerritFelix Edel proposed zuul/zuul master: PF4: Update builds and buildsets tables + use newest patternfly release  https://review.opendev.org/74611209:22
openstackgerritSorin Sbarnea (zbr) proposed zuul/zuul master: Consolidate js/jsx indentation  https://review.opendev.org/74737709:52
*** bhavikdbavishi has quit IRC10:01
*** bhavikdbavishi has joined #zuul10:02
*** bhavikdbavishi has quit IRC10:17
*** bhavikdbavishi has joined #zuul10:35
*** bhavikdbavishi has quit IRC10:47
*** jhesketh has quit IRC10:53
*** jhesketh has joined #zuul10:55
*** rfolco has joined #zuul11:14
*** rfolco has quit IRC11:15
*** rfolco has joined #zuul11:15
*** jpena is now known as jpena|lunch11:38
*** hashar has quit IRC11:47
*** bhavikdbavishi has joined #zuul11:50
*** _erlon_ has joined #zuul11:57
*** rlandy has joined #zuul11:57
*** hashar has joined #zuul12:02
openstackgerritMatthieu Huin proposed zuul/zuul master: web UI: add Autoholds Requests page  https://review.opendev.org/72930712:28
*** jpena|lunch is now known as jpena12:47
*** Goneri has joined #zuul13:16
openstackgerritMatthieu Huin proposed zuul/zuul master: REST API: improve tenant scoping of autohold, authorizations  https://review.opendev.org/72811813:25
openstackgerritMatthieu Huin proposed zuul/zuul master: REST API: improve tenant scoping of autohold, authorizations  https://review.opendev.org/72811813:25
*** bhavikdbavishi has quit IRC13:46
openstackgerritSorin Sbarnea (zbr) proposed zuul/zuul master: Consolidate js/jsx indentation  https://review.opendev.org/74737714:24
*** avass has quit IRC14:27
mhuhey, can I get this merged please? https://review.opendev.org/#/c/728118/ some improvements on the REST API14:32
*** chandankumar is now known as raukadah14:52
clarkbmhu: I still find the list response incredibly awkward there15:32
clarkbit implies that you can get authorizations outside of your tenant scope there15:32
*** jpena is now known as jpena|off15:37
openstackgerritClark Boylan proposed zuul/nodepool master: Load diskimage configs before building them  https://review.opendev.org/74727715:57
clarkbthat should fix thefailing test_secure_file test15:57
*** tosky has quit IRC16:00
*** rlandy is now known as rlandy|brb16:04
*** rlandy|brb is now known as rlandy16:20
zbrclarkb: https://review.opendev.org/#/c/747377/ -- as someone asked me to change indentation on the pref file, i think we need to do this first.16:44
zbrthis should avoid having noise in reviews about style ;)16:44
*** bhavikdbavishi has joined #zuul16:50
*** hashar has quit IRC16:59
clarkbzbr: I'm not opposed, but really the wrong person to weigh in on js style16:59
clarkbanother option is simply to agree we won't nit pick style too much16:59
fungiand then ignore comments which are style nitpicks17:15
zbrthe ident proposed is based on current practice in the rest of the files, mainly minimal change to make it consistent17:36
zbrlucky for us eslint can also reformat itself, so I did not waste time on it17:36
zbrmy only concern is that this change would not get any support as others will want to avoid having to rebase their own patches :D17:37
openstackgerritClark Boylan proposed zuul/nodepool master: Load diskimage configs before building them  https://review.opendev.org/74727717:38
clarkband that fixes another test issue (it was a race that we had noted in comments even, but this change trips it frequently)17:38
*** dustinc has quit IRC17:45
*** saneax has quit IRC18:09
*** vishalmanchanda has quit IRC18:27
*** nils has quit IRC18:38
*** bhavikdbavishi has quit IRC18:56
openstackgerritMerged zuul/nodepool master: Add image-pause CLI command  https://review.opendev.org/74730618:59
*** rfolco has quit IRC19:29
*** rfolco has joined #zuul19:29
*** yoctozepto1 has joined #zuul19:30
*** yoctozepto has quit IRC19:32
*** yoctozepto1 is now known as yoctozepto19:32
*** tdasilva has quit IRC19:34
*** tdasilva has joined #zuul19:35
*** jhesketh_ has joined #zuul19:38
*** jhesketh has quit IRC19:42
clarkbtobiash: any reason to not approve https://review.opendev.org/#/c/719599/ ? it already has two +2's19:57
clarkbtobiash: also left a -1 on https://review.opendev.org/#/c/747142/ for a small but important detail (I think)20:05
tobiashclarkb: ack and no reason I remember20:08
*** tosky has joined #zuul20:30
*** rfolco has quit IRC20:49
openstackgerritMerged zuul/nodepool master: config: add environment variable substitution  https://review.opendev.org/71959920:57
*** rlandy has quit IRC21:53
*** _erlon_ has quit IRC22:46
*** tosky has quit IRC23:48

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