Thursday, 2018-09-06

fungiclarkb: sure thing00:03
*** sthussey has quit IRC00:04
*** rlandy is now known as rlandy|bbl00:04
*** threestrands has joined #openstack-infra00:05
*** threestrands has joined #openstack-infra00:05
*** tosky has quit IRC00:07
clarkbthe downside to using the env var for ansible forks value is now I can't jus ask ps for the info on whether or not we are using default or 50 :)00:11
clarkbI don't think it is used on the current run00:11
*** owalsh has joined #openstack-infra00:11
*** longkb has joined #openstack-infra00:12
ianwif you caught ps -aef | grep ssh | wc -l at > 20 that would indicate it?00:13
*** owalsh- has quit IRC00:15
clarkbianw: when it was on the command line -f included it, but now that it is an env var I think you hvae to check /proc/$pid/environ00:15
clarkbI don't think ps will dump that but maybe I am wrong00:15
ianwoh i just meant if you saw more than 20 ssh processes, that would indicate the limit was > 20 and ergo @ 5000:16
clarkboh00:17
clarkblooks like there are plenty, I think due to control persist? not sure that works00:19
clarkbat least cross checking against /proc doesn't seem to be set to 50 yet00:19
*** dchen has joined #openstack-infra00:20
clarkbnodepool fix should be just a few minutes away now00:25
*** owalsh_ has joined #openstack-infra00:25
*** owalsh has quit IRC00:28
openstackgerritGhanshyam Mann proposed openstack-infra/devstack-gate master: Update grenade settings for stable/rocky  https://review.openstack.org/59159400:29
*** spartakos has quit IRC00:30
*** jamesmcarthur has joined #openstack-infra00:33
*** ijw has quit IRC00:34
*** ijw has joined #openstack-infra00:34
*** ijw has quit IRC00:35
*** ijw has joined #openstack-infra00:35
*** owalsh has joined #openstack-infra00:36
clarkbremote puppet else is running now, when that is done we should pick up both the forks change and the nodepool fix in the next run (assuming nodepool chnage merges in the next few minutes)00:36
*** owalsh_ has quit IRC00:40
*** owalsh_ has joined #openstack-infra00:40
*** owalsh- has joined #openstack-infra00:41
*** owalsh has quit IRC00:43
openstackgerritMerged openstack-infra/system-config master: Add a run-nodepool job  https://review.openstack.org/59840300:44
*** owalsh_ has quit IRC00:45
*** owalsh has joined #openstack-infra00:47
*** owalsh_ has joined #openstack-infra00:50
*** rcernin has quit IRC00:50
*** owalsh- has quit IRC00:51
*** rcernin has joined #openstack-infra00:51
*** owalsh has quit IRC00:52
*** gyee has quit IRC00:53
clarkbnext ansible run in ~6 minutes00:54
*** owalsh has joined #openstack-infra00:55
*** yamamoto has joined #openstack-infra00:55
*** owalsh_ has quit IRC00:56
*** agopi|brb is now known as agopi00:56
*** owalsh_ has joined #openstack-infra00:57
*** threestrands has quit IRC00:57
*** threestrands has joined #openstack-infra00:58
*** threestrands has joined #openstack-infra00:58
*** owalsh- has joined #openstack-infra01:00
*** owalsh has quit IRC01:01
*** owalsh has joined #openstack-infra01:02
openstackgerritKaifeng Wang proposed openstack-infra/project-config master: Add api-ref job to ironic-inspector  https://review.openstack.org/59989001:03
*** owalsh_ has quit IRC01:03
*** owalsh_ has joined #openstack-infra01:04
*** owalsh- has quit IRC01:05
*** owalsh- has joined #openstack-infra01:07
*** tonyb has quit IRC01:07
*** owalsh has quit IRC01:07
*** tonyb has joined #openstack-infra01:09
*** owalsh_ has quit IRC01:10
*** owalsh has joined #openstack-infra01:10
*** slaweq has joined #openstack-infra01:11
*** owalsh_ has joined #openstack-infra01:12
*** owalsh- has quit IRC01:13
*** owalsh- has joined #openstack-infra01:15
*** slaweq has quit IRC01:16
*** owalsh has quit IRC01:16
*** owalsh has joined #openstack-infra01:17
*** owalsh_ has quit IRC01:18
*** fnordahl has quit IRC01:19
*** fnordahl has joined #openstack-infra01:20
*** timothyb89 has joined #openstack-infra01:20
*** owalsh_ has joined #openstack-infra01:21
*** tonyb_ has joined #openstack-infra01:21
*** eernst has quit IRC01:21
*** owalsh- has quit IRC01:21
*** owalsh has quit IRC01:24
*** owalsh has joined #openstack-infra01:25
*** eernst has joined #openstack-infra01:26
*** tonyb_ has quit IRC01:27
*** tonyb has quit IRC01:27
*** owalsh- has joined #openstack-infra01:27
*** eernst has quit IRC01:27
*** eernst has joined #openstack-infra01:28
*** owalsh_ has quit IRC01:28
beisnerhi all - seems like the post jobs are forever-stuck. is there a known issue?01:30
*** owalsh_ has joined #openstack-infra01:30
*** owalsh has quit IRC01:30
clarkbbeisner: post jobs have the lowest priority behindgate and check01:30
clarkbbeisner: this means that with the check and gate queues being busy they are getting all of the nodes starving post. this is normal01:31
beisnerhi clarkb - ah ok, thx01:31
*** tonyb has joined #openstack-infra01:31
*** eernst has quit IRC01:32
*** owalsh- has quit IRC01:34
*** owalsh has joined #openstack-infra01:39
*** owalsh_ has quit IRC01:40
*** aeng has quit IRC01:42
*** lbragstad has quit IRC01:43
*** jamesmcarthur has quit IRC01:43
*** lbragstad has joined #openstack-infra01:45
*** aeng has joined #openstack-infra01:45
*** jamesmcarthur has joined #openstack-infra01:45
*** annp has joined #openstack-infra01:48
*** hongbin has joined #openstack-infra01:51
*** rlandy|bbl is now known as rlandy01:54
*** armax has quit IRC01:54
*** camunoz has quit IRC01:55
*** jamesmcarthur has quit IRC01:56
*** owalsh_ has joined #openstack-infra01:56
*** aeng has quit IRC01:59
*** owalsh has quit IRC01:59
*** lbragstad has quit IRC02:09
*** spartakos has joined #openstack-infra02:10
*** slaweq has joined #openstack-infra02:11
openstackgerritMerged openstack-infra/zuul-jobs master: Add guidelines on variable naming  https://review.openstack.org/59592002:12
*** slaweq has quit IRC02:15
*** nicolasbock has quit IRC02:18
*** anteaya has quit IRC02:18
*** anteaya has joined #openstack-infra02:18
*** armax has joined #openstack-infra02:30
*** ijw has quit IRC02:34
*** diablo_rojo has quit IRC02:35
*** ijw_ has joined #openstack-infra02:37
*** ijw_ has quit IRC02:41
*** psachin has joined #openstack-infra02:45
*** ijw has joined #openstack-infra03:05
*** ijw has quit IRC03:10
*** slaweq has joined #openstack-infra03:11
*** rlandy has quit IRC03:13
*** gcb_ has quit IRC03:14
*** slaweq has quit IRC03:15
*** anteaya has quit IRC03:16
*** gcb_ has joined #openstack-infra03:17
*** ramishra has joined #openstack-infra03:19
*** ijw has joined #openstack-infra03:26
*** ijw has quit IRC03:31
*** spartakos has quit IRC03:35
*** ramishra has quit IRC03:38
*** ramishra has joined #openstack-infra03:39
*** eernst has joined #openstack-infra03:46
*** jamesmcarthur has joined #openstack-infra03:56
*** hongbin has quit IRC04:01
ianwcorvus / clarkb : https://github.com/ansible/ansible/pull/45265 ... fixes a bunch of profile issues and adds option to just print profile ... off the to great pull request dumping ground in the sky.  see if anyone bites04:01
*** yamamoto has quit IRC04:05
*** yamamoto has joined #openstack-infra04:05
*** markvoelker has quit IRC04:06
*** markvoelker has joined #openstack-infra04:06
*** threestrands has quit IRC04:10
*** markvoelker has quit IRC04:11
*** janki has joined #openstack-infra04:13
*** yamamoto has quit IRC04:16
*** gcb_ has quit IRC04:17
*** threestrands has joined #openstack-infra04:21
*** threestrands has joined #openstack-infra04:21
mordredianw: that pr adds 'import logging' but doesn't seem to use it04:26
mordredotherwise lgtm04:27
ianwoh, yep, sorry, i found the internal logging, let me repush04:27
ianwmordred: if you want a fun one, that i think we'll be pretty unique in triggering -> https://github.com/ansible/ansible/pull/4509904:31
*** ijw has joined #openstack-infra04:33
*** yboaron_ has joined #openstack-infra04:34
*** dave-mccowan has quit IRC04:36
*** eernst has quit IRC04:36
*** ijw has quit IRC04:37
*** longkb has quit IRC04:40
*** longkb has joined #openstack-infra04:40
*** stakeda has joined #openstack-infra04:42
*** Bhujay has joined #openstack-infra04:45
*** gcb_ has joined #openstack-infra04:53
*** kjackal has joined #openstack-infra05:04
*** markvoelker has joined #openstack-infra05:07
*** ijw has joined #openstack-infra05:11
*** slaweq has joined #openstack-infra05:11
openstackgerritIan Wienand proposed openstack-infra/system-config master: [wip] Separate out ansible logs in puppet apply-test  https://review.openstack.org/60033105:14
*** ijw has quit IRC05:15
*** slaweq has quit IRC05:15
openstackgerritSimon Westphahl proposed openstack-infra/nodepool master: Implement liveness probe for static driver  https://review.openstack.org/60008305:18
openstackgerritSimon Westphahl proposed openstack-infra/nodepool master: Cleanup of leaked resource for static driver  https://review.openstack.org/60008405:18
*** r-mibu has joined #openstack-infra05:28
*** ijw has joined #openstack-infra05:31
*** noama has joined #openstack-infra05:32
*** links has joined #openstack-infra05:33
*** hashar has joined #openstack-infra05:35
openstackgerritMerged openstack-infra/zuul-jobs master: Remove trigger-readthedocs job  https://review.openstack.org/58383605:38
*** graphene has joined #openstack-infra05:38
*** udesale has joined #openstack-infra05:38
*** ijw has quit IRC05:39
openstackgerritIan Wienand proposed openstack-infra/system-config master: [wip] Separate out ansible logs in puppet apply-test  https://review.openstack.org/60033105:40
AJaegerconfig-core, please review https://review.openstack.org/598383 and https://review.openstack.org/581000  https://review.openstack.org/#/c/600240/05:44
*** quiquell has joined #openstack-infra05:46
*** d0ugal_ has quit IRC05:49
*** d0ugal_ has joined #openstack-infra05:53
*** gcb_ has quit IRC05:53
*** links has quit IRC05:57
*** links has joined #openstack-infra05:57
*** quiquell is now known as quiquell|rover05:58
*** chandankumar is now known as chkumar|ruck05:58
openstackgerritMerged openstack-infra/zuul master: Remove restriction on add_host  https://review.openstack.org/60020505:59
*** d0ugal_ has quit IRC06:05
*** Bhujay has quit IRC06:07
openstackgerritIan Wienand proposed openstack-infra/system-config master: [wip] Separate out ansible logs in puppet apply-test  https://review.openstack.org/60033106:16
*** gcb_ has joined #openstack-infra06:17
*** Bhujay has joined #openstack-infra06:18
openstackgerritMerged openstack-infra/project-config master: Change neutron grafana dashboard to 2-column format  https://review.openstack.org/60024006:22
*** dchen has quit IRC06:24
*** seongsoocho has joined #openstack-infra06:25
*** pcaruana has joined #openstack-infra06:25
*** udesale_ has joined #openstack-infra06:27
*** udesale_ has quit IRC06:27
*** udesale_ has joined #openstack-infra06:29
*** udesale has quit IRC06:29
*** jtomasek has joined #openstack-infra06:32
*** d0ugal_ has joined #openstack-infra06:32
*** udesale_ has quit IRC06:39
*** aojea has joined #openstack-infra06:40
*** udesale has joined #openstack-infra06:43
AJaegerinfra-root, I had usermode +R - only registred users can talk privately to me and removed that and got spammed privately this time ;( So, spam is still around - but might be different modus operandi06:44
*** markvoelker has quit IRC06:45
*** apetrich has quit IRC06:48
*** caphrim007_ has quit IRC06:50
*** slaweq has joined #openstack-infra06:50
*** caphrim007 has joined #openstack-infra06:50
*** alexchadin has joined #openstack-infra06:54
*** e0ne has joined #openstack-infra06:58
*** rcernin has quit IRC07:00
*** Miouge is now known as miouge07:00
*** graphene has quit IRC07:02
*** tmorin has joined #openstack-infra07:03
*** graphene has joined #openstack-infra07:04
*** shardy has joined #openstack-infra07:04
*** sshnaidm|off is now known as sshnaidm07:06
*** quiquell|rover has quit IRC07:06
*** Emine has joined #openstack-infra07:06
*** quiquell has joined #openstack-infra07:07
*** pguimaraes has quit IRC07:07
*** d0ugal_ has quit IRC07:07
*** priteau has joined #openstack-infra07:09
*** quiquell|rover has joined #openstack-infra07:09
*** quiquell|rover has joined #openstack-infra07:10
*** alexchadin has quit IRC07:10
*** quiquell|rover has quit IRC07:11
*** quiquell|rover has joined #openstack-infra07:12
*** quiquell has joined #openstack-infra07:12
*** quiquell has joined #openstack-infra07:13
*** quiquell has quit IRC07:13
*** ccamacho has joined #openstack-infra07:14
*** quiquell has joined #openstack-infra07:14
*** quiquell has quit IRC07:14
*** quiquell has joined #openstack-infra07:15
*** masber has joined #openstack-infra07:16
*** quiquell is now known as quiquell|rover07:16
*** masber has quit IRC07:16
*** masber has joined #openstack-infra07:17
*** zhangfei has joined #openstack-infra07:18
*** tesseract has joined #openstack-infra07:21
dirkinfra-root: anyone around who can tell me what/how/why the opensuse-mirror-update.sh is failing?07:23
dirkhttp://mirror.gra1.ovh.openstack.org/opensuse/repositories/Cloud:/OpenStack:/Master/openSUSE_Leap_42.3/repodata/ is about a week old, so it doesn't sync from the master (which is more current)07:23
*** pguimaraes has joined #openstack-infra07:24
*** alexchadin has joined #openstack-infra07:27
*** udesale has quit IRC07:27
evrardjpdirk: what do you call master? Is that the repo infra is mirroring from, or the original repos of opensuse.org?07:27
*** udesale has joined #openstack-infra07:27
*** ssbarnea|bkp2 has quit IRC07:27
evrardjpdirk: let me rephrase, is kernel.org/gwdg.de up to date?07:28
hwoarangyes07:28
evrardjpok so that last change didn't hurt.07:29
*** d0ugal_ has joined #openstack-infra07:34
AJaegerinfra-root, http://logs.openstack.org/51/576351/3/gate/openstack-infra-base-integration-ubuntu-bionic/9734484/job-output.txt.gz#_2018-09-06_06_41_34_575058 has "Failed to fetch http://mirror.mtl01.inap.openstack.org/ ...." - any problems with that mirror?07:35
evrardjphwoarang: I am concerned that mirror.pypi releasetimes is now 2 months ago07:35
*** pcaruana has quit IRC07:36
hwoarangevrardjp: i don't understand :)07:36
evrardjpI see on grafana.openstack.org that the latest mirror.debian/ubuntu was done an hour ago, mirror.opensuse 11 days ago, mirror.pypi 2 months ago07:37
evrardjpexcept if I can't read the dashboard properly, which is possible too07:38
*** ssbarnea|bkp has joined #openstack-infra07:40
jheskethdirk: rsync: failed to connect to rsync.opensuse.org (62.146.92.202): Connection timed out (110). I'm also struggling to connect to rsync.opensuse.org from my laptop, so I'm not sure if there's something wrong there?07:41
*** tosky has joined #openstack-infra07:41
ianwevrardjp: we've stopped using that mirror and now just use a reverse proxy to upstream pypi07:42
ianwit started to exceed the 2tb limit of the underlying AFS volume07:42
ianwand we had no practical way to prune it07:42
dirkevrardjp: gwdg.de is up to date07:42
dirkjhesketh: but thats the openSUSE Tubmbleweed rsync.. the rsync for repositories should happen before07:42
dirkjhesketh: rsync.opensuse.org has a hardware defect, so its down atm07:43
*** markvoelker has joined #openstack-infra07:43
dirkit is known to be down atm07:43
dirkjhesketh: the /repositories (e.g. OBS_MIRROR in opensuse-mirror-update.sh ) runs before that code path though..07:43
dirkjhesketh: or does it only release the new stuff when *all* rsyncs finished successfully?07:43
evrardjpianw: oh that's the reason. Maybe it should be removed from the AFS dashboard?07:43
ianwdidn't we switch mirror.opensuse to kernel.org?  or back from it or something ... istr we've had trouble finding a stable upstream07:43
*** holser_ has joined #openstack-infra07:44
evrardjpianw: yes, but that's for a part of those, the OBS hasn't changed07:44
ianwevrardjp: we should clean up the volume and do that.  i think we were in a bit of a fire-drill moving away from it, but things seem ok with the reverse proxy07:44
evrardjpso we are wondering how come :)07:44
jheskethdirk: ah, right.. it looks like the repositories have sync'd to the update-mirror.. have you checked other region mirrors?07:45
dirkjhesketh: no, I don't know how to look them up07:45
dirkovh is the only one I know and its out dated07:45
openstackgerritDirk Mueller proposed openstack-infra/system-config master: Skip occassional issues with mirroring openSUSE:Factory:Update  https://review.openstack.org/60036007:45
evrardjpianw: so all references to bandersnatch will also get removed in the future?07:46
jheskethdirk: looks like others are equally out of date http://mirror.ord.rax.openstack.org/opensuse/repositories/Cloud:/OpenStack:/Master/openSUSE_Leap_42.3/repodata/07:46
evrardjpjhesketh: yeah on AFS it looks like mirror is 11 days out, but I am not sure how to read that07:46
*** jpich has joined #openstack-infra07:47
evrardjps/days out/days old/07:47
*** pcaruana has joined #openstack-infra07:48
*** jpena|off is now known as jpena07:49
jheskethunfortuantely I was just on my way out for the day so I can't look any further right now. I should be back in a few hours though07:49
jheskethmaybe ianw or somebody else can take a quick look if not07:49
evrardjpyeah it must be late jhesketh ! we'll deal with that with the people available, do not worry ;)07:51
*** priteau has quit IRC07:52
ianwthe date on that page is the last time the volume released, and the volume releases when the script works07:56
ianw+ k5start -t -f /etc/opensuse.keytab service/opensuse-mirror -- timeout -k 2m 30m rsync -rlptDvz --delete --stats --delete-excluded --exclude=i586 rsync://rsync.opensuse.org/buildservice-repos-main/openSUSE:/Factory:/Update/standard/ /afs/.openstack.org/mirror/opensuse/update/tumbleweed07:57
ianwrsync: failed to connect to rsync.opensuse.org (62.146.92.202): Connection timed out (110)07:57
openstackgerritDirk Mueller proposed openstack-infra/system-config master: Skip occassional issues with mirroring openSUSE:Factory:Update  https://review.openstack.org/60036007:57
ianwthat's the failure (which i guess we already knew?)07:57
openstackgerritJean-Philippe Evrard proposed openstack-infra/project-config master: Remove references to pypi mirroring from dashboard  https://review.openstack.org/60036407:57
dirkianw: see the patch above that I just proposed. that would keep the script going to the release07:57
evrardjpthanks dirk07:58
hwoarangsneaky problem :)08:02
hwoarangthanks dirk08:02
evrardjpianw: I posted a patch about the grafana afs dashboard, but I can help further if required. https://review.openstack.org/#/c/600364/108:02
evrardjphwoarang: sneaky is the right term indeed08:02
evrardjpdirk: on our side, when will that hardware be fixed? :D08:03
*** gcb_ has quit IRC08:10
*** david-lyle has quit IRC08:14
*** dklyle has joined #openstack-infra08:15
*** kaisers has quit IRC08:19
*** longkb has quit IRC08:19
*** longkb has joined #openstack-infra08:21
*** olivierb has joined #openstack-infra08:22
*** kaisers has joined #openstack-infra08:25
*** dtantsur|afk is now known as dtantsur08:25
*** sambetts|afk has joined #openstack-infra08:27
dirkianw: do you have suggestions on how to detect "systemic failure" ?08:29
dirkianw: for FActory:Update I really don't care actually..08:29
dirkwant me to try to come up with a code soluton or just change the sloppy comment in the commit message?08:30
*** sshnaidm is now known as sshnaidm|afk08:31
hwoarangi guess for systemic failure we always get that info straight from opensuse so we know when that happens08:36
*** udesale has quit IRC08:47
*** udesale has joined #openstack-infra08:47
*** udesale has quit IRC08:48
*** udesale has joined #openstack-infra08:48
*** electrofelix has joined #openstack-infra08:48
*** d0ugal_ has quit IRC08:49
evrardjphwoarang: wouldn't that be the 'occasional failure' ?08:49
evrardjpenglish is hard :p08:49
*** d0ugal has joined #openstack-infra08:49
evrardjpanyway ... I am not sure this matters right now?08:50
hwoarangoh i read 'systematic'08:51
cgoncalvesAJaeger, dhellmann: hi. considering py3 goal, would it make sense to rename jobs from *py35* to just *py3*? octavia is an example where we have py35 in job names but we don't force v3.5 explicitly08:56
*** alexchadin has quit IRC08:58
ianwhwoarang / evrardjp : i don't think i care that much :)  i mean how do you tell the difference between an occasional failure and "repo moved and is never coming back".  i don't think you can, really09:04
evrardjpianw: that's how I understood it -- and yes, with this code we can't, and I think whatever code we will produce will not replace a good ol' human communication... "this has failed for x days, because of y"09:05
evrardjpcgoncalves: curious about how you'll later see the difference with py36... but wasn't there a thread about it?09:06
*** gfidente has joined #openstack-infra09:07
*** hwoarang_ has joined #openstack-infra09:11
cgoncalvesevrardjp, I'd say that if the job doesn't explicitely set a python 3 version that it should be named just 'py3'. if 3.6 is set then 'py36'09:12
*** udesale_ has joined #openstack-infra09:13
*** alexchadin has joined #openstack-infra09:14
*** hwoarang has quit IRC09:15
openstackgerritIan Wienand proposed openstack-infra/system-config master: [wip] Separate out ansible logs in puppet apply-test  https://review.openstack.org/60033109:15
*** udesale has quit IRC09:16
*** sshnaidm|afk is now known as sshnaidm09:16
*** threestrands has quit IRC09:16
evrardjpcgoncalves: on yeah I didn't mean mutually exclusive things -- I just wanted to raise the fact that there was maybe a discussion about those :)09:20
cmurphyzuul has quite the backlog today09:20
*** cshastri has joined #openstack-infra09:21
*** derekh has joined #openstack-infra09:21
ianwcmurphy: yeah, it's been busy all .au day too09:21
*** hwoarang_ is now known as hwoarang09:23
*** alexchadin has quit IRC09:24
cgoncalvesevrardjp, ah, got it! thanks for letting me know. I will try to look for that email thread09:27
*** mgoddard has joined #openstack-infra09:27
mgoddardhi, seeing connection failures to http://mirror.mtl01.inap.openstack.org/pypi/simple in some jobs. Is it a transient issue or should I hold off rechecking?09:28
*** pblaho has joined #openstack-infra09:29
evrardjpcgoncalves: hoping my memory doesn't play me tricks though :D09:29
ianwmgoddard: hrm, it doesn't seem to be responding09:33
*** markmcd has joined #openstack-infra09:34
ianwmgoddard: interesting, the entire server appears to have disappeared09:36
openstackgerritMerged openstack/ptgbot master: Add channel feedback on 'book' command  https://review.openstack.org/60010009:37
openstackgerritIan Wienand proposed openstack-infra/project-config master: Revert "Revert "Disable inap-mtl01 provider""  https://review.openstack.org/60039509:40
*** dabukalam has joined #openstack-infra09:41
ianwproject-config : ^ that hurts with current backup, but i don't have time to rebuild the mirror right now09:41
*** panda has quit IRC09:48
mgoddardthanks for checking ianw09:49
ianwmgagne: ^^ unless you have any other thoughts?09:49
*** d0ugal has quit IRC09:51
openstackgerritMark Goddard proposed openstack-infra/project-config master: Add new project config for tenks  https://review.openstack.org/60039709:51
*** udesale_ has quit IRC09:52
*** udesale has joined #openstack-infra09:53
*** hashar is now known as hasharAway09:53
*** aojea has quit IRC09:54
*** quiquell|rover is now known as quiquell|rover|a09:57
*** quiquell|rover|a is now known as quique|rover|afk09:57
*** xinliang has quit IRC09:58
*** xinliang has joined #openstack-infra09:58
*** annp has quit IRC10:00
*** d0ugal has joined #openstack-infra10:01
*** xinliang has quit IRC10:03
*** xinliang has joined #openstack-infra10:03
*** xinliang has quit IRC10:05
*** wolverineav has joined #openstack-infra10:05
*** xinliang has joined #openstack-infra10:05
*** xinliang has joined #openstack-infra10:05
*** xinliang has left #openstack-infra10:07
*** xinliang has joined #openstack-infra10:08
*** xinliang has joined #openstack-infra10:08
*** xinliang has quit IRC10:12
*** d0ugal has quit IRC10:14
*** xinliang has joined #openstack-infra10:14
*** xinliang has joined #openstack-infra10:14
*** e0ne has quit IRC10:15
*** xinliang has joined #openstack-infra10:15
*** xinliang has joined #openstack-infra10:15
openstackgerritMerged openstack-infra/project-config master: Revert "Revert "Disable inap-mtl01 provider""  https://review.openstack.org/60039510:17
*** owalsh_ is now known as owalsh10:19
*** quique|rover|afk is now known as quiquell|rover10:19
*** d0ugal has joined #openstack-infra10:22
*** graphene has quit IRC10:26
*** graphene has joined #openstack-infra10:28
*** nicolasbock has joined #openstack-infra10:31
*** stakeda has quit IRC10:31
*** e0ne has joined #openstack-infra10:39
*** dave-mccowan has joined #openstack-infra10:46
*** wolverineav has quit IRC10:54
*** pbourke has quit IRC10:59
*** pbourke has joined #openstack-infra11:00
*** jpena is now known as jpena|lunch11:02
*** dhajare has joined #openstack-infra11:03
*** ccamacho has quit IRC11:07
*** kjackal_ has joined #openstack-infra11:12
openstackgerritMerged openstack/diskimage-builder master: Fix bootloader packages for aarch64  https://review.openstack.org/59859511:17
* jhesketh is back for a little bit11:21
jhesketh+w on the skipping factory update issues, hopefully that fixes the rest of the repo being out of date11:21
openstackgerritMerged openstack/diskimage-builder master: Replace assertRaisesRegexp with assertRaisesRegex  https://review.openstack.org/59677411:22
*** Emine has quit IRC11:23
*** mgoddard has left #openstack-infra11:24
*** tmorin has left #openstack-infra11:24
openstackgerritMerged openstack/diskimage-builder master: Fix CentOS image build failure when dib runs on system where audit disabled  https://review.openstack.org/55948511:25
openstackgerritMerged openstack/diskimage-builder master: Remove legacy-opensuse-423 nodeset  https://review.openstack.org/59178811:25
*** Emine has joined #openstack-infra11:25
*** aojea has joined #openstack-infra11:26
*** longkb has quit IRC11:28
*** zhangfei has quit IRC11:29
openstackgerritMerged openstack/diskimage-builder master: Remove unsued opensuse jobs  https://review.openstack.org/59179711:29
openstackgerritMark Goddard proposed openstack-infra/project-config master: Add new project config for tenks  https://review.openstack.org/60039711:34
*** rh-jelabarre has joined #openstack-infra11:38
*** eharney has quit IRC11:39
AJaegerconfig-core, senlin is ready for python3-first - please review https://review.openstack.org/#/c/597502/11:41
*** hasharAway is now known as hashar11:49
*** psachin` has joined #openstack-infra11:50
*** psachin has quit IRC11:51
*** rfolco has joined #openstack-infra11:52
openstackgerritAndreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Remove unused job legacy-grenade-dsvm-sahara  https://review.openstack.org/60042011:57
*** jpena|lunch is now known as jpena11:59
*** dhajare has quit IRC12:00
*** trown|outtypewww is now known as trown12:00
openstackgerritAndreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Remove unused job legacy-grenade-dsvm-sahara  https://review.openstack.org/60042012:04
*** alexchadin has joined #openstack-infra12:11
*** boden has joined #openstack-infra12:12
*** bharath_t has joined #openstack-infra12:12
*** rh-jelabarre has quit IRC12:14
openstackgerritMerged openstack-infra/system-config master: Skip occassional issues with mirroring openSUSE:Factory:Update  https://review.openstack.org/60036012:18
*** links has quit IRC12:22
*** markvoelker has quit IRC12:22
*** markvoelker has joined #openstack-infra12:23
*** links has joined #openstack-infra12:28
*** yboaron_ has quit IRC12:32
*** yboaron_ has joined #openstack-infra12:32
*** ccamacho has joined #openstack-infra12:38
*** ccamacho has quit IRC12:38
*** rlandy has joined #openstack-infra12:38
openstackgerritNguyen Hai proposed openstack-infra/project-config master: remove job settings for senlin repositories  https://review.openstack.org/59750212:38
*** ccamacho has joined #openstack-infra12:38
*** eharney has joined #openstack-infra12:38
*** alexchadin has quit IRC12:41
*** e0ne has quit IRC12:41
*** ansmith has joined #openstack-infra12:43
*** kgiusti has joined #openstack-infra12:46
*** dhajare has joined #openstack-infra12:47
*** r-mibu has quit IRC12:47
*** sthussey has joined #openstack-infra12:49
*** alexchadin has joined #openstack-infra12:49
*** jamesmcarthur has quit IRC12:52
*** jamesmcarthur has joined #openstack-infra12:52
*** lbragstad has joined #openstack-infra12:54
*** e0ne has joined #openstack-infra12:59
*** camunoz has joined #openstack-infra13:01
*** jamesmcarthur has quit IRC13:03
*** bobh has joined #openstack-infra13:05
*** links has quit IRC13:08
*** mriedem has joined #openstack-infra13:08
openstackgerritTobias Henkel proposed openstack-infra/zuul master: WIP: Match tag items against containing branches  https://review.openstack.org/57855713:08
*** lbragstad has quit IRC13:09
*** bobh has quit IRC13:09
*** auristor has quit IRC13:12
*** links has joined #openstack-infra13:13
*** yboaron_ has quit IRC13:14
*** links has quit IRC13:14
*** lbragstad has joined #openstack-infra13:15
*** links has joined #openstack-infra13:15
*** caphrim007 has quit IRC13:20
fungidirk: evrardjp: hwoarang: ianw: we'd need to put those in separate afs volumes if we want to release updates for different suse mirrors independently13:21
efriedGood UGT morning. Can anyone (AJaeger) tell me how to add zuul doc jobs? My first two attempts in https://review.openstack.org/#/c/600268/ were merely guesses, clearly wrong.13:25
fungimgagne: huh, our inap-17301 project in mtl01 is indicating there are no servers. we should have a mirror instance there13:28
fungiany idea what's happened?13:28
fungiahh, i see ianw has already turned us back off there13:29
*** auristor has joined #openstack-infra13:32
*** kjackal has quit IRC13:33
*** kjackal_ has quit IRC13:34
*** bobh has joined #openstack-infra13:39
*** kjackal has joined #openstack-infra13:40
*** kjackal_ has joined #openstack-infra13:41
openstackgerritFabien Boucher proposed openstack-infra/zuul-jobs master: ensure-sphinx: do not attempt a gettext install if exists  https://review.openstack.org/59902813:42
*** eharney has quit IRC13:43
*** chkumar|ruck is now known as chkumar|out13:44
*** bobh has quit IRC13:46
*** caphrim007 has joined #openstack-infra13:46
*** dhajare has quit IRC13:47
*** sshnaidm is now known as sshnaidm|mtg13:49
*** caphrim007 has quit IRC13:51
*** rh-jelabarre has joined #openstack-infra13:53
*** anteaya has joined #openstack-infra13:57
*** eharney has joined #openstack-infra13:58
*** psachin` has quit IRC13:59
*** hashar is now known as hasharAway13:59
*** kjackal_ has quit IRC14:02
*** kjackal has quit IRC14:02
*** d0ugal has quit IRC14:05
*** d0ugal has joined #openstack-infra14:09
ssbarnea1corvus: https://review.openstack.org/#/c/570546/ oneliner on devstack. thanks.14:13
*** bobh has joined #openstack-infra14:18
AJaegerefried: reviewed and commented14:19
efriedAJaeger: Thanks!14:19
*** bharath_t has quit IRC14:27
openstackgerritAndreas Jaeger proposed openstack-infra/infra-manual master: Mention that docs-on-readthedocs should stay in project-config  https://review.openstack.org/60045314:28
openstackgerritAndreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Small edits for descriptions  https://review.openstack.org/60045414:33
openstackgerritPaul Belanger proposed openstack-infra/puppet-zuul master: Add docker-py dependency for ansible-playbook  https://review.openstack.org/60045614:34
*** Bhujay has quit IRC14:34
openstackgerritAndreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Use tox-docs for specs repositories  https://review.openstack.org/60045714:35
pabelangerinfra-root: To follow up on topic in #zuul, ^ is needed to allow jobs to use docker modules with zuul-executor. I am not sure how best to manage ansible dependencies in the long run14:35
openstackgerritAndreas Jaeger proposed openstack-infra/project-config master: Use tox-docs for specs publishing  https://review.openstack.org/60045814:38
*** cshastri has quit IRC14:39
AJaegerefried: let me send a followup change to use templates ;)14:40
efriedAJaeger: That would be great, thank you.14:40
*** graphene has quit IRC14:44
AJaegerefried: done...14:44
mgagnefungi: bad news, looks like someone deleted the mirror VM by mistake in inap-mtl01. good news, volumes still exist.14:45
AJaegerefried: don't you want lower-constraints?14:45
*** graphene has joined #openstack-infra14:46
fungimgagne: no worries, we'll get a fresh one built14:46
mgagneclarkb: as for boot timeout, could it be that since we updated (in fact, reinstalled some of) the compute nodes, all images need to be downloaded, cached and used to boot new VMs?14:46
fungimgagne: "accidentally deleted" is way better than "just vanished" at least ;)14:47
clarkbmgagne: possibly14:47
mgagnefungi: someone assumed that "inap-mtl01 is disabled" means there is no VMs whatsoever left.14:47
clarkbfungi mgagne we can delete the volume and start over too if necessary, that is all cache data14:47
mgagneclarkb: whatever works for you, I'm just gonna shame the guy even more =)14:48
clarkbmight be good for someone other than me to go through launvh node process on bridge.o.o to see what I missed in trying to get that working14:49
efriedAJaeger: You mean the gate job? Yes, I would think so.14:49
fungimgagne: sure, that's also one of the reasons we have two separate tenants/projects so that persistent control plane stuff can be isolated to one and the ephemeral stuff which is safe to wipe goes in the other14:49
AJaegerefried: patches sent, I'm good now ;)14:49
efriedAJaeger: Our l-c file is still nova's, but I still don't understand how to trim it tbh because all the deps seem to be transitive.14:49
AJaegerefried: best ask dhellmann14:50
efriedack14:50
*** d0ugal has quit IRC14:50
fungimgagne: also i accidentally delete things with some regularity, don't give him too hard of a time ;)14:50
*** Swami has joined #openstack-infra14:50
mgagnefungi: but it's the only pleasure I can get these times! ;)14:51
openstackgerritMarkus Hosch proposed openstack-infra/zuul master: Add --check-config option to zuul scheduler  https://review.openstack.org/54216014:51
*** d0ugal has joined #openstack-infra14:52
AJaegerefried: I think having the job will make that job easier ;)14:54
AJaegerdhellmann: I think kolla is ready for python3-first, want to remove WIP from https://review.openstack.org/#/c/597590/ ?14:55
AJaegerconfig-core, two cleanups: https://review.openstack.org/#/c/600420/ removes migrated template and senlin is migrated with python3-first https://review.openstack.org/597502 . please review14:58
*** quiquell|rover has quit IRC15:02
*** jamesmcarthur has joined #openstack-infra15:03
*** janki has quit IRC15:04
*** hamzy has quit IRC15:04
*** jamesmcarthur has quit IRC15:08
*** kjackal has joined #openstack-infra15:09
*** kjackal_ has joined #openstack-infra15:09
*** dhajare has joined #openstack-infra15:10
*** gyee has joined #openstack-infra15:11
*** r-daneel has quit IRC15:17
*** sshnaidm|mtg is now known as sshnaidm15:20
*** d0ugal has quit IRC15:21
AJaegerdhellmann: I think ec2-api and kolla are ready for python3-first, want to remove WIP from the project-config changes?15:22
*** spartakos has joined #openstack-infra15:24
*** hamzy has joined #openstack-infra15:27
*** bobh has quit IRC15:29
*** bobh has joined #openstack-infra15:29
*** psachin` has joined #openstack-infra15:29
*** graphene has quit IRC15:30
*** psachin` has quit IRC15:31
*** ssbarnea|bkp has quit IRC15:32
*** graphene has joined #openstack-infra15:32
*** d0ugal has joined #openstack-infra15:34
*** udesale has quit IRC15:35
*** tosky has quit IRC15:36
*** alexchadin has quit IRC15:38
*** ccamacho has quit IRC15:38
*** ccamacho1 has joined #openstack-infra15:38
*** pblaho has quit IRC15:39
*** e0ne has quit IRC15:42
*** pcaruana has quit IRC15:42
*** r-daneel has joined #openstack-infra15:44
corvusinfra-root: it looks like we may have broken iptables on adns1 which is impacting dns for zuul-ci.org15:44
fungicorvus: so were we previously allowing 53/tcp+udp to adns1 from the v4 and v6 addresses of ns1 and ns2?15:45
*** r-daneel has quit IRC15:45
corvusfungi: that's my recollection15:45
*** anteaya has quit IRC15:46
corvusi'm going to try to find the old puppet config15:46
pabelangeryes, that looks right: http://git.openstack.org/cgit/openstack-infra/system-config/tree/manifests/site.pp?id=dde24421d0ce2053d525db601594a9e3dcca9c44#n85815:47
corvuspabelanger: ++15:47
corvusthere's a really interesting thing we're doing there with hostnames, but that's beside the point for now :)15:48
pabelangerso agree, adding missing firewall rules will like be fix15:48
corvusi don't see any rules for adns115:49
corvusi'll work on a patch to add15:49
*** jamesmcarthur has joined #openstack-infra15:49
fungiyeah, i guess it would be in playbooks/group_vars/adns.yaml which doesn't exist?15:49
corvusfungi: yep15:49
fungiand just needs an iptables_extra_allowed_hosts list i guess to get this working15:51
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Allow ns servers to connect to adns  https://review.openstack.org/60046915:51
corvusfungi, clarkb, pabelanger: ^15:51
*** caphrim007 has joined #openstack-infra15:52
clarkbis adns an group already?15:52
clarkbit is15:53
clarkband we don't need udp because the transfers are too big for udp right?15:53
corvusclarkb: yeah, the transfers use tcp and that's all we had open before15:53
fungiit's axfr which will always use tcp15:53
clarkbrelated: is there a way to tell dig to return the actual records and not do validation and fudge the results for you? I guess +trace +additional and going through the entire recursive list is that15:53
fungii was going to ask about udp as well until i looked at the old puppet manifest15:54
fungiclarkb: and +nodnssec potentially15:54
corvusclarkb: well, an individual query will do that.  just "dig +norec ns1.zuul-ci.org @something"15:54
clarkbcorvus: I had done dig NS zuul-ci.org last night and it returned no results15:55
*** vdrok has joined #openstack-infra15:55
clarkbeven though the NS record was there, dig was excluding those that failed dnssec validation apparently15:55
clarkbor maybe that was the server I was digging against?15:55
fungiif you were relying on your recursive resolver it was likely filtering expired signed results15:55
corvusclarkb: right, but that was either to a recursive resolver which was validating, or it was to a server that wasn't authoritative15:56
corvusso i started with dig @a.root-servers.net and worked down from there15:56
corvusapparently there's a +sigchase option that no one has compiled in by default15:56
corvusclarkb: maybe tumbleweed does? :)15:57
*** ramishra has quit IRC15:57
clarkbcorvus: tumbleweed manpage says that feature is deprecated in favor of delv(1)15:57
corvushrm.  debian says i can get delv by installing 'bind9'.  :(15:58
fungiyeah, dig is being pruned back and delv is the new thing15:58
*** Swami has quit IRC15:58
fungidebian provides delv in its dnsutils package15:58
fungiso same package as dig15:59
corvusoh maybe i need to upgrade15:59
openstackgerritPaul Belanger proposed openstack-infra/puppet-zuul master: Add docker dependency for ansible-playbook  https://review.openstack.org/60045615:59
fungii'm on unstable. i haven't checked stable15:59
corvusit looks like we're rather backed up, should i direct-enqueue that?15:59
fungiyes, that seems prudent15:59
corvusdone16:00
*** pcaruana has joined #openstack-infra16:01
fungithanks!16:01
clarkbcorvus: ianw followup on the ansible forks change, looks like we run ansible every hour and 15 minutes with the full ansible run taking just over an hour16:03
corvusclarkb: that's after the fix landed?16:04
clarkbcorvus: yes, this is the last handful of runs and I double checked the fix is landed on disk. I haven't confirmed that ansible is using that config though16:04
corvusclarkb: since that's no change, i think we should revert back to "-f 20" and check the time.  then change that to "-f 50"16:05
clarkb`grep 'Make sure system-config repo is up to date' /var/log/ansible/ansible.log` is how I'm checking the period on this16:05
clarkbcorvus: that seems reasonable16:05
corvusi'll do it16:05
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Revert "Fix ansible forks env variable"  https://review.openstack.org/60047116:07
*** hasharAway is now known as hashar16:08
*** sshnaidm is now known as sshnaidm|off16:10
openstackgerritAdam Coldrick proposed openstack-infra/storyboard master: Rationalise worklist ordering code  https://review.openstack.org/60047216:10
clarkbinfra-root please consider https://review.openstack.org/#/c/600251/ removes +rf from all our irc channels since we have yet to see spam in this channel16:10
fungiclarkb: AJaeger mentioned in scrollback that when he undid his usermode +R he started getting privmsg spam. not sure if that should be taken as a sign to delay or not16:11
*** ccamacho has joined #openstack-infra16:12
*** verdurin has quit IRC16:13
corvuswhat is legacy-system-config-puppet-syntax-3 ?16:13
*** links has quit IRC16:14
AJaegerclarkb: I approved 600251 - we can quickly revert...16:14
*** jamesmcarthur has quit IRC16:14
clarkbcorvus: it runs `bundle exec rake syntax` whcih I thin runs the puppet linter?16:14
*** ssbarnea|bkp has joined #openstack-infra16:14
clarkbcorvus: pep8 for puppet316:15
corvusi guess we still need that until we're fully puppet4?16:15
AJaegerdhellmann: we need to merge https://review.openstack.org/#/c/597598/ first to finish storlets - ocata change is not ready for py35, see https://review.openstack.org/#/c/59618316:15
clarkbya probably worth keeping16:15
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Move project config in repo  https://review.openstack.org/60047516:16
openstackgerritJames E. Blair proposed openstack-infra/project-config master: Remove system-config project config  https://review.openstack.org/60047616:17
clarkbfungi: ya we should watch it, overall though I think +r has been problematic for many users and at least in testing here so far things are quiet16:17
clarkbwe aren't going to avoid all spam but as long as it isn't persistent we are probably fine16:17
*** verdurin has joined #openstack-infra16:18
*** mriedem is now known as mriedem_away16:19
openstackgerritMerged openstack-infra/project-config master: remove job settings for senlin repositories  https://review.openstack.org/59750216:20
*** eernst has joined #openstack-infra16:20
*** jamesmcarthur has joined #openstack-infra16:21
*** ccamacho has quit IRC16:22
openstackgerritMerged openstack-infra/system-config master: Allow ns servers to connect to adns  https://review.openstack.org/60046916:24
clarkbanother followup on ansible related items from yseterday: we are writing clouds.yaml on nodepool nodes with ansible now16:25
clarkbseems to be working fine16:25
fungiexcellent16:25
clarkbif another infra-root can review https://review.openstack.org/#/c/597728/ (thanks corvus) I can watch that go in and make sure we are cloud launching properly16:26
openstackgerritMerged openstack-infra/project-config master: Stop requiring users be registered on openstack channels  https://review.openstack.org/60025116:26
clarkbinfra-root: also with the merging of the nodepool testinfra tests I think we have a good framework in place for testing arbitrary nodes (even with secrets) as we make changes. I think we should adopt a practice of requiring tests for these chagnes as they go in16:27
fungiwhy is there an uncommitted playbooks/conn-check.yaml file in /opt/system-config on bridge.o.o?16:27
Shrewsi thought we had already adopted that practice  :)16:28
clarkbShrews: we sort of were but sort of weren't (and maybe we discussed it more in the meeting I missed due to vacation), calling it out as I know I failed at it myself16:28
clarkbfungi: I'm guessing monty was using that to test ansible16:28
ssbarnea1clarkb: I am wondering if you found a workaround for https://github.com/pypa/pip/issues/4759 bug as seeing this error in build logs is really anoying. your pip bug is almost one year old....16:29
fungiclarkb: have an example? like, how would you recommend testing adns1.o.o allows 53/tcp from ns1 and ns2 now that 600469 has merged?16:29
clarkbssbarnea1: to be honest I gave up on it, I tried to push that PR through responded to reviewers then 9 months later get an email asking why I ignored the reviewer16:30
clarkbssbarnea1: if it was an actual breaking bug I would put more effort into it but it is just confusing noise so meh. If you want to pick it up and address the comments feel free16:31
clarkbfungi: https://review.openstack.org/#/c/597728/8/testinfra/test_bridge.py is the test I added for checking the cron entry was added to the node properly. But you would add a test_dns.py file and either a new job or node to existing nodeset with dns group'd nodes16:31
clarkbthen in the testinfra test_dns.py file you can shell out to iptables -L -n (though there may be more native primitives)16:32
fungiclarkb: cool, thanls for the example!16:32
clarkbhttps://testinfra.readthedocs.io/en/latest/modules.html#iptables there is actually an iptables type you can use to check things there16:32
corvusfungi: i was about to write that job/test... do you want me to do that, or did you want to try your hand on it?16:32
*** akhila has joined #openstack-infra16:32
*** florianf has quit IRC16:32
fungii can give it a shot16:33
openstackgerritMerged openstack-infra/project-config master: Add stx-distcloud and stx-distcloud-client repos  https://review.openstack.org/59838316:33
corvusfungi: cool, i'd suggest a new 2-node job like run-nodepool with nodes named 'adns1.openstack.org' and 'ns1.openstack.org'16:33
corvusfungi: er, well, a 3 node job cause you also need bridge.openstack.org16:33
Shrewsclarkb: what is installing run_cloud_launcher.sh into /opt/system-config ?16:33
corvusShrews: it's actually in the repo16:34
openstackgerritAdam Coldrick proposed openstack-infra/storyboard master: Rationalise worklist ordering code  https://review.openstack.org/60047216:34
corvusShrews: /opt/system-config is a checkout of the system-config repo, and it's here: http://git.openstack.org/cgit/openstack-infra/system-config/tree/run_cloud_launcher.sh16:35
fungiclarkb: corvus: a new 3-node job just to test dns transfers? i wonder how many jobs will be running against system-config changes in the future16:35
clarkbShrews: https://git.openstack.org/cgit/openstack-infra/system-config/tree/playbooks/update-system-config.yaml runs as part of https://git.openstack.org/cgit/openstack-infra/system-config/tree/run_all.sh#n3616:35
clarkbfungi: we've been using file matches to scope them to the files that are relevant, the nodepool job should have examples of that as well16:36
corvusfungi: i've been pretty agressive with file matchers for these.16:36
*** jpena is now known as jpena|off16:36
fungigot it16:36
fungiconcerns alleviated in that case16:36
corvusthe file organization makes it really nice actually16:36
pabelangerclarkb: +1 to testinfra for system-config.  Think it has been working great so far16:37
*** aojea has quit IRC16:37
*** jamesmcarthur has quit IRC16:37
*** jamesmcarthur has joined #openstack-infra16:39
corvusfungi, clarkb: related, i proposed https://review.openstack.org/600475 so that when we're making new jobs, we can temporarily stop running the 20 other puppet jobs which aren't relevant.  maybe we should go ahead and get that moving?16:39
Shrewsclarkb: +Ad16:40
clarkbShrews: thanks16:40
fungicorvus: yep, i was looking to see if i needed to base this job on that change16:40
fungibut other than risking merge conflicts, i suppose not16:40
clarkbcorvus: we could also add file matchers to run the puppet jobs when .pp files and some of the support things change16:40
corvusfungi: yeah, not much point since it requires the corresponding project-config change to be useful16:41
corvusclarkb: yeah, that sounds good, though i can't wrap my head around that myself just yet :)16:41
*** jpich has quit IRC16:41
* clarkb wanders off to find tea while cloud launcher change goes through the gate16:43
*** trown is now known as trown|lunch16:47
AJaegerconfig-core, some reviews for your consideration, please: a small update for descriptions in ozj https://review.openstack.org/600454 removal of unused job https://review.openstack.org/600420 and removal of unused irc channel https://review.openstack.org/59847716:48
*** dtantsur is now known as dtantsur|afk16:49
*** gfidente has quit IRC16:50
clarkbAJaeger: that clarification on the docs jobs is useful. Do we keep that job around for older branches?16:51
*** derekh has quit IRC16:55
*** apetrich has joined #openstack-infra16:55
*** holser_ has quit IRC16:56
*** olivierb has quit IRC16:59
*** shardy has quit IRC17:00
*** jaosorior has quit IRC17:00
*** jaosorior has joined #openstack-infra17:01
AJaegerthanks, clarkb and corvus17:02
AJaegerclarkb: yes, we keep it around for older branches17:02
openstackgerritMerged openstack-infra/project-config master: Remove #openstack-product  https://review.openstack.org/59847717:03
AJaegerclarkb: I also switched specs jobs to use the new jobs see https://review.openstack.org/600458 and https://review.openstack.org/600457 . I don't think it's worth for those to add a new template and then rename templates for each repo...17:04
*** jamesmcarthur has quit IRC17:04
*** ccamacho has joined #openstack-infra17:05
clarkbAJaeger: I'm trying to think if they are generally compatible. We don't use branches on the spec repos so if they aren't the fix doesn't have to go to multiple repos at least17:05
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Small edits for descriptions  https://review.openstack.org/60045417:06
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Remove unused job legacy-grenade-dsvm-sahara  https://review.openstack.org/60042017:06
clarkbAJaeger: the changes lgtm, we should probably send email to the dev lsit about it when the changes go in17:07
AJaegerclarkb: the difference is that we in the sphinx job "tox -e venv python setup.py build_sphinx"  - while tox-docs runs "tox -e docs". Yes, there are no branches..17:07
clarkbAJaeger: ya so requires that there be a docs target in the specs repos (not sure if they all have one currently)17:08
AJaegerclarkb: yes, wanted to send an email once I had some thumbs up here ;) Will do.17:08
AJaegerclarkb: let me check quicly...17:08
*** noama has quit IRC17:09
AJaeger... all have docs - and manual checking of first 20 it all looks fine.17:09
*** spartakos has quit IRC17:11
*** r-daneel has joined #openstack-infra17:11
clarkbdoes anyone have time to work on the inap mirror rebuild today? I'm happy to help, but do think a second set of eyes on the process would be valuable (also 190 more test nodes would help with our backlogs)17:13
*** bobh has quit IRC17:14
*** jamesmcarthur has joined #openstack-infra17:17
*** jamesmcarthur_ has joined #openstack-infra17:17
openstackgerritMerged openstack-infra/system-config master: Run cloud launcher on bridge.o.o  https://review.openstack.org/59772817:17
corvusclarkb: what's required?17:17
clarkbthe channel mode change has been applied globally now looks like17:17
clarkbcorvus: running through the launch-node steps on bridge.openstack.org and working out where things don't quite work according to the docs. One case will be needing root/sudo17:18
*** psachin has joined #openstack-infra17:18
clarkbcorvus: I think I fixed the python bugs in the process last pass through and we have up to date clouds.yaml content so it should be very close now17:18
fungii think my dance card is full, also worried about the remaining unknown-unknowns with launch-node right now leading me down even deeper rabbit holes17:19
clarkbmostly I want to avoid the case where it only works for monty and I because we've worked through whatever bugs and failed to push all the fixes back despite our best efforts :)17:19
*** bhavikdbavishi has joined #openstack-infra17:21
*** jamesmcarthur has quit IRC17:21
*** tesseract has quit IRC17:21
openstackgerritAndreas Jaeger proposed openstack-infra/bindep master: Update docs building  https://review.openstack.org/59975517:23
*** jcoufal has joined #openstack-infra17:23
*** bobh has joined #openstack-infra17:24
*** psachin has quit IRC17:26
*** pcaruana has quit IRC17:27
corvusclarkb: what's the basic task?  why are we rebuilding the server and does it currentyl exist?17:30
*** jcoufal has quit IRC17:31
clarkbcorvus: we are rebuilding the server and it does not exist (inap accidently/mistakenly deleted it). The server's old volume does still exist17:31
corvusclarkb: i don't know anything about volumes on mirrors -- is that documented?17:32
clarkbcorvus: a quick scan of system config docs says it is not. We use a volume if flavor sizes are too small to provide an afs cache and a apache cache17:33
fungii think it's not documented particularly because not all our mirrors need the extra space17:33
EmilienMone of the proxies (the cbs.centos) is out to date, what can we do? see: https://cbs.centos.org/repos/virt7-container-common-candidate/x86_64/os/Packages/atomic-registries-1.22.1-25.git5a342e3.el7.x86_64.rpm and http://mirror.bhs1.ovh.openstack.org:8080/cbs.centos/virt7-container-common-candidate/x86_64/os/Packages/atomic-registries-1.22.1-23.git5a342e3.el7.x86_64.rpm17:34
EmilienMmwhahaha: ^17:34
fungiwe started adding volumes to existing mirrors as our data storage needs across them grew and outstripped available free space on the rootfs of various flavors17:34
corvusclarkb, fungi: where should i mount it?17:34
*** e0ne has joined #openstack-infra17:34
clarkbcorvus: looking at mirror.dfw.rax.openstack.org we have /dev/mapper/main-proxycache on /var/cache/apache2 and /dev/mapper/main-afscache on /var/cache/openafs each ~100GB total17:35
corvusokay, i can pattern after that17:35
clarkbthe lvs share the same pv according to pvs there17:35
*** spartakos has joined #openstack-infra17:35
clarkbcorvus: in this case if hyou want to reuse the same volume you should be able to import the existing lvs into lvm then update fstab?17:36
*** prad_ has joined #openstack-infra17:36
fungiyeah, because we added the afs and apache caches at different times, some mirrors have/had them as separate cinder volumes and others as logical volumes on a single cinder volume17:36
fungistarted out with some of them having a volume mounted at /var/cache/openafs so the second volume ended up at /var/cache/apache217:37
clarkbEmilienM: http://mirror.bhs1.ovh.openstack.org:8080/cbs.centos/virt7-container-common-candidate/x86_64/os/Packages/atomic-registries-1.22.1-25.git5a342e3.el7.x86_64.rpm is there too since its just a proxy17:37
clarkbEmilienM: is the problem that some index hasn't updated yet?17:37
fungii've considered just doing a single volume at /var/cache instead, but it can be nice to avoid runaway apache proxy cache starving afs cache and causing bizarre errors17:38
*** links has joined #openstack-infra17:38
clarkbif that is the problem you likely have to wait for the ttl to run out on the cached data so that apache will check if it has changed and refetch it. We have a limit of a day on those ttls iirc17:38
*** links has quit IRC17:38
*** jamesmcarthur_ has quit IRC17:39
EmilienMclarkb: most probably yes17:39
corvusclarkb: ok i'll give it a shot17:39
EmilienMclarkb: ok gotcha17:39
EmilienMcan we reduce the ttl?17:39
EmilienMwell once a day should be fine17:39
clarkbEmilienM: those values are provided by the source server, we just set a sanity check of a day max on our end17:39
clarkbit may be shorter if the source says so, but cannot be longer17:39
*** diablo_rojo has joined #openstack-infra17:40
EmilienMack17:41
EmilienMmwhahaha: ^*17:41
*** mriedem_away is now known as mriedem17:41
*** auristor has quit IRC17:43
*** auristor has joined #openstack-infra17:43
fungilooks like the dns fix merged at 16:24z, latest ansible run started at 17:3017:44
fungiso should be taking effect any time now17:44
fungiprevious run started at 16:00 so was already underway when the fix went in17:45
corvusclarkb: do you have any WIP documentation changes related to launching?17:46
clarkbcorvus: no it looks like they merged, I think one issue is that I didn't update the docs to assume root but to assume you could run as a normal user since that is what we had done and ianw's change was proposed to fix that17:47
corvusclarkb: so https://git.openstack.org/cgit/openstack-infra/system-config/tree/launch/README but without the sudos?  what about the rax dns stuff?17:48
*** prad_ has quit IRC17:49
clarkboh dns, that is a good question. I punted on that because we don't have python2 on bridge.o.o and the rackdns client doesn't do python3 currently17:49
clarkbcorvus: ya you don't need the group stuff at the beginning but will need sudo to run launch node iirc17:50
corvusclarkb: rackdns-venv has a python217:50
corvusso i guess that's expected to work17:50
*** eharney has quit IRC17:51
clarkbcorvus: I want to say that was just rsynced off of puppetmaster and doesn't work?17:51
clarkbI ended up fiddling with it for a bit then just did it through the web ui because I couldn't figure it out with the current system constraints17:51
corvusoh yeah, it doesn't work17:51
corvusclarkb: okay, so what's the goal here?  launch the server, or figure out how to use bridge to launch servers?17:52
corvusbecause bridge is clearly not ready to launch servers, and doing so may be a very long process17:52
corvusif we just want to launch the server, we should use puppetmaster or something17:52
*** prad_ has joined #openstack-infra17:52
clarkba little of both? this is already useful as it reminds us that we havne't figured out rackdns on bridge. We can install python217:53
corvusi feel like a lot of effort has gone into not installing python217:53
clarkbif we install python2 we likely need to rebuild the rackdns venv as python versions will have changed between trusty and xenial and venvs tend to not be portable across those changes17:53
corvusso if the goal is figure out how to launch servers on bridge, i think we need to figure something else out.17:53
clarkbcorvus: is edit dns manually by hand a reasonable short term something else?17:54
clarkb(I think our other options are not use rackspace dns or make rackdns run under pytho3n)17:54
*** prad_ has quit IRC17:54
corvusclarkb: using the web ui is not an option in my mind17:54
fungiwe probably have our work cut out for us redesigning our dns cut out for us anyway once we pick a winterscale domain and start hosting it on our dns infrastructure17:55
corvusfungi: yes, though i feel confident we can find a way to automate updates :)17:55
* fungi apparently had a split thought in the middle of writing that not-sentence, but you get my meaning17:55
fungicorvus: agreed. there are protocols with words like that in them, even17:56
*** jamesmcarthur has joined #openstack-infra17:56
funginot that we necessarily want to use them, but anyway17:56
clarkbhttps://github.com/linickx/rsdns is potentially another option17:56
corvusclarkb: if you want me to try to figure out how to use rax dns on bridge, i can work on that17:57
clarkbseems to only require a handful of standardish linux/gnu tools17:57
corvusi was mostly trying to figure out if we wanted to get the server up today to get those 200 nodes, or we wanted to leave it for next week after we figure all this out17:57
*** olivierb has joined #openstack-infra17:57
clarkbI think both things would be nice :) zuul does have a fairly large backlog, but we are also likely heading into a quiet period with the ptg happening17:58
clarkbif we had to prioritize, fixing the systems to launch nodes on bridge.o.o is probably more important and will be most useful during the ptg17:58
corvusok, will do17:58
*** electrofelix has quit IRC17:59
corvusum.  how do i make a virtualenv?18:00
fungiheh, i had the same question18:00
clarkbcorvus: I did document that https://git.openstack.org/cgit/openstack-infra/system-config/tree/launch/README#n1918:00
zxiiroanyone have permissions for JJB ReadTheDocs? none of the current maintainers have access so we cannot do the action required http://lists.openstack.org/pipermail/openstack-dev/2018-August/132836.html18:00
fungii tried `python3 -m venv foo` and it told me i don't have the necessary additional packages installed because ubuntu apparently likes to strip ensure-pip out of their interpreter package18:01
clarkbfungi: see link above, that did work for me18:01
*** e0ne has quit IRC18:01
clarkbzxiiro: hrm let me see if I can find out18:01
corvuszxiiro: does rtd say who the owners are?18:02
clarkbcorvus: zxiiro it does, hashar and zaro18:02
*** pcaruana has joined #openstack-infra18:02
clarkbzxiiro: maybe send them email and feel free to cc me if you like, but I think you can ask them to add you as maintainer18:02
zxiiroYeah18:02
clarkb(I would +1 that)18:02
hasharhi. I will be around in roughly 1 hour and a half18:02
zxiirook i'll dig up their emails from gerrit i guess18:02
*** links has joined #openstack-infra18:02
fungiclarkb: wow, okay so we explicitly install the python3-virtualenv deb ubuntu provides, i guess?18:03
hasharwill be happy to add more poeple to rtd as needed (if I can do that)18:03
zxiirohashar: oh great, you're here. sounds good to me18:03
clarkbhashar: hello can you add zxiiro as maintainer on https://readthedocs.org/projects/jenkins-job-builder/ ?18:03
* hashar tries18:03
clarkbfungi: yup and that doesn't install a command in a path so you have to run the .py directly18:03
fungiclarkb: i guess we could install the python3-venv package too/instead18:03
fungiwhich would allow `python3 -m venv ...` to work18:04
clarkbcorvus: fungi reading rsnds the shell scripts are pretty straightforward if we want to give that a go. Not a spaghetti mess from my reading so far18:04
*** openstackgerrit has quit IRC18:05
hasharzxiiro: I have added https://readthedocs.org/profiles/zxiiro/ :)18:05
clarkbnot quite as readable as python probably since it is large curl commands with fun tr involved18:05
hasharzxiiro: feel free to add more maintainers as may fit. I have not been active on openstack projects for quite a while unfortunately :\\18:06
zxiirohashar: can confirm. Thanks!18:06
zxiirohashar: i'll add the other folks18:06
*** e0ne has joined #openstack-infra18:06
hasharthank you !!18:06
*** olivierb has quit IRC18:06
clarkbhashar: thanks18:06
fungicorvus: clarkb: pabelanger: looks like the adns1 iptables fix was applied in the last minute or two, and zuul-ci.org records are resolving for me normally again18:08
*** e0ne has quit IRC18:08
corvusfungi: me too, thx18:08
clarkbfungi: I confirm the new iptables rules in place18:08
clarkbas well as A record resoultion for zuul-ci.org18:08
*** openstackgerrit has joined #openstack-infra18:10
openstackgerritAndreas Jaeger proposed openstack-infra/project-config master: Move coverage jobs to non-voting gate jobs  https://review.openstack.org/60049518:10
clarkbrsdns lacks a license :/18:10
*** links has quit IRC18:11
openstackgerritAndreas Jaeger proposed openstack-infra/project-config master: Move coverage jobs to non-voting gate jobs  https://review.openstack.org/43283618:11
openstackgerritAndreas Jaeger proposed openstack-infra/project-config master: Move coverage jobs to non-voting gate jobs  https://review.openstack.org/43283618:13
AJaegerianw: I updated your change ^18:13
*** bharath_t has joined #openstack-infra18:13
*** gfidente has joined #openstack-infra18:14
bharath_tcan you please review https://review.openstack.org/#/c/599709/ ?18:15
bharath_tAnd also please let me know the next steps for creating new project18:15
*** spartakos has quit IRC18:15
clarkbbharath_t: that will create 4 repositories for you whose only contents are the .gitreview configuration file to talk to our gerrit server. Once created you can clone them, make changes and push them back18:17
clarkbbharath_t: you will want to request that you get added to the initial groups as well so that you can approve cahnges18:17
openstackgerritAndreas Jaeger proposed openstack-infra/project-config master: Move coverage jobs to non-voting gate jobs  https://review.openstack.org/43283618:17
fungibharath_t: the next steps are also outlined in https://docs.openstack.org/infra/manual/creators.html18:18
*** melwitt is now known as melwitt_awaysick18:18
AJaegerbharath_t: I found two small problems - could you update quickly and then I'll give my +2?18:19
bharath_tclarkb, we need to raise separate request to add me to that group?18:21
bharath_tfungi, sure  :) will check the guide18:21
bharath_tAJaeger, will update it now18:22
AJaegerbharath_t: yes, you need to ask here to get added, this is a manual task - as explained in the guide18:22
openstackgerritbharaththiruveedula proposed openstack-infra/project-config master: Add new project Gyan  https://review.openstack.org/59970918:26
*** jaosorior has quit IRC18:27
*** ccamacho has quit IRC18:28
AJaegerbharath_t: LGTM - clarkb want to +2A? ^18:28
clarkbya I'll take another look18:29
corvusclarkb: i think python3ifying rackdns is feasible.  i'm going to do that to minimize the delta.  i'd rather save a full overhaul for when we have our new dns system18:29
*** bharath_t has quit IRC18:29
clarkbcorvus: ok18:30
clarkbcorvus: assuming licenses allow us, we may want to consider making PRs upstream or forking it on our gerrit18:30
clarkbcorvus: just so that the delta isn't stuck on bridge.o.o18:30
corvusclarkb: i'd rather just continue to pretend it doesn't exist for a few more months and then do a real thing18:31
*** bharath_t has joined #openstack-infra18:31
bharath_tAJaeger, okay18:31
AJaegerbharath_t: our CI system is really busy, might take two hours or so until this merges (assuming it passes tests)18:33
bharath_tAJaeger, sure18:34
bharath_tAJaeger, few questions ... as you said to add people to the group, we need to request manually. How can I see the groups members, and if we need to add new members to the group, we follow the same procedure18:37
*** Swami has joined #openstack-infra18:37
AJaegerbharath_t: all done via gerrit, find your project - e.g. https://review.openstack.org/#/admin/projects/openstack-dev/pbr and then click on access and then on the groups18:38
AJaegerbharath_t: we will add you as initial member and then it's self-maintenance18:39
*** jamesmcarthur has quit IRC18:40
*** akhila has quit IRC18:40
kmallocclarkb: did we remove the forward to #openstack-unregistered recently or something similar? I noticed a change in channel modes in -keystone and checking that it was intended (aka, spam flood has died off?)18:42
kmalloc(or anyone else who knows)18:43
clarkbkmalloc: yes18:43
kmallocclarkb: cool. :)18:43
bharath_tAJaeger, got it, thank you18:43
clarkbkmalloc: https://review.openstack.org/#/c/600251/ was the change to do it. We ran this channel for two days without spam so figured we would extend ti globally18:43
clarkbkmalloc: if we start seeing spam again we can revert18:43
AJaegerconfig-core, https://review.openstack.org/597590 and https://review.openstack.org/597582 are two repos that have migrated jobs now as part of python3-first, please +2A18:47
*** dhajare has quit IRC18:48
kmallocclarkb: yeah i'm not worried, i'm pleased the spam stopped18:49
*** jamesmcarthur has joined #openstack-infra18:50
*** bhavikdbavishi has quit IRC18:51
*** prad_ has joined #openstack-infra18:56
*** prad_ has quit IRC18:56
*** camunoz has quit IRC18:57
*** graphene has quit IRC18:57
*** trown|lunch is now known as trown18:57
*** graphene has joined #openstack-infra18:59
*** AJaeger_ has joined #openstack-infra19:00
openstackgerritKendall Nelson proposed openstack-infra/puppet-ptgbot master: Unschedule FC-SIG  https://review.openstack.org/60051619:00
*** pblaho has joined #openstack-infra19:00
*** AJaeger has quit IRC19:02
*** AJaeger_ is now known as AJaeger19:03
*** e0ne has joined #openstack-infra19:05
*** eharney has joined #openstack-infra19:06
openstackgerritMatt Riedemann proposed openstack-infra/devstack-gate master: Update grenade settings for stable/rocky  https://review.openstack.org/59159419:09
*** olivierb has joined #openstack-infra19:09
*** camunoz has joined #openstack-infra19:12
openstackgerritKendall Nelson proposed openstack/ptgbot master: Update unbook documentation  https://review.openstack.org/60052019:15
clarkbcloud launcher is running now fyi, but failing. Something about a key hash not matching so it won't update (have to delete first)19:16
clarkbI'll dig into that after lunch19:16
openstackgerritMerged openstack-infra/system-config master: Revert "Fix ansible forks env variable"  https://review.openstack.org/60047119:17
*** jtomasek has quit IRC19:20
AJaegerdhellmann: we need to merge https://review.openstack.org/#/c/597598/ first to finish storlets - ocata change is not ready for py35, see https://review.openstack.org/#/c/596183 - could you un-WIP 597598, please?19:23
dhellmannAJaeger : done19:24
AJaegerthanks19:24
AJaegerconfig-core, https://review.openstack.org/#/c/597598/ https://review.openstack.org/597590 and https://review.openstack.org/597582 are  repos that have migrated jobs now as part of python3-first, please +2A19:25
*** spartakos has joined #openstack-infra19:30
*** bharath_t has quit IRC19:31
corvusAJaeger, timburke: regarding 591528 -- it looks like the idea is that we want to run (1) swift's functional tests with a git checkout of swiftclient, and (2) swiftclient's functional tests with a git checkout of swiftclient.  both of those jobs inherit from the basic swift functional job which normally uses released swiftclient and runs swift's functional test.  is all that correct?19:32
*** bharath_t has joined #openstack-infra19:33
AJaegercorvus: yes that's my understanding19:33
timburkeyep19:33
*** pblaho has quit IRC19:33
*** hamzy has quit IRC19:34
corvusAJaeger, timburke: i think the change as written is the best approach.  you can override both required-projects and vars within the project stanza (just like 'voting' is on the job that's removed).  but you can't do that twice for the same job with different values.  really these new jobs are doing new things compared to the parent, so they merit their own names, even just for the sanity of folks19:35
corvuslooking at them (so you know what the job does).19:35
*** pcaruana has quit IRC19:35
timburkeand they should both be using tip-of-$branch swift and keystone, and the functional tests (whether swift's or swiftclient's) should be run once with tempauth and once with keystone19:35
*** olivierb has quit IRC19:35
corvusAJaeger, timburke: on that note, i do suggest adding a 'description:' field to them so you can communicate that to folks looking at them.  it took me a few minutes of cross-referencing to determine the intent of the jobs.19:36
AJaegercorvus: I just wonder whether the first one is needed - do we need to explicitely add "required-projects" for swiftclient if this is run on swiftclient19:36
*** Emine has quit IRC19:37
corvusAJaeger: oh i see.  1 sec.19:37
AJaegercorvus: or could we use the parent as is directly?19:37
*** Emine has joined #openstack-infra19:38
timburkeas i recall, i saw the parent job pass on patchset 1 of https://review.openstack.org/#/c/591530/ (which should have caused everything except docs and pep8 to fail)19:40
*** camunoz has quit IRC19:43
corvusAJaeger: there is a distinction.  devstack will add a project to LIBS_FROM_GIT if it is in required-projects *or* it's the project under test.  so a similar job using devstack shouldn't need to make a new job for that.  tox-siblings will install sibling packages only if they are in required-projects, not if they are the project under test.  of course, typically, the project under test is always19:44
corvusinstalled from source in a tox job, so it's effectively the same thing.  the only difference would be if some job used tox but did not install the project under test, yet did use tox-siblings.19:44
AJaegertimburke: could you add descriptions to the jobs in change 591528, please?19:44
corvusAJaeger, timburke: i don't know how swift-dsvm-functional works, so i don't know whether it should be expected to make a difference or not.19:45
AJaegercorvus: swift-dsvm-functional has devstack-minimal as parent - instead of devstack. So, that might be the difference here....19:45
timburkeAJaeger: sure19:46
corvusAJaeger: i think it still does the same libsfromgit thing19:46
corvusAJaeger: in which case, i *think* the first job isn't necessary.19:47
timburkecorvus: it *does* use tox, and it uses it inside of *swift* while testing swiftclient. i think that might be the trouble. maybe the definition in swift should include swiftclient as a required-project...19:47
AJaegercorvus: checking devstack, I agree it should not be needed.19:48
AJaegercorvus, timburke, let me write a DNM test change to check...19:48
johnsomNow that we are using a bunch of templates for py27, etc. can I set a nodeset on a template? I want to kick the tires on ubuntu-bionic nodes, but it's not clear to me I can specify a nodeset for the template jobs.19:50
corvustimburke: hrm, yeah, if the tox inside the functional job is pulling in its own dependencies outside of what devstack set up (seems quite likely), then that might be it.19:51
corvustimburke, AJaeger: so devstack would be installing swiftclient from source, but without swiftclient being in required-projects, tox-siblings (which is being run because i see that it's using the tox role) won't be installing swiftclient from source within the tox env19:52
AJaegercorvus: AH!19:52
corvustimburke, AJaeger: it is exactly the 'run tox in a repo other than the repo under test' scenario i outlined above19:52
timburkeaaaand that's why i hate legacy-swift-dsvm-functional :-(19:53
timburke(inside of swiftclient, that is)19:53
corvustimburke: as for adding swiftclient to the base job -- that's an option, but it depends on whether you always want to test swiftclient from source.  many projects want to test against released libs as well.  so some run two jobs: one that tests against released libs, and a second that tests the lib from git.19:54
AJaegertimburke: all those legacy jobs are legacy and need to be replaced...19:54
AJaegercorvus: I've pushed https://review.openstack.org/600540 - I really want to double check...19:55
corvustimburke, AJaeger: so aside from any consideration of changing swift-dsvm-functional, i think the current patchset is a good solution.  and probably warrants a note why the project-under-test is being added to required-projects.  :)19:55
AJaegertimburke: will you update - or shall I?19:55
timburkecorvus: yeah, that's part of why i took the path i did -- i see the value in using the released lib, but i *definitely* want to be making sure that some swiftclient release doesn't break swift's gate19:56
*** anteaya has joined #openstack-infra19:56
corvusAJaeger: good idea.  i mean, i've spent a whole 10 minutes digging into a pretty complicated combination of things and making a bunch of assumptions.  i could easily be wrong about something.  :)19:56
timburkeAJaeger: i can update it, but it might be a bit. i'm buried in other things at the moment19:56
AJaegertimburke: ok, I'll do it quickly - if you disagree, please take over again ;)19:57
AJaegercorvus: thanks a lot for all the digging and explanation!19:57
timburkesounds like a plan. thanks AJaeger!19:57
*** johanssone has joined #openstack-infra19:59
*** e0ne has quit IRC20:02
corvusAJaeger, timburke: i left a summary comment on 591528.20:02
pabelangercorvus: clarkb: fungi: any concerns about pip installing docker onto zuul-executor? Context is have a jobs that would like to use docker modules in ansible, but we need them on executor side: https://review.openstack.org/600456/20:05
AJaegercorvus, thanks - I've updated the change with description and some comments20:07
clarkbpabelanger: I think we need a better long term plan for how to manage that but in the short term that is probably fine to add the one offs like this as we run into them20:08
clarkbopenstackci-vexxhost-mtl1 seems to be the problem with run_cloud_launcher. I'm going to run it against only that cloud region manually and see if I can sort it out (may end up deleting the existing infra-root-keys and let it redo it20:09
fungipabelanger: "use docker modules in ansible..." to do what? that change doesn't really outline the use case20:10
clarkbfungi: use the docker module in ansible to manage containers instead of execing docker cli20:10
fungicontainers on server instances, or somewhere else?20:10
corvuswhat jobs?20:10
pabelangersure, I can update which jobs.20:11
* AJaeger waves good night20:11
clarkbAJaeger: good night20:11
fungihave a good evening AJaeger!20:11
AJaegerthanks20:11
corvusi'm confused because this would only affect stuff running on the executor20:11
corvusand i feel like i've missed a conversation if we're talking about trusted jobs running docker20:12
openstackgerritPaul Belanger proposed openstack-infra/puppet-zuul master: Add docker dependency for ansible-playbook  https://review.openstack.org/60045620:13
pabelanger^ updated commit with example job and log20:13
pabelangerbut yes, not running docker on the executor side, just the client for use on remote nodes20:14
fungipabelanger: why can't the client be executed _on_ the remote notes?20:14
pabelangerfungi: I don't know, https://docs.ansible.com/ansible/latest/modules/docker_image_module.html#requirements is more info about dependency20:15
pabelangerI can look at module and make sure not an issue with documentation20:15
pabelangeror, maybe I am reading this wrong20:16
pabelangerhost that executes this module, could be the nodepool node20:16
pabelangerI can test that easyily20:16
pabelangereasily*20:16
corvusyes that's what it should be20:16
corvusyou'll need to install the requirements on the remote node20:16
pabelangerokay, in that case, I can abandon puppet-zuul patch20:18
clarkbcorvus: pabelanger oh so ansible isn't vendoring over those libs from the local node to the remote? for some reason I thought it did that20:19
*** tosky has joined #openstack-infra20:19
pabelangertesting, but I thought it was doing that too20:19
pabelangerbut, believe corvus is right20:20
clarkbit appears that the two accounts in vexxhost mtl1 exhibit the sshe keys error but toehr clouds don't with cloud launcher20:20
clarkbI'm going to see if I can find out more info about the keys in that region20:20
mnaserclarkb: anything our side?20:22
corvusclarkb, pabelanger: it does not do that, and it's a very good thing too.  otherwise the requirements on the controller node would be immense and unresolvable (how could we have the necessary dependencies for all possible remote os versions).  this is much easier to reason about.  need to run docker on remote node?  install docker on remote node.20:22
clarkbmnaser: no I think it has more to do with how the ansible modules won't update an existing key by deleting it and creating it again20:22
clarkbthe valeus do differ20:22
clarkbinfra-root I am going to delete the infra-root-keys value in vexxhost-mtl1 manually then rerun cloud launcher against that region to make sure it is happy now20:23
*** caphrim007 has quit IRC20:23
clarkbthen automated cron will rerun at top of hour and we can check it is happy still20:23
clarkbmnaser: thank you for checking though20:27
*** jamesmcarthur has quit IRC20:27
pabelangerclarkb: +1 to delete infra-root-keys20:28
clarkbI've gone ahead and done that, both accounts in that region should have up to date infra-root-keys20:28
clarkbI'll check in half an hour if the full run of cloud launcher succeeds now20:28
pabelangerclarkb: https://github.com/ansible/ansible/pull/30565 likely is our fix for keys20:29
clarkbpabelanger: we'll have to update the cloud launcher playbooks too but ya that looks like what we want. Also there are a bunch of warnings emitted about our use of jinja in when states we should clean up20:30
pabelangerack20:30
clarkbhopefully that is one more item checked off the migration list though20:31
*** nicolasbock has quit IRC20:34
clarkbI'm going to go ahead and look at fixing the warnings while they are top of mind20:34
corvusclarkb: mirrors need to be xenial?20:37
openstackgerritMerged openstack-infra/project-config master: Add new project Gyan  https://review.openstack.org/59970920:38
clarkbcorvus: yes, they are puppeted currently20:38
corvusclarkb: i should be creating an 8g xenial node in openstackci-inap region mtl01, right?20:40
clarkbcorvus: that sounds right to me. We use the test node flavor in a few clouds because they only have that flavor.20:41
openstackgerritMerged openstack-infra/zuul-jobs master: Log output and exception when traceroute fails  https://review.openstack.org/57635120:41
corvusmgagne: i just got an error creating that^.  uuid 759283c0-c1ce-4c4f-90bd-9d5c0742af10 {u'message': u'No valid host was found. There are not enough hosts available.', u'code': 500, u'created': u'2018-09-06T20:38:10Z'}20:41
clarkbcorvus: and that seems to be plenty big as far as resources go other than disk in most cases20:41
mgagnecorvus: was it for the mirror instance?20:41
corvusmgagne: yes20:42
*** tonytan4ever has quit IRC20:43
*** kgiusti has left #openstack-infra20:44
mgagnecorvus: I freed one node for the other project/tenant.20:44
mgagneall nodes are part of an aggregate limited to the nodepool tenant and none were left for the other projects.20:44
mgagneshould be ok now20:44
corvusmgagne: thanks, retrying20:44
corvusclarkb: i see 4 volumes.  i assume i should attach mirror.mtl01.inap.openstack.org/main01 and main02.  verify it looks okay, and then delete mirror.mtl01.internap.openstack.org/main01 and main02.  sound right?20:45
corvusclarkb: (note inap/internap names)20:45
*** akhila has joined #openstack-infra20:46
*** ansmith has quit IRC20:47
mgagneb5beb67b-e231-4f45-8aa2-1371f854553a and ae5fd3fb-bcdd-45bf-9de9-9eeafb471da9 were used according to deleted rows in block_device_mapping20:48
clarkbcorvus: yes inap should be the new volume and internap volumes likely can go away20:48
mgagneinternap volumes were used with instance UUID b0953052-6dd6-48e3-b333-90a11e276edd which got deleted on 2017-08-17 16:08:4320:49
clarkbTIL ansible with_items flattened lists so if you move to loop well you have to flatten to maintain compat20:49
openstackgerritClark Boylan proposed openstack/ansible-role-cloud-launcher master: Don't use deprecated when jinja2 syntax  https://review.openstack.org/60055120:53
clarkbpabelanger: mordred ^ I haven't tested that however reading docs I think the fix is as simple as that20:53
*** nicolasbock has joined #openstack-infra20:54
*** anteaya has quit IRC20:55
*** dims has quit IRC20:55
*** rfolco has quit IRC21:00
*** rfolco has joined #openstack-infra21:00
*** rh-jelabarre has quit IRC21:02
mnaserclarkb: no problem :)21:02
mnaserfwiw21:03
mnasercloud launcher ci is broken, i tried to do some stuff but yeah21:03
*** kjackal has quit IRC21:03
clarkbmnaser: hrm is that going to be the new item on the thread I am pulling on?21:04
*** kjackal_ has quit IRC21:04
mnaserclarkb: yup -- https://review.openstack.org/#/q/project:openstack/ansible-role-cloud-launcher21:04
mnaserso os_router has an issue which we fixed upstream21:04
mnaserbut i kinda ran out of steam and time21:05
*** bobh has quit IRC21:05
*** bharath_t has quit IRC21:05
mnaserhttps://github.com/ansible/ansible/pull/4444821:05
mnaserjust merged to 2.621:05
clarkbmnaser: running the cloud launcher ci against ansible 2.6 should fix it then?21:06
mnaserclarkb: i dont know if that fix is released or not yet as it was only 2 days ago21:06
clarkbah21:06
mnaserclarkb: but there was some other ci changes too like actually deploying devstack to run it21:06
clarkbso 2.6.next21:06
mnaserin the bottom of the stack21:06
mnaseryeah21:06
mnaseri commented out the os_router stuff but it failed for some other stuff and yeah21:06
clarkbmnaser: is the floating ip failue the os_router problem? have an example of the failures after commenting things out?21:10
mnaserclarkb: nope, after commenting it out i'm not sure why that timeout happened21:11
mnaserit could really just be a transient thing21:11
mnaserOR because there is no router it cant allocate a floating ip21:11
mnaserso maybe that has to be commented out too21:11
*** dims has joined #openstack-infra21:11
clarkbya routers are where fips are terminated I think those may be related21:11
*** trown is now known as trown|outtypewww21:12
clarkbthe next ansible run should run with the explicit -f20 and we can compare runtimes against that baseline21:17
corvusmgagne: https://identity.api.cloud.iweb.com isn't responding for me21:23
clarkbnew cloud launcher error fatal: [localhost]: FAILED! => {"changed": false, "msg": "Multiple matches found for default"} when setting up default security groups in packethost (I'll debug this one too)21:24
mgagnecorvus: ok, looks like some people are fixing stuff atm, it should be back now.21:25
mgagnecorvus: same guy as last time :D21:26
ianwhey, i see we're on the internap issue, lmn if i can help21:27
clarkbthere are indeed many default security groups visible to openstackci-packethost. I believe the reason is openstackci-packethost is an admin account21:31
mgagnecorvus: are you seeing problems on your side with API endpoints?21:33
mgagneI'm getting conflicted reports on the system status21:33
*** eernst_ has joined #openstack-infra21:34
*** eernst has quit IRC21:34
clarkbmnaser: ^ have you used cloud launcher with admin accounts? any idea if there is a workaround for this problem?21:34
mnaserclarkb: i have not used cloud launcher with admin accounts because the idea terrifies me :D21:34
mnaserbut if you have an admin account, maybe you can use that to make a non-admin one and give it member role for the project and call it a day :)21:34
mnaserbecause the only other workaround i can imagine is referencing by uuid21:35
corvusmgagne: yes, compute endpoint now21:35
*** sshnaidm|off has quit IRC21:36
corvusmgagne: host='compute.api.mtl01.cloud.iweb.com'21:36
mgagneok, people on working on it right now.21:37
openstackgerritClark Boylan proposed openstack-infra/system-config master: Don't set security groups in openstackci-packethost  https://review.openstack.org/60055721:38
clarkbmnaser: its kind of annoying that the default api behavior for an admin account is to show everything rather than account specific info21:38
clarkbI'm guessing we can't change that behavior easily at this point so meh21:38
mnaserclarkb: top that -- this is neutron specific behaviour21:39
mgagneunless you introduce the all_tenants=1 parameter =)21:39
mnasernova doesn't do that21:39
mnasernope, neutron always sends it all out21:39
clarkbugh21:39
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Update launch README for bridge  https://review.openstack.org/60055821:39
clarkbcorvus: did you get rackdns speaking python3?21:40
*** ijw has joined #openstack-infra21:43
*** gfidente has quit IRC21:44
*** bobh has joined #openstack-infra21:45
mgagnecorvus: so people are telling me it's back online, can you confirm?21:45
corvusclarkb: yes21:46
clarkbneat21:46
corvusmgagne: yes, server list works for me now21:46
corvusclarkb: 2to3 did most of the work21:46
mgagnecorvus: good, sorry for the inconveniences. I don't have much excuses. :-/21:47
corvusmgagne: heh, that's okay.  i ran out of those a long time ago too.21:47
openstackgerritMerged openstack-infra/system-config master: Move project config in repo  https://review.openstack.org/60047521:49
*** ijw has quit IRC21:50
*** eernst_ has quit IRC21:50
*** eernst has joined #openstack-infra21:50
*** bobh has quit IRC21:50
corvusi think ansible is doing its thing on the new mirror now21:52
clarkbcorvus: thsi run started with -f2021:52
clarkb`grep 'Playbook run took' /var/log/ansible/ansible.log | tail -50` is my rough comparison grep21:53
*** boden has quit IRC21:54
corvusclarkb: you can also use ianw's script in /var/log/ansible to see the whole time (should be ~45 mins if the theory is correct)21:54
corvusonce it's finished21:55
clarkbya the above grep gives you intermediate playbook results21:58
clarkbthe really short playbooks are shorted so far21:58
clarkbbut I'm not sure their run time is long enough to be reliable21:58
*** ijw has joined #openstack-infra21:58
*** dave-mccowan has quit IRC22:00
clarkbcorvus: I think -f20 took base.yaml from ~32-33 minutes down to 27 minutes22:01
clarkbseems to confirm the effort to use 50 threads was not working22:01
*** mriedem is now known as mriedem_away22:02
*** mriedem_away has quit IRC22:02
fungidefault value if no -f specified is 10, right?22:06
corvusfungi: 522:06
fungiahh22:06
corvusnext we should do -f 5022:06
fungiso possible most of those playbooks were actually running the equivalent of -f5 because ~handwave~ envvar not being seen/used22:07
openstackgerritClark Boylan proposed openstack-infra/system-config master: Run puppet apply test serially on centos  https://review.openstack.org/60027022:07
clarkbianw: ^ fyi updated based on teh suggestion to serialize only centos22:07
clarkbfungi: yup22:08
corvusyeah, that's the theory.  but we should see about the whole run.  the difference was very noticeable (45->65 mins)22:08
ianwclarkb: yeah, i think that's probably the way to go, i don't think we want to expend a lot of effort debugging.  i also started to try and clean up the jumbled mess you get running all those playbooks in parallel with https://review.openstack.org/#/c/600331/22:11
*** ijw has quit IRC22:11
ianwsomething is, however, being rather persistent at evading my shell redirects22:11
ianwvery much complicated by running it under xargs22:12
*** ijw has joined #openstack-infra22:12
clarkbianw: we do get different stdout for each apply written to disk iirc22:13
clarkbhttp://logs.openstack.org/70/600270/1/check/infra-puppet-apply-3-centos-7/52847d7/applytest/puppetapplytest26.final.out for example22:14
ianwyeah, but the ansible around the puppet all gets jumbled in console.log22:15
clarkbah the ansible itself22:15
clarkbwhich is where we see the race I guess that happens before we run puppet22:15
ianwhttp://logs.openstack.org/31/600331/4/check/infra-puppet-apply-4-ubuntu-xenial/b05b84d/job-output.txt.gz#_2018-09-06_10_12_07_013789 like all that stuff that is somehow still coming out22:16
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Write ansible log to fileout in apply test  https://review.openstack.org/60056222:16
corvusclarkb, ianw: ^ think that might work?22:16
*** gema has quit IRC22:18
ianwcould do.  turning off "-x" in the script probably worth it for signal:noise ratio too22:19
clarkbthat is a unique config per run so should work22:19
*** gema has joined #openstack-infra22:20
*** gema has quit IRC22:20
*** gema has joined #openstack-infra22:20
*** ijw has quit IRC22:22
*** dave-mccowan has joined #openstack-infra22:23
*** ijw has joined #openstack-infra22:27
*** rcernin has joined #openstack-infra22:29
*** hashar has quit IRC22:29
*** ijw has quit IRC22:33
*** jamesdenton has quit IRC22:34
openstackgerritThanh Ha (zxiiro) proposed openstack-infra/project-config master: Add JJB rtd webhook id  https://review.openstack.org/60056922:35
*** ijw has joined #openstack-infra22:36
clarkbzxiiro: do you need to add the rtd job as well?22:37
zxiiroclarkb: selective reading, missed that part thanks :)22:37
zxiirofix that in a bit22:37
openstackgerritThanh Ha (zxiiro) proposed openstack-infra/project-config master: Add JJB rtd webhook id  https://review.openstack.org/60056922:38
zxiirothere we go22:38
corvusclarkb, ianw: does mirror01.mtl01.inap.openstack.org look right now?22:44
clarkbcorvus: http://mirror01.mtl01.inap.openstack.org/ubuntu/ ya that lgtm22:46
clarkbI think I've found one of the problems with our ansible runtime, we run remote_puppet_afs against groups and hosts matching afs* and that runs with -f 1 now22:48
clarkbproblem is mirrors and zuul executors and others all match afs*22:48
clarkbI'm working on a patch now22:48
corvusoof22:48
*** ijw has quit IRC22:48
clarkbcorvus: the comment in run_all.sh says we run with -f 1 to ensure we maintain afs quorum. Do you know which hosts we need to run one at a time to maintain quorum?22:49
clarkbcorvus: is it afs0X.region.openstack.org and afsdb0X.openstack.org ?22:50
corvusclarkb: yeah.  the db servers to keep the ubiq database available, but also we probably don't want to do anything on more than one fileserver at a time, just out of caution.22:51
clarkbthanks22:51
*** diablo_rojo has quit IRC22:54
openstackgerritClark Boylan proposed openstack-infra/system-config master: Serialize puppet on afs servers properly  https://review.openstack.org/60057522:56
clarkbcorvus: ianw ^ I think I got that right and expect it to speed things up a bit22:56
clarkb(if anyone wants to know how I figured that out I was running ps against ansible while running and basically doing my best to follow it manually while in progress. Noticed that ze04 was being ansibled during remote_puppet_afs which seemed weird and went from there)22:56
corvusclarkb: am i ready to delete the old volumes and put mtl01 back in service?22:57
corvusclarkb: ah.  hopefully when we get ara it'll be easier to see what's run where...22:57
*** elod has quit IRC22:58
clarkbcorvus: yes I think so22:59
clarkblet me ssh in real quick22:59
*** tosky has quit IRC22:59
clarkbya new mirror lgtm thanks22:59
clarkbas far as methods of debugging go I'm not sure its the best one :) but reading the log file was becoming difficult23:00
openstackgerritClark Boylan proposed openstack-infra/system-config master: Use git-servers group in remote_puppet_else  https://review.openstack.org/60057623:00
openstackgerritJames E. Blair proposed openstack-infra/project-config master: Revert "Revert "Revert "Disable inap-mtl01 provider"""  https://review.openstack.org/60057723:00
clarkbthis chagne is a related cleanup, we were inconsistent in our selection of groups23:00
clarkbI don't think that one fixes a production issue though, just a sanity fix23:01
*** dave-mccowan has quit IRC23:01
corvus#status log rebuilt mirror.mtl01.inap.openstack.org and removed unused volumes mirror.mtl01.internap.openstack.org/main02 and mirror.mtl01.internap.openstack.org/main0123:01
openstackstatuscorvus: finished logging23:01
clarkbianw can you review https://review.openstack.org/600575 as I am sure it intersects with some of your afs work and want to make sure that works for you too (though we should find somethign that works for only serializing the bare minimum set of nodes)23:02
corvusianw, clarkb, mgagne: should be ready to re-enable inap in  https://review.openstack.org/60057723:02
*** pcrews has joined #openstack-infra23:02
clarkbcorvus: I +2d thanks23:02
*** slaweq_ has joined #openstack-infra23:02
*** tpsilva has quit IRC23:02
*** slaweq has quit IRC23:03
clarkbre Ara our ansible.log file contains a lot of data that shouldn't be public I've noticed23:03
corvusclarkb: the afs change looks good; we might make an 'afs-server' group that holds afs and afsdb.  but that's borderline bikeshedding right now.23:03
clarkbwe should be careful ara doesn't expose those details23:04
ianwcorvus: is it 198.72.125.8 mirror.mtl01.inap.openstack.org ? or do i have old dns23:04
corvusclarkb: it does? (aside from the openstacksdk logging?)23:04
corvusianw: i think that's old.  did that come from dns?23:04
corvusi may need to check the web dashboard and delete an old a record23:05
clarkbcorvus: ya the openstacksdk stuff23:05
ianwcorvus: yep, just tried pinging it from here23:05
corvusclarkb: that shouldn't end up in ara.  that also should be removed from the log; i think mordred ran out of time for that before he left.23:05
clarkbcorvus: ianw looks like we have an A record instead of a cname at mirror.mtl01.inap.openstack.org23:05
clarkbcorvus: gotcha23:06
ianwoh, old fashioned23:06
clarkbcorvus: ianw I think we can fix this by replacing that A record with a cname to new 01 servers23:06
corvusclarkb: i will do that23:06
corvusold record had a ttl of 3600; the cname is in place now and should be propagated by 00:10 utc23:11
clarkbI'm going to pop out for a break now, but if we get that afs change in this evening I can help watching it23:18
clarkb*help watch it23:18
*** elod has joined #openstack-infra23:18
clarkbthen tomorrow is likely a day of errands for me to help prep family for being out of town for a week23:18
*** diablo_rojo has joined #openstack-infra23:20
*** bharath_t has joined #openstack-infra23:26
ianw"close & reopen your pull request" seems a pretty sucky way to get around an unrelated CI failure23:30
bharath_tAJaeger, regarding https://review.openstack.org/#/c/599709/, can you please me to those groups?23:30
clarkbbharath_t: I'll get them23:36
bharath_tthanks clarkb23:37
clarkbbharath_t: done23:38
*** Swami has quit IRC23:48
*** bharath_t has quit IRC23:48
openstackgerritMerged openstack-infra/puppet-ptgbot master: Unschedule FC-SIG  https://review.openstack.org/60051623:53
*** camunoz has joined #openstack-infra23:59

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