Friday, 2017-10-13

jeblairianw: okay, i just commented out the vos release in a local copy of the script00:00
openstackgerritIan Wienand proposed openstack-infra/openstack-zuul-jobs master: unbound: Add check for undefined default_ipv6  https://review.openstack.org/51165300:00
ianwpabelanger / clarkb: ^ i feel like this is bug https://github.com/ansible/ansible/issues/23675 but i guess it's clear enough00:00
ianwjeblair: ++, fingers crossed00:01
openstackgerritPaul Belanger proposed openstack-infra/system-config master: Bump reprepro timeout up to 90m  https://review.openstack.org/51165400:01
pabelangerjeblair: clarkb: ianw: ^reprepro timeout bump00:01
ianwpabelanger: is the cron update an hour though?00:01
fungiclarkb: find will spend its time iterating through directories too. i expect the shell glob to be an up-front cost vs find traversing the entire tree. in the end maybe they're similarly efficient00:01
jeblairianw: 2h -- 0 */2 * * *00:01
ianwoh, that's reasonable then00:02
openstackgerritClark Boylan proposed openstack-infra/openstack-zuul-jobs master: Reparent ozj integration jobs to base-test for testing  https://review.openstack.org/51164600:02
clarkbianw: I rebased ^ on your change which should end up testing it00:02
openstackgerritMerged openstack-infra/devstack-gate master: Stop it with the moving then copying of cache files  https://review.openstack.org/51126000:02
mnaser^^^ yay00:02
ianwclarkb: thanks00:02
jeblairwe're past the update and on to 'checking state of mirror'00:02
fungiclarkb: basically i avoided find because we could provide an exact set of globs to filter vs find having to look inside every directory for a potential match00:03
clarkbfungi: ya00:03
pabelangerianw: why the fact? and not just use both statements in the task below?00:03
ianwpabelanger: avoid duplicating it, thought it was a bit clearer00:03
jeblair'checking state of mirror' takes about 15-20 mins.00:04
pabelangerianw: you could make that a block too00:04
fungiclarkb: at least in my experience, with very wide and deep file trees, a specific shell glob will win over a subdirectory match with find00:04
jeblairclarkb: you want to do one mirror or all of em?00:04
pabelangerianw: +3, we can refactor later00:04
clarkbjeblair: we probably want to do all of them? Since any of them could still be broken?00:04
jeblairclarkb: sorry, i meant switch one mirror host to read-write or all mirror hosts00:05
jeblairMissing file pool/main/a/apparmor/apparmor_2.10.95-0ubuntu2.6_amd64.deb00:05
jeblairFiles are missing for 'apparmor'!00:05
jeblairThere have been errors!00:05
clarkbjeblair: oh we probably want to flip all of them to get data back quicker00:05
jeblairianw: how about i perform checksum.db removal on that file then re-run?00:06
clarkbbut can also flip in the biggest region too and probably get data back pretty quick00:06
*** sbezverk has quit IRC00:06
ianwjeblair: ok ... odd that it didn't happen before?00:06
fungii know i shouldn't jinx us, but here's hoping we don't have files spontaneously disappearing out of afs00:08
jeblairthe script runs deleteifunereferenced00:08
jeblairi suspect we just deleted it ourselves00:08
fungiyeah, that seems far more likely00:08
ianwbut only after the "update" phase?00:08
clarkbianw: pabelanger http://logs.openstack.org/46/511646/2/infra-check/base-integration-centos-7/4d11e4f/job-output.txt.gz#_2017-10-13_00_05_07_944763 will we not test that premerge?00:08
fungiin other news, we're just about to hit the 2% free inodes mark00:09
clarkbok I've got to take the kids for a walk. Back later to check on the ara on failure stuff00:09
ianwclarkb: do you mean unworkflow it?  i'm ok with that00:10
jeblairianw, fungi: that file is not in my dump of checksums.list from earlier00:11
fungistrange00:11
jeblairianw, fungi: so that means that it hasn't been in the filesystem for a while00:11
openstackgerritPaul Belanger proposed openstack-infra/project-config master: Set v3 nodepool citycloud timeout to 600  https://review.openstack.org/51165600:11
jeblair(we didn't just delete it, and it didn't disappear)00:11
fungiyeah, extra strange00:11
pabelangerjeblair: fungi: clarkb: added launch-timeout to citycloud also. ^ See to be having issues building nodes, but when working usually 30 second boots00:11
jeblairi don't know why the check thinks we need it though00:12
jeblairit's not in the list of unreferenced files00:12
*** smatzek has quit IRC00:13
fungidoes it exist in the source site reprepro copies from?00:13
*** smatzek has joined #openstack-infra00:13
jeblairfungi: i don't know00:13
fungii'll check an upstream mirror00:14
pabelangerHmm, debian-jessie failed again on mirrors00:15
pabelangerI'm going to hold one and see what is going on for zuulv300:15
jeblairpabelanger: can you be more specific?00:15
pabelangerhttp://logs.openstack.org/53/511653/1/infra-gate/base-integration-debian-jessie/25c9216/ara/result/983d76c0-aee7-4213-9036-30f45539b0c1/00:15
pabelangerwe should not be using upstream mirrors there00:16
pabelangerbut our AFS00:16
*** gmann_afk is now known as gmann00:16
jeblairk00:16
jeblairfungi: that file is referenced by the xenial-security package list00:17
ianwhmm, yes one of the first things we tried was removing xenial-security & xenial-updates yesterday from the update00:17
ianwif things in them are borked, that might be related00:18
*** smatzek has quit IRC00:18
*** xinliang has quit IRC00:18
fungijeblair: i see it at http://security.ubuntu.com/ubuntu/pool/main/a/apparmor/00:18
fungiso the security mirror should in theory be wanting it00:19
jeblairianw: we're telling it to mirror xenial-security now, yeah?00:19
jeblairso i'm surprised that reprepro update did not download it00:19
jeblairi'm considering running that again00:19
ianwjeblair: yes, that should definitely be in there00:19
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Move tox_envlist into job variables for releasenote jobs  https://review.openstack.org/51162800:19
ianwi removed it for one run as we tried that, but put the original file back00:19
*** sdague has quit IRC00:19
jeblairrunning update made no changes00:21
openstackgerritPaul Belanger proposed openstack-infra/zuul-jobs master: Use sources.list.d for Debian with APT  https://review.openstack.org/51165800:27
pabelangerjeblair: dmsimard: clarkb: ianw: ^ should be our fix for debian mirrors on zuulv300:27
pabelangerwill confirm in logs first00:27
ianwhuh, yeah that seems right00:28
*** thorst has joined #openstack-infra00:29
jeblairi'm trying to figure out what database files are involved in the 'check' command to see if we can figure out if we're missing something from one of them00:30
tonybHuh, it turns out I don't actually understand the zuulv2 config well enough to write that tool. Rats.00:31
jeblairtonyb: here's an idea: take the console output of a layout check job and parse it00:31
jeblairtonyb: zuul outputs its "compiled" configuration into the log00:32
*** thorst has quit IRC00:32
jeblairtonyb: it'll tell you what jobs run for what projects under what pipelines.  it will also have branch matching information in there00:32
jeblairtonyb: it's not intended to be parsable -- it's a debugging aid, but with a few heuristic regex matches, it may get pretty close to what you want00:33
jeblairtonyb: or, you know, forget i said anything and go do something fun.  :)00:33
ianwclarkb: that didn't appear to use the unbound change? https://review.openstack.org/#/c/511646/00:33
clarkbianw: ya I wonder if it has to be post merge00:35
clarkbbecause it is prerun?00:35
tonybjeblair: hehe :) Thanks for the idea I'll look at it00:35
*** cody-somerville has joined #openstack-infra00:36
ianwclarkb: yeah, might be too early00:36
*** hamzy has joined #openstack-infra00:36
pabelangerianw: clarkb: which patch?00:37
clarkbpabelanger: 511646 and its parent00:37
jeblairianw: i'm worried we now need to do step 4 -- packages.db.00:38
*** aeng has quit IRC00:39
clarkbI've approved the debian package fix which shold allow us to get in ianw's change and then test the ara on failure thing00:39
ianwjeblair: yeah, touching that dists/ directory i was very worried about :/00:39
*** Goneri has quit IRC00:39
pabelangerlooking00:39
*** aeng has joined #openstack-infra00:39
jeblairianw: hrm, it's in packages.db00:40
ianwjeblair: security / updates aren't that big, should we drop them & reload?00:41
ianwi think that means remove from the config file00:41
pabelangerclarkb: ya, base-test won't be able to use unmerged role. You'd need to write a test in ozj, which then uses unbound role00:41
ianwit tells you they're not there, clear them out00:41
ianwrun delete unreferenced00:41
ianwthen start again?00:42
jeblairianw: is there any chance when you ran rereference you did so with a config without xenial-security?00:42
ianwi did not intentionally do that, but isn't not impossible00:42
ianwthat was a terrible sentence00:43
ianwi may have done that00:43
clarkbpabelanger: best test is what I want to test though, we'll just merge ianws fix and recheck00:43
*** claudiub|2 has quit IRC00:43
jeblairianw: hrm.  i was wondering if running rereference might be a way to fix this, but that's only likely if there was something missing from your earlier run of it.00:43
pabelangerclarkb: right, but that is in project-config (trusted playbook) it won't be able to use the unmerged role00:43
*** kaisers1 has joined #openstack-infra00:44
pabelangerya, merge / recheck should work00:44
jeblairianw: xenial-security|main|amd64 pool/main/a/apparmor/apparmor_2.10.95-0ubuntu2.6_amd64.deb00:44
jeblairthat's from references.db00:45
jeblairianw: so that looks correct00:45
ianwjeblair: to flesh out my idea, it's remove them from /etc/reprepro/ubuntu/distributions, then run "reprepro --delete clearvanished", then an update, then put them back00:46
jeblairianw: your idea is sounding like the best approach.00:46
*** kaisers has quit IRC00:46
*** LindaWang has joined #openstack-infra00:47
*** kiennt26 has joined #openstack-infra00:47
jeblairreprepro --confdir /etc/reprepro/ubuntu -VVVV --delete clearvanished00:47
jeblairianw: ^?00:47
ianwyep00:47
ianwit will tell you to run that if you modify distributions anyway00:48
ianwat least it did for me00:48
*** kiennt26 has quit IRC00:48
jeblair403 files lost their last reference.00:49
jeblair(dumpunreferenced lists such files, use deleteunreferenced to delete them.)00:49
*** kiennt26 has joined #openstack-infra00:49
jeblairi will not run deleteunreferenced00:49
ianwno, yeah i never ran thta00:49
jeblairianw: so now, still with xenial-security omitted, run update?00:49
ianwi think so, and see if it works?00:50
*** eroux has quit IRC00:50
ianwhopefully it just finishes all ok ...00:50
jeblairokay that's done00:50
jeblairno mention of xenial-security00:50
pabelangerze05 doesn't appear to be running zuul-merger00:50
jeblairpabelanger: executors are not supposed to run mergers00:50
pabelangerso, zm0500:51
pabelangeralso zm08 too00:51
ianwjeblair: that seems good?!  in theory we put it back in, it starts fresh ...00:51
pabelangerand zm06 and zm07 have hung git processes00:51
*** cody-somerville has quit IRC00:51
openstackgerritMerged openstack-infra/zuul-jobs master: Use sources.list.d for Debian with APT  https://review.openstack.org/51165800:51
jeblairianw: okay, replaced distributions file, running update again00:52
pabelangeradding to zuulv3 etherpad so we can look at00:52
jeblairianw: it's getting lots of debs00:53
ianw403 deb's hopefully, from the prior output00:53
jeblairi lost count after 300:54
openstackgerritMerged openstack-infra/system-config master: Bump reprepro timeout up to 90m  https://review.openstack.org/51165400:54
*** huanxie has joined #openstack-infra00:55
pabelangerokay, added zuul-merger issues to zuulv3-issues etherpad, getting late for me to look into them. Can help in the morning on those00:56
jeblairianw: done.  i'll run checkpool and check now manually?00:56
ianwjeblair: ++ ... have we fixed it?!00:57
ianw(we being you :)00:57
jeblairthe error happened at the end of the check command last time, so 15m away from finding out00:58
jeblairprobably shoulda dropped the -VVVV for check01:00
*** ramishra has quit IRC01:00
*** ramishra has joined #openstack-infra01:01
*** cody-somerville has joined #openstack-infra01:02
*** cody-somerville has joined #openstack-infra01:02
jeblairianw: exit 0!01:04
ianwthat's the best exit of them all01:04
jeblairianw: i'm going to remove the unreferenced file for the previous run, and run my modified (no vos release) script again?01:04
ianwas in the output of dumpunreferenced?01:06
ianwis there much in there?01:06
jeblairianw: er, i think there was.  i just deleted it.01:06
ianwmaybe just generate it and make sure we're not going to somehow delete everything01:07
*** cody-somerville has quit IRC01:07
jeblairianw: the script generates the list and uses it on the *next* run01:07
jeblairwhich is why i deleted it this time, i didn't want to delete things from our previous run with bad data01:08
jeblairi'm running the script now.  it has complete the update (a noop), and generated a new unreferenced list.  it has 795 files in it.01:08
jeblairit's running a check now.01:08
ianwright, just looking at those files, seeing if they have a common thread01:09
jeblairif this works, i'm thinking run the script again. it will delete the unferenced files and then run another check.  if that works, maybe we're ok?  (or should we run it one more time?)01:09
*** thorst has joined #openstack-infra01:10
*** stakeda has joined #openstack-infra01:10
ianwjeblair: ok, so looking at that list, yeah it seems reasonable that they're just old packages01:10
ianw(rather than packages we want but have somehow decided to delete)01:11
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Switch to RW afs mirror  https://review.openstack.org/51166001:11
*** cody-somerville has joined #openstack-infra01:11
jeblairianw: it just finished successfully!01:11
jeblairianw: i'd like to grab dinner now... may i make the following suggestion:01:11
jeblairianw: you run "/root/corvus/reprepro-mirror-update /etc/reprepro/ubuntu mirror.ubuntu" as many times as you like to feel comfortable, then merge 511660.01:12
ianwok, let it do the vos release?01:12
jeblairianw: /root/corvus/reprepro-mirror-update does not have the vos release01:13
jeblair511660 will let us test the mirror without a vos release01:13
ianwoh, right yeah i see01:13
jeblaironce we have jobs that have run with 511660 in place, we can revert it.  if they worked, we can vos release and return to normal.  if they did not work, we will still have the old read-only volume to keep things going while we regroup.01:14
ianwjeblair: ok, is cron turned off?01:15
jeblairi think clarkb maybe wanted to let that run for a while to increase the chance of random jobs hitting some bad package.01:15
jeblairianw: yes, the cron entry is commented out, and mirror-update is in the emergency file.01:15
ianwok, cool01:16
jeblairianw: great, thanks!  i'm off to dinner now.01:16
*** Apoorva_ has joined #openstack-infra01:17
clarkbya was hoping to try and shake any problems loose before doing a vos release01:19
clarkbsince at that point we are committed aiuu01:19
*** aeng has quit IRC01:19
*** Apoorva has quit IRC01:20
*** ijw has quit IRC01:20
*** thorst has quit IRC01:20
*** Apoorva_ has quit IRC01:21
ianwclarkb: yep, i think if we commit that change, even then just running apt-get update should pull in everything new, we can check the problem packages, gnutls, curl etc01:21
ianwok, i ran the update and it removed the unreferenced files, the ones that all appeared to be just from regular updates01:21
ianwover the past few days01:22
ianwi'll run again, and it should basically do nothing now01:22
ianwreprepro is as confusing as hell, but surprisingly robust it seems01:24
ianwyay, as expected the update has done nothing01:25
ianwergo, i believe we have a completely up-to-date mirror01:25
*** cuongnv has joined #openstack-infra01:27
clarkbnice01:28
clarkbI +2'd the rw path change01:28
ianwi think i'm going to write some of this down while it's still fresh in case it happens again01:30
*** hongbin has joined #openstack-infra01:32
*** felipemonteiro has quit IRC01:36
openstackgerritMerged openstack-infra/system-config master: Switch to RW afs mirror  https://review.openstack.org/51166001:36
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: unbound: Add check for undefined default_ipv6  https://review.openstack.org/51165301:39
*** esberglu has joined #openstack-infra01:39
*** esberglu has quit IRC01:39
*** cody-somerville has quit IRC01:39
*** esberglu has joined #openstack-infra01:39
clarkbI just rechecked the ozj check for ara only on failures01:40
*** esberglu has quit IRC01:44
clarkbno ara at http://logs.openstack.org/46/511646/2/infra-check/base-integration-ubuntu-xenial/d768565/01:44
clarkbI think it works01:44
*** psachin has joined #openstack-infra01:46
pabelangeryah, looks good01:46
*** markvoelker has joined #openstack-infra01:47
pabelangershould see if we can do a failure next01:47
*** iyamahat has quit IRC01:47
*** yamahata has quit IRC01:47
clarkbwe have failure already from when ipv6 didnt work01:49
clarkblook at the retry limit runs01:49
pabelangerisn't that before role merged?01:52
*** dhinesh has quit IRC01:56
*** ihrachys_ has quit IRC01:59
*** thorst has joined #openstack-infra01:59
*** thorst has quit IRC02:00
clarkbno the role we are testing already merged02:03
clarkbianws change was ipv6 fix but we are testing ara on failure02:03
*** daidv has joined #openstack-infra02:07
*** kiennt26 has quit IRC02:18
*** mgagne has quit IRC02:19
*** kiennt26 has joined #openstack-infra02:19
*** markvoelker has quit IRC02:21
*** thingee has quit IRC02:22
*** comstud has quit IRC02:22
*** mgagne has joined #openstack-infra02:22
*** mgagne is now known as Guest5368002:22
*** thingee has joined #openstack-infra02:22
*** comstud has joined #openstack-infra02:23
openstackgerritIan Wienand proposed openstack-infra/system-config master: firehose.rst: trivial syntax  https://review.openstack.org/51167702:32
openstackgerritIan Wienand proposed openstack-infra/system-config master: Add reprepo documentation  https://review.openstack.org/51167802:32
ianwjeblair / pabelanger: ^ i hope that is never needed!02:32
*** iyamahat has joined #openstack-infra02:32
ianwok, one thing about switching these to the .openstack.org is that i guess we've lost caching instantly02:36
ianwfresh trusty update -> http://paste.openstack.org/show/623515/02:37
clarkboh hrm02:38
clarkblooks happy though?02:39
ianwfresh xenial update -> http://paste.openstack.org/show/623516/02:39
ianwboth look happy, and that was the packages in question i think?02:39
clarkbya gnutls02:39
ianwi'd feel confident running an update with vos release at this point02:40
ianwand switching back, so we get those caches going02:40
clarkbI havent been directly involved so will defer to you and jeblair's judgement on that02:41
clarkbbut sounds like probably a reasonable idea because caches02:41
*** dave-mccowan has quit IRC02:41
ianwlet's see if jeblair checks back within the next 20 mins with any other ideas, otherwise i think we go.  i'll prep the revert02:42
fungiyeah, watching from the late-night sidelines, but sounds promising at any rate02:42
jeblairo/02:43
openstackgerritIan Wienand proposed openstack-infra/system-config master: Revert "Switch to RW afs mirror"  https://review.openstack.org/51168102:43
jeblairianw: ++ revert and release02:45
ianwok, i'll manually run the full /usr/local/bin/ update script now02:45
ianwlogging to /tmp/ubuntu-update.log02:46
*** hongbin has quit IRC02:50
*** links has joined #openstack-infra02:51
ianw... and done02:52
ianwand thus the mirrors should be healthy again!02:52
fungiand we have 20m inodes free on the logs site now02:53
fungitomorrow's shaping up to be a quiet day for a change (i hope)02:53
ianwjeblair: should we let cron loose on it again?  we could wait, at this point02:54
*** bobh has quit IRC03:03
*** rosmaita has quit IRC03:06
*** iyamahat has quit IRC03:06
*** liusheng has quit IRC03:09
*** liusheng has joined #openstack-infra03:09
*** thorst has joined #openstack-infra03:10
openstackgerritMerged openstack-infra/system-config master: firehose.rst: trivial syntax  https://review.openstack.org/51167703:11
*** udesale has joined #openstack-infra03:12
*** hongbin has joined #openstack-infra03:14
*** thorst has quit IRC03:15
jeblairianw: if you wanted to leave cron disabled, i (or maybe fungi?) could get that first thing in our morning03:15
*** nicolasbock has quit IRC03:16
jeblairianw: though if you are inclined to re-enable it, it sounds like it ought to be safe now.03:16
jeblairyour choice :)03:16
ianwjeblair: let's do it now, tomorrow will have enough to deal with :)  i'll be around to keep and eye on the next few runs03:17
ianwi'll remove it from emergency03:17
*** markvoelker has joined #openstack-infra03:18
*** yamahata has joined #openstack-infra03:21
*** smatzek has joined #openstack-infra03:22
openstackgerritMerged openstack-infra/system-config master: Revert "Switch to RW afs mirror"  https://review.openstack.org/51168103:23
*** hongbin has quit IRC03:25
*** aeng has joined #openstack-infra03:27
*** priteau has joined #openstack-infra03:31
*** smatzek has quit IRC03:32
*** priteau has quit IRC03:36
*** yamahata has quit IRC03:36
*** edmondsw has joined #openstack-infra03:38
*** coolsvap has joined #openstack-infra03:41
*** jaosorior has joined #openstack-infra03:45
*** edmondsw has quit IRC03:45
*** markvoelker has quit IRC03:51
*** yamahata has joined #openstack-infra03:53
*** dbecker has quit IRC03:54
SamYaplewould https://github.com/openstack-infra/zuul/blob/feature/zuulv3/zuul/ansible/callback/zuul_stream.py#L69 be useable with async tasks?03:57
*** daidv has quit IRC04:00
*** daidv has joined #openstack-infra04:01
*** dhinesh has joined #openstack-infra04:01
*** cuongnv has quit IRC04:03
*** shu-mutou-AWAY is now known as shu-mutou04:04
*** cuongnv has joined #openstack-infra04:04
*** annp has joined #openstack-infra04:05
*** ykarel|afk has joined #openstack-infra04:06
*** dbecker has joined #openstack-infra04:09
*** kiennt26 has quit IRC04:15
*** huanxie has quit IRC04:18
*** gildub has quit IRC04:20
*** esberglu has joined #openstack-infra04:20
*** esberglu has quit IRC04:25
*** pcaruana has joined #openstack-infra04:27
*** sree has joined #openstack-infra04:29
*** ramishra has quit IRC04:37
*** ramishra has joined #openstack-infra04:39
openstackgerritIan Wienand proposed openstack-infra/system-config master: Create reprepro lockdir  https://review.openstack.org/51138004:41
*** markvoelker has joined #openstack-infra04:49
*** armax has quit IRC04:50
*** armax has joined #openstack-infra04:51
*** armax has quit IRC04:51
*** armax has joined #openstack-infra04:51
*** huanxie has joined #openstack-infra04:52
*** armax has quit IRC04:52
*** armax has joined #openstack-infra04:52
*** armax has quit IRC04:53
*** bobh has joined #openstack-infra04:54
*** pcaruana has quit IRC04:56
openstackgerritAndreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Use native propose-translation jobs  https://review.openstack.org/51143504:56
*** AJaeger has joined #openstack-infra04:57
*** AJaeger has quit IRC04:57
*** bobh has quit IRC04:58
*** claudiub|2 has joined #openstack-infra04:58
openstackgerritAndreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Add branches to all periodic jobs  https://review.openstack.org/51153304:58
*** cody-somerville has joined #openstack-infra05:00
*** cody-somerville has joined #openstack-infra05:00
*** udesale has quit IRC05:01
*** AJaeger has joined #openstack-infra05:01
*** vaidy has quit IRC05:02
*** isviridov_away has quit IRC05:02
AJaegerianw, if you have some time to review a few of my open changes for project-config and openstack-zuul-jobs, please, it would be great...05:02
*** cody-somerville has quit IRC05:04
*** thorst has joined #openstack-infra05:11
*** thorst has quit IRC05:15
ianwAJaeger: you always provide for an interesting tour around some lesser known parts of these jobs :)05:15
*** jaosorior has quit IRC05:17
openstackgerritMerged openstack-infra/project-config master: Remove legacy job from openstackdocstheme  https://review.openstack.org/51128705:17
*** chlong has joined #openstack-infra05:20
*** ykarel|afk has quit IRC05:21
*** markvoelker has quit IRC05:22
*** gouthamr has quit IRC05:24
*** edmondsw has joined #openstack-infra05:27
*** pcaruana has joined #openstack-infra05:30
*** jaosorior has joined #openstack-infra05:30
*** ykarel has joined #openstack-infra05:32
*** edmondsw has quit IRC05:32
*** slaweq_ has joined #openstack-infra05:32
*** priteau has joined #openstack-infra05:32
*** slaweq_ has quit IRC05:33
*** smatzek has joined #openstack-infra05:33
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Convert legacy-api-ref and -guide to native v3  https://review.openstack.org/51129005:35
*** priteau has quit IRC05:37
openstackgerritMerged openstack-dev/pbr master: Add reno for release notes management  https://review.openstack.org/51067905:37
*** smatzek has quit IRC05:38
*** vaidy has joined #openstack-infra05:38
*** isviridov_away has joined #openstack-infra05:39
*** armaan has joined #openstack-infra05:39
*** pcaruana has quit IRC05:39
*** rcernin has joined #openstack-infra05:41
*** armaan has quit IRC05:43
*** nikhil_k has quit IRC05:46
*** andreas_s has joined #openstack-infra05:53
*** spectr has quit IRC05:58
*** spectr has joined #openstack-infra05:58
*** Hal has joined #openstack-infra05:59
*** Hal is now known as Guest9048605:59
*** claudiub|2 has quit IRC06:01
*** cshastri has joined #openstack-infra06:06
*** dfflanders has joined #openstack-infra06:08
*** slaweq_ has joined #openstack-infra06:09
*** esberglu has joined #openstack-infra06:11
*** esberglu has quit IRC06:11
*** esberglu has joined #openstack-infra06:11
*** esberglu has quit IRC06:11
*** jbadiapa has joined #openstack-infra06:11
*** esberglu has joined #openstack-infra06:12
*** ykarel_ has joined #openstack-infra06:16
*** esberglu has quit IRC06:16
*** ykarel has quit IRC06:17
*** aeng has quit IRC06:18
*** markvoelker has joined #openstack-infra06:19
*** kiennt26 has joined #openstack-infra06:21
*** e0ne has joined #openstack-infra06:32
*** e0ne has quit IRC06:35
*** e0ne has joined #openstack-infra06:35
AJaegerianw: they were broken, so I had to learn something new ;)06:38
AJaegerianw: thanks06:38
evrardjptonyb: updated the review https://review.openstack.org/#/c/511479/1 with a comment as you asked.06:39
*** ccamacho has joined #openstack-infra06:39
*** slaweq_ has quit IRC06:40
*** vsaienk0 has joined #openstack-infra06:41
*** slaweq_ has joined #openstack-infra06:44
openstackgerritMerged openstack-infra/project-config master: Use build-openstack-api-ref for openstackdocstheme  https://review.openstack.org/51131806:44
openstackgerritAndreas Jaeger proposed openstack-infra/project-config master: Fix v2 shade setup  https://review.openstack.org/51173306:47
*** eumel8 has joined #openstack-infra06:48
AJaegerianw, yolanda, zuul v2 files are currently broken, we need the change above ^06:48
openstackgerritAndreas Jaeger proposed openstack-infra/project-config master: Set v3 nodepool citycloud timeout to 600  https://review.openstack.org/51165606:48
eumel8morning06:48
yolandaAJaeger, looking06:50
AJaegermorning, eumel8 and yolanda06:50
*** martinkopec has joined #openstack-infra06:51
yolandapypi jobs will be enough to fix shade?06:51
AJaegeryolanda: should be ;) want to +2 and then I'll wait for test results?06:51
yolandaok +206:51
AJaegeryolanda: if you have time, could you review https://review.openstack.org/#/c/511435/ and https://review.openstack.org/511404 and https://review.openstack.org/511563 as well, please?06:52
*** markvoelker has quit IRC06:52
*** dfflanders has quit IRC06:52
AJaegerand https://review.openstack.org/51155506:53
*** shardy has joined #openstack-infra06:54
eumel8yolanda: maybe it's a good the to get a merge: https://review.openstack.org/#/c/506795/ :)06:54
*** mpaolino has joined #openstack-infra06:54
*** ykarel has joined #openstack-infra06:55
*** ykarel_ has quit IRC06:57
*** huanxie has quit IRC06:57
*** threestrands has quit IRC06:58
openstackgerritMerged openstack-infra/project-config master: Fix v2 shade setup  https://review.openstack.org/51173306:59
*** bauzas is now known as bauwser07:01
*** vhosakot has quit IRC07:06
AJaegermordred: now you can release shade ^07:06
*** pcaruana has joined #openstack-infra07:07
*** dhajare has joined #openstack-infra07:08
openstackgerritAndreas Jaeger proposed openstack-infra/project-config master: Fix publish-releasenotes  https://review.openstack.org/51173907:10
AJaegermnaser: followup to your change ^07:11
*** thorst has joined #openstack-infra07:12
*** florianf has joined #openstack-infra07:13
*** priteau has joined #openstack-infra07:15
*** edmondsw has joined #openstack-infra07:15
*** gcb has quit IRC07:16
*** thorst has quit IRC07:17
*** slaweq_ has quit IRC07:17
*** armaan has joined #openstack-infra07:17
*** vsaienk0 has quit IRC07:18
*** edmondsw has quit IRC07:20
*** priteau has quit IRC07:20
openstackgerritAndreas Jaeger proposed openstack-infra/project-config master: Use system-required instead of merge-check  https://review.openstack.org/51174507:20
*** slaweq_ has joined #openstack-infra07:20
*** tesseract has joined #openstack-infra07:22
openstackgerritAndreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Add system-required template  https://review.openstack.org/51174607:22
openstackgerritAndreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Remove merge-check  https://review.openstack.org/51174707:22
AJaegermonty, jeblair, I've done the change for merge-check -> system-required ^07:23
AJaegerUse topic:system-required for reviewing07:23
openstackgerritAndreas Jaeger proposed openstack-infra/infra-manual master: Use system-required instead of merge-check  https://review.openstack.org/51174807:25
*** adarazs has quit IRC07:25
*** dtantsur|afk has quit IRC07:26
*** spectr has quit IRC07:26
*** tesseract has quit IRC07:26
*** rcernin has quit IRC07:26
*** pcaruana has quit IRC07:26
*** martinkopec has quit IRC07:27
*** dtantsur has joined #openstack-infra07:31
*** adarazs has joined #openstack-infra07:32
*** e0ne has quit IRC07:33
openstackgerritAndreas Jaeger proposed openstack-infra/infra-manual master: Document v3 periodic jobs  https://review.openstack.org/51175007:36
AJaegerjeblair, mordred: Let's document periodic jobs ^07:36
openstackgerritAndreas Jaeger proposed openstack-infra/infra-manual master: Document v3 periodic jobs  https://review.openstack.org/51175007:38
*** martinkopec has joined #openstack-infra07:39
*** spectr has joined #openstack-infra07:40
*** pcaruana has joined #openstack-infra07:40
*** rcernin has joined #openstack-infra07:40
*** jpena|off is now known as jpena07:41
*** armaan has quit IRC07:41
*** spectr has quit IRC07:43
*** pcaruana has quit IRC07:44
*** pcaruana has joined #openstack-infra07:44
*** spectr has joined #openstack-infra07:44
*** Guest90486 has quit IRC07:46
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Remove pin pip from beaker legacy jobs  https://review.openstack.org/51136107:46
*** markvoelker has joined #openstack-infra07:50
*** slaweq_ has quit IRC07:53
*** andreas_s has quit IRC07:54
*** jpich has joined #openstack-infra07:55
yolandaAJaeger, how is zuul v3 status now? are we ready to review things?07:55
AJaegeryolanda: we can review v3 things for sure - but v3 only runs on infra stuff...07:56
AJaegerSo, let's not suggest to folks to merge adding .zuul.yaml to their repos or update them - no tests are run currently for "normal projects"07:56
AJaegeryolanda: so, go ahead and review project-config, openstack-zuul-jobs etc. please07:56
yolandaok07:57
*** andreas_s has joined #openstack-infra07:57
AJaegeryolanda: https://etherpad.openstack.org/p/zuulv3-issues has on the top of list of reviews if you like to use that list - or go directly over the repos07:59
*** rossella_s has joined #openstack-infra08:01
*** slaweq_ has joined #openstack-infra08:02
yolandaok let me look08:03
*** egonzalez has joined #openstack-infra08:05
openstackgerrityolanda.robla proposed openstack-infra/openstack-zuul-jobs master: convert deploy-guide to native zuul v3  https://review.openstack.org/51156408:07
AJaegeryolanda: why did you push 511564 again? A recheck should be enough - but only once project-config change is in08:07
yolandai clicked the rebase button08:08
yolandafor that -1 from zuul08:08
*** rossella_s has quit IRC08:08
AJaegeryolanda: that -1 from zuul is normal for any change that dpeends on project-config08:08
AJaegerSo, feel free to +2 - and I'll recheck and +A once the dependency is in.08:09
tonybevrardjp: +2W08:09
openstackgerritMerged openstack-infra/project-config master: Add in merge-check templates for LOCI  https://review.openstack.org/51064908:09
AJaegeryolanda: See the message in 511564 "This change depends on a change to a config project.08:09
yolandaAJaeger , so is normal to see -1 without logs, if the project-config change is still not merged?08:09
AJaegerThe syntax of the configuration in this change has been verified to be08:09
AJaegercorrect once the config project change upon which it depends is08:09
AJaegermerged, but it can not be used until that occurs.08:09
AJaeger"08:09
AJaegeryolanda: yes!08:09
yolandaok08:10
openstackgerritMerged openstack-infra/irc-meetings master: Moving the OpenStack-Ansible meeting time and channel  https://review.openstack.org/51147908:10
SamYaplewould love to get a review +2+W for this bindep patch https://review.openstack.org/#/c/506502/08:10
*** rossella_s has joined #openstack-infra08:10
SamYapleif anyone has some time08:10
*** yamahata has quit IRC08:11
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: remove legacy openstackdocstheme jobs  https://review.openstack.org/51128908:11
*** thorst has joined #openstack-infra08:12
AJaegeryolanda: since project-config is a trusted repo, some of these depends-on will not work and we have to do the dance of merge one change, recheck the next etc. So, feel free to +2 and I'll recheck once dependines are in...08:14
openstackgerritMerged openstack-infra/project-config master: Add publish-deploy-guide job  https://review.openstack.org/51156308:14
*** armaan has joined #openstack-infra08:15
openstackgerritAndreas Jaeger proposed openstack-infra/project-config master: Use system-required instead of merge-check  https://review.openstack.org/51174508:15
*** rossella_s has quit IRC08:16
*** thorst has quit IRC08:17
*** rossella_s has joined #openstack-infra08:18
openstackgerritMerged openstack-infra/project-config master: Removing collectd-ceilometer-plugin definition from zuul.d/projects.yaml  https://review.openstack.org/51008608:19
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Use native propose-translation jobs  https://review.openstack.org/51143508:20
*** andreas_s has quit IRC08:21
openstackgerritMerged openstack-infra/project-config master: Update some legacy jobs  https://review.openstack.org/51155508:21
*** gcb has joined #openstack-infra08:21
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Add branches to all periodic jobs  https://review.openstack.org/51153308:22
*** markvoelker has quit IRC08:23
*** andreas_s has joined #openstack-infra08:23
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Remove unused kuryr-libnetwork jobs  https://review.openstack.org/51140408:23
openstackgerritDuong Ha-Quang proposed openstack-infra/openstack-zuul-jobs master: Remove legacy jobs in Mixmatch  https://review.openstack.org/51176308:25
openstackgerritDuong Ha-Quang proposed openstack-infra/project-config master: Remove legacy jobs in Mixmatch  https://review.openstack.org/51176408:25
*** andreas_s has quit IRC08:26
*** derekh has joined #openstack-infra08:26
*** andreas_s has joined #openstack-infra08:28
*** lucas-afk is now known as lucasagomes08:28
*** rossella_s has quit IRC08:28
*** coolsvap has quit IRC08:30
*** rossella_s has joined #openstack-infra08:31
*** dhinesh has quit IRC08:31
openstackgerritMerged openstack-infra/irc-meetings master: Update chair for doc team meeting  https://review.openstack.org/51148408:31
*** ralonsoh has joined #openstack-infra08:32
*** slaweq_ has quit IRC08:35
*** jascott1 has quit IRC08:35
*** jascott1 has joined #openstack-infra08:35
*** kjackal_ has joined #openstack-infra08:35
*** ykarel is now known as ykarel|lunch08:36
tonybI'm trying to use owners.py and it's failing to get https://review.openstack.org/gitweb?p=openstack/governance.git;a=blob_plain;f=reference/projects.yaml;hb=oct-2017-elections with a 404 shoudl that still work?08:36
*** johnthetubaguy has quit IRC08:39
*** jascott1 has quit IRC08:39
*** johnthetubaguy has joined #openstack-infra08:40
*** thorst has joined #openstack-infra08:41
*** e0ne has joined #openstack-infra08:41
*** slaweq_ has joined #openstack-infra08:44
*** armaan has quit IRC08:44
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: convert deploy-guide to native zuul v3  https://review.openstack.org/51156408:44
*** thorst has quit IRC08:45
*** huanxie has joined #openstack-infra08:46
*** kiennt26 has quit IRC08:50
AJaegertonyb: the updated gerrit caused a few changes, this could be one part of it. Best discuss with rest of team later...08:54
*** electrofelix has joined #openstack-infra08:56
*** kaisers1 has quit IRC08:57
*** dtantsur has quit IRC08:57
*** dtantsur has joined #openstack-infra08:58
*** adarazs has quit IRC08:58
*** rcernin has quit IRC08:59
*** spectr has quit IRC08:59
*** panda|rover|off is now known as panda|rover08:59
*** martinkopec has quit IRC08:59
*** pcaruana has quit IRC08:59
ianwtonyb: we switched to cgit, so ... no09:00
ianwi would get that from git.openstack.org, rather than review.o.o09:00
ianwthat way it's load balanced & more reliable09:00
*** andreas_s has quit IRC09:02
*** kiennt26 has joined #openstack-infra09:03
*** edmondsw has joined #openstack-infra09:04
*** adarazs has joined #openstack-infra09:05
*** andreas_s has joined #openstack-infra09:07
*** edmondsw has quit IRC09:08
*** ccamacho has left #openstack-infra09:09
*** andreas_s has quit IRC09:12
*** martinkopec has joined #openstack-infra09:12
*** rcernin has joined #openstack-infra09:12
*** spectr has joined #openstack-infra09:12
*** andreas_s has joined #openstack-infra09:12
*** pcaruana has joined #openstack-infra09:13
*** ccamacho has joined #openstack-infra09:14
*** jgr has joined #openstack-infra09:14
*** priteau has joined #openstack-infra09:16
*** andreas_s has quit IRC09:17
*** slaweq_ has quit IRC09:17
*** kiennt26 has quit IRC09:17
*** ykarel|lunch is now known as ykarel09:18
*** markvoelker has joined #openstack-infra09:20
*** priteau has quit IRC09:20
*** slaweq_ has joined #openstack-infra09:21
openstackgerritDuong Ha-Quang proposed openstack-infra/openstack-zuul-jobs master: Remove legacy jobs in Mogan  https://review.openstack.org/51178009:23
openstackgerritDuong Ha-Quang proposed openstack-infra/project-config master: Remove legacy jobs in Mogan  https://review.openstack.org/51178109:23
openstackgerritTovin Seven proposed openstack-infra/project-config master: Remove legacy oslo.versionedobjects job  https://review.openstack.org/51178309:24
openstackgerritTovin Seven proposed openstack-infra/openstack-zuul-jobs master: Remove legacy oslo.versionedobjects job  https://review.openstack.org/51178409:24
*** gcb has quit IRC09:27
*** tosky has joined #openstack-infra09:28
toskyAJaeger: morning, thanks for the review09:31
AJaegermorning, tosky - you're welcome09:31
tosky(as usual :)09:31
toskydid you see this error, which seems unrelated to tripleo? http://logs.openstack.org/54/509554/2/check/gate-tripleo-ci-centos-7-scenario004-multinode-oooq/8daa9af/logs/devstack-gate-setup-workspace-new.txt.gz#_2017-10-12_23_46_57_06709:31
AJaegertosky: best ask others on the team later for that - might have been a networking issue09:32
toskyAJaeger: uhm, so should I not feel guilty if I try to recheck?09:33
AJaegertosky: with that timing, infra was still not green, so go for the recheck...09:35
toskyoh, thanks!09:35
*** Hal has joined #openstack-infra09:36
*** Hal is now known as Guest1938809:36
*** andreas_s has joined #openstack-infra09:36
openstackgerritDuong Ha-Quang proposed openstack-infra/project-config master: Remove legacy jobs in Cliff  https://review.openstack.org/51178809:37
openstackgerritDuong Ha-Quang proposed openstack-infra/openstack-zuul-jobs master: Remove legacy jobs in Cliff  https://review.openstack.org/51178909:37
*** chlong has quit IRC09:40
*** egonzalez has quit IRC09:42
openstackgerritLuigi Toscano proposed openstack-infra/project-config master: sahara-dashboard: use the openstack-python-jobs template  https://review.openstack.org/51179109:43
*** priteau has joined #openstack-infra09:44
*** andreas_s has quit IRC09:45
*** andreas_s has joined #openstack-infra09:46
*** egonzalez has joined #openstack-infra09:46
*** huanxie has quit IRC09:46
*** shu-mutou is now known as shu-mutou-AWAY09:47
*** markvoelker has quit IRC09:53
*** slaweq_ has quit IRC09:55
*** andreas_s has quit IRC09:55
*** armaan has joined #openstack-infra09:58
toskyyolanda: I know that this is a minor review, but can you please take some time to check small IRC notification change for sahara channels: https://review.openstack.org/#/c/503936/ ?09:58
toskyespecially now that we are in the time before two storms :)09:59
*** shardy has quit IRC09:59
*** slaweq_ has joined #openstack-infra10:00
*** andreas_s has joined #openstack-infra10:00
openstackgerritRaissa Sarmento proposed openstack-infra/project-config master: Add initial jobs for manila-tempest-plugin  https://review.openstack.org/51054810:02
*** cuongnv has quit IRC10:03
*** sambetts|afk is now known as sambetts10:05
openstackgerritLuigi Toscano proposed openstack-infra/project-config master: sahara-dashboard: use the openstack-python-jobs template  https://review.openstack.org/51179110:08
openstackgerritRaissa Sarmento proposed openstack-infra/project-config master: Add initial jobs for manila-tempest-plugin  https://review.openstack.org/51054810:09
*** andreas_s has quit IRC10:09
*** florianf has quit IRC10:09
*** florianf has joined #openstack-infra10:09
openstackgerritAndreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Remove openstack-api-ref jobs  https://review.openstack.org/51179610:13
*** sbezverk has joined #openstack-infra10:14
*** andreas_s has joined #openstack-infra10:14
*** daidv has quit IRC10:15
*** armaan has quit IRC10:17
*** pbourke has quit IRC10:17
*** pbourke has joined #openstack-infra10:18
*** andreas_s has quit IRC10:19
*** andreas_s has joined #openstack-infra10:19
*** annp has quit IRC10:22
*** sdague has joined #openstack-infra10:24
*** boden has joined #openstack-infra10:27
*** andreas_s has quit IRC10:28
*** andreas_s has joined #openstack-infra10:29
openstackgerritAndreas Jaeger proposed openstack-infra/project-config master: v3 cleanup: Use openstack-python-jobs  https://review.openstack.org/51180310:30
*** psachin has quit IRC10:33
*** gmann is now known as gmann_afk10:33
*** andreas_s has quit IRC10:34
*** huanxie has joined #openstack-infra10:34
*** andreas_s has joined #openstack-infra10:36
*** jascott1 has joined #openstack-infra10:37
*** sree has quit IRC10:41
*** thorst has joined #openstack-infra10:42
*** sree has joined #openstack-infra10:42
*** esberglu has joined #openstack-infra10:43
*** esberglu has quit IRC10:43
*** thorst_ has joined #openstack-infra10:45
*** sree has quit IRC10:46
*** jaosorior has quit IRC10:47
*** thorst has quit IRC10:47
*** pcaruana has quit IRC10:48
*** yamamoto has quit IRC10:48
*** thorst_ has quit IRC10:49
*** ociuhandu has quit IRC10:49
*** markvoelker has joined #openstack-infra10:50
AJaegerjeblair, mordred, pabelanger, we have an infra-post job running for 5 h - apparently waiting for node. Job is publish-openstack-sphinx-docs-infraqueue, repo openstack-zuul-jobs, hash is 8c1d4e1b40a3f1a47e511785d5a54a1d3e11451f .10:54
*** claudiub has joined #openstack-infra10:59
mordredAJaeger: cool- I'll look in just a few11:00
*** slaweq_ has quit IRC11:03
*** aperri has joined #openstack-infra11:07
*** nicolasbock has joined #openstack-infra11:09
*** slaweq_ has joined #openstack-infra11:10
*** armaan has joined #openstack-infra11:10
mordredAJaeger: hrm. I'm not sure I'm awake enough to debug this  yet11:10
*** adarazs has left #openstack-infra11:13
*** nicolasbock has quit IRC11:14
AJaegermordred: it's not urgent - just wanted to point it out. Get awake first, please ;)11:14
*** lucasagomes is now known as lucas-hungry11:14
AJaegermordred: once you're awake, hope you like https://review.openstack.org/511746 ;)11:15
*** hashar has joined #openstack-infra11:19
openstackgerritAndreas Jaeger proposed openstack-infra/project-config master: Stop translation of trove  https://review.openstack.org/51181311:20
AJaegereumel8, ianychoi, kumarmn ^11:20
*** edmondsw has joined #openstack-infra11:20
*** markvoelker has quit IRC11:23
*** erlon has joined #openstack-infra11:23
*** sree has joined #openstack-infra11:24
*** alexchadin has joined #openstack-infra11:24
*** rosmaita has joined #openstack-infra11:25
*** Qiming has quit IRC11:26
*** nicolasbock has joined #openstack-infra11:26
eumel8AJaeger: I heart it, trove is in an unmaintanable state11:26
*** aviau has quit IRC11:28
*** Qiming has joined #openstack-infra11:28
*** aviau has joined #openstack-infra11:28
openstackgerritAndreas Jaeger proposed openstack-infra/project-config master: Stop translation of trove  https://review.openstack.org/51181311:28
AJaegereumel8: just updated commit message to explain this can be changed back ^11:28
AJaegereumel8: could you +1 if you agree - it's not worth spending resources on it *currently*11:29
eumel8AJaeger: sure, I would also set the projects read-only in Zanata to prevent useless translations11:30
mordredAJaeger: love it!11:31
*** ldnunes has joined #openstack-infra11:34
AJaegereumel8: yes, exactly - make it read-only...11:34
AJaegermordred: three changes for this ;( https://review.openstack.org/#/c/511745 would be the next one...11:35
mordredAJaeger: yah - the story around moving/renaming is not as fun as other things11:35
AJaegermordred: yeah ;(11:36
*** smatzek has joined #openstack-infra11:37
AJaegeryolanda and ianw reviewed nicely our open list - now we need one more review basically. mordred, if you have time to go over the etherpad reviews, it would be nice . But nothing urgent...11:37
*** esberglu has joined #openstack-infra11:37
mordredAJaeger: YOU KNOW - we could just locate the system-required project-template in project-config - there's no real need for it to be in openstack-zuul-jobs11:37
eumel8AJaeger: trove-dashboard ist still in use? https://review.openstack.org/#/c/504749/ https://review.openstack.org/#/c/493761/11:37
mordredAJaeger: cool - will look11:37
AJaegereumel8: yeah - but they merge at least some stuff...11:37
eumel8not for stable/pike...11:38
AJaegermordred: shall I moe it over?11:38
AJaegermordred: I quickly gave a -2 o nthe change in case we want to have the template there11:39
AJaegerright now we have no templates in project-config for zuul v311:39
mordredAJaeger: hrm. actually - no, if we leave it in ozj then we can do depends-on with chanes to it11:39
mordredAJaeger: so I think I'm wrong and the way it is is correct11:39
AJaegerok11:40
*** yamamoto has joined #openstack-infra11:40
AJaegerlunch time - bbl11:41
*** huanxie has quit IRC11:41
*** esberglu has quit IRC11:41
*** slaweq_ has quit IRC11:43
*** armaan has quit IRC11:44
*** armaan has joined #openstack-infra11:44
*** yamamoto has quit IRC11:45
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Add system-required template  https://review.openstack.org/51174611:47
*** yamamoto has joined #openstack-infra11:48
*** baoli has joined #openstack-infra11:49
*** pblaho has joined #openstack-infra11:53
*** thorst has joined #openstack-infra11:53
*** yamamoto has quit IRC11:54
*** martinkopec has quit IRC11:54
*** dprince has joined #openstack-infra11:54
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Collapse job header lines into a single statement  https://review.openstack.org/51182111:57
alexchadinhi. how can I ask Zuul to recheck commit instead of Jenkins?11:57
AJaegeralexchadin: you cannot.11:58
AJaegeralexchadin: a recheck would recheck both systems11:58
AJaegerbut currently zuul v3 is only running on a few repos due to the problems we had elsewhere the last days.11:59
alexchadinAJaeger: oh, thanks for quick reply11:59
AJaegeralexchadin: best wait until zuul v3 is running check on all jobs again and then recheck11:59
AJaegeralexchadin: why did you recheck 510848 ?12:00
AJaegeralexchadin: ah, sorry - that's fine...12:01
alexchadinAJaeger: Zuul shows error with one job: multinode-integration-debian-jessie12:01
AJaegeralexchadin: didn't remember that dependencies were in...12:01
alexchadinAJaeger: +112:01
*** hashar has quit IRC12:01
*** armaan has quit IRC12:01
*** baoli_ has joined #openstack-infra12:02
*** armaan has joined #openstack-infra12:02
*** andreas_s has quit IRC12:02
*** tpsilva has joined #openstack-infra12:02
*** baoli has quit IRC12:04
openstackgerritMerged openstack-infra/project-config master: Fix publish-releasenotes  https://review.openstack.org/51173912:05
*** LindaWang has quit IRC12:06
openstackgerritMerged openstack-infra/project-config master: sahara-dashboard: use the openstack-python-jobs template  https://review.openstack.org/51179112:07
odyssey4mehowdy all - IIRC someone on the ML mentioned an environment variable which was specific to zuul v2 and would not be carried forward... but I can't seem to find it - does anyone know which env var I'm referring to?12:07
mordredAJaeger, ianw, yolanda: great work while the rest of us slept! I've +A'd all the things I can +A12:08
mordredodyssey4me: probably ZUUL_REF12:08
mordredI think12:08
mordredhang on ...12:08
mordredodyssey4me: http://git.openstack.org/cgit/openstack-infra/zuul/tree/zuul/ansible/filter/zuul_filters.py?h=feature/zuulv312:09
openstackgerritMerged openstack-infra/project-config master: Add initial jobs for manila-tempest-plugin  https://review.openstack.org/51054812:09
mordredodyssey4me: that is the filter plugin that turns things in to legacy env vars12:09
*** felipemonteiro has joined #openstack-infra12:09
odyssey4meok, so ZUUL_URL, ZUUL_REF and ZUUL_COMMIT are v2 only12:09
mordredodyssey4me: that said - the intent is for ALL of the ZUUL_* env vars to go away - the things in that are just for transition purposes12:10
odyssey4mewe're going to build a bit of a shim to detect the version and use z-c for v2 only12:10
mordredodyssey4me: woot!12:10
*** baoli_ has quit IRC12:10
odyssey4methat'll help us get jobs working with both active12:10
odyssey4mecool - thanks a mill12:10
*** lucas-hungry is now known as lucasagomes12:10
*** hashar has joined #openstack-infra12:11
mordredI'm a fan of this plan12:11
openstackgerritMerged openstack-infra/infra-manual master: Use system-required instead of merge-check  https://review.openstack.org/51174812:11
openstackgerritMerged openstack-infra/infra-manual master: Document v3 periodic jobs  https://review.openstack.org/51175012:12
odyssey4mewe're also trying to be better citizens - https://review.openstack.org/#/q/topic:infra-log-inode-reduction - apologies for hogging the inodes :/12:12
*** bobh has joined #openstack-infra12:13
*** martinkopec has joined #openstack-infra12:14
*** felipemonteiro has quit IRC12:14
*** markvoelker has joined #openstack-infra12:15
*** baoli has joined #openstack-infra12:17
*** shardy has joined #openstack-infra12:19
*** sree has quit IRC12:20
*** sree has joined #openstack-infra12:21
*** armaan has quit IRC12:21
*** armaan has joined #openstack-infra12:21
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Fix doc typo that missed important words  https://review.openstack.org/50990512:21
*** eharney has joined #openstack-infra12:22
*** slaweq_ has joined #openstack-infra12:23
mordredodyssey4me: no worries! thanks for cleaning that up!12:23
mordredodyssey4me: (turns out it's a constant learning experience around here)12:24
*** LindaWang has joined #openstack-infra12:24
*** martinkopec has quit IRC12:24
odyssey4memordred yep, that's what we love about it!12:24
*** sree has quit IRC12:25
*** hashar has quit IRC12:25
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Remove legacy-.*python{34,35} jobs  https://review.openstack.org/51155712:25
*** efried is now known as fried_rice12:26
AJaegerthanks, mordred!12:27
*** trown|outtypewww is now known as trown12:28
AJaegerodyssey4me: see http://lists.openstack.org/pipermail/openstack-dev/2017-October/123049.html12:28
*** yamamoto has joined #openstack-infra12:28
odyssey4meAJaeger ah, LOG_PATH12:28
*** baoli has quit IRC12:29
odyssey4metyvm12:29
*** yamamoto has quit IRC12:29
*** martinkopec has joined #openstack-infra12:29
*** baoli has joined #openstack-infra12:29
*** ociuhandu has joined #openstack-infra12:30
*** hashar has joined #openstack-infra12:32
*** martinkopec has quit IRC12:32
*** mat128 has joined #openstack-infra12:32
*** baoli has quit IRC12:34
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Add role to ensure per-node output dirs exist  https://review.openstack.org/51182312:35
*** gfidente has joined #openstack-infra12:36
openstackgerritAndreas Jaeger proposed openstack-infra/project-config master: Use system-required instead of merge-check  https://review.openstack.org/51174512:36
AJaegermordred: rebased since a just merged change introduced merge-check again ^. Do you want to +2 again?12:36
*** ociuhandu has quit IRC12:39
*** hashar has quit IRC12:40
*** Qiming has quit IRC12:41
mordredAJaeger: I do!12:41
AJaegerthanks!12:41
*** Qiming has joined #openstack-infra12:42
*** dave-mccowan has joined #openstack-infra12:42
*** stakeda has quit IRC12:43
AJaegermordred: do we need a rebase of https://review.openstack.org/495426 ?12:43
*** jpena is now known as jpena|lunch12:43
openstackgerritMonty Taylor proposed openstack-infra/project-config master: Set zuul_output_dir in site-variables  https://review.openstack.org/51182712:44
openstackgerritMonty Taylor proposed openstack-infra/project-config master: Use ensure-output-dirs in base-test  https://review.openstack.org/51182812:44
openstackgerritMonty Taylor proposed openstack-infra/project-config master: Use ensure-output-dirs in base  https://review.openstack.org/51182912:44
*** hashar has joined #openstack-infra12:45
*** links has quit IRC12:45
*** esberglu has joined #openstack-infra12:45
*** jcoufal has joined #openstack-infra12:47
dmsimardodyssey4me: thanks for inodes :D12:47
openstackgerritMonty Taylor proposed openstack-infra/project-config master: Mark publish-openstack-artifacts and pipelines as post-review  https://review.openstack.org/49542612:47
mordredAJaeger: ^^ yes!12:47
*** dave-mccowan has quit IRC12:47
dmsimardodyssey4me: fwiw ara is part of zuul v3 by default, perhaps there is something to do there once you move to native v3 jobs -- the equivalent to https://review.openstack.org/#/c/511818/ was implemented in https://review.openstack.org/#/c/511622/12:48
dmsimardodyssey4me: that said, we'll be exploring different options on how to best serve ara with the least strain on the infrastructure12:48
*** pcaruana has joined #openstack-infra12:48
pabelangermorning12:50
*** alexchadin has quit IRC12:50
*** alexchadin has joined #openstack-infra12:51
mordredodyssey4me, dmsimard: yah - I'm quite interested in what the integration story is for openstack-ansible and v3 ... once the dust settles I have this hunch the openstack-ansible repos are going to be great case studies to figure out how to make the story great for projects that are already natively ansible12:51
AJaegermordred: there're a couple of more changes by you in project-config, zuul-jobs, and zuul - some of these look needed as well. Any further hidden gems in there that need rebasing and push?12:51
mordredmorning pabelanger !12:51
AJaegermorning, pabelanger !12:51
*** eumel8 has quit IRC12:52
*** tlaxkit has joined #openstack-infra12:52
mordredAJaeger: that's an excellent quesiton :)12:52
dmsimardmordred: I'm also interested. Interestingly, we (RDO) have an Ansible framework for running gate jobs outside the gate -- tl;dr, it installs dependencies, sets up RDO trunk repos, clones a project such as puppet-openstack-integration and then uses run_tests.sh. With v3 and native ansible, the lines blur quite a bit.12:53
mordreddmsimard: yah. that's all stuff I'd expect v3 to do - especially the cloning repos part12:54
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Removes legacy collectd-ceilometer-plugin jobs  https://review.openstack.org/51008712:54
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Remove openstack-api-ref jobs  https://review.openstack.org/51179612:54
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Remove legacy watcher jobs  https://review.openstack.org/51084812:54
AJaegerSame question for pabelanger - just saw https://review.openstack.org/#/c/486199/ and wondered whether we need that...12:55
mordredpabelanger, AJaeger: yah - that does look like a thign that would be nice to merge12:55
AJaegerpabelanger: question was: there're a couple of more changes by you in project-config, zuul-jobs, and zuul - some of these look needed as well. Any further hidden gems in there that need rebasing and push?12:56
*** slaweq_ has quit IRC12:56
AJaegermordred, pabelanger: Let me rebase 486199...12:56
*** spectr has quit IRC12:57
openstackgerritAndreas Jaeger proposed openstack-infra/project-config master: Limit add-build-sshkey to localhost  https://review.openstack.org/48619912:58
AJaegerdone ^12:58
*** ociuhandu has joined #openstack-infra12:58
odyssey4memordred yeah, I'm looking forward to thinning out some of these extra layers - right now we do job -> tox -> script -> ansible... we should at least be able to do job (ansible) -> script -> ansible to start with... then see where we can also ditch the script bits if that makes sense12:58
odyssey4meour script bits are usually to get ansible there - we may be able to only use those bits if you're testing outside of zuul12:59
odyssey4meanyway - we'll see... early days12:59
mordredodyssey4me: yah - I also want to explore making some elements of ansible -> ansible richer/better in some manner12:59
openstackgerritboden proposed openstack-infra/project-config master: zuul v3 gate changes for vmware-nsx  https://review.openstack.org/51183613:00
*** slaweq_ has joined #openstack-infra13:01
evrardjpmordred: odyssey4me maybe in the future, we can rely on directly using our playbooks, and we just set a variable of our python interpreter, and we are good.13:01
*** spectr has joined #openstack-infra13:01
evrardjpfuture!13:01
odyssey4meevrardjp hmm, that could be interesting - use a pre task to get our ansible there, then flip to doing the run with our venv and our version of ansible13:04
odyssey4meneat idea13:04
*** mriedem has joined #openstack-infra13:05
evrardjpyeah, simple too.13:05
evrardjpbasically reproducing work locally just says: "Installs ansible and those dependencies"13:05
odyssey4meevrardjp yeah, and we have run_tests.sh to do that for humans13:06
evrardjpwho needs humans when we can have Zuul!13:06
evrardjpwe can haz*13:06
AJaegermordred: abandon https://review.openstack.org/#/c/501475/ ? rebase: https://review.openstack.org/#/c/500615/ and https://review.openstack.org/#/c/500614/ ?13:06
*** jistr is now known as jistr|mtg13:08
openstackgerritThierry Carrez proposed openstack-infra/infra-specs master: Flat Git Namespace  https://review.openstack.org/46187813:11
mordredAJaeger: abandoned the first - the second two are post-v3 things to discuss with folks whether they like it or not13:12
AJaegerok13:12
AJaegerJust rechecked an ansible linters job - see http://logs.openstack.org/63/495463/3/infra-check/openstack-tox-linters/0e539b6/job-output.txt.gz13:12
AJaegerERROR! the role 'fetch-javscript-tarball' was not found in /home/zuul/src/git.openstack.org/openstack-infra/project-config/playbooks/javascript/roles:/home/zuul/src/git.openstack.org/openstack-infra/project-config/.tox/linters/src/zuul-jobs/roles:/home/zuul/src/git.openstack.org/openstack-infra/project-config/playbooks/javascript13:12
AJaegerah, see the error - will fix!13:12
*** ociuhandu has quit IRC13:14
alexchadinAJaeger: do we also need to add .zuul files to appropriate python clients?13:14
AJaegeralexchadin: you can - if you run legacy jobs, see https://docs.openstack.org/infra/manual/zuulv3.html13:14
*** hashar has quit IRC13:15
ttxfungi: posted a second rev on the flat-git-namespace spec. I listed a number of analysis tasks that I haven't completed yet. One of them is to explore if smart rewriting/redirecting on git.o.o would not spare us from having to fix all those hardcoded references at once.13:15
alexchadinAJaeger: yeah, we have tempest job for client13:15
openstackgerritAndreas Jaeger proposed openstack-infra/project-config master: Fix typo in role  https://review.openstack.org/51184013:16
*** baoli has joined #openstack-infra13:20
alexchadinAJaeger: should projects take appropriate rally jobs into their repos or rally team does it into its own repo?13:21
alexchadinAJaeger: they have a lot of project specific jobs13:21
AJaegeralexchadin: something to discuss with rally team ;) Might make sense for them to have all in one place...13:22
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Add role to fetch output from nodes  https://review.openstack.org/51184313:22
AJaegerfor now: I suggest you leave them in project-config, move other jobs, convert them - and discuss...13:23
AJaegerNo need to move the rally jobs *now*13:23
*** kgiusti has joined #openstack-infra13:23
alexchadinAJaeger: waiting for an answer from rally ptl side :)13:24
mordredalexchadin, AJaeger: yah - I think ultimately having a set of good v3 rally jobs in the rally repo would be the pattern we'd advocate for13:25
mordredsimilarly - we're aiming to have base devstack jobs in the devstack repo and then base tempest jobs in the tempest repo13:26
AJaegermordred: something to document? ^13:27
mordredAJaeger: yah - probably so ... there is a difference between "you can define jobs in your own repo - andyou can also use jobs from someone else's repo" and "we recommend that projects like tempest and rally define jobs in their repos that other projects can consume"13:28
AJaegermordred: yes, there is - I know. We could still say that projects can define jobs for multiple projects etc. - without naming them.13:29
*** baoli has quit IRC13:30
*** smatzek has quit IRC13:30
*** yamamoto has joined #openstack-infra13:30
*** nikhil_k has joined #openstack-infra13:30
mordredAJaeger: oh - no, sorry - I was saying I think we shoujld spell out the second thing because people might not infer it correctly just from documenting the first thing13:31
*** baoli has joined #openstack-infra13:31
mordredthe first is 'you can do this' - the second is 'you should do this'13:31
*** Qiming has quit IRC13:31
*** baoli has quit IRC13:32
*** chlong has joined #openstack-infra13:33
*** slaweq has quit IRC13:33
AJaegerah, I see ;) /me should not multi-task...13:33
AJaegerbbl13:33
*** Qiming has joined #openstack-infra13:33
*** slaweq_ has quit IRC13:34
openstackgerritMonty Taylor proposed openstack-infra/project-config master: Add fetch-output to base-test  https://review.openstack.org/51185013:36
openstackgerritMonty Taylor proposed openstack-infra/project-config master: Add fetch-output to base job  https://review.openstack.org/51185113:36
*** baoli has joined #openstack-infra13:39
*** yamamoto has quit IRC13:40
*** slaweq has joined #openstack-infra13:41
*** jpena|lunch is now known as jpena13:41
*** armaan has quit IRC13:41
*** armaan has joined #openstack-infra13:42
odyssey4memordred what's the path where the git repositories are put by v3?13:42
odyssey4mez-c (v2) had /opt/git ... IIRC the path in zuulv3 is something along the line of /home/zuul/src/openstack.org/<repo> ?13:43
odyssey4meoh, that'd be <org>/<repo>13:44
*** rcernin has quit IRC13:44
dmsimardAJaeger: I think https://review.openstack.org/#/c/504788/ is good to go13:44
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Update upload-logs to process docs as well  https://review.openstack.org/51185313:44
AJaegerdmsimard: wanted jeblair give a chance to review 504788 again - but if he doesn't manage it today, I'll do again...13:45
dmsimardworks for me13:45
dmsimardmordred: commented on https://review.openstack.org/#/c/511843/13:45
*** andreas_s has joined #openstack-infra13:47
*** kaisers has joined #openstack-infra13:47
*** dansmith is now known as superdan13:47
*** d0ugal has quit IRC13:48
*** dave-mccowan has joined #openstack-infra13:49
*** gouthamr has joined #openstack-infra13:49
*** yamamoto has joined #openstack-infra13:50
*** gouthamr has quit IRC13:51
*** andreas_s has quit IRC13:51
*** gouthamr has joined #openstack-infra13:51
*** dave-mcc_ has joined #openstack-infra13:55
*** dave-mccowan has quit IRC13:57
*** smatzek has joined #openstack-infra13:59
*** smatzek has quit IRC13:59
*** smatzek has joined #openstack-infra13:59
*** smatzek has quit IRC13:59
*** eumel8 has joined #openstack-infra14:00
*** smatzek has joined #openstack-infra14:00
*** bnemec is now known as beekneemech14:00
pabelangermordred: AJaeger: re: 486199 I don't think we need that any more, it was more of an optimization.  So, I can WIP for now, and maybe revisit in the future14:01
AJaegerpabelanger: sure14:01
*** mriedem has quit IRC14:03
*** ociuhandu has joined #openstack-infra14:03
pabelangerianw: jeblair: great work on ubuntu mirror14:03
*** d0ugal has joined #openstack-infra14:03
pabelangermordred: looks like new shade was released, did we want to restart nodepool-builders?14:04
*** aperri has quit IRC14:05
AJaegerpabelanger: let's first debug why we have an infra-post job for 8+ hours...14:05
*** andreas_s has joined #openstack-infra14:05
AJaeger(or at least discuss that;)14:05
pabelangerlooking14:05
*** claudiub has quit IRC14:06
* AJaeger needs to step out for a bit...14:06
pabelangerwow14:06
*** mriedem has joined #openstack-infra14:06
pabelangerwe have 31 held debian-jessie nodes14:06
pabelangerfor zuulv314:06
pabelangerI suspect, we no longer have capicity for launch the needed node for the job14:07
pabelangerya, I suspect nodepool v3 is wedged14:08
pabelangerwe have plenty of ready nodes for xenial14:08
*** trown is now known as trown|brb14:08
pabelangerYah, for some reason the autohold I did for debian-jessie last night, has held 31 instances of debian-jessie14:09
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Upload fetched artifacts to log server  https://review.openstack.org/51185814:09
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Only publish artifacts when zuul.change is not defined  https://review.openstack.org/51185914:09
pabelangerShrews: ^do you have a moment to help debug why?14:09
openstackgerritMonty Taylor proposed openstack-infra/project-config master: Handle basic artifacts in base-test job  https://review.openstack.org/51186014:09
openstackgerritMonty Taylor proposed openstack-infra/project-config master: Process artifacts in base and base-minimal  https://review.openstack.org/51186114:09
Shrewspabelanger: looking14:10
*** armax has joined #openstack-infra14:11
*** alexchadin has quit IRC14:12
Shrewspabelanger: they are held because you've put an autohold on somewhere14:12
openstackgerritThierry Carrez proposed openstack-infra/infra-specs master: Flat Git Namespace  https://review.openstack.org/46187814:12
Shrewspabelanger: the only comment is "pabelanger" so that's all the info i have14:12
pabelangerShrews: ya, I put an autohold o zuulv3 last night14:13
pabelangerbut, I assumed it was for 1 node14:13
*** slaweq has quit IRC14:13
pabelangerShrews: Did I have to delete my autohold?14:13
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Add role to fetch output from nodes  https://review.openstack.org/51184314:13
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Update upload-logs to process docs as well  https://review.openstack.org/51185314:13
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Upload fetched artifacts to log server  https://review.openstack.org/51185814:13
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Only publish artifacts when zuul.change is not defined  https://review.openstack.org/51185914:13
Shrewspabelanger: what command did you use?14:13
pabelanger1 sec14:13
pabelangersudo zuul autohold --tenant openstack --project openstack-infra/openstack-zuul-jobs --job base-integration-debian-jessie --reason pabelanger14:14
pabelangerthat was on zuulv3.o.o14:14
*** trown|brb is now known as trown14:15
*** hongbin has joined #openstack-infra14:16
pabelangermordred: https://review.openstack.org/511656/ when you have a moment14:16
Shrewspabelanger: that would hold any nodes requested by that job for 1 run of that job. i don't know how many nodes that job uses, but you shouldn't need to delete your autohold (unless you specified a count arg). but you can try if you want using --count=014:18
Shrewspabelanger: but you'll need to manually delete those nodes14:18
pabelangerShrews: ya, that's how I understood it too. But, not sure why autohold held more then 1 node, in fact 31 :)14:19
*** Qiming has quit IRC14:19
erlon@guys, I'm trying to increase the amount of log my CI can store. After systemd, all devstack logfiles are trimmed and usually we loose a lot of useful information. Does anyone knows how to increase that?14:19
pabelangerShrews: okay, I can delete them now, but is that something we want to add to zuulv3-issues and debug with you and jeblair?14:19
*** Qiming has joined #openstack-infra14:19
pabelangerShrews: before I delete, I'd also wouldn't mind you seeing my 100-0000416513 has been pending for some time14:21
*** huanxie has joined #openstack-infra14:21
*** yamamoto has quit IRC14:21
*** sree has joined #openstack-infra14:21
openstackgerritMerged openstack-infra/project-config master: Use system-required instead of merge-check  https://review.openstack.org/51174514:22
*** slaweq has joined #openstack-infra14:23
*** andreas_s has quit IRC14:23
pabelangerShrews: okay, looks like 100-0000416513 has been stuck, retrying in citycloud-kna114:23
pabelangerwhich, doesn't have quota any more14:23
openstackgerritJames E. Blair proposed openstack-infra/infra-manual master: Don't use in-line yaml lists  https://review.openstack.org/51187614:24
tinwoodsorry, very quick question to anyone you knows.  Is zuul v3 running, or is it back to zuul v2?   Sorry, if this a dumb question; not sure where to look to see what is the version.  e.g. status.openstack.org/zuul/ shows v2.6.0 which means I'm guessing v2 is running.14:24
pabelangertinwood: yah, we are still on zuulv2.5 ATM.14:25
pabelangertinwood: we're looking to start rollout again, this coming Sunday 22:00 UTC14:25
*** sree has quit IRC14:25
tinwoodpabelanger, thanks for the update.14:26
Shrewspabelanger: yeah, no quota. but nodes are deleting14:27
pabelangerShrews: okay, and we'd never release the request to another provider right?14:28
*** andreas_s has joined #openstack-infra14:28
Shrewspabelanger: nope14:28
AJaegerjeblair: thanks for fixing my typos and grammar!14:28
*** egonzalez has quit IRC14:29
AJaegerpabelanger: but the debian jessie is not the note the post job needs...14:29
*** ramishra has quit IRC14:29
pabelangerAJaeger: mordred: ^ see discussion with Shrews. We're basically blocked on infra-post job, until we get more quota in citycloud-kna114:29
*** yamahata has joined #openstack-infra14:30
*** rbrndt has joined #openstack-infra14:31
pabelangerI admit, I'd don't fully understand why we try forever for a single provider, but seems like we are more fragile to getting into these blocking scenarios. Where, if we know we no longer have quota, why not release the request for another provider to try?14:31
pabelangerbecause, we do have other providers with ready xenial nodes, but not getting used14:31
jeblairpabelanger: why is there no quota?14:32
Shrewsjeblair: out of 10 total, 8 are deleting, 1 in hold, 1 is ready but of the wrong label14:33
pabelangerwe have 8 nodes in deleting state, 1 held and 1 ready currently in kna1, which gets us 10 max-servers14:33
jeblairare the deleting nodes stuck?14:33
pabelangerI think they are14:34
pabelangerlooking at logs now14:34
Shrewsthe deletes are timing out14:34
*** Dinesh_Bhor has quit IRC14:34
pabelangeryah, see that now14:35
Shrewsno idea why. currently searching scheduler logs for a hold issue, so if someone else could look at the timeout, that would be helpful14:35
*** dave-mcc_ is now known as dave-mccowan14:35
*** andreas_s has quit IRC14:36
*** cshastri has quit IRC14:38
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Remove merge-check  https://review.openstack.org/51174714:38
pabelangerHm14:38
pabelanger3432242d-846c-4b5a-9fd8-309edb280cd5 in citycloud-kna1, is currently BUILD state14:39
pabelangerand see when we send a delete, it is not affecting it14:39
pabelangerhowever, citycloud-kna1 is working currently. I can see nodepool v2 launching servers14:40
*** yamamoto has joined #openstack-infra14:41
openstackgerritBen Nemec proposed openstack-infra/tripleo-ci master: Support pike->master undercloud upgrades  https://review.openstack.org/51188514:42
*** caphrim007_ has quit IRC14:43
*** rcernin has joined #openstack-infra14:45
*** LindaWang has quit IRC14:45
*** spectr has quit IRC14:46
*** LindaWang has joined #openstack-infra14:46
Shrewsjeblair: hrm, did we have a recent gear release maybe? Seeing this for autohold jobs: http://paste.openstack.org/show/623582/14:47
Shrewsoh, that may not be gear related14:47
*** e0ne has quit IRC14:47
Shrewsgrrr14:47
jeblairShrews: we did have a recent gear release, but it only had one change, and that was a send mutex14:47
*** bobh has quit IRC14:48
Shrewsjeblair: yeah, i saw JSON decode error and asked too quickly14:49
*** claudiub has joined #openstack-infra14:50
*** andreas_s has joined #openstack-infra14:50
*** huanxie has quit IRC14:51
*** slaweq has quit IRC14:56
AJaegermordred: do you want to abandon https://review.openstack.org/#/c/508767/ - seeing jeblair's comment?14:56
*** jistr|mtg is now known as jistr14:58
Shrewspabelanger: I think I see the bug causing the autohold issue (though I don't understand the underlying cause yet). I believe you'll need to do the autohold clear (using --count=0) to stop this from recurring for now14:59
fungitonyb: ianw: oh, wow, sorry about that... i knew we'd switched to linking cgit instead of gitweb in the gerrit webui but didn't realize we'd actually disabled access to the gitweb interface on review.o.o entirely. i agree that'll break a number of scripts which were depending on it (especially ones which want to retrieve things like configuration from the all-projects "repo" which we don't mirror15:00
fungianywhere)15:00
*** vhosakot has joined #openstack-infra15:01
openstackgerritAndreas Jaeger proposed openstack-infra/project-config master: Re-enable v3 pipelines  https://review.openstack.org/51189915:01
AJaegerteam, here's change to enable the pipelines again once we're ready to start ^15:02
pabelangerShrews: k, let me do that now15:02
pabelangerShrews: I can then start the clean up process for debian-jessie15:02
*** rcernin has quit IRC15:03
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Update upload-logs to process docs as well  https://review.openstack.org/51185315:03
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Upload fetched artifacts to log server  https://review.openstack.org/51185815:03
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Only publish artifacts when zuul.change is not defined  https://review.openstack.org/51185915:03
*** slaweq has joined #openstack-infra15:03
mordreddmsimard: thanks for the reviews!15:03
mordredAJaeger: looking15:03
clarkbfungi: we didnt so much disable gitweb as it completely stopped working :(15:04
*** andreas_s has quit IRC15:04
pabelangerShrews: okay, held nodes have been deleted15:05
mordredpabelanger: re: BUILDING/delete -- yah, deleting a node in building state is unpossible15:07
clarkberlon: the systemd switch in devstack shouldnt have changed what you are  logging if you are using devstack-gate. It grabs the service logs out of journalctl15:08
clarkberlon: generally though look to journalctl to find the logs15:08
*** mpaolino has quit IRC15:08
*** andreas_s has joined #openstack-infra15:09
*** huanxie has joined #openstack-infra15:09
erlonclarkb: yeap, but by default systemd uses Nopersistent logging, which I think is the reason why our logs are so short (2M vs 15M on infra)15:10
erlonclarkb: may be is there any configuration on your nodepool scripts to set that?15:10
mordredwhat is nopersistent?15:11
openstackgerritDavid Shrewsbury proposed openstack-infra/zuul feature/zuulv3: Better exception handling during autohold  https://review.openstack.org/51190215:12
Shrewsjeblair: pabelanger: ^^15:12
erlonmordred: does not use disk to store logs, saves on /run15:12
clarkberlon: our image builds force systemd to write to disk rather than its ring buffer15:12
openstackgerritChandan Kumar proposed openstack-infra/project-config master: Added initial jobs for python-tempestconf  https://review.openstack.org/51119415:13
clarkberlon: you can create the dir it looks for or change the config from auto to persistent15:13
erlonclarkb: hmm, ok, other than that is there anything else?15:13
chandankumarAJaeger: tosky is this correct now https://review.openstack.org/#/c/511194/ ?15:13
clarkbno thats it15:13
*** andreas_s has quit IRC15:13
chandankumarmordred: hello15:13
erlonclarkb: good, ill try that15:13
toskychandankumar: remove also publish-openstack-python-branch-tarball, it is part of openstack-python-jobs15:14
openstackgerritChandan Kumar proposed openstack-infra/project-config master: Added initial jobs for python-tempestconf  https://review.openstack.org/51119415:15
*** smarcet has joined #openstack-infra15:15
smarcetfungi: morning how are u doing ? :) just wanna let u know that this one https://review.openstack.org/#/c/497616/ seems that got stuck15:16
chandankumarmordred: on review.openstack.org people are not able to add me as a reviewer15:16
chandankumarmordred: and on this https://review.openstack.org/#/admin/groups/1842,members and also i am not able add new members here15:17
chandankumarmordred: ianw found that on my email-id <chkumar@redhat.com> there are two account, that's why people are not able to add me as a reviewer15:17
chandankumarmordred: my Account ID 12393 is this please have a look, thanks :-)15:18
chandankumartosky: done15:18
*** LindaWang1 has joined #openstack-infra15:20
*** LindaWang has quit IRC15:22
*** alex_xu has quit IRC15:22
*** LindaWang1 is now known as LindaWang15:22
*** florianf has quit IRC15:23
*** florianf has joined #openstack-infra15:23
*** annp has joined #openstack-infra15:24
*** LindaWang has quit IRC15:26
rajinirSetting up new workspace by the builds is taking an hour now. Used to take 3-5 mins. What could be the cause?15:26
openstackgerritDavid Shrewsbury proposed openstack-infra/zuul feature/zuulv3: Better exception handling during autohold  https://review.openstack.org/51190215:26
rajinirhttps://www.irccloud.com/pastebin/iIslATLA/Settung%20up%20workspace15:27
clarkbchandankumar: I'll take a look in a few minutes, need to get to real keyboard15:27
*** LindaWang has joined #openstack-infra15:27
clarkbrajinir: you should look in the workspace setup log to see where it spent its time15:27
clarkbrajinir: is this from an infra run job or third party ci?15:28
rajinirclarkb: Thirdparty CI15:28
*** Qiming has quit IRC15:29
rajinirclarkb: thanks let me check the logs15:30
*** alex_xu has joined #openstack-infra15:31
openstackgerritAndreas Jaeger proposed openstack-infra/project-config master: Added initial jobs for python-tempestconf  https://review.openstack.org/51119415:32
*** Qiming has joined #openstack-infra15:33
*** fanzhang_ has joined #openstack-infra15:34
AJaegeranybody for a quick +3 ^ - I just added a newline15:34
chandankumarAJaeger: thanks you fix it :-)15:34
*** smatzek has quit IRC15:35
dmsimardAJaeger: hm, I wonder if we should be doing this at all or if we should instead make it go straight to v315:35
dmsimardAJaeger: once we switch to v3, this project won't have any legacy jobs.15:35
AJaegerdmsimard: wait, wait - that change is exactly the policy we want ;)15:36
AJaegerdmsimard: read https://docs.openstack.org/infra/manual/zuulv3.html#what-not-to-convert15:36
AJaegerdmsimard: there's no legacy job in that one15:36
*** slaweq has quit IRC15:36
dmsimardAJaeger: first time I see system-required o_O /me searches15:37
AJaegerdmsimard: we merged it today ;)15:37
dmsimardoh no wonder then15:37
AJaegerdmsimard: new name for merge-check15:37
dmsimardAJaeger: oh, okay15:37
AJaegerdmsimard: we want to have it easy to add more to it later15:37
AJaegerdmsimard: see backscroll from 17hours earlier15:37
dmsimard:(15:37
*** smatzek has joined #openstack-infra15:38
*** dtantsur is now known as dtantsur|afk15:39
*** huanxie has quit IRC15:40
pabelangermordred: jeblair: clarkb: AJaeger: Now that we have an idea of the infra-post jobs being stuck, I'd like to see if we are ready to do a restart on nb04.o.o for new shade release and rax uploads15:41
*** slaweq has joined #openstack-infra15:41
*** annp has quit IRC15:41
dmsimardAJaeger, chandankumar: +1 in https://review.openstack.org/#/c/511194/ but added a comment15:43
clarkbdmsimard: did you see I think I got ara on failure tested last night and it is working. So next step is to add it to base if that hasnt happened yet15:44
dmsimardclarkb: yup, I saw. Let me see if it's in base yet, otherwise I can submit it.15:44
toskydmsimard: no jobs defined? I thought that adding the template you get the jobs defined there automatically15:45
Shrewspabelanger: did you do anything for cleaning up the instances stuck 'deleting'?15:45
dmsimardtosky: the jobs that are being added are zuul v2 jobs, which will work and run today (or whenever it merges)15:46
dmsimardtosky: however, since the v3 "migration script" has already ran and we won't be running it again, these v2 jobs won't be migrated to v315:46
pabelangerShrews: only clean up was when I deleted held debian-jessie node15:46
pabelangerthat opened quota for xenial to build15:46
pabelangerhowever, we still have stuck nodes in delete state15:46
dmsimardtosky: therefore you need to setup a zuul.yaml file in python-tempestconf with your jobs and layout there15:47
*** bobh has joined #openstack-infra15:47
toskydmsimard: where is openstack-python-jobs defined?15:47
dmsimardtosky: there is an equivalent template for that in v3, hang on15:47
Shrewspabelanger: and those are because we tried to delete them while they were building?15:47
*** dhajare has quit IRC15:47
pabelangerShrews: yah, according to mordred, which will never delete15:48
pabelangerso, should like we might need to get citycloud to help15:48
toskydmsimard: doesn't zuul.d/project.yaml contain zuul v3 jobs definitions? Otherwise the legacy- jobs defined there wouldn't make sense15:48
dmsimardtosky: there is openstack-python35-jobs and openstack-python-jobs (py27+pep8)15:49
toskydmsimard: and that review adds openstack-python-jobs15:49
dmsimardtosky: the jobs exists, they just aren't applied to openstack/python-tempestconf15:49
dmsimardtosky: that adds it for zuul v315:50
dmsimardzuulv2***15:50
dmsimardoh fuck15:50
dmsimardit adds it to zuul v315:50
* dmsimard hides15:50
toskyexactly :)15:50
pabelangerbut, like I asked above, couldn't we release the request to another provider if we know quota is full? And have provider B fullfill the request? In the case this morning, we had plenting of xenial nodes available over all15:50
toskyI went through that already, so I know the feeling15:50
dmsimardtosky: yeah.. I'll blame that on friday and lack of caffeine.15:51
dmsimardtosky: sorry if I confused you, I was totally off track15:52
*** chlong has quit IRC15:53
toskydmsimard: no, no problem, really, better clarify this now15:53
toskynext week it will be (another) roller coaster15:53
* AJaeger sends coffee and weekend to dmsimard ;)15:53
fungiclarkb: oh, so the gitweb in 2.13 was basically broken and that's what urged us to switch our linking to git.o.o? that makes more sense but i guess i had forgotten it already15:53
toskyspeaking of which... what is the current plan for the re-deployment of zuul v3? Monday?15:53
dmsimardAJaeger: weekend, yes, please send me more weekends15:53
dmsimardtosky: not today :D15:53
*** caphrim007 has joined #openstack-infra15:53
toskydefinitely :D15:54
dmsimardtosky: so far it's looking like sunday, there'll be an email sent out I suspect.15:54
AJaegerpabelanger: go ahead!15:54
*** Qiming has quit IRC15:55
fungismarcet: thanks for the heads up, i've rechecked it so hopefully will merge shortly15:55
clarkbfungi: ya15:55
fungii'll keep a closer eye on it this time, sorry!15:55
*** LindaWang has quit IRC15:56
*** Qiming has joined #openstack-infra15:57
clarkbchandankumar: when you login what account id shows up at https://review.openstack.org/#/settings/ is it 12393?15:58
*** lucasagomes is now known as lucas-afk15:58
clarkbchandankumar: or is it 8944?15:59
*** Apoorva has joined #openstack-infra16:00
clarkbchandankumar: it looks like you may have ended up using both accounts? If you can clarify which one is the one you are currently using and whether or not that is the one you would like to continue using that would be helpful16:01
pabelangerokay, stopping nb04 to pickup latest shade16:01
*** Apoorva has quit IRC16:01
fungiclarkb: one common pattern there too is users who end up submitting changes with the username/ssh keys on their earlier account but logging into the webui and reviewing with their later account16:02
Shrewspabelanger: i'm going to add a check to nodepool to not try to delete an instance that is building, but yeah, we might need provider help to cleanup the ones that are still building16:03
clarkbfungi: chandankumar interesting, both accounts have an ssh username set16:03
Shrewspabelanger: oh, wait plz16:03
clarkbchandankumar: so maybe you can clarify which ssh user you are using as well16:03
fungiclarkb: but also sometimes they end up switching usernames with the new account, so double-checking both the account id number showing in the webui _and_ the ssh username they're using to submit changes can be useful16:03
Shrewspabelanger: i want to watch builder logs but i'm stuck doing too much at the same time. gimme a minute?16:03
*** reed has quit IRC16:04
pabelangerShrews: sorry, builder already stopped16:04
Shrewspabelanger: ok, go ahead and start it16:05
pabelangerShrews: is it builder you want or nodepool-launcher?16:05
Shrewspabelanger: builder. that's where i discovered the shade bug16:05
pabelangerkk16:05
pabelangerHmm16:05
pabelangeri see a lot of open mounts from DIB16:05
openstackgerritAndreas Jaeger proposed openstack-infra/infra-manual master: Recommendations and examples for v3 job usage  https://review.openstack.org/51191516:05
AJaegermordred: I tried it myself ^16:05
pabelangerlet me see if I can manually clean them up16:05
pabelangerianw: when you are back, we might want to audit nb03 and see why we are leaving mounts from dib^16:06
clarkbchandankumar: to summarize I think there are at least two pieces of info that we need to untangle which account to use going forward. First is what account id yo usee in the web ui settings page and second what ssh user you use to push code. And so that its written down somewhere the two accounts are 8944 and 1239316:06
*** reed has joined #openstack-infra16:07
Shrewsmordred: fyi, nodepool builder being restarted with newest shade.16:07
pabelangermordred: Shrews: I think it might be faster to reboot nb04 then manually unmount all the open mount points from dib16:08
pabelangerany objections?16:08
pabelangerinfra-root^16:08
Shrewspabelanger: ok, i'm off16:08
clarkbdmsimard: after a quick skim of project-config changes I'm not seeing one for doing ara on failure on base.16:09
*** makowals has quit IRC16:09
*** baoli_ has joined #openstack-infra16:09
pabelangerokay, rebooting nb0416:10
clarkbif we can get that in I think the next step is considering reenabling check/periodic on v316:10
AJaegerclarkb: https://review.openstack.org/511899 is the patch for that16:10
clarkbAJaeger: ty16:10
pabelangerShrews: okay, back16:11
*** Apoorva has joined #openstack-infra16:11
*** baoli__ has joined #openstack-infra16:12
*** kjackal_ has quit IRC16:12
*** ykarel is now known as ykarel|afk16:12
*** kjackal_ has joined #openstack-infra16:12
*** baoli has quit IRC16:13
*** claudiub has quit IRC16:13
*** slaweq has quit IRC16:14
*** baoli_ has quit IRC16:15
pabelangerOpenStackCloudException: Multiple matches found for ubuntu-trusty-150778183516:15
*** yamahata has quit IRC16:15
pabelangerthat was same as yesterday16:15
pabelangerand think mordred suggested we write a shade script to clean up leaked images?16:16
AJaegerclarkb: I squeezed the three changes we did for disabling together into one in 51189916:16
*** lnxnut_ has quit IRC16:16
*** slaweq has joined #openstack-infra16:16
mordredpabelanger: yah - I can do that real quick16:16
*** slaweq has quit IRC16:18
*** slaweq has joined #openstack-infra16:18
*** ykarel|afk has quit IRC16:20
*** kiennt26 has joined #openstack-infra16:20
dmsimardclarkb: I thought I had sent a change for base failure for ara, guess not. I'll send one now.16:21
*** pcaruana has quit IRC16:23
openstackgerritDavid Shrewsbury proposed openstack-infra/nodepool feature/zuulv3: WIP: Do not delete instances that are 'building'  https://review.openstack.org/51191716:25
*** huanxie has joined #openstack-infra16:26
Shrewspabelanger: i believe that ^^^ will prevent us from trying to delete a building instance until it changes status. I need a test for it still.16:26
*** kjackal_ has quit IRC16:27
*** felipemonteiro has joined #openstack-infra16:27
*** jpich has quit IRC16:28
*** fried_rice is now known as fried_rice_injer16:28
*** fried_rice_injer is now known as friedrice_injera16:29
*** yamamoto has quit IRC16:29
openstackgerritDavid Moreau Simard proposed openstack-infra/project-config master: Only generate ARA reports on failure for base jobs  https://review.openstack.org/51191816:32
*** felipemonteiro has quit IRC16:32
*** apetrich has quit IRC16:32
dmsimardinfra-root: ^ enabling limited report generation for v3 jobs as per discussed16:32
*** apetrich has joined #openstack-infra16:32
pabelangerShrews: I'll defer to both jeblair and mordred on 511917. I'm not sure how best to handle it16:33
*** derekh has quit IRC16:33
openstackgerritMerged openstack-infra/tripleo-ci master: Configure OVB jobs to use local mirrors for images  https://review.openstack.org/51143416:33
*** xarses has joined #openstack-infra16:33
Shrewspabelanger: k, but if deletes for a building instance get lost in the ether, then without that code, we'll have deleter threads that stick around forever (or until a restart)16:34
smarcetfungi: thx u :)16:34
*** snapiri has quit IRC16:34
*** rpittau has quit IRC16:34
fungismarcet: yw, looks like it should merge at any moment now16:35
*** fanzhang_ has quit IRC16:35
openstackgerritMerged openstack-infra/system-config master: OpenStackID Release 1.0.23  https://review.openstack.org/49761616:35
mordredpabelanger, Shrews: I have written a script in ~nodepool on nl01.openstack.org called cleanup.py16:36
*** dhajare has joined #openstack-infra16:36
pabelangermordred: looks good, did you want to run it now?16:39
*** camunoz has joined #openstack-infra16:41
Shrewspabelanger: hrm, actually, i think the delete may timeout and we'd try to delete again later, which makes my change unnecessary (though slightly more efficient). I might abandon it16:42
mordredpabelanger: biggest question - it will delete all copies of an image that has duplicates - which might leave us without a particular image on a cloud - I think that should be ok thought, right?16:44
Shrewsthat is, in fact, what it's currently doing16:44
fungiShrews: i manually delete instances in a building state semi-regularly, so at least some deployments work just fine that way16:44
fungiand waiting into it reaches active before issuing the delete would therefore be at least marginally less efficient in those16:45
fungis/into/until16:45
*** trown is now known as trown|lunch16:45
pabelangermordred: yah, I think so. It would be good to audit rackspace again after we run your script to see if any other leaked images are found16:46
mordredpabelanger: kk. I run now16:46
Shrewsfungi: manual intervention is needed for these in citycloud-kna116:48
fungithat's pretty wacky. i wonder if they got stuck for "other" reasons16:49
fungiand not simply because we tried to delete them while they were building16:49
Shrewsfungi: there are 8.16:49
*** jtomasek has quit IRC16:49
Shrewsfungi: all fedora-26, interestingly16:49
fungii've seen that happen when new instances get scheduled to a broken compute node16:50
fungii wonder if whatever was broken was related to that particular image (maybe with nova's image cache?)16:50
pabelangerfungi: yah, I am thinking a hypervisor issue myself16:50
fungithough probably pointless to speculate16:50
pabelangerwe should reach out to citycloud however16:50
Shrewsi have to afk for a bit16:51
pabelangerShrews: mordred just seen a new (for me) exception on nb04.o.o debug.log16:51
pabelangerwhen you have time16:51
fungibasically you have one compute node with something wrong with the particular cached image, where any time you ask it to boot that it never exits building... then it becomes a tarpit slowly accumulating broken building instances over some span of time16:51
fungiuntil eventually most of your quota is taken up by them16:52
mordredpabelanger: awesome!16:52
clarkbcan we get reviews on https://review.openstack.org/#/c/511918/1 to only run ara on failure, then we should consider https://review.openstack.org/511899 to reenable v3 pipelines16:53
pabelangerlooking16:54
*** huanxie has quit IRC16:56
mordredpabelanger: OpenStackCloudHTTPError: (409) Client Error for url: https://sto2.citycloud.com:9292/v2/images/4a8ccead-b90a-47a7-a968-178f2ec79994/file Conflict (Inner Exception: Expecting value: line 1 column 1 (char 0))16:56
pabelangerclarkb: left comment16:56
mordredpabelanger: is that the new exception of which you speak?16:56
pabelangermordred: yah, that's it16:56
mordredkk. thanks16:56
pabelangerdmsimard: ^comment on 51189916:57
clarkbdmsimard: see pabelanger's comment16:57
*** wolverineav has joined #openstack-infra16:58
clarkbpabelanger: dmsimard I could imagine there may be cases where overriding that to true could be useful for some reason, I doubt projects would go overriding it, but wouln't be the first time I've been suprirsed by job configs :)16:58
*** epende has quit IRC16:59
pabelangerclarkb: dmsimard: ya, I think we should just set it at http://git.openstack.org/cgit/openstack-infra/project-config/tree/playbooks/base/post-logs.yaml#n5 just to be safe, then we don't have to find jobs and track down extra inode usage again16:59
dmsimardclarkb: ara runs by default and requires to be trusted, we can't run "manually" in base-minimal integration tests.16:59
pabelangerthis is only short term any ways16:59
clarkbdmsimard: one of the things that make base-minimal minimal is it does not run ara17:00
dmsimardI'm not really worried about singular jobs overriding the config, if anything, it leaves us an easy way to test things17:00
*** AJaeger has left #openstack-infra17:00
*** AJaeger has quit IRC17:00
clarkbdmsimard: so setting the var there is mostly noop17:00
*** AJaeger has joined #openstack-infra17:00
*** epende has joined #openstack-infra17:00
pabelangerwell, if jobs override it, we have no idea, until we run out of inodes17:01
pabelangerthat would be my concern17:01
dmsimardclarkb: we do run ara in minimal (exactly because we can't test it anyway) http://git.openstack.org/cgit/openstack-infra/project-config/tree/playbooks/base-minimal/post-logs.yaml#n517:01
*** dhajare has quit IRC17:01
*** jascott1 has quit IRC17:01
dmsimardpabelanger: let's not exaggerate the impact of ara on inodes17:01
clarkbdmsimard: huh the comment is wrong above base minimal then "doesn't validate the node and does not generate an ARA report"17:01
*** jascott1 has joined #openstack-infra17:02
clarkbdmsimard: we can clean that up in a followup17:02
dmsimardclarkb: oops ?17:02
pabelangerdmsimard: sure, but this is the main reason why we are doing this today for ARA, otherwise, why bother?17:02
dmsimardpabelanger: it would require projects to consciously and explicitely override that variable, I feel if someone does that, they know what they're doing, and that someone could be one of us17:03
*** jpena is now known as jpena|off17:03
pabelangerI think in general, we shouldn't allow jobs to override anything in base17:04
*** e0ne has joined #openstack-infra17:04
pabelangerbut will defer to others17:04
clarkbya I think generally not overriding base is a good idea. Since its the highly opinionated base for everyting17:05
dmsimardAny vars in ansible can be overridden with extra vara17:05
*** shardy has quit IRC17:05
dmsimardIf we want people to not override anything in base, we need to think about a lot of different things, like preventing vars from being passed as extra vars for trusted content17:06
*** slaweq has quit IRC17:06
dmsimardUnless that's already the case, but I don't know.17:06
*** shardy has joined #openstack-infra17:06
clarkbextra vars are controlled by zuul directly iirc?17:07
clarkblike its locked up in the zuul config because thos eare passed on ansible invocation17:07
*** Goneri has joined #openstack-infra17:07
dmsimardHmmm, vars are set in the inventory, not as extra vars, you're right17:08
dmsimardextra-vars are also 'almighty' in the sense that even if you try to override it with a set_fact, it won't take effect17:08
pabelangeronly secrets and site vars are extra_vars today17:08
*** yamahata has joined #openstack-infra17:09
*** yamamoto has joined #openstack-infra17:09
*** yamamoto has quit IRC17:09
*** jtomasek has joined #openstack-infra17:09
*** kiennt26 has quit IRC17:09
pabelanger2017-10-13 17:09:48,426 INFO nodepool.builder.UploadWorker.9: Image build opensuse-423-0000000525 in rax-dfw is ready17:10
pabelangeryay17:10
pabelangerShrews: mordred ^ upload to rax17:10
mordredpabelanger: woot17:10
* dirk wonders why opensuse was exciting17:12
pabelangerwhat isn't exciting about opensuse :D17:12
clarkbmordred: did you want to weigh in on the ara on failure where to set the var thing? chnage 51191817:12
pabelangerdirk: we've been having issues uploading to rax over the last few days, new shade release appears to have fixed it17:13
*** vhosakot has quit IRC17:13
*** jascott1 has quit IRC17:13
dirkpabelanger: ah, thanks for info17:14
*** dhajare has joined #openstack-infra17:16
*** sambetts is now known as sambetts|adk17:20
*** sambetts|adk is now known as sambetts|afk17:20
*** ijw has joined #openstack-infra17:20
mordredclarkb: looking17:20
*** andreas_s has joined #openstack-infra17:21
openstackgerritAndreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Remove api-guide remains  https://review.openstack.org/51192417:21
odyssey4memordred another one - our pike tests are still frequent, so I ported the patch back: https://review.openstack.org/51192317:21
*** dhinesh has joined #openstack-infra17:22
dmsimardclarkb, pabelanger: so, circling back to the base jobs and ara report generation. My method allows it to be overridden on a need basis but we can enforce if it we want. Which do we want ? IMO enforcing it just makes it harder to perhaps selectively tests things (such as an error not showing up in a false positive, for example, or whatever it could be)17:23
dmsimardI'll settle with the majority here17:23
clarkbdmsimard: I think I'm fine either way, mordred was looking at it as of a couple minutes ago so curious what he says17:23
clarkbI do think we should get that in and then enable v3 pipelines again though17:24
*** dhinesh_ has joined #openstack-infra17:25
*** andreas_s has quit IRC17:25
pabelangerclarkb: before we bring zuulv3 back online, we should look at zuulv3 mergers, they are currently down (all). I noted on the zuulv3-issues etherpad last night17:26
pabelangerzm05 to zm0817:26
clarkbpabelanger: says that jeblair is looking at it?17:27
mordredpabelanger, clarkb, dmsimard: I'm fine with it on the job and allowing a job to override it - there might be a valid reasons it needs to be set on a case by case basis17:28
pabelangerclarkb: yah, see that now. Just wanted to make sure we maybed solved it before start up with jobs again17:28
clarkbpabelanger: we should probably make sure jeblair is looking at it (but maybe that is why jeblair has been so quiet, busy digging into that?)17:28
*** dhinesh has quit IRC17:29
pabelangersure17:29
pabelangerwe're also making good process on rackspace uploads now on nb04.o.o17:29
pabelangermordred: did your shade script on nb04 also finish?17:29
mordredpabelanger: still runnning17:31
AJaegertrivial fix for a typo in a job: https://review.openstack.org/511840  - reviews welcome17:32
AJaegermordred: I worked some more on zuulv3 manual, what do you think of https://review.openstack.org/511915  ? And let's approve https://review.openstack.org/#/c/511876/ , please17:33
AJaegerthanks, clarkb and pabelanger !17:34
clarkbpabelanger: you good with mordreds comment?17:34
*** camunoz has quit IRC17:36
*** slaweq has joined #openstack-infra17:37
*** camunoz has joined #openstack-infra17:37
*** dhajare has quit IRC17:37
*** e0ne has quit IRC17:37
openstackgerritMerged openstack-infra/project-config master: Fix typo in role  https://review.openstack.org/51184017:38
pabelangerclarkb: yah, just wanted to point it out and make sure all on board.17:38
pabelanger+317:39
clarkbpabelanger: thanks17:39
openstackgerritDavid Moreau Simard proposed openstack-infra/project-config master: Improve the comment for the base-minimal job  https://review.openstack.org/51193217:41
dmsimardclarkb: ^ fixed your comment17:41
*** tosky has quit IRC17:41
*** trown|lunch is now known as trown17:41
clarkbdmsimard: thanks17:42
*** rbrndt has quit IRC17:42
*** armax has quit IRC17:42
*** nicolasbock has quit IRC17:42
inc0good morning, are you planning Friday afternoon deployment?17:42
openstackgerritMerged openstack-infra/project-config master: Only generate ARA reports on failure for base jobs  https://review.openstack.org/51191817:42
dmsimardinc0: for zuul v3? no.17:43
inc0just checking, any chances you can enable some nodepool for zuulv3 so we can work on our jobs over weekend?17:43
openstackgerritAndreas Jaeger proposed openstack-infra/project-config master: WIP: Fix api-guide/api-ref publishing  https://review.openstack.org/51193317:43
*** gfidente has quit IRC17:43
AJaegermordred: the publish jobs for api-guide/api-ref are wrong ^17:43
AJaegermordred: that change is incomplete - do you have a good idea on how to get the servicename? ^17:44
pabelangerinc0: were talking about enabling zuulv3 check pipelines again now, but unsure just yet17:44
dmsimardclarkb, mordred: did we settle on sunday ? are we sending an email out ? There was some concerns, here for example: http://lists.openstack.org/pipermail/openstack-dev/2017-October/123584.html17:44
pabelangerbut mostly wanting to make sure were stable from recent CI outage17:44
*** jtomasek has quit IRC17:45
clarkbpabelanger: we seemto have stabilized around 62 million free inodes. its been floating up and down around that number today17:46
clarkbdmsimard: I thought so, mordred did email go out? I saw ianw had a followup email on the outages17:46
pabelangerdmsimard: yah, I think I'd rather rollout zuulv3 and help projects get jobs working.17:47
dmsimardpabelanger: rollout as in, enable check and keep v2 gating ?17:47
AJaegerclarkb: haven't seen an annoucement by mordred today17:47
*** jascott1 has joined #openstack-infra17:47
AJaegerdmsimard: switch off v317:47
pabelangerdmsimard: no, rollout zuulv3 and stop zuulv217:47
AJaegerdmsimard: switch off v2 I mean17:47
pabelangerI don't think logs.o.o will be able to keep up with 2 zuuls17:48
inc0how about roll out partially, without legacy jobs, and first thing after weekend roll 3?17:48
dmsimardinc0: rolling without legacy jobs would be a bit complicated, I think ?17:49
inc0that will allow few silent days (although over weekend) for no lifers like me to work on gates17:49
*** dosaboy has quit IRC17:49
inc0we can just comment *a lot* of lines in one file;)17:49
inc0https://github.com/openstack-infra/openstack-zuul-jobs/blob/master/zuul.d/zuul-legacy-jobs.yaml17:50
clarkbI've got to pop out for lunch shortly (driving into town) but if mordred (or someone) can send that email saying plan is for sunday that would be great17:50
clarkbinc0: rough plan is turn on pipelines again today since friday and weekend is low volume allowing people to iterate on jobs17:50
clarkbthen about 2200UTC Sunday we will flip things back to v317:51
jlvillalNot sure if anybody would like to review: https://review.openstack.org/#/c/509670/  Support the log level "CRITICAL"17:51
inc0cool, that'd be awesome, thanks!17:51
*** armax has joined #openstack-infra17:52
*** camunoz has quit IRC17:53
openstackgerritMerged openstack-infra/project-config master: Improve the comment for the base-minimal job  https://review.openstack.org/51193217:53
*** camunoz has joined #openstack-infra17:54
*** baoli__ has quit IRC17:54
fungiclarkb: i expect we'll start gaining ground on the inode count again once activity trails off over the next few hours17:55
*** zaneb has quit IRC17:55
openstackgerritKen Dreyer proposed openstack/python-jenkins master: build_job: return queue item identifier  https://review.openstack.org/51193717:56
openstackgerritKen Dreyer proposed openstack/python-jenkins master: add get_queue_item() method  https://review.openstack.org/51193817:56
openstackgerritKen Dreyer proposed openstack/python-jenkins master: better get_build_info() example  https://review.openstack.org/51193917:56
fungibut yeah, we've been hovering at 93% used inodes since i woke up17:56
clarkbok I'v egot to pop out now, I'll be back around probably 1:30 PDT or so. Would be great if someone can send an email with status update17:57
*** ralonsoh has quit IRC18:01
*** panda|rover is now known as panda|rover|off18:01
openstackgerritKen Dreyer proposed openstack/python-jenkins master: build_job: return queue item identifier  https://review.openstack.org/51193718:04
openstackgerritKen Dreyer proposed openstack/python-jenkins master: add get_queue_item() method  https://review.openstack.org/51193818:04
openstackgerritKen Dreyer proposed openstack/python-jenkins master: better get_build_info() example  https://review.openstack.org/51193918:04
*** baoli has joined #openstack-infra18:05
openstackgerritAndreas Jaeger proposed openstack-infra/project-config master: WIP: Fix api-guide/api-ref publishing  https://review.openstack.org/51193318:07
*** yamamoto has joined #openstack-infra18:10
*** pblaho has quit IRC18:12
*** tosky has joined #openstack-infra18:13
*** armaan has quit IRC18:13
*** Guest53680 is now known as mgagne18:14
*** mgagne has quit IRC18:14
*** mgagne has joined #openstack-infra18:14
mgagneWe would like to schedule a maintenance next Tuesday (October 17) on inap-mtl01 to perform an upgrade on Nova at around ~6:00pm EDT. Would that be ok with you? API will be unavailable but existing instances will continue to be reachable.18:15
*** smarcet has quit IRC18:16
clarkbmgagne: should be fine, should we expect to disable nodepool against that cloud beforehand?18:18
mgagneclarkb: what do you recommend? IIRC, last time, we didn't do anything to that regard.18:18
clarkbnodepool should gracefully handle it it will just hit your api (or try to) and fail18:19
clarkbif that is ok on your end we may just leave it to run on its own18:19
fungiit's not strictly necessary, just helps if you want our load to go away before you start working18:19
*** yamamoto has quit IRC18:19
mgagnewe are updating Nova, we won't update anything related to hypervisor themselves so instances can stay18:20
*** eharney has quit IRC18:20
thingeeam I understanding right that the post jobs for publishing docs is leaning on the instructions with tox?18:20
fungithingee: yeah, they basically run `tox -e venv python setup.py build_sphnix`18:21
fungier, sphinx18:21
fungihelps to spell it correctly18:22
thingeeAJaeger: sounds perfect to me18:22
fungii think technically it ends up being `tox -e venv -- python setup.py build_sphinx` but basically the same18:22
thingeeAJaeger: whoops sorry wrong channel :)18:22
AJaegerthingee: we can talk here as well ;)18:23
fungimordred has a proposal out to change the pti to expect projects to just run sphinx-build without needing tox18:23
fungibut the our jobs won't reflect that until the tc has had a chance to debate and approve something along those lines18:24
*** rbrndt has joined #openstack-infra18:26
*** r0llerd3rby has joined #openstack-infra18:26
dmsimardfungi: on my end I just run tox -e docs ?18:26
fungidmsimard: yeah, a lot of projects include that in their tox.ini today as a convenience18:27
fungibut it's technically not what our pti demands official projects support18:27
*** ldnunes has quit IRC18:27
*** ldnunes has joined #openstack-infra18:28
*** mriedem1 has joined #openstack-infra18:29
dmsimardlooking at what python-jobs provides, it circles back to https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/macros.yaml#L314 and then ultimately https://github.com/openstack-infra/project-config/blob/master/jenkins/scripts/run-docs.sh which sounds way more complicated than it should be18:29
fungiinfra-root: since 511918 (Only generate ARA reports on failure for base jobs) has merged, are we missing anything else before we reenable check and tripleo-check for zuulv3.o.o?18:30
fungii can send the announcement about sunday, but it would be nice if the announcement could mention we're running v3 check jobs again18:30
AJaegerfungi, AFAIK we're ready to +2A https://review.openstack.org/511899 ;)18:31
fungithanks AJaeger!18:31
*** mriedem has quit IRC18:31
clarkbpabelanger had concerns about the zuul merger18:31
clarkbso that might need debugging too18:31
*** eharney has joined #openstack-infra18:31
fungioh, right, jeblair may or may not be looking into the mergers18:31
*** mriedem1 is now known as mriedem18:32
*** eharney has quit IRC18:32
pabelangeryah, I think we need input from jeblair on zuulv3 mergers, zm05 to zm08 are not running anything ATM18:32
*** eharney has joined #openstack-infra18:32
*** electrofelix has quit IRC18:32
pabelangernb04.o.o is pretty much recovered, rackspace has been getting new images uploaded since we restarted with lastest shade18:33
mordredpabelanger: awesome!18:33
*** dprince has quit IRC18:33
pabelangerand we're just building a new xenial DIB now with latest AFS mirror content18:34
fungiyeah, looks like jeblair's shell sessions on most of those mergers are only idle for a few hours, so presumably he was doing something on them at least that recently18:34
pabelangerthat should go live in the next hour18:34
*** zaneb has joined #openstack-infra18:34
dmsimardclarkb: where is https://review.openstack.org/#/c/501887/ used ? it mentions puppet, afaik we no longer use puppet on nodepool nodes so that would be for infrastructure nodes ?18:35
pabelangerand looking at reprepro on mirror-update.o.o, it appears to be happy again18:35
*** claudiub has joined #openstack-infra18:35
fungidmsimard: correct18:36
fungidmsimard: related to the mechanism we use for adding and disabling our individual admin shell accounts on our control-plane and similarly persistent servers18:37
*** pcrews has quit IRC18:38
fungidmsimard: and he's repeatedly rechecking it in hopes of triggering a nondeterministic failure we get in those validation jobs18:38
fungiso we can hopefully get additional clues as to what's causing it18:38
dmsimardokay18:39
*** armax_ has joined #openstack-infra18:40
fungidmsimard: when it fails, it looks like this http://logs.openstack.org/16/497616/1/gate/gate-infra-puppet-apply-3-centos-7/154b032/puppetapplytest29.final.out.FAILED18:41
dmsimardfungi: that sounds faulty indeed, the key should lay on a single line and start with ssh-rsa, ssh-dsa etc18:42
fungiyep!18:42
fungiso what we want to know is why it does this, and why it's only occasional (usually the job succeeds)18:42
*** armax has quit IRC18:42
*** armax_ is now known as armax18:42
*** lihi has quit IRC18:42
dmsimarddo we want to merge that ? or is it self tested ?18:43
*** slaweq has quit IRC18:43
*** kjackal_ has joined #openstack-infra18:43
dmsimardoh, it's self tested, okay http://logs.openstack.org/87/501887/2/check/gate-infra-puppet-apply-3-centos-7/52c6356/console.html#_2017-10-13_00_00_27_43276618:44
fungiyeah, it's fine for us to merge it, i think, because then the next time any of our changes hits that nondeterministic error, we can get more detail to help us track it down18:44
dmsimardmaybe we should merge it and add an e-r query18:44
*** lihi has joined #openstack-infra18:45
*** gouthamr has quit IRC18:47
fungii don't think we index those logs18:47
fungiinto logstash/elasticsearch18:47
fungibut i could be mistaken18:47
*** slaweq has joined #openstack-infra18:48
*** dosaboy has joined #openstack-infra18:48
*** felipemonteiro has joined #openstack-infra18:48
*** dosaboy has quit IRC18:48
*** rhallisey has quit IRC18:49
*** dosaboy has joined #openstack-infra18:50
*** felipemonteiro_ has joined #openstack-infra18:50
openstackgerritMike Perez proposed openstack-infra/project-config master: Adding check/publish jobs for contributor guide  https://review.openstack.org/51194618:53
*** felipemonteiro has quit IRC18:53
thingeeAJaeger: I think I did that right? ^18:54
thingeeI didn't update zuul/layout.yml ... it looks like that would involve creating a legacy job.18:54
thingeebut I'm also very confused. :)18:55
fungipabelanger: without running zuul-merger processes, how is zuulv3 running a job for 511946,1 now?18:57
pabelangerfungi: each executor also has a merger process, so we do some merging18:58
fungiahh18:58
pabelangerbut will be a scaling issue without main18:58
fungigot it18:58
AJaegerthingee: just use the template and you#re good to go.18:59
*** jcoufal has quit IRC18:59
AJaegerThis is v3 setup- which will be fully used by Monday, no need to hazzle with v2 until then18:59
jeblairfungi, pabelanger: i put my name next to the merger issue on the etherpad.  :)19:00
jeblairfungi: i understand the issue, it's not a blocker.19:01
mordredpabelanger, Shrews: fwiw, I'm still poking at the image cleanup thing19:05
*** ijw has quit IRC19:05
*** baoli has quit IRC19:05
*** baoli has joined #openstack-infra19:06
* AJaeger signs off and wishes everybody a nice evening, weekend etc19:10
*** AJaeger has quit IRC19:10
*** baoli has quit IRC19:10
*** florianf has quit IRC19:10
*** florianf has joined #openstack-infra19:11
*** baoli has joined #openstack-infra19:12
*** friedrice_injera is now known as fried_rice19:12
*** baoli has quit IRC19:13
fungijeblair: so we can approve 511899 safely without waiting to have merger daemons running on zm05-08 again?19:14
openstackgerritDavid Moreau Simard proposed openstack-infra/openstack-zuul-jobs master: Add integration tests for emit-job-header and set-zuul-log-path-fact  https://review.openstack.org/51194819:14
*** baoli has joined #openstack-infra19:15
jeblairfungi: yep. i just did so.19:15
fungithanks!19:15
*** florianf has quit IRC19:15
dmsimardmordred: see this patch ^ otherwise https://review.openstack.org/#/c/511821/ could have landed and broken every gate19:15
fungii'll get the announcement drafted up in an etherpad and link it in here in a moment19:15
*** florianf has joined #openstack-infra19:15
mordreddmsimard: yay!19:15
fungiunless mordred already had one in progress?19:16
* dmsimard proposes we volounteer mordred announcer in chief19:16
*** armaan has joined #openstack-infra19:17
openstackgerritMerged openstack-infra/project-config master: Re-enable v3 pipelines  https://review.openstack.org/51189919:17
fungiwe need infra-branded bullhorns/megaphones19:17
* fungi watches zuulv3 for signs of life in check pipelines19:18
*** baoli has quit IRC19:19
*** r0llerd3rby has quit IRC19:20
*** slaweq has quit IRC19:21
*** slaweq has joined #openstack-infra19:21
*** andreas_s has joined #openstack-infra19:21
openstackgerritDavid Moreau Simard proposed openstack-infra/openstack-zuul-jobs master: Add integration tests for emit-job-header and set-zuul-log-path-fact  https://review.openstack.org/51194819:21
fungizuulv3 check lives again19:22
pabelangerfungi: https://review.openstack.org/511656/ would be a nice one to land also. caps launch-timeout on citycloud to 10mins19:22
*** eggshell has joined #openstack-infra19:24
*** gouthamr_ has joined #openstack-infra19:24
dmsimardpabelanger: 10 minutes ? wow19:29
*** florianf has quit IRC19:30
*** florianf has joined #openstack-infra19:30
openstackgerritMike Perez proposed openstack-infra/project-config master: Adding check/publish jobs for contributor guide  https://review.openstack.org/51194619:31
fungiinfra-root: proposed announcement is https://etherpad.openstack.org/p/X6d2vXrmDm19:32
*** ijw has joined #openstack-infra19:33
*** andreas_s has quit IRC19:34
mordredpabelanger, Shrews: ok. dupicate images should be all taken care of19:36
*** florianf has quit IRC19:37
openstackgerritDavid Moreau Simard proposed openstack-infra/openstack-zuul-jobs master: Add integration tests for fetch-zuul-cloner  https://review.openstack.org/51195319:37
*** florianf has joined #openstack-infra19:37
dmsimardfungi: so this means that sunday v2 is no longer gating, correct ? I feels it needs an explicit mention19:40
dmsimardbrb19:40
fungidmsimard: starting at 22:00 sunday, yes19:41
SamYapletwo days until i can publish to dockerhub19:41
fungii'll clarify, thanks19:41
*** Swami has joined #openstack-infra19:41
fungiSamYaple: and 2.5 days until dockerhub humbly requests that you cease and desist? ;)19:41
SamYaplefungi: haha no. my images are like 40mb, ill be the least of thier worries19:42
SamYaplebut if they do, quay.io is totally cool wit hthis19:42
jeblairfungi: ++19:43
fungidmsimard: does my extra sentence in the tl;dr provide the necessary illumination?19:43
dmsimardfungi: yes19:44
*** ccamacho has quit IRC19:46
fungiawesome19:47
bodenhi, where can I find zuul v3 logs if I have a -1V from Zuul, but no jobs (logs) listed on the patch?  https://review.openstack.org/#/c/511542/19:49
openstackgerritKen Dreyer proposed openstack/python-jenkins master: build_job: return queue item identifier  https://review.openstack.org/51193719:49
openstackgerritKen Dreyer proposed openstack/python-jenkins master: add get_queue_item() method  https://review.openstack.org/51193819:49
openstackgerritKen Dreyer proposed openstack/python-jenkins master: better get_build_info() example  https://review.openstack.org/51193919:49
fungiboden: hit the "toggle ci" button at the bottom and expand the commend from the "Zuul" user19:49
pabelangermordred: yah, nodepool looks happy again19:50
bodenfungi thank you19:50
mtreinishfungi: is there any plans to fix that somehow. (probably add on to the js for the tbale) I've see a bunch of people confused by that19:51
fungiboden: looks like it should be a - instead of a _19:51
bodenfungi: ya, my bad :(19:51
pabelangerfungi: ++19:52
fungimtreinish: depends on what you mean by "plans" but i think most of us assume we'll amend the ci results table to show a row for "syntax error" or something as well as teach it about some of the new job failure types19:52
*** Swami has quit IRC19:53
mtreinishfungi: I guess that assumption was it. I just wanted to know if it's on the radar for a future work item (I'm assuming there is a long to do list for all the things that need to be updated post migration)19:54
fungimtreinish: yep, i expect if anyone who's not totally turned off by hacking on javascript wanted to take that up, none of us would object19:54
fungiotherwise, we'll likely get around to it soon-ish19:55
fungiokay, licking the stamp for the announcement now19:55
*** smarcet has joined #openstack-infra19:56
*** andreas_s has joined #openstack-infra19:57
*** Swami has joined #openstack-infra19:58
mtreinishfungi: heh, well my js is terrible. So I'm probably not volunteering for that :)19:58
openstackgerritKen Dreyer proposed openstack/python-jenkins master: get_build_console_output: fix docs for "number" parameter  https://review.openstack.org/51195419:59
*** smarcet has quit IRC20:00
fungimtreinish: you could add it to the zuul section of the zuulv3-issues etherpad if you want, so we don't forget it needs doing at least20:00
openstackgerritDavid Moreau Simard proposed openstack-infra/openstack-zuul-jobs master: Add integration tests for fetch-zuul-cloner  https://review.openstack.org/51195320:01
mtreinishfungi: done20:01
dmsimardpabelanger: btw I had a comment on https://review.openstack.org/#/c/510155/20:02
*** andreas_s has quit IRC20:02
pabelangerdmsimard: possible, I haven't had a change to test that playbook yet. will update shortly20:03
fungithanks mtreinish20:05
jeblairthat's not really an issue with zuul20:06
fungigood point, we could create a gerrit section20:06
*** felipemonteiro__ has joined #openstack-infra20:06
fungii'll do that now20:06
*** smatzek has quit IRC20:09
*** felipemonteiro_ has quit IRC20:09
*** iyamahat has joined #openstack-infra20:10
fungilooking through the issues pad, "Devstack-gate jobs do not find cached files in ~zuul when running under the zuul user in v3" is solved now, yes?20:11
fungii'm double-checking a v3 job now to confirm20:12
fungier, build i mean20:12
clarkbya 511260 fixed that20:12
*** sdague has quit IRC20:13
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Remove -e from requirements  https://review.openstack.org/51195720:15
jeblairpabelanger, mordred, fungi: ^ that should fix the mergers20:15
jeblair(they were never started with the new git timeout code because of that error)20:15
*** andreas_s has joined #openstack-infra20:16
pabelangerah.. +220:16
fungicompletely agree with that fix20:18
*** AJaeger has joined #openstack-infra20:19
AJaegerteam, please check the zuul v3 failures in https://review.openstack.org/#/c/511501/ - this looks really broken, we should not get RETRY_LIMIT on a simple pep8 anymore....20:19
AJaegerSorry, no time to investigate - I'm gone, just saw this and wanted to pass on...20:20
* AJaeger waves good night20:20
*** AJaeger has quit IRC20:20
*** kgiusti has left #openstack-infra20:20
jeblairand really gone!20:20
fungiclarkb: yeah, confirmed we have a v3 devstack run here which didn't need to download get-pip.py, so i'll move that issue to the archive pad: http://logs.openstack.org/97/509697/4/check/congress-devstack-py35-api-mysql/dce7f2d/logs/devstack-early.txt.gz#_2017-10-13_19_41_45_67620:20
jeblairthe thing aj pointed out is it apparently needs horizon to run its pep8 jobs20:22
pabelangerlooking20:22
jeblairhttp://logs.openstack.org/01/511501/2/check/openstack-tox-pep8/8d9a420/job-output.txt.gz#_2017-10-13_19_38_16_44493320:22
*** thorst has quit IRC20:23
pabelangerodd20:23
*** thorst has joined #openstack-infra20:23
pabelangerah, sahara-dashboard20:25
pabelangermaybe it is using horizon apis20:25
fungicurious why that retried instead of failing20:25
pabelangerhttp://git.openstack.org/cgit/openstack/sahara-dashboard/tree/tools/tox_install.sh20:26
pabelangerseems to have its own tox installer20:26
pabelangerunsure why that would be20:27
jeblairfungi: in order to support the tox-siblings functionality, we do a tox env creation with no commands first, in the pre-playbook.  that's what failed.20:27
toskyuh20:27
fungipabelanger: most projects have a pip wrapper like that one, called from tox, to shoehorn in constraints files so that local devs can optionally use them20:27
*** thorst has quit IRC20:28
pabelangerfungi: interesting, didn't know that20:28
fungipabelanger: most of the constraints-using projects anyway20:28
toskyuhm, maybe we can kill it20:28
toskyor maybe not20:29
toskyit was added recently, and from outside the sahara team20:29
*** andreas_s has quit IRC20:29
toskyadded by tonyb20:29
*** trown is now known as trown|outtypewww20:30
jeblairremote:   https://review.openstack.org/511501 WIP Initial Zuul v3 jobs20:30
jeblairi pushed a revision20:30
*** Apoorva_ has joined #openstack-infra20:30
toskyjeblair: oh, interesting20:31
toskyI couldn't proceed with the experiments because zuul v3 stopped voting20:31
jeblairoh, i didn't add pep8 there, i'll push another20:31
jeblairi'm pretty sure this is going to prompt an interesting conversation with ajaeger and mordred next week20:32
fungipabelanger: oh, i should clarify, specifically most constraints-using libraries which are themselves represented in the constraints list need a wrapper like that around pip to first edit themselves out of the constraints list before installing. see commit message on https://review.openstack.org/413354 for details20:32
toskyjeblair: one question: if the job is defined locally, is its definition merged with the definition from project-config, or does it replace that definition completely?20:32
jeblairtosky: but i think it's useful to at least push this forward till it's working so we have something more definitive to talk about :)20:32
jeblairtosky: merged20:32
jeblairtosky: so i'm about to add pep8 to that change, which will basically alter the pep8 definition that comes from project-config to add the required project20:33
jeblairtosky: pushed ps3 -- hope you don't mind20:34
jeblairer, ps4 rather20:34
*** Apoorva has quit IRC20:34
toskyI will complete it with the dashboard job20:34
toskybut probably on Monday20:34
*** andreas_s has joined #openstack-infra20:34
toskyso does it mean that zuul v3 is voting again on all projects?20:35
toskyI have another recheck to launch then20:35
jeblairtosky: voting in check, yes20:35
pabelangerfungi: ya, I wonder if tox-siblings might help with that now20:35
fungitosky: yes, and come monday it'll be gating on them20:35
pabelangerI haven't look much at tox-siblings myself just yet20:35
fungipabelanger: maybe within the ci, but not for the local developer use case20:35
pabelangerfungi: true20:35
fungipabelanger: oh, except maybe it does once we have a good story around local devs firing their tests with ansible?20:36
fungiso i think the answer to that is, not so much just yet, but perhaps soon20:36
pabelangeryah, will be interesting to see if more devs embrace ansible locally, or continue along with bash scripts20:38
pabelangerwait and see approach20:38
*** pcrews has joined #openstack-infra20:38
toskyjeblair: would it make sense to define a set of openstack-tox-{pep8,py27,py35}-dashboards jobs with the added dependency on horizon that can be used by all horizon dashboards?20:39
mordredfungi, pabelanger: yes - tox-siblings should help with the tox/pip/constraints thing20:39
*** andreas_s has quit IRC20:39
fungijeblair: thanks for the clarification on the RETRY_LIMIT for that one... that means we should occasionally expect RETRY_LIMIT failures on changes adding/modifying job configuration when it runs afoul of the tox-siblings pre task, but not for changes which aren't touching job configuration?20:39
mordredalso, tonyb and I looked a little bit in to potentially using -c$UPPER_CONSTRAINTS_FILE in the tox.ini in the requirements line - which should make the need for the wrapper script go away for hte libraries that are themselves in constraints20:39
mordredso there are 2 different things that are different in the new world order that will help us simplify how this works today20:40
mordredI think we were waiting to push on the 'use -c in tox.ini' approach until v3 settled down to reduce confusion20:40
openstackgerritMerged openstack-infra/puppet-zuul master: Add statsd to zuul.conf  https://review.openstack.org/51101820:40
fungimordred: how does that solution you discussed with tonyb work around the need to filter the project itself out of teh constraints list?20:41
mordredyes20:41
mordredbecaues it's not putting it into the install command entry liek we do today20:41
mordredbut instead only in the requirements entry20:41
jeblairtosky: yeah, that would work.  let's run it by andreas monday20:41
fungiohhh20:41
mordredso that it gets applied when tox installs the requirements20:41
mordredbut not when it installs the repo itself20:41
openstackgerritIhar Hrachyshka proposed openstack-infra/project-config master: Revert "neutron: Make grenade-neutron-dvr-multinode job non-voting"  https://review.openstack.org/50531820:42
fungimordred: got it, but not when tox installs other things (unless also explicitly copied to commands in the other tox envs which don't use the normal requirements?20:42
*** ihrachys has joined #openstack-infra20:42
jeblairwow.  git just *completely* bombed on a merge conflict resolution.  like, i think i'm going to need to rewrite the patch.20:43
fungiwha? that's unfortunate20:43
jeblairi guess maybe i could try redoing it with another merge resolve strategy20:44
mordredtosky, jeblair: might also want to consider an openstack-tox-dashboards project-template that adds requirements into the requirements in the pipeline definitions20:44
jeblairi've never done that before20:44
*** thorst has joined #openstack-infra20:44
*** wolverineav has quit IRC20:44
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Remove -e from requirements  https://review.openstack.org/51195720:44
jeblairmordred, tosky: ^ yeah that's better i think20:44
fungijeblair: i've done that very occasionally when i want to emulate the behavior of gerrit/zuul's git merge conflict resolution choices20:45
toskymordred, jeblair: template which uses the normal jobs instead of single extended jobs?20:45
toskywhy not20:45
jeblairtosky: yeah, it'd be the project-template version of what i did in that patch20:45
*** edmondsw has quit IRC20:45
*** Apoorva_ has quit IRC20:46
mordredtosky: yah - that way it's still the 'openstack-tox-pep8' job that's getting run, just with specific additions (mostly I don't think the difference matters, but in this particular case (the core tox jobs) it might be nicer for the jobs themselves to be 'the same')20:46
mordredphysically each option will do th eright thing - this is more about managing how we think about things20:46
*** thorst has quit IRC20:48
pabelanger+1 openstack-tox-dashboards20:49
*** andreas_s has joined #openstack-infra20:52
*** smatzek has joined #openstack-infra20:53
*** wolverineav has joined #openstack-infra20:59
clarkblooks like v3 is alive again!21:00
pabelangerokay, nb04 is done with images for today! Which means we ubuntu DIBs now live, for a few hours, using latest reprepro ubuntu.mirrors21:00
pabelangerand I haven't seen any issues21:01
pabelanger\o/21:01
*** iyamahat has quit IRC21:02
*** iyamahat_ has joined #openstack-infra21:02
pabelangerclarkb: yah, just seen https://review.openstack.org/510991/ pass all zuulv3 jobs (requirements)21:02
clarkblooks like we are still at 62 million ish inodes free21:02
pabelangerthat is great21:02
clarkbwow21:02
*** slaweq has quit IRC21:04
*** andreas_s has quit IRC21:05
jeblairfungi: strategy=resolve strategy-options=patience :)21:08
mordred\o/21:09
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Switch statsd config to zuul.conf  https://review.openstack.org/51101721:09
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Emit some stats from executor  https://review.openstack.org/51107321:09
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Emit some nodepool stats  https://review.openstack.org/51108521:09
jeblairhopefully that did the right thing.  i think tests will tell us if it chewed it up undetectably21:09
dmsimardjeblair: can we start rewriting zuul v3 in puppet now ? https://twitter.com/laserllama/status/91888466336362086521:09
*** andreas_s has joined #openstack-infra21:10
jeblairi don't even know why i follow twitter links :)21:11
*** huanxie has joined #openstack-infra21:12
*** ihrachys has quit IRC21:12
dmsimardok, here's the direct one: https://puppet.com/products/puppet-bolt21:12
dmsimardalso, bit of friday trolling never hurt anyone :D21:12
dmsimardclarkb: do we have/should we have a cacti graph for inodes ?21:13
jeblairdmsimard: no/yes21:13
*** ldnunes has quit IRC21:13
clarkb++21:13
tonybfungi: Correct it does result in a small ammount of duplication but if gets rid of 1 reason to use tox_install.sh21:13
tonybmordred: Yeah while it isn't directly related to the v3 work I am waiting for the switch befoer I more forward.21:14
dmsimardok let's see if I can figure it out21:14
fungitonyb: i'm 200% in favor of any option which gets rid of even some of the random shell scripts people are cargo culting from repo to repo21:14
*** smatzek has quit IRC21:14
* tonyb should send an email saying 'we're chaning the way we use constratints and would like testers'21:15
jeblairdmsimard: it' probably requires admin privs to add; unfortunately our cacti system largely pedates our automation, so about the only thing that's been automated is adding new hosts21:15
* tonyb kinda started with shade ;P21:15
dmsimardjeblair: ah, I was trying to see if the configuration was stored in system-config cacti things21:15
fungidmsimard: it _should_ be but likely isn't. if you want to hack on some sort of templating we're missing there, i'm happy to get you copies of configuration21:16
tonybpabelanger, tosky there's a review for dragonflow? that make tox_install work with both v2 and v321:17
dmsimardfungi: last time I've done any meaningful amount of cacti was years ago, so something to pattern off of would be great, yes :)21:17
tonybHavign said that maybe it's worth doign the constratints before v3?21:17
fungidmsimard: lmk when you have bandwidth to work on it and i'll dig up the configs on the server21:17
dmsimardfungi: it was either that or trying to figure something out for our ara situation mid term21:18
fungitonyb: "before v3" is... well, the next 48 hours and change21:18
fungi(if all goes well)21:18
jeblair48 hours and 42 minutes21:18
jeblairnot that anyone's counting21:18
fungi;)21:19
clarkb"the final countdown!"21:19
jeblairbig red clock21:19
fungina-na-na-naaaa21:19
dmsimardfungi: looks like it's readily available over snmp so it shouldn't be very hard to add http://www.net-snmp.org/docs/mibs/UCD-SNMP-MIB.txt21:19
*** andreas_s has quit IRC21:19
jeblairyeah, i think the way to do it would be to add it to the "data query" that's used for partitions21:19
fungidmsimard: yes, i'm painfully aware, we simply hadn't been querying those oids previously because we didn't have frequent issues with inode capacity anywhere21:19
dmsimardfungi: oh, sure, just saying I would add it if I could :D21:20
jeblairthat way it pulls in a data template for each partition and you get set up with a data source automatically21:20
tonybfungi: Yeah the 'plan' was to use proposal bit to keep the cargo culted scripts in sync but that seemed unpopular so I never implemented it21:21
tonybfungi: Well it wont happen in 48 hours ;P21:21
jeblairit looks like the '-e' fix has gone in and things have been reinstalled correctly on the mergers, so i will start them now21:22
*** wolverineav has quit IRC21:22
jeblairdone21:22
tonybfungi: to be clear the proposal mordred and I are proposing will only remove the ommon case the sibling projects thing is still up in the air21:23
tonybfungi: and I don't have a good solution that works with depends-on and local development21:24
fungitonyb: yep, i think the compelling story there is to make it easy for people to invoke job playbooks locally21:24
fungipulling in the same tox-sibling playbook we use in ci21:24
tonybfungi: Interesting, that kinda moves away from the make OpenStack more std. but seems like a move in the right direction21:25
openstackgerritMonty Taylor proposed openstack-infra/shade master: Handle glance image pagination links better  https://review.openstack.org/51196321:25
*** dhinesh_ has quit IRC21:25
tonybfungi: so WRT gitweb should I just switch owners.py to using cgit?21:25
*** armax has quit IRC21:25
clarkbtonyb: yes I think so, gitweb was unfunctioning upon upgrade and review-dev had been running cgit (it was something we had planned on doing a long time ago just didn't show up until we were more forced into it)21:26
fungitonyb: yes21:26
fungisorry for not realizing that before today21:26
tonybokay patch incomming ;P21:26
tonybfungi: np there's a lot going on ;P21:26
fungiwe didn't intentionally take down gitweb, it was a casualty in the war on gerrit21:27
tonyb:)21:27
clarkbI feel like gerrit is winning21:27
jeblairtonyb: while we have a moment -- is the elections site okay?  do we need to do any forced publication or anything like that?  just wanted to check in on it specifically because things have been hectic recently.21:27
mordredtonyb: which thing becomes harder from a 'run locally' perspective? the 'run with the tip of a different project' part?21:27
tonybthat may be the least positive things I've heard clarkb say ;P21:28
fungidmsimard: so poking around a bit, i think what you'll want is a copy of our /var/lib/cacti/linux_host.xml file21:28
clarkbtonyb: ha21:28
tonybjeblair: election is okay for right now but in ~week we need the publish jobs working21:28
dmsimardfungi: there's one already in system-config but it dates back to 5 years ago, is that the same one ?21:28
fungidmsimard: oh! yes, probably21:28
fungiit's 6867 lines long21:28
tonybjeblair: I can push some trivial chnages next week to test that and thenreact accordingly21:28
dmsimardfungi: https://github.com/openstack-infra/system-config/blob/master/modules/openstack_project/files/cacti/linux_host.xml21:29
tonybjeblair: but mordred had patches that looked to fix it?21:29
mordredI did?21:29
tonybmordred: Yup install this gitrepo at this SHA21:29
jeblairdmsimard: i at least tried not to have it all be server side magic :)21:30
tonybmordred: the static-site publishing things you did ~2 weeks ago21:30
dmsimardjeblair: you write all that XML by hand? :p21:30
mordredtonyb: ah nod21:30
dmsimardfungi: I've already entered an ara rabbit hole worth exploring so I'll circle back to inodes in cacti unless someone beats me to it21:30
jeblairdmsimard, fungi: that's an export of all the custom goodness that we do to actually get correct data.  it will at the least, let us recreate all the templates if needed, as well as replicate it on a local machine.21:30
jeblair(you import that template into cacti, and it creates a bunch of database records for those things)21:32
*** mat128 has quit IRC21:32
tonybmordred: this is the hack we're currently using: https://git.openstack.org/cgit/openstack/castellan-ui/tree/tools/tox_install.sh?id=198f56de0dd67d1e404beebd971ec2d9eb291696#n7221:33
*** andreas_s has joined #openstack-infra21:33
mordredtonyb: ah - wild :)21:33
mordredtonyb: so - fwiw, when I want to run tox tests with a git state of a different project ...21:34
mordredtonyb: I tend to do: .tox/py35/bin/pip install -e ../other-project ... and then just run tox -epy35 again as normal21:34
mordredtonyb: but I also don't use git branch and instead use git reset - so I'm a little weird21:34
*** dave-mccowan has quit IRC21:34
tonybmordred: Hmm okay that's cute.  I wonder how the devs in affected projects will feel about it?21:35
*** felipemonteiro__ has quit IRC21:36
mordredtonyb: well - it shouldn't matter too much - that script won't break tox-siblings - it just won't use the exact same mechanism to get the other repo21:36
*** armax has joined #openstack-infra21:36
mordredtonyb: so if we want to keep that tox_install script to enable that usage - I don't think anything will be amiss21:37
*** slaweq has joined #openstack-infra21:37
mordredShrews, clarkb, SamYaple, pabelanger: https://review.openstack.org/511963 'fixes' the issue seen with extra /v2 url segments21:38
tonybmordred: okay.  the zuul-cloner shim will work once the require-projects is setup correctly, but fail before that correct?21:38
*** thorst has joined #openstack-infra21:38
mordredtonyb: that's right21:38
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Emit some nodepool stats  https://review.openstack.org/51108521:39
mordredtonyb: but also the zuul-cloner shim is not needed in the gate once required-projects are set up21:39
mordredin v321:39
*** anupn has joined #openstack-infra21:39
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Specify duplicate project defn behavior  https://review.openstack.org/51102521:39
tonybmordred: okay that's what I thought.21:40
*** esberglu has quit IRC21:40
anupnHi all, is it possible to run pxe_ipmitool_tinyipa job without swift service?21:41
* tonyb will try to clean all that up next week'ish21:41
mordredOH21:41
mordredhttps://git.openstack.org/cgit/openstack/castellan-ui/tree/tox.ini#n2021:41
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: WIP: experiment with late-binding inheritance  https://review.openstack.org/51135221:41
mordredI fully see what's going on there21:41
mordrednow21:41
anupnI understand for agent_* drivers in Ironic, i have to use swift service21:41
*** Apoorva has joined #openstack-infra21:41
mordredwow ... SO ...21:41
tonybmordred: sorry yeah I shoudl have included that url too21:42
fungitonyb: lmk if you need help with the gitweb->cgit patch for owners.py, though if memory serves i corralled all the gitweb interaction into one function for ease of updating21:42
*** thorst has quit IRC21:42
*** huanxie has quit IRC21:42
*** camunoz has quit IRC21:42
fungitonyb: and happy to review any patch for that, just give me a heads up in case i miss it21:43
tonybfungi: Yeah, I was spliting it out again but I'll see what that looks like.21:43
mordredtonyb: if I can make a recommendation - we should have that os: logic (or maybe the install_from_git logic) look to see if ~/src/git.openstack.org/$PROJ exists, and if so use it ... or, hrm. lemme think about this a little more21:43
tonybmordred: Yup exactly, that's what drgaonflow does21:44
mordredtonyb: (I obviously totally see the issue this is solving for - mostly trying to help think about it in ways that don't continue to use zuul-cloner)21:44
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Use weakref for change cache  https://review.openstack.org/51135521:44
tonybmordred: I think the zuul-cloner stuff can go once we pass the point of never going back to v221:44
mordredtonyb: cool. I'd also love to, if we can, avoid double-installing required-projects21:45
tonybmordred: +121:45
mordredtonyb: I think we may be back to "it's really unfortunate that we don't publish server projects to pypi"21:45
tonybmordred: in *theory* we could pull them from the .zuul.yaml21:45
mordredtonyb: can't count on that - it might also be set in a central repo21:46
tonybmordred: Yeah I'm looking at doign that and had good conversations at the PTG along those lines where I was given a heap of probelms to consider21:46
mordredtonyb: yah - I think I gave you some of those :)21:46
*** andreas_s has quit IRC21:46
tonybmordred: Yeah (to problems) and Yeah to things bing in the central repos21:47
mordredtonyb: neutron and horizon at least both have pypi records21:47
mordred(that we own)21:47
tonybmordred: True that may be enough for the near term.21:47
mordredkeystone is the one that has someone else on it21:47
tonybmordred: Yeah but it hasn't been used since ~201221:48
mordredyah - maybe we could convince Dan Crosta to release it to us21:48
tonybmordred: possibly, or ask the cheeseshop to do it.  I want to generate the list of all conflicts before we look to hard at that21:49
mordred++21:49
jeblairmordred: tox-siblings does normal tox installation, then over-installs siblings, right?21:49
mordredyes21:49
mordredtonyb: I'd like to see if we can just start doing that - because the MAIN issue with these is that there is no other way for tox to install horizon into castellan-ui currently other than via this kind of thing - so normal dev work that isn't trying to simular depends-on locally is still borked21:50
jeblairmordred: and when you say you want to avoid double installing, you mean you want to avoid the step where we install from git.openstack.org, and then install from ~zuul21:51
*** boden has quit IRC21:51
mordredjeblair: yah21:51
jeblairmordred: publication to pypi means we'd still be on a double-install path, but we're installing from mirorrs, so the ci cost is small.21:51
tonybmordred: Yup.  it *is* a requirement that we can't express :( so we need to look at fixing that anyway21:51
mordredtonyb: ++21:51
mordredjeblair: exactly21:51
mordredjeblair: the normal case is 'install from pypi' which is how we install things in tox21:51
*** andreas_s has joined #openstack-infra21:51
mordredthe exceptional case that's likely mostly used in the gate and not locally is "please use this newer version"21:52
mordredso if we published even just neutron and horizon we could avoid openstack-tox-dashboard-jobs and openstack-tox-jobs-networking since those projects could all actually express their actual requirements21:52
jeblairi suspect the 'newer version' thing may be much more used for projects like this than other types, but still yes.21:53
mordredjeblair: ++21:53
jeblairmordred: we *could* avoid those templates, but i'm not certain it's desirable -- if these projects *are* depending on in-development software, then they'd want to keep the required-projects entries and make use of tox-siblings.21:54
jeblairit is, however, a *choice* at that point which is desirable in all cases :)21:54
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Better exception handling during autohold  https://review.openstack.org/51190221:54
mordredjeblair: yes - I'd like for them to be able to test both things if they want21:55
*** Nil_ has quit IRC21:55
mordred"I want to test against the most recent release" and "I want to test against master"21:55
jeblairya21:55
*** andreas_s has quit IRC21:56
mordredZOMG I ACTUALLY LOGGED IN TO STORYBOARD!!!!21:56
*** iyamahat_ has quit IRC21:57
*** iyamahat has joined #openstack-infra21:57
jeblairmordred: celebrate with a trivial +3?  :)  https://review.openstack.org/51187621:58
SamYaplemordred: celebrate with a non-trival +3? :) https://review.openstack.org/#/c/506502/22:01
openstackgerritTony Breeds proposed openstack-infra/system-config master: Switch to cgit from gitweb  https://review.openstack.org/51197022:04
*** bobh has quit IRC22:07
tonybfungi: I'm going AFK for $some_time feel free to take over/replace ^^ whichever works for you22:07
openstackgerritMerged openstack-infra/infra-manual master: Don't use in-line yaml lists  https://review.openstack.org/51187622:07
*** florianf has quit IRC22:09
*** ijw has quit IRC22:09
*** florianf has joined #openstack-infra22:10
*** andreas_s has joined #openstack-infra22:10
*** slaweq has quit IRC22:10
*** tpsilva has quit IRC22:10
*** florianf has quit IRC22:10
*** zaneb has quit IRC22:11
*** tlaxkit has quit IRC22:11
openstackgerritMerged openstack-infra/bindep master: Add new syntax to allow matching multiple profile  https://review.openstack.org/50650222:13
*** andreas_s has quit IRC22:14
*** andreas_s has joined #openstack-infra22:14
fungitonyb: thanks, i'll take a gander at it22:16
*** slaweq has joined #openstack-infra22:19
jeblairthe statsd patches for zuulv3 are ready for review.  it's a series of 3 patches starting at https://review.openstack.org/51101722:21
jeblairit might be handy to have those in soonish22:21
*** gouthamr_ has quit IRC22:23
mordredjeblair: looking22:23
*** cody-somerville has joined #openstack-infra22:24
SamYaplew00t! thanks mordred. been waiting for that patch for a while22:24
*** Swimingly has quit IRC22:25
mordredSamYaple: it's like I'm your personal santa claus this week22:25
SamYaplethe one time im thankful for chrismas in october22:26
*** esberglu has joined #openstack-infra22:26
*** Swimingly has joined #openstack-infra22:26
*** andreas_s has quit IRC22:27
mordred:)22:28
openstackgerritMonty Taylor proposed openstack-infra/shade master: Handle glance image pagination links better  https://review.openstack.org/51196322:31
*** esberglu has quit IRC22:31
*** dbecker has quit IRC22:32
mordredjeblair: also - you might find https://review.openstack.org/#/c/511963/ amusing22:32
mordredjeblair: for some definitions of 'amusing'22:32
jeblairmordred: you had me at 'pagination'22:34
*** baoli has joined #openstack-infra22:34
*** andreas_s has joined #openstack-infra22:37
fungijeblair: was 511073 the one you had conflict resolution issues with? the diff for it is... nigh unreadable22:40
fungias best i can tell it's shuffling a lot of class definitions around22:41
fungii'm going to have to trust that the tests prove all the bits are still there22:42
*** Swami has quit IRC22:45
*** andreas_s has quit IRC22:45
*** baoli has quit IRC22:45
*** andreas_s has joined #openstack-infra22:46
*** rbrndt has quit IRC22:49
jeblairfungi: yeah.. it's *basically* just moving the Job class before the Executor class so that the executor can reference it in a class var22:49
fungiahh, the gerrit webui does a little better job at showing that22:49
jeblairfungi: tbh, i could probably split that into two changes if you want -- one that only reorders the classes, then one that makes the actual changes (there aren't many)22:49
fungibut having to keyword search for statsd to find what got added22:49
fungithat would make it easier to see what's going on (i hate to be _that_ reviewer, but...)22:50
openstackgerritAnup Navare proposed openstack-infra/devstack-gate master: [Test] DNM Checking if tinyIPA builds with py3  https://review.openstack.org/50964122:50
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: WIP: experiment with late-binding inheritance  https://review.openstack.org/51135222:51
*** slaweq has quit IRC22:52
*** Apoorva has quit IRC22:53
*** Apoorva has joined #openstack-infra22:53
*** andreas_s has quit IRC22:54
*** andreas_s has joined #openstack-infra22:55
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Emit some stats from executor  https://review.openstack.org/51107322:57
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Emit some nodepool stats  https://review.openstack.org/51108522:57
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Re-order executor/job classes  https://review.openstack.org/51197722:57
fungijeblair: thanks!22:57
jeblairfungi, mordred: ^ there we go22:57
jeblairwe'll see if i extracted that correctly :)22:57
mordredjeblair: oh - that's much better - I stand by my +2 - but I'm glad I was right from eyeballing22:58
*** slaweq has joined #openstack-infra23:00
*** baoli has joined #openstack-infra23:00
openstackgerritMonty Taylor proposed openstack-infra/shade master: Add devstack jobs for zuul v3  https://review.openstack.org/50036523:02
openstackgerritMonty Taylor proposed openstack-infra/shade master: Add devstack jobs for zuul v3  https://review.openstack.org/50036523:03
mordredok - now that shade's gate jobs are passing again normally - let's see how those fancy new v3 devstack jobs do23:03
jeblairoh where's that move devstack to devstack change?23:03
mordredoh - good question23:04
mordredjeblair: https://review.openstack.org/#/c/508906/23:04
mordredjeblair: I haven't updated it in a while - might need to re-sync with devstack-gate repo ... should we go ahead and delete from d-g and add to devstack real quick before the rollout?23:05
*** priteau has quit IRC23:05
jeblairmordred: ++23:06
openstackgerritLin Yang proposed openstack-infra/project-config master: Add OpenStack client check to python-rsdclient  https://review.openstack.org/51165023:06
jeblairmordred: you want to prep the remove patch real quick?23:07
fungijeblair: you've got 2x +2 on the statsd stack though i left comments/questions on a couple23:07
*** xarses has quit IRC23:08
mordredjeblair: I need to afk real quick - will be back in an hour and will do so then23:09
jeblairmordred: k23:09
SamYapleWhat is the release schedule for bindep? i realize it is a non-standard release cycle23:10
SamYapleis it a "whnever we feel like it" deal?23:10
*** wolverineav has joined #openstack-infra23:12
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Make statsd import failure raise exception  https://review.openstack.org/51197823:12
jeblairfungi: thanks, responded!23:13
fungiSamYaple: it has a release frequency proportional to the frequency with which people bug us about it, though i'm unsure of the scalar constant involved23:13
*** andreas_s has quit IRC23:13
jeblairi have a feeling we're about to collect more experimental data on that contast23:13
jeblairconstant23:13
fungiindeed!23:13
SamYapleyouve always been good at predictions jeblair :)23:15
SamYapleas long as there is a tagged release from now to anytime before queens I will be happy23:16
*** wolverineav has quit IRC23:18
*** priteau has joined #openstack-infra23:18
*** eumel8 has quit IRC23:20
*** esberglu has joined #openstack-infra23:20
*** priteau has quit IRC23:21
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Switch statsd config to zuul.conf  https://review.openstack.org/51101723:22
*** andreas_s has joined #openstack-infra23:22
*** smatzek has joined #openstack-infra23:23
*** ilpianista has quit IRC23:23
*** fried_rice is now known as efried23:23
*** shu-mutou-AWAY has quit IRC23:23
*** esberglu has quit IRC23:25
*** smatzek has quit IRC23:26
*** ilpianista has joined #openstack-infra23:27
*** claudiub has quit IRC23:27
*** gouthamr has joined #openstack-infra23:29
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Re-order executor/job classes  https://review.openstack.org/51197723:31
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Emit some stats from executor  https://review.openstack.org/51107323:31
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Emit some nodepool stats  https://review.openstack.org/51108523:31
*** andreas_s has quit IRC23:31
*** slaweq has quit IRC23:33
*** hongbin has quit IRC23:35
*** andreas_s has joined #openstack-infra23:36
*** kzaitsev_pi has quit IRC23:36
*** zaneb has joined #openstack-infra23:36
*** bobh has joined #openstack-infra23:37
*** kzaitsev_pi has joined #openstack-infra23:38
*** gouthamr_ has joined #openstack-infra23:43
*** slaweq has joined #openstack-infra23:43
*** gouthamr has quit IRC23:46
*** Apoorva has quit IRC23:46
*** superdan is now known as dansmith23:47
pabelangernice^23:48
*** markvoelker has quit IRC23:49
*** Goneri has quit IRC23:54
*** andreas_s has quit IRC23:54
jeblairpower outages here23:55
*** gouthamr_ has quit IRC23:57
*** andreas_s has joined #openstack-infra23:59
*** smatzek has joined #openstack-infra23:59
openstackgerritPaul Belanger proposed openstack-infra/system-config master: Remove zuulv3-dev.o.o  https://review.openstack.org/51198623:59
clarkbjeblair: because fire?23:59

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