Monday, 2019-10-14

*** swest has quit IRC01:08
*** swest has joined #zuul01:22
*** bhavikdbavishi has joined #zuul03:04
*** bhavikdbavishi1 has joined #zuul03:08
*** bhavikdbavishi has quit IRC03:10
*** bhavikdbavishi1 is now known as bhavikdbavishi03:10
*** bhavikdbavishi has quit IRC03:53
*** bhavikdbavishi has joined #zuul03:53
*** bolg has joined #zuul04:04
*** swest has quit IRC04:34
*** sanjayu_ has joined #zuul04:53
*** mhu has quit IRC04:53
*** swest has joined #zuul05:26
*** bolg has quit IRC05:27
*** bolg has joined #zuul05:28
*** bhavikdbavishi has quit IRC05:38
*** AJaeger has quit IRC05:52
*** AJaeger has joined #zuul06:14
*** pcaruana has joined #zuul06:43
*** bhavikdbavishi has joined #zuul06:54
*** aluria has joined #zuul06:57
*** bhavikdbavishi1 has joined #zuul06:57
*** bhavikdbavishi has quit IRC06:59
*** bhavikdbavishi1 is now known as bhavikdbavishi06:59
*** raukadah is now known as chandankumar07:11
*** gtema has joined #zuul07:16
*** themroc has joined #zuul07:28
*** jangutter has joined #zuul07:42
*** jpena|off is now known as jpena07:46
*** hashar has joined #zuul08:08
*** themroc has quit IRC09:04
*** themroc has joined #zuul09:06
*** themroc has quit IRC09:15
*** themroc has joined #zuul09:15
*** themroc has quit IRC09:23
*** jamesmcarthur has joined #zuul09:40
*** jamesmcarthur has quit IRC09:45
*** bhavikdbavishi has quit IRC10:11
*** hashar has quit IRC10:13
*** themroc has joined #zuul11:00
*** hashar has joined #zuul11:11
*** sanjayu_ has quit IRC11:25
*** jpena is now known as jpena|lunch11:31
*** pcaruana has quit IRC11:34
*** avass has joined #zuul11:36
*** pcaruana has joined #zuul11:41
*** themr0c has joined #zuul11:56
*** themroc has quit IRC11:59
*** rfolco has joined #zuul12:08
openstackgerritSorin Sbarnea proposed zuul/zuul-jobs master: configure-mirrors: add support for centos-8  https://review.opendev.org/68811812:10
*** rfolco is now known as rfolco|ruck12:12
avassI think I found a but with the gerrit driver12:17
avassIt seems that setting a label in a pipeline configuration is case insensitive but checking which labels are allowed to be missing in a dependent pipeline is case sensitive12:18
openstackgerritAlbin Vass proposed zuul/zuul master: Label check should be case insensitive  https://review.opendev.org/68839312:18
avassit also only checked that the missing labels is a proper subset, meaning if the missing labels and the labels that were allowed to be missing were equal it found also fail12:20
*** jpena|lunch is now known as jpena12:27
*** bhavikdbavishi has joined #zuul12:30
AJaegeravass: could you add tests when you fix such issues, please?12:31
avassAJeager: Sure12:32
*** bhavikdbavishi1 has joined #zuul12:33
*** bhavikdbavishi has quit IRC12:34
*** bhavikdbavishi1 is now known as bhavikdbavishi12:34
openstackgerritSorin Sbarnea proposed zuul/zuul-jobs master: configure-mirrors: add support for centos-8  https://review.opendev.org/68811812:36
pabelangerclarkb: I'll create a PR today for people to review12:51
*** gtema has quit IRC13:00
*** gtema has joined #zuul13:00
avassAJeager: Looks like I'm gonna be busy for a couple of hours. I will fix it later tonight.13:03
*** openstackstatus has quit IRC13:15
*** sanjayu_ has joined #zuul13:17
openstackgerritSorin Sbarnea proposed zuul/zuul-jobs master: configure-mirrors: add support for centos-8  https://review.opendev.org/68811813:19
*** gtema has quit IRC13:42
*** sanjayu_ has quit IRC13:45
*** chandankumar is now known as raukadah13:56
tobiashavass: gerrit changed its behavoir a year ago or so, so we made this case sensitive. Thus actually everything should be case sensitive13:58
*** themroc has joined #zuul13:59
tobiashBecause case insensitive reporting doesn't work correctly13:59
*** themr0c has quit IRC13:59
openstackgerritSorin Sbarnea proposed zuul/zuul-jobs master: configure-mirrors: add support for centos-8  https://review.opendev.org/68811814:01
*** themr0c has joined #zuul14:02
*** themroc has quit IRC14:04
corvusmordred: was that 404 the same missing image manifest as before, or a different one?  (trying to figure out if we should go see if the logs caught anything)14:06
*** yolanda__ has joined #zuul14:06
*** yolanda has quit IRC14:09
corvusmordred: looks like it's referring to an image upload on oct 9; i think that's before we made the changes to the registry; so i'll recheck 678875 to get a new build uploaded to the intermediate registry14:10
*** jamesmcarthur has joined #zuul14:13
*** jamesmcarthur has quit IRC14:14
*** jamesmcarthur has joined #zuul14:14
mordredcorvus: ok. good idea14:14
*** hashar has quit IRC14:14
*** michael-beaver has joined #zuul14:18
*** bolg has quit IRC14:27
*** themr0c has quit IRC14:43
*** themr0c has joined #zuul14:43
*** ffloreth__ has joined #zuul14:51
*** themr0c has quit IRC14:54
*** hashar has joined #zuul15:07
*** ttx has quit IRC15:18
clarkbtristanC: not sure if you saw but ianw tracked down the NetworkManager problem to glean turning interfaces up whcih led to NM ignoring them even if NM was told to manage them :/15:19
clarkbtristanC: we've updated glean to not turn instances up in the systemd case which relies on udev to trigger glean on a specific interface (deciding that if udev runs glean against the interface we manage it with NM and let it bring it up)15:20
tristanCclarkb: alright, so iiuc my glean changes are not necessary and i can abandon them?15:22
clarkbtristanC: yes that is my undertsanding15:22
tristanCcool, glad this is getting fixed :-)15:22
clarkbif zuulians can keep an eye out for those fortnebula ip issues just to confirm we don't see them anymoer that would be great15:25
*** mattw4 has joined #zuul15:26
*** ttx has joined #zuul15:29
openstackgerritSorin Sbarnea proposed zuul/zuul-jobs master: configure-mirrors: add support for centos-8  https://review.opendev.org/68811815:31
openstackgerritJames E. Blair proposed zuul/project-config master: Add third-party-check pipeline  https://review.opendev.org/68842815:31
Shrewsclarkb: oh, that's great news. that should help with nodepool stability a bit15:32
Shrewstests, that is15:32
*** bhavikdbavishi has quit IRC15:37
*** bolg has joined #zuul15:43
openstackgerritSorin Sbarnea proposed zuul/zuul-jobs master: configure-mirrors: add support for centos-8  https://review.opendev.org/68811815:46
*** bolg has quit IRC15:50
*** mattw4 has quit IRC15:52
*** mattw4 has joined #zuul15:56
jangutterquick question for the native English speakers. "will be cleared after the cleanup interval has passed" or "will be cleared after the cleanup interval has elapsed"?15:57
jangutteror neither.15:57
clarkbI think both are correct? elapsed is specific to time and carried slightly more info as a result15:59
*** openstackstatus has joined #zuul16:04
*** ChanServ sets mode: +v openstackstatus16:04
*** hashar has quit IRC16:05
openstackgerritSorin Sbarnea proposed zuul/zuul-jobs master: configure-mirrors: add support for centos-8  https://review.opendev.org/68811816:07
openstackgerritJan Gutter proposed zuul/nodepool master: Fix typo in port-cleanup-interval description  https://review.opendev.org/68844016:07
*** hashar has joined #zuul16:07
*** ffloreth__ has quit IRC16:08
jangutterclarkb: thanks, I "passed" that into the review.16:11
mnaserim building docker images for geard (adding the jobs etc)16:16
mnasershould i aim to use opendevorg/gear ?16:16
clarkbit is such a simple service I doubt there is much reason to need to customize anything16:16
clarkbmaybe for ssl config, but that should already be configurable via bind mounts16:17
mnaserclarkb: well i just kinda want to get it published on dockerhub16:17
mnaserto be able to consume the images16:17
mnaser(its not there)16:17
clarkbopendevorg/gear does or does not exist?16:18
mnaserdoes not16:18
clarkb(your initial question implied ti did exist)16:18
*** hashar has quit IRC16:18
mnasersorry, i was wondering if it was a good place to live16:18
clarkbI'm fine with it living there16:18
mnaserlooks like its prefixed opendev/ but the code lives in the openstack tenant16:19
clarkbmost of opendev/ is still tested in the openstack/ tenant16:20
*** igordc has joined #zuul16:21
mnaserhm i might have signed myself up for a tad bit more work16:22
clarkbthe tenancy in zuul shouldn't affect that16:23
mnaserright but the jobs to upload to opendevorg seem to live in the opendev tenant16:23
mnasernot seeing any child jobs other than help ones for build/upload16:23
clarkbthose need to be inherited from on a per projcet basis aiui16:23
clarkbbut also those jobs are in base-jobs which are loaded by all tenants16:23
clarkblook in opendev/system-config16:23
clarkbor zuul/zuul for examples16:24
*** armstrongs has joined #zuul16:26
*** michael-beaver has quit IRC16:37
corvusi bet it's easy to move gear to opendev16:39
corvusi'll take a quick look at that16:40
openstackgerritSorin Sbarnea proposed zuul/zuul-jobs master: configure-mirrors: add support for centos-8  https://review.opendev.org/68811816:43
*** hashar has joined #zuul16:45
armstrongshey is there any info on when we expect the bitbucket driver to land? My company may be switching from github to bitbucket so i'm keen that zuul can still be used if they do16:47
corvusarmstrongs: i think it's pretty close, however, ofosos hasn't had time to work on it in the past 3 months.  whenever he or someone else has time to address the review comments in https://review.opendev.org/657837 i think it could be merged in short order.16:49
corvuser, 2 months, sorry16:49
corvusmnaser: https://review.opendev.org/68845116:51
armstrongscool i noticed it only works for self-hosted and not the cloud one. I'm by no means a bitbucket expert but if it were to have the cloud version do you have any idea what is significantly different. Given how invested we are in zuul if i had a bitbucket could probably get the team to help out with it16:52
mnasercorvus: ok cool in that case it might be good for me to push up a patch which runs the tox jobs because there is no in-repo .zuul.yaml16:52
corvusmnaser: oh sorry i meant to link you to that one: https://review.opendev.org/68845516:53
mnaserah yes okay even better then :)16:53
corvusmnaser: but it's going to throw errors until 688451 lands16:53
mnasercorvus: might need to be squashed eventually with https://review.opendev.org/#/c/688450/16:54
mnasercorvus: unless you want to merge that quickly now to avoid squashing things :)16:54
corvusmnaser: was it broken?16:54
mnaseryes, see earlier commit https://review.opendev.org/#/c/671674/16:55
mnaserfrom 3 days ago16:55
corvusmnaser: ah neat16:55
corvuslet's merge your fix now then16:55
mnasersphinxcontrib.programoutput depends on newer sphinx features and it was running ancient sphinx so i just fixed that up now16:55
mnaserso i did a bunch of clean up while i was at it too16:55
*** jpena is now known as jpena|off16:55
corvusmordred: want to +3 https://review.opendev.org/688450 ?16:56
openstackgerritSorin Sbarnea proposed zuul/zuul-jobs master: configure-mirrors: add support for centos-8  https://review.opendev.org/68811816:56
corvusso we'll get that in, then the tenant move, then fix whatever we broke in the tenant move, then off to the races with docker16:56
mnaser++16:57
corvusarmstrongs: i don't know the answer to that -- maybe ofosos does, or maybe noorul (who is not online right now)16:58
armstrongsthanks lets see what they say. :)16:59
mnasermordred: did you mean to CR+2 without workflow on the doc fix change? :)16:59
corvusarmstrongs: i do know that we talked with ofosos about the fact that it's written based on a polling driver, and that ultimately it should be expanded to support webhook events (whatever bitbucket calls those); i imagine that might be something one would want for the cloud version17:00
openstackgerritSorin Sbarnea proposed zuul/zuul-jobs master: configure-mirrors: fix mirror_fqdn variable  https://review.opendev.org/68846817:02
mnaserlooks like the docker image built cleanly https://review.opendev.org/#/c/688446/4 so thats nice17:02
armstrongsyeah defo. If we do decide to pivot we should be able to get some dev resource to help as the alternative of removing zuul isnt looking like an option given nothing else does what it does17:02
armstrongs:)17:02
corvusarmstrongs: great, i think ofosos has said he'd appreciate help in the past, and i certainly think it'd be better for the community overall to have multiple people familiar with the driver, so i'm sure that would be welcome all around :)17:03
corvusmnaser: i added a +W to 45017:05
mnaserok ill wait for that to merge and +W the project-config change17:05
corvusmnaser: sounds good... the tenant move will require an ansible run, so could take 15-30m after it lands to take effect; then a recheck on the .zuul.yaml change should show real results.  i'm going to afk for a bit17:06
mnasercorvus: cool, ill try to check in here and there17:08
openstackgerritSorin Sbarnea proposed zuul/zuul-jobs master: config-mirrors: Consolidate dnf/yum handler  https://review.opendev.org/68847917:09
ofososarmstrongs: I could definitely use some help17:09
*** hashar has quit IRC17:44
*** gtema has joined #zuul18:03
corvustristanC: https://review.opendev.org/687986 already has reviews, but i wanted to remind you that you wanted to take a look at it since we were talking about how that works last week.  i'm not blocked or anything -- i still have more testing i want to set up.18:21
*** hashar has joined #zuul18:22
openstackgerritTobias Henkel proposed zuul/zuul master: Don't install zuul in the bindep tox testenv  https://review.opendev.org/65225118:43
openstackgerritTobias Henkel proposed zuul/zuul master: Add nodesets API route  https://review.opendev.org/60587718:51
openstackgerritTobias Henkel proposed zuul/zuul master: Extract an abstract base Parser class  https://review.opendev.org/54561019:10
openstackgerritMerged zuul/project-config master: Add third-party-check pipeline  https://review.opendev.org/68842819:22
AJaegernow we can merge https://review.opendev.org/688427 ^19:23
openstackgerritMerged zuul/nodepool master: Fix typo in port-cleanup-interval description  https://review.opendev.org/68844019:23
*** jamesmcarthur has quit IRC20:21
*** pcaruana has quit IRC20:26
*** jamesmcarthur has joined #zuul20:30
*** hashar has quit IRC20:42
openstackgerritMerged zuul/zuul master: Don't install zuul in the bindep tox testenv  https://review.opendev.org/65225120:45
*** mattw4 has quit IRC21:02
*** mattw4 has joined #zuul21:02
*** mattw4 has quit IRC21:25
*** mattw4 has joined #zuul21:25
corvustristanC, mordred, clarkb: i think i understand enough about oci/crio/podman/containers/kubernetes to do the multi-registry zuul-registry thing with k8s.  it may take a few changes to zuul-jobs roles to get there; i'm going to work on that next.21:26
fungiosf staff are putting together a community newsletter this week... anybody have zuul-specific news we should be including/pointing out? i suppose an ansiblefest redux is good to include, if any of you have articles or blog posts you want linked from that21:27
corvusfungi: would the zuul-related vexxhost product announcement from ansiblefest be appropriate to include?21:28
fungilikely, i'll mention it to the editors21:32
fungithanks!21:32
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: install-kubernetes: add support for cri-o runtime  https://review.opendev.org/68857821:47
*** gtema has quit IRC21:56
*** avass has quit IRC22:12
*** armstrongs has quit IRC23:00
*** armstrongs has joined #zuul23:00
*** jamesmcarthur has quit IRC23:17
*** jamesmcarthur has joined #zuul23:19
*** jamesmcarthur has quit IRC23:30
*** jamesmcarthur has joined #zuul23:35
*** jamesmcarthur has quit IRC23:48
*** armstrongs has quit IRC23:50
*** mattw4 has quit IRC23:50
*** jamesmcarthur has joined #zuul23:55

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