Thursday, 2020-09-24

*** freenzyfriday has joined #zuul00:34
*** freenzyfriday has quit IRC00:43
*** saneax has quit IRC00:57
*** Goneri has quit IRC00:59
*** freenzyfriday has joined #zuul01:28
*** mordred1 has joined #zuul01:33
*** mordred1 is now known as mordred01:34
*** freenzyfriday has quit IRC01:37
*** freenzyfriday has joined #zuul02:22
*** freenzyfriday has quit IRC02:32
*** saneax has joined #zuul02:35
openstackgerritJan Kubovy proposed zuul/zuul master: Scheduler's pause/resume functionality  https://review.opendev.org/70973504:08
openstackgerritJan Kubovy proposed zuul/zuul master: Separate connection registries in tests  https://review.opendev.org/71295804:08
openstackgerritJan Kubovy proposed zuul/zuul master: Prepare Zookeeper for scale-out scheduler  https://review.opendev.org/71726904:08
openstackgerritJan Kubovy proposed zuul/zuul master: Mandatory Zookeeper connection for ZuulWeb in tests  https://review.opendev.org/72125404:08
openstackgerritJan Kubovy proposed zuul/zuul master: Driver event ingestion  https://review.opendev.org/71729904:08
openstackgerritJan Kubovy proposed zuul/zuul master: Connect merger to Zookeeper  https://review.opendev.org/71622104:08
openstackgerritJan Kubovy proposed zuul/zuul master: Connect fingergw to Zookeeper  https://review.opendev.org/71687504:08
openstackgerritJan Kubovy proposed zuul/zuul master: Connect executor to Zookeeper  https://review.opendev.org/71626204:08
openstackgerritJan Kubovy proposed zuul/zuul master: Improve typings in context of 744416  https://review.opendev.org/75357804:08
openstackgerritJan Kubovy proposed zuul/zuul master: WIP: Switch to using zookeeper instead of gearman for jobs  https://review.opendev.org/74441604:08
*** freenzyfriday has joined #zuul04:11
*** bhavikdbavishi has joined #zuul04:15
*** freenzyfriday has quit IRC04:21
*** evrardjp has quit IRC04:33
*** evrardjp has joined #zuul04:33
*** freenzyfriday has joined #zuul05:06
*** freenzyfriday has quit IRC05:11
*** reiterative has quit IRC05:44
*** reiterative has joined #zuul05:45
*** saneax has quit IRC05:51
*** freenzyfriday has joined #zuul06:00
*** freenzyfriday has quit IRC06:09
*** saneax has joined #zuul06:15
*** jfoufas1 has joined #zuul06:22
*** vishalmanchanda has joined #zuul06:30
*** freenzyfriday has joined #zuul06:30
*** bhavikdbavishi has quit IRC06:42
*** bhavikdbavishi has joined #zuul06:42
*** jpena|off is now known as jpena06:43
*** hashar has joined #zuul06:49
*** jcapitao has joined #zuul07:09
*** bhavikdbavishi has quit IRC07:14
*** smyers has quit IRC07:21
*** saneax has quit IRC07:28
noonedeadpunkAJaeger: Hi! Can you check https://review.opendev.org/#/c/751611/ ? If it is better now?)07:31
AJaegernoonedeadpunk: let me do some quick English nit edits...07:33
noonedeadpunkyeah, sure. that's not my strength:(07:34
openstackgerritAndreas Jaeger proposed zuul/zuul-jobs master: Add support to use stow for ensure-python  https://review.opendev.org/75161107:39
AJaegernoonedeadpunk: did some more then nits - could you review, please?07:40
AJaegernoonedeadpunk: do we use stow in our images and thus could test that this works?07:40
noonedeadpunkAJaeger: we don't yet:(07:41
noonedeadpunkand there was https://review.opendev.org/751610 to fix element a bit07:41
noonedeadpunkAJaeger: changes look good07:42
noonedeadpunkbut we wanted to use it iirc to save up time for building pyenv07:43
AJaegerinteresting. Gave my +2, hope others review 751611 and you get the images working ;)07:44
openstackgerritJan Kubovy proposed zuul/zuul master: Switch to using zookeeper instead of gearman for jobs  https://review.opendev.org/74441607:59
openstackgerritMerged zuul/zuul-jobs master: Add support to use stow for ensure-python  https://review.opendev.org/75161108:01
*** tosky has joined #zuul08:04
*** saneax has joined #zuul08:21
*** yolanda has joined #zuul08:22
*** saneax has quit IRC08:38
*** smyers has joined #zuul08:47
openstackgerritFelix Edel proposed zuul/zuul master: Make reporting asynchronous  https://review.opendev.org/69125308:47
openstackgerritFelix Edel proposed zuul/zuul master: Make reporting asynchronous  https://review.opendev.org/69125308:49
openstackgerritJan Kubovy proposed zuul/zuul master: Improve typings in context of 744416  https://review.opendev.org/75357808:55
*** smyers has quit IRC08:59
*** smyers has joined #zuul09:00
*** hashar has quit IRC09:03
openstackgerritFelix Edel proposed zuul/zuul master: Make reporting asynchronous  https://review.opendev.org/69125309:05
*** saneax has joined #zuul09:10
*** hashar has joined #zuul09:17
*** AJaeger has quit IRC09:27
*** yolanda has quit IRC09:38
*** yolanda has joined #zuul09:39
openstackgerritMatthieu Huin proposed zuul/zuul master: REST API: improve tenant scoping of autohold, authorizations  https://review.opendev.org/72811809:42
openstackgerritMatthieu Huin proposed zuul/zuul-client master: "Defactor" CLI code from zuul project  https://review.opendev.org/75309309:44
openstackgerritMatthieu Huin proposed zuul/zuul-client master: Add cross testing with Zuul  https://review.opendev.org/75126409:48
openstackgerritMatthieu Huin proposed zuul/zuul-client master: Add encrypt subcommand  https://review.opendev.org/75070909:56
*** AJaeger has joined #zuul10:01
*** wuchunyang has joined #zuul10:07
*** wuchunyang has quit IRC10:12
*** jcapitao is now known as jcapitao_lunch10:48
*** sanjayu_ has joined #zuul11:00
*** saneax has quit IRC11:02
*** bhavikdbavishi has joined #zuul11:10
*** wuchunyang has joined #zuul11:22
*** bhavikdbavishi1 has joined #zuul11:23
*** bhavikdbavishi has quit IRC11:25
*** bhavikdbavishi1 is now known as bhavikdbavishi11:25
openstackgerritJan Kubovy proposed zuul/zuul master: Improve typings in context of 744416  https://review.opendev.org/75357811:34
*** jpena is now known as jpena|lunch11:37
*** freenzyfriday has quit IRC12:01
*** wuchunyang has quit IRC12:02
*** jcapitao_lunch is now known as jcapitao12:02
openstackgerritFelix Edel proposed zuul/zuul master: Make reporting asynchronous  https://review.opendev.org/69125312:04
*** rlandy has joined #zuul12:07
openstackgerritJan Kubovy proposed zuul/zuul master: Switch to using zookeeper instead of gearman for jobs  https://review.opendev.org/74441612:13
*** bhavikdbavishi has quit IRC12:29
*** jpena|lunch is now known as jpena12:38
*** ikhan has joined #zuul12:46
*** rfolco has joined #zuul12:49
*** rfolco is now known as rfolco|ruck12:54
*** Goneri has joined #zuul13:00
*** jpena is now known as jpena|off13:15
*** jpena|off is now known as jpena13:21
*** ikhan has quit IRC13:28
*** sanjayu_ has quit IRC13:35
openstackgerritMatthieu Huin proposed zuul/zuul master: [DNM] test zuul-client encrypt subcommand  https://review.opendev.org/75410313:55
openstackgerritMatthieu Huin proposed zuul/zuul master: [DNM] test zuul-client encrypt subcommand  https://review.opendev.org/75410313:59
*** hashar has quit IRC14:18
*** jfoufas1 has quit IRC14:18
openstackgerritBenoît Bayszczak proposed zuul/nodepool master: [provider][aws] now performs a multi-describe at once  https://review.opendev.org/74032814:36
openstackgerritMatthieu Huin proposed zuul/zuul master: [DNM] test zuul-client encrypt subcommand  https://review.opendev.org/75410314:38
openstackgerritBenoît Bayszczak proposed zuul/nodepool master: [provider][aws] now performs a multi-describe at once  https://review.opendev.org/74032814:38
*** benoit_lbc has joined #zuul14:44
*** sshnaidm is now known as sshnaidm|afk14:48
corvusmhu: can you give me a quick summary of what i should do to help you get zuul-client out?  (sorry, i've been distracted last few days).  i'll try to get that done this morning.14:57
mhucorvus, thanks! The ones needed are https://review.opendev.org/#/c/751291/ and https://review.opendev.org/#/c/753093/14:58
mhuwith these and a release, we get a working zuul-client on PyPI that covers the same features as the Zuul CLI when using the REST API14:59
mhuThis one will add the doc online: https://review.opendev.org/#/c/751312/14:59
mhuThen there's adding functional testing which would be nice: https://review.opendev.org/#/c/752039/ https://review.opendev.org/#/c/751264/ and https://review.opendev.org/#/c/753096/15:01
openstackgerritMatthieu Huin proposed zuul/zuul master: [DNM] test zuul-client encrypt subcommand  https://review.opendev.org/75410315:08
*** jcapitao is now known as jcapitao_afk15:23
corvusmhu: i left comments on those first two -- let's keep the release notes to actionable information for end users (eg "added new command", or "deprecated argument")15:29
*** hashar has joined #zuul15:30
fungialso i would question the need for detailed release notes in the client before the tool becomes usable (maybe even not until 1.0?)15:39
fungithough perhaps we should tag 1.0.0 of the client before starting to remove anything from the zuul repo15:39
corvusfungi, mhu: yeah, i think the only note we should have right now is "made initial release" just to put something on the release notes page15:43
corvus(which, of course, will be true once we make the next release)15:43
*** jcapitao_afk is now known as jcapitao15:44
mhucorvus, fungi okay makes sense15:51
fungirelease notes are essentially upgrade guidance to the user, so with no users yet there's nobody to warn about changes they'll encounter when upgrading15:53
*** jpena is now known as jpena|off15:56
openstackgerritMatthieu Huin proposed zuul/zuul master: [DNM] test zuul-client encrypt subcommand  https://review.opendev.org/75410315:59
openstackgerritTobias Henkel proposed zuul/zuul master: Make reporting asynchronous (alternative)  https://review.opendev.org/75412715:59
openstackgerritMatthieu Huin proposed zuul/zuul-client master: Fix integration with Zuul before release  https://review.opendev.org/75129116:01
openstackgerritMatthieu Huin proposed zuul/zuul-client master: "Defactor" CLI code from zuul project  https://review.opendev.org/75309316:03
*** benoit_lbc has quit IRC16:06
tristanCcorvus: fungi: my bad, i told mhu we shouldn't be doing a new release without a release note. I understood there was concerns that users may already tried using zuul-client-0.0.116:13
fungigiven it didn't even install the library, probably not successful attempts16:13
tristanCfungi: I was hoping a releasenote would clear this up for eventual existing user16:15
fungii still feel like there's probably little benefit to release notes prior to a 1.0.0 release (but also am in favor of 1.0.0 as soon as the project is usable)16:17
mhufungi, should the release versions match zuul's (at least Major.Minor) or are we fine with keeping them independent?16:21
*** jcapitao has quit IRC16:24
corvusindependent is fine16:24
fungiyeah, i don't see any point in trying to link them. doing that would suggest that we only support using like-versioned combinations16:32
fungiif we decide we only support using like-versioned combinations, then we should consider bumping the major on the client to match zuul's at that time, but doing that now would be premature16:32
corvustristanC: https://review.opendev.org/753093 and parent lgtm if you want to +316:38
mhuI'm just a bit worried about API versioning but we can cross that bridge when we get there16:49
corvusyeah, if we feel like we should jump versions, that's fine too16:53
openstackgerritMerged zuul/zuul-client master: Fix integration with Zuul before release  https://review.opendev.org/75129117:02
corvusoops, looks like the second change wasn't rebased right17:41
openstackgerritJames E. Blair proposed zuul/zuul-client master: "Defactor" CLI code from zuul project  https://review.opendev.org/75309317:42
*** iurygregory has quit IRC17:45
*** jamesmcarthur has joined #zuul17:48
openstackgerritMerged zuul/zuul-client master: "Defactor" CLI code from zuul project  https://review.opendev.org/75309317:53
*** Eighth_Doctor is now known as Conan_Kudo17:58
*** Conan_Kudo is now known as Eighth_Doctor17:58
*** iurygregory has joined #zuul17:58
*** hamalq has joined #zuul18:04
*** mach1na has joined #zuul18:19
*** jamesmcarthur has quit IRC18:21
*** mach1na has quit IRC18:21
*** mach1na has joined #zuul18:22
*** tosky has quit IRC18:22
*** mach1na has quit IRC18:23
*** jamesmcarthur has joined #zuul18:23
*** jamesmcarthur has quit IRC18:31
*** jamesmcarthur has joined #zuul18:38
*** hashar has quit IRC18:56
*** openstackgerrit has quit IRC19:14
*** jamesmcarthur has quit IRC19:21
*** jamesmcarthur has joined #zuul19:23
clarkbtristanC: AJaeger see comment on https://review.opendev.org/#/c/752757/5 I think its reasonably tested due to https://review.opendev.org/#/c/752743/ but if we want to test it going forward doing more is probably a good idea19:39
*** rfolco|ruck is now known as rfolco|ruck|brb19:53
*** jamesmcarthur has quit IRC20:13
*** jamesmcarthur has joined #zuul20:14
*** rfolco|ruck|brb has quit IRC20:36
*** rfolco|ruck|brb has joined #zuul20:37
*** rfolco|ruck|brb is now known as rfolco|ruck20:37
*** rfolco|ruck has quit IRC21:11
*** jamesmcarthur has quit IRC21:24
ianwzuul-maint: https://review.opendev.org/#/c/741385/ is a rework of the search bar for pf4, would be nice for a +2 and will clear another pf4 update21:32
*** jamesmcarthur has joined #zuul21:41
*** jamesmcarthur has quit IRC21:42
*** jamesmcarthur_ has joined #zuul21:42
*** openstackgerrit has joined #zuul22:00
openstackgerritTristan Cacqueray proposed zuul/zuul-jobs master: ensure-zookeeper: add role to setup zookeeper  https://review.opendev.org/75275722:00
*** armstrongs has joined #zuul22:12
*** armstrongs has quit IRC22:21
*** freenzyfriday has joined #zuul22:23
*** freenzyfriday has quit IRC22:27
*** vishalmanchanda has quit IRC22:39
*** jamesmcarthur has joined #zuul22:41
openstackgerritMerged zuul/zuul master: PF4: Update filter toolbar on builds and buildsets page  https://review.opendev.org/74138522:44
*** jamesmcarthur_ has quit IRC22:45
openstackgerritMerged zuul/zuul master: PF4: Update builds and buildsets tables + use newest patternfly release  https://review.opendev.org/74611222:51
*** freenzyfriday has joined #zuul23:13
*** freenzyfriday has quit IRC23:18
clarkbtristanC: thats an interesting linting failure on the ensure-zookeeper change "git in dirty status, reporting as failure23:22
ianwzuul-main: https://review.opendev.org/754230 promote-deployment: fix synchronize should fix the opendev-promote-javascript-deployment that has been failing for ... a long time :/23:24
ianwzuul-maint: ^23:24
openstackgerritTristan Cacqueray proposed zuul/zuul-jobs master: ensure-zookeeper: add role to setup zookeeper  https://review.opendev.org/75275723:33
tristanCclarkb: indeed, hope next ps is the way to go23:34
tristanCianw: it doesn't seem like zuul-maint can approve change on this project23:34
ianwclarkb: i think the auto-job generator runs, and if "git diff" is non-zero it indicates there's something not committed?23:38
clarkbah23:38
ianwtristanC: oh, yeah probably not.  i wonder if we should expand it because if you're familiar with zuul-jobs then it's all very similar23:40
*** rlandy has quit IRC23:44
clarkbianw: if https://review.opendev.org/752757 looks good to you that is the change we need to land https://review.opendev.org/#/c/752743/2 and remove fedora30 from nodepool. Then dib will be the last project using f3023:48
ianwok, LGTM; i don't know how we keep that latest_version field as the actual latest_version ...23:55
*** jamesmcarthur has quit IRC23:59

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