Tuesday, 2017-09-26

clarkbansipuppet has started00:00
*** dingyichen has joined #openstack-infra00:01
jeblairi started the executor on ze01 (since cloning finished... a while ago)00:01
*** baoli has joined #openstack-infra00:01
*** xarses has quit IRC00:01
*** jaypipes has quit IRC00:02
clarkbrnning on afs nodes now00:03
*** SumitNaiksatam has quit IRC00:05
*** baoli has quit IRC00:06
*** jascott1_ has quit IRC00:06
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Cap ansible<2.4.0 for linters job  https://review.openstack.org/50730000:06
*** rlandy is now known as rlandy|bbl00:07
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Merge project pipeline definitions  https://review.openstack.org/50716300:12
openstackgerritMerged openstack-infra/zuul feature/zuulv3: It's branches, not branch  https://review.openstack.org/50728500:13
clarkbok I think we are reverted at this point based on syslog on zuulv300:14
*** jkilpatr has quit IRC00:16
*** yamamoto has quit IRC00:16
clarkband all the changes we listed have merged00:17
*** Swami has quit IRC00:17
*** nunchuck has quit IRC00:20
*** yamamoto has joined #openstack-infra00:21
*** srobert has quit IRC00:22
*** jdandrea_ has joined #openstack-infra00:27
*** mikal_ has quit IRC00:32
*** mikal has joined #openstack-infra00:34
fungireset and ready for tomorrow's launch window00:35
*** gildub has joined #openstack-infra00:40
*** zhurong has joined #openstack-infra00:41
*** aeng has quit IRC00:44
*** LindaWang has joined #openstack-infra00:45
*** jamesmcarthur has joined #openstack-infra00:53
openstackgerritRico Lin proposed openstack-infra/irc-meetings master: Change heat meeting schedule to 1300 UTC Wed.  https://review.openstack.org/50690100:56
openstackgerritRico Lin proposed openstack-infra/irc-meetings master: Change heat meeting schedule to 1300 UTC Wed.  https://review.openstack.org/50690100:56
*** Apoorva_ has joined #openstack-infra00:57
*** Apoorva has quit IRC01:00
*** aeng has joined #openstack-infra01:00
*** ijw has quit IRC01:02
*** Apoorva_ has quit IRC01:02
*** ijw has joined #openstack-infra01:02
*** ijw has quit IRC01:07
*** aeng has quit IRC01:08
*** kjackal_ has joined #openstack-infra01:18
*** xarses has joined #openstack-infra01:18
*** xarses has quit IRC01:19
*** xarses has joined #openstack-infra01:19
*** cshastri has joined #openstack-infra01:20
*** aeng has joined #openstack-infra01:21
*** dhill_ has quit IRC01:21
*** dhill_ has joined #openstack-infra01:21
*** esberglu has quit IRC01:33
*** rbergeron has joined #openstack-infra01:33
*** hongbin has joined #openstack-infra01:36
*** liujiong has joined #openstack-infra01:39
zhenguohi infras, regarding this https://www.mail-archive.com/openstack-dev@lists.openstack.org/msg111088.html , could some core please help us? thanks!01:39
*** camunoz has quit IRC01:41
*** aeng has quit IRC01:42
*** tpsilva has quit IRC01:42
*** yamamoto has quit IRC01:50
*** yamamoto has joined #openstack-infra01:50
*** yamamoto_ has joined #openstack-infra01:51
*** gcb has joined #openstack-infra01:52
*** yamamoto has quit IRC01:55
*** pahuang has quit IRC02:07
*** rwsu has joined #openstack-infra02:20
*** pahuang has joined #openstack-infra02:23
*** kjackal_ has quit IRC02:26
*** pblaho has quit IRC02:29
*** xarses_ has joined #openstack-infra02:30
*** xarses has quit IRC02:33
*** mwarad has joined #openstack-infra02:35
*** shu-mutou-AWAY is now known as shu-mutou02:39
*** erlon has quit IRC02:46
*** mwarad has quit IRC02:47
*** thorst has quit IRC02:51
*** thorst has joined #openstack-infra02:51
*** hongbin has quit IRC02:51
*** hongbin has joined #openstack-infra02:52
*** Dinesh_Bhor has joined #openstack-infra02:54
*** thorst has quit IRC02:56
*** SumitNaiksatam has joined #openstack-infra02:56
*** kiennt26 has joined #openstack-infra02:56
*** Dinesh_Bhor has quit IRC02:57
*** Dinesh_Bhor has joined #openstack-infra02:57
*** mikal has quit IRC02:59
*** mikal has joined #openstack-infra03:02
*** rlandy|bbl is now known as rlandy03:06
*** pblaho has joined #openstack-infra03:07
*** dave-mcc_ has quit IRC03:09
*** rlandy has quit IRC03:09
*** ramishra has joined #openstack-infra03:18
*** lewo` has quit IRC03:18
*** lewo` has joined #openstack-infra03:19
*** Dinesh_Bhor has quit IRC03:24
*** Dinesh_Bhor has joined #openstack-infra03:28
*** rwsu has quit IRC03:30
*** Apoorva has joined #openstack-infra03:30
*** jdandrea_ has quit IRC03:31
*** Dinesh_Bhor has quit IRC03:34
*** gouthamr has quit IRC03:35
*** rwsu has joined #openstack-infra03:37
*** hongbin has quit IRC03:40
*** Dinesh_Bhor has joined #openstack-infra03:46
*** aeng has joined #openstack-infra03:51
*** thorst has joined #openstack-infra03:53
*** udesale has joined #openstack-infra03:59
*** rwsu has quit IRC04:02
*** rhallisey has quit IRC04:02
*** bhavik1 has joined #openstack-infra04:08
*** jamesmcarthur has quit IRC04:09
*** rwsu has joined #openstack-infra04:15
*** ykarel_ has joined #openstack-infra04:15
*** aviau has quit IRC04:19
*** aviau has joined #openstack-infra04:20
*** mtreinish has quit IRC04:24
*** psachin has joined #openstack-infra04:26
*** mwarad has joined #openstack-infra04:34
*** mtreinish has joined #openstack-infra04:34
*** Apoorva has quit IRC04:36
*** Apoorva has joined #openstack-infra04:37
*** cuongnv has joined #openstack-infra04:38
*** jamesmcarthur has joined #openstack-infra04:40
*** Apoorva has quit IRC04:41
*** Apoorva has joined #openstack-infra04:45
*** bhavik1 has quit IRC04:46
*** mwarad has quit IRC04:47
*** gcb has quit IRC04:48
*** claudiub has joined #openstack-infra04:49
*** gildub has quit IRC04:55
*** Apoorva has quit IRC05:02
openstackgerritTobias Henkel proposed openstack-infra/nodepool feature/zuulv3: Add username to build and upload information  https://review.openstack.org/45396805:04
openstackgerritTobias Henkel proposed openstack-infra/nodepool feature/zuulv3: Rename ssh_port to connection_port  https://review.openstack.org/50080005:04
openstackgerritTobias Henkel proposed openstack-infra/nodepool feature/zuulv3: Support username also for unmanaged cloud images  https://review.openstack.org/50080805:04
openstackgerritTobias Henkel proposed openstack-infra/nodepool feature/zuulv3: Add connection_type to build and upload information  https://review.openstack.org/50314805:04
openstackgerritTobias Henkel proposed openstack-infra/nodepool feature/zuulv3: Don't gather host keys for non ssh connections  https://review.openstack.org/50316605:04
openstackgerritTobias Henkel proposed openstack-infra/nodepool feature/zuulv3: Add connection_port to build and upload information  https://review.openstack.org/50411205:04
*** Apoorva has joined #openstack-infra05:05
*** Apoorva has quit IRC05:05
*** dhajare has joined #openstack-infra05:05
*** sree has joined #openstack-infra05:06
*** Apoorva has joined #openstack-infra05:08
openstackgerritTobias Henkel proposed openstack-infra/nodepool feature/zuulv3: Improve test case node_assignment_at_quota  https://review.openstack.org/50613405:09
openstackgerritTobias Henkel proposed openstack-infra/nodepool feature/zuulv3: Honor cloud quotas before launching nodes  https://review.openstack.org/50383805:09
openstackgerritTobias Henkel proposed openstack-infra/nodepool feature/zuulv3: Don't fail on quota exceeded  https://review.openstack.org/50305105:09
*** sree has quit IRC05:09
openstackgerritTobias Henkel proposed openstack-infra/nodepool feature/zuulv3: Make max-servers optional  https://review.openstack.org/50428205:09
openstackgerritTobias Henkel proposed openstack-infra/nodepool feature/zuulv3: Support cores limit per pool  https://review.openstack.org/50428305:09
openstackgerritTobias Henkel proposed openstack-infra/nodepool feature/zuulv3: Support ram limit per pool  https://review.openstack.org/50428405:09
*** sree has joined #openstack-infra05:09
*** mriedem has quit IRC05:12
*** adreznec has quit IRC05:13
*** gildub has joined #openstack-infra05:13
*** bnemec has quit IRC05:14
*** mwarad has joined #openstack-infra05:14
*** Apoorva has quit IRC05:18
*** gcb has joined #openstack-infra05:24
*** felipemonteiro has joined #openstack-infra05:35
*** yamamoto_ has quit IRC05:37
*** yamamoto has joined #openstack-infra05:37
*** mat128 has joined #openstack-infra05:38
smcginniszhenguo: A think a lot of the infra cores are offline for the next several hours05:39
smcginniszhenguo: If you can, it might be best to reply to that ML thread and ask there what the next steps would be to move that along.05:39
*** mat128 has quit IRC05:41
*** mat128 has joined #openstack-infra05:43
*** mat128 has quit IRC05:46
*** mat128 has joined #openstack-infra05:47
*** armax has joined #openstack-infra05:59
*** felipemonteiro has quit IRC06:00
*** adreznec has joined #openstack-infra06:01
*** thorst has quit IRC06:03
*** pgadiya has joined #openstack-infra06:04
*** ihrachys has quit IRC06:05
*** mat128 has quit IRC06:09
zhenguosmcginnis: thanks for the information06:11
*** jaosorior has quit IRC06:12
*** aeng has quit IRC06:17
*** pcaruana has joined #openstack-infra06:20
*** andreas_s has joined #openstack-infra06:21
*** jascott1 has joined #openstack-infra06:22
*** zhurong has quit IRC06:22
*** iyamahat has joined #openstack-infra06:23
*** iyamahat has quit IRC06:24
*** iyamahat has joined #openstack-infra06:24
*** eranrom has quit IRC06:27
*** eranrom has joined #openstack-infra06:32
*** makowals has joined #openstack-infra06:33
*** jaosorior has joined #openstack-infra06:35
openstackgerritTobias Henkel proposed openstack-infra/nodepool feature/zuulv3: Add connection_port to build and upload information  https://review.openstack.org/50411206:35
*** kiennt26 has quit IRC06:36
*** rcernin has joined #openstack-infra06:47
*** eranrom has quit IRC06:57
*** thorst has joined #openstack-infra06:59
*** _mwarad_ has joined #openstack-infra07:00
*** sree has quit IRC07:02
*** mwarad has quit IRC07:02
*** sree has joined #openstack-infra07:02
*** eranrom has joined #openstack-infra07:06
*** sree has quit IRC07:07
*** iyamahat has quit IRC07:07
*** florianf has joined #openstack-infra07:09
*** makowals has quit IRC07:11
*** hashar has joined #openstack-infra07:11
*** dtantsur|afk is now known as dtantsur07:11
*** tesseract has joined #openstack-infra07:14
*** eranrom_ has joined #openstack-infra07:16
*** eranrom__ has joined #openstack-infra07:19
*** aviau has quit IRC07:19
*** aviau has joined #openstack-infra07:19
*** eranrom has quit IRC07:19
*** e0ne has joined #openstack-infra07:22
*** jtomasek has joined #openstack-infra07:22
*** eranrom_ has quit IRC07:23
*** armax has quit IRC07:29
*** rossella_s has joined #openstack-infra07:37
*** sree has joined #openstack-infra07:41
*** egonzalez has joined #openstack-infra07:42
*** apuimedo has joined #openstack-infra07:42
*** sree has quit IRC07:46
openstackgerritAndreas Scheuring proposed openstack/diskimage-builder master: Add zipl element as s390x architecture bootloader  https://review.openstack.org/44354807:48
*** rossella_s has quit IRC07:49
*** gildub has quit IRC07:53
*** eranrom__ has quit IRC07:54
*** xinliang has quit IRC07:55
*** ramishra has quit IRC07:56
*** hrubi has quit IRC07:57
*** hrubi has joined #openstack-infra07:57
*** ccamacho has joined #openstack-infra07:59
*** alexchadin has joined #openstack-infra08:00
*** sree has joined #openstack-infra08:02
*** jpich has joined #openstack-infra08:04
*** jascott1 has quit IRC08:05
*** markvoelker has quit IRC08:06
*** timrc has quit IRC08:07
*** eranrom has joined #openstack-infra08:07
*** dizquierdo has joined #openstack-infra08:07
*** timrc has joined #openstack-infra08:08
*** alexchadin has quit IRC08:08
*** xinliang has joined #openstack-infra08:08
*** dhajare has quit IRC08:12
*** iyamahat has joined #openstack-infra08:12
*** iyamahat_ has joined #openstack-infra08:14
*** iyamahat has quit IRC08:14
*** eranrom has quit IRC08:18
*** iyamahat_ has quit IRC08:20
*** zhurong has joined #openstack-infra08:21
*** dhajare has joined #openstack-infra08:21
*** ykarel_ is now known as ykarel|lunch08:22
*** _mwarad_ has quit IRC08:22
*** electrofelix has joined #openstack-infra08:26
*** pgadiya has quit IRC08:30
*** ralonsoh has joined #openstack-infra08:31
*** yamamoto has quit IRC08:31
*** alexchadin has joined #openstack-infra08:31
*** gildub has joined #openstack-infra08:34
*** yamamoto has joined #openstack-infra08:34
*** yamamoto has quit IRC08:39
*** udesale has quit IRC08:45
*** pgadiya has joined #openstack-infra08:46
*** yamamoto has joined #openstack-infra08:47
*** yamamoto has quit IRC08:47
*** udesale has joined #openstack-infra08:51
*** ykarel|lunch is now known as ykarel08:54
*** jpena|off is now known as jpena08:57
*** sree has quit IRC09:00
*** sree has joined #openstack-infra09:00
*** derekh has joined #openstack-infra09:00
*** dingyichen has quit IRC09:02
*** sree has quit IRC09:05
*** yamamoto has joined #openstack-infra09:10
*** udesale has quit IRC09:11
*** udesale has joined #openstack-infra09:11
*** yamamoto has quit IRC09:12
fricklerfungi: there is still more cleanup to do regarding apps.o.o, see https://ask.openstack.org/en/question/110400/is-the-openstack-community-app-catalog-dead/. I added a comment on the wiki, but someone with more background should do a complete update of that page09:14
*** yamamoto has joined #openstack-infra09:14
fricklerfungi: also please pull the cname record for storage.apps.o.o, that site still does a js-redirect to apps.o.o09:14
*** yamamoto has quit IRC09:16
*** yamamoto has joined #openstack-infra09:16
*** dtantsur is now known as dtantsur|lunch09:21
*** pbourke has quit IRC09:23
*** pbourke has joined #openstack-infra09:23
*** tosky has joined #openstack-infra09:29
*** stakeda has quit IRC09:30
*** yamamoto has quit IRC09:31
openstackgerritAlfredo Moralejo proposed openstack-infra/tripleo-ci master: DNM: troubleshoot ocata periodic jobs  https://review.openstack.org/50744809:31
*** yamamoto has joined #openstack-infra09:35
*** pgadiya has quit IRC09:45
*** zhurong has quit IRC09:52
openstackgerritCarlos Camacho proposed openstack-infra/tripleo-ci master: Add sudo to ping  https://review.openstack.org/50746309:58
*** shu-mutou is now known as shu-mutou-AWAY10:00
*** sdague has joined #openstack-infra10:00
*** jascott1 has joined #openstack-infra10:06
*** markvoelker has joined #openstack-infra10:07
*** ijw has joined #openstack-infra10:08
*** jascott1 has quit IRC10:10
*** hrubi has quit IRC10:11
*** rhallisey has joined #openstack-infra10:11
*** sree has joined #openstack-infra10:11
*** ijw has quit IRC10:13
*** armax has joined #openstack-infra10:15
*** LindaWang has quit IRC10:15
*** sree has quit IRC10:16
*** liceoprova has joined #openstack-infra10:19
*** pblaho has quit IRC10:22
*** cshastri has quit IRC10:26
*** gildub has quit IRC10:32
*** zhurong has joined #openstack-infra10:35
*** iyamahat has joined #openstack-infra10:35
*** iyamahat has quit IRC10:36
*** iyamahat has joined #openstack-infra10:37
*** udesale has quit IRC10:37
openstackgerritSagi Shnaidman proposed openstack-infra/tripleo-ci master: Set repo setup release in playbook  https://review.openstack.org/50493910:40
*** cuongnv has quit IRC10:41
*** markvoelker has quit IRC10:41
*** iyamahat has quit IRC10:43
*** iyamahat has joined #openstack-infra10:45
*** iyamahat has quit IRC10:46
*** iyamahat_ has joined #openstack-infra10:46
*** liceoprova has quit IRC10:48
*** liceoprova has joined #openstack-infra10:48
*** liujiong has quit IRC10:51
konghi, is it a right place to ask devstack related questions here?10:53
*** udesale has joined #openstack-infra10:53
*** iyamahat_ has quit IRC10:54
*** jkilpatr has joined #openstack-infra10:54
kongok, just try my luck. I wonder how to add an additional service in an existing devstack environment?10:55
liceoprovahttp://echootv.org/live/showsat_PegD8u/2KzUQW6Y/10341.ts10:56
*** liceoprova has quit IRC10:57
*** alexchadin has quit IRC10:58
*** alexchadin has joined #openstack-infra10:59
*** jkilpatr has quit IRC10:59
dimsinfra-root : please take a look at this git repo swap request when you get a chance - https://www.mail-archive.com/openstack-dev@lists.openstack.org/msg111037.html11:01
dimsfungi : ^^ ( cc @zhenguo )11:02
zhenguodims: thanks :)11:03
*** sulo has left #openstack-infra11:04
*** nicolasbock has joined #openstack-infra11:07
*** armax has quit IRC11:12
fricklerkong: for devstack please use #openstack-qa11:13
kongfrickler: ok, thanks11:14
*** alexchadin has quit IRC11:14
*** jkilpatr has joined #openstack-infra11:15
*** ralonsoh_ has joined #openstack-infra11:21
*** ykarel is now known as ykarel|afk11:21
*** e0ne has quit IRC11:21
*** rhallisey has quit IRC11:22
*** rossella_s has joined #openstack-infra11:22
openstackgerritFlavio Percoco proposed openstack-infra/irc-meetings master: Allow for meetings to be hosted in project channel  https://review.openstack.org/50748911:23
openstackgerritFlavio Percoco proposed openstack-infra/irc-meetings master: Reflect TC's office hours in the meetings calendar  https://review.openstack.org/50749011:23
*** rhallisey has joined #openstack-infra11:24
*** ralonsoh has quit IRC11:24
*** iyamahat has joined #openstack-infra11:27
*** e0ne has joined #openstack-infra11:28
*** iyamahat has quit IRC11:29
*** alexchadin has joined #openstack-infra11:30
*** iyamahat has joined #openstack-infra11:31
*** pblaho has joined #openstack-infra11:31
*** thorst has quit IRC11:31
*** rossella_s has quit IRC11:33
*** zhurong has quit IRC11:35
*** markvoelker has joined #openstack-infra11:39
*** zhurong has joined #openstack-infra11:39
*** iyamahat has quit IRC11:48
*** bobh has joined #openstack-infra11:55
*** ykarel|afk is now known as ykarel11:57
*** dprince has joined #openstack-infra11:57
*** tpsilva has joined #openstack-infra12:01
*** jpena is now known as jpena|lunch12:03
*** timrc has quit IRC12:05
*** timrc has joined #openstack-infra12:06
*** dave-mccowan has joined #openstack-infra12:07
*** mat128 has joined #openstack-infra12:08
*** dave-mcc_ has joined #openstack-infra12:10
*** thorst has joined #openstack-infra12:11
*** dave-mccowan has quit IRC12:12
*** markvoelker has quit IRC12:12
*** dtantsur|lunch is now known as dtantsur12:15
*** raissa_ has quit IRC12:17
*** hrubi has joined #openstack-infra12:17
*** raissa_ has joined #openstack-infra12:20
*** sree has joined #openstack-infra12:20
*** rlandy has joined #openstack-infra12:22
*** wolverineav has joined #openstack-infra12:27
*** bobh has quit IRC12:27
*** udesale has quit IRC12:29
*** armax has joined #openstack-infra12:30
*** markvoelker has joined #openstack-infra12:32
*** hemna__ has joined #openstack-infra12:34
*** armax has quit IRC12:37
*** yamamoto has quit IRC12:38
mordred16345212:40
*** trown|outtypewww is now known as trown12:45
*** zhurong has quit IRC12:46
*** jaypipes has joined #openstack-infra12:49
*** mat128 has quit IRC12:52
*** bnemec has joined #openstack-infra12:52
funginow mordred is a numbers station12:52
mordredI'm always a numbers station12:52
*** srobert has joined #openstack-infra12:54
*** yamamoto has joined #openstack-infra12:55
*** mat128 has joined #openstack-infra12:55
*** erlon has joined #openstack-infra12:57
*** esberglu has joined #openstack-infra12:57
*** mat128 has quit IRC12:57
*** sree has quit IRC12:58
*** dizquierdo has quit IRC12:58
*** sree has joined #openstack-infra12:58
*** LindaWang has joined #openstack-infra12:58
*** sree has quit IRC12:59
*** sree has joined #openstack-infra12:59
*** Dinesh_Bhor has quit IRC12:59
fungifrickler: thanks for the suggestion. i've cleaned up dns records for storage.apps and also apps-beta and apps-dev12:59
*** sree has quit IRC13:00
*** sree has joined #openstack-infra13:00
*** jpena|lunch is now known as jpena13:01
fricklerfungi: cool, thx13:01
*** alexchadin has quit IRC13:01
*** jcoufal has joined #openstack-infra13:04
fungii also tweaked the warning heading on that wiki article to "no longer in service"13:05
*** yamamoto has quit IRC13:06
*** mat128 has joined #openstack-infra13:07
*** Dinesh_Bhor has joined #openstack-infra13:08
*** kuromagi has quit IRC13:09
*** kuromagi has joined #openstack-infra13:10
*** mat128 has quit IRC13:10
fungifrickler: a bit of poking in search engines points out that we don't clean up the published copies on the governance site when we move/remove files in the build: https://governance.openstack.org/tc/reference/projects/community-app-catalog.html13:10
*** mriedem has joined #openstack-infra13:11
*** bobh has joined #openstack-infra13:11
fungimaybe after zuul v3 is behind us, we can start looking at leveraging our docs site mechanisms for a lot of our other static content sites too13:11
mordredfungi: ++13:12
*** mat128 has joined #openstack-infra13:12
*** Goneri has joined #openstack-infra13:14
*** mat128 has quit IRC13:15
*** kgiusti has joined #openstack-infra13:17
*** bh526r has joined #openstack-infra13:20
*** kjackal_ has joined #openstack-infra13:20
*** amoralej is now known as amoralej|lunch13:21
*** mat128 has joined #openstack-infra13:22
*** ykarel has quit IRC13:22
*** baoli has joined #openstack-infra13:23
mordredinfra-root: just pushed up four patches to zuul that didn't report here- but they pick up the timeout thing from yesterday and the file rename - as well as add two sorted calls13:25
mordredhttps://review.openstack.org/507535 https://review.openstack.org/507536 https://review.openstack.org/507537 https://review.openstack.org/50753813:25
Shrewsalready reviewing13:26
mordredinfra-root: also - yes, there was some leftover cruft from previous runs in the playbook dir (sorry) - I'm re-running the migration script with everything rebased and clean and whatnot13:27
Shrewsmordred: is sorting required-projects a visual thing, or a necessity?13:27
mordredShrews: yah - just visual13:27
*** udesale has joined #openstack-infra13:27
mordredShrews: it stood out when I was comparing diffs from a previous run - and python decided to reorder the entries for no reason13:27
mordred(because they go through a set() I believe)13:27
*** ihrachys has joined #openstack-infra13:27
Shrewsi like predictability13:27
mordredyah13:28
mnaserhttps://review.openstack.org/#/c/507538 isnt this technically removing the sorting?13:29
mnaseror does ordereddict not always serialize order properly13:29
mordredmnaser: I have found that passing kwargs to ordereddict constructor does not preserve order of the kwargs13:32
mordredwhich makes sense, since python **kwargs are a normal unordered dict13:32
mnaserTIL13:32
mordredyah - I've learned many things in this process13:32
*** felipemonteiro has joined #openstack-infra13:32
mordredmnaser: if you wanna see some real crack - check out the http://git.openstack.org/cgit/openstack-infra/project-config/tree/tools/add-projects-to-main.py script :)13:33
mordredmnaser: it uses ruamel.yaml to do yaml things while preserving comments ... and is COMPLETELY insane13:33
*** mat128 has quit IRC13:33
Shrewsi was about to ask about the ordereddict thing13:34
mnasershoulda just used mechnical turk13:34
mnaser:P13:34
*** felipemonteiro_ has joined #openstack-infra13:34
mnasermy brains stops functioning when anything involving tokens is involved13:35
Shrewsmordred: oops, need to update the zuul-migrate job13:35
Shrewsstill uses the -m option13:35
mordredwhoops13:36
*** felipemonteiro has quit IRC13:38
*** dmsimard|off is now known as dmsimard13:39
*** mat128 has joined #openstack-infra13:40
*** armax has joined #openstack-infra13:41
*** mat128 has quit IRC13:41
*** mat128 has joined #openstack-infra13:42
*** liujiong has joined #openstack-infra13:46
*** srobert has quit IRC13:46
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Rename outfiles to zuul- to match rollout  https://review.openstack.org/50753513:46
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Sort list of required projects  https://review.openstack.org/50753613:46
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Set a max on timeout  https://review.openstack.org/50753713:46
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Sort fields in post playbook  https://review.openstack.org/50753813:46
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Add local file copying steps  https://review.openstack.org/50754413:46
*** srobert has joined #openstack-infra13:46
mordredShrews: therewego13:46
*** camunoz has joined #openstack-infra13:47
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Add local file copying steps  https://review.openstack.org/50754413:47
*** ykarel has joined #openstack-infra13:48
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Add local file copying steps  https://review.openstack.org/50754413:48
*** camunoz has quit IRC13:55
*** camunoz has joined #openstack-infra13:55
*** eranrom has joined #openstack-infra13:56
*** eharney has joined #openstack-infra13:58
*** armax has quit IRC13:58
*** eranrom has quit IRC14:00
*** eranrom has joined #openstack-infra14:05
*** yamamoto has joined #openstack-infra14:07
*** armax has joined #openstack-infra14:07
*** jascott1 has joined #openstack-infra14:08
openstackgerritDoug Hellmann proposed openstack-infra/project-config master: switch whereto to publish docs on docs.o.o  https://review.openstack.org/50724314:11
*** amoralej|lunch is now known as amoralej14:12
*** yamamoto has quit IRC14:14
*** jascott1 has quit IRC14:16
openstackgerritMonty Taylor proposed openstack-infra/openstack-zuul-jobs master: Migrate jobs from zuulv2 to zuulv3  https://review.openstack.org/50717614:16
openstackgerritMonty Taylor proposed openstack-infra/openstack-zuul-jobs master: Add jobs for project-config  https://review.openstack.org/50755114:16
*** baoli has quit IRC14:17
*** baoli has joined #openstack-infra14:17
*** eranrom has quit IRC14:17
*** mwarad has joined #openstack-infra14:19
*** rossella_s has joined #openstack-infra14:20
mnaserhmm14:20
mnaseri can consistently get gerrit to give me a 50014:20
mnaserhttps://review.openstack.org/#/c/507435/ > rebase with ui > enter "507555"14:21
mnaserand im seeing timeouts to git.openstack.org from ovh14:21
*** felipemonteiro_ has quit IRC14:22
mnaserhttp://logs.openstack.org/50/507550/1/check/gate-puppet-ceph-puppet-unit-4.8-centos-7/8af24a7/console.html14:22
*** rbrndt has joined #openstack-infra14:23
openstackgerritPaul Belanger proposed openstack-infra/openstack-zuul-jobs master: Limit base-integration jobs  https://review.openstack.org/50755814:24
*** mwarad has quit IRC14:24
*** sree has quit IRC14:24
*** dizquierdo has joined #openstack-infra14:25
openstackgerritMonty Taylor proposed openstack-infra/openstack-zuul-jobs master: Add jobs for project-config  https://review.openstack.org/50755114:30
*** rossella_s has quit IRC14:31
*** makowals has joined #openstack-infra14:34
*** xarses has joined #openstack-infra14:42
openstackgerritGage Hugo proposed openstack-infra/project-config master: Skip ansible upgrade job in keystone  https://review.openstack.org/50542614:42
*** lbragstad has quit IRC14:44
clarkbmnaser: I think if the rebase has conflicts you get a 50014:45
clarkbmnaser: maybe try it locally and see if that is the case?14:45
*** srobert has quit IRC14:46
mnaserif i remember correctly it'll tell me there's a merge conflict in the UI usually14:46
mnaserlet me see14:46
clarkbmordred: did you see there was confusion over the source of your migration data?14:46
mnaserhttp://logs.openstack.org/72/507472/1/gate/gate-puppet-openstack-integration-4-scenario002-tempest-ubuntu-xenial/c73dbf0/console.html14:47
mnasermore timeouts over at ovh14:47
mnaserunable to access git.o.o14:47
mordredclarkb: yes - I'm finishing up getting it cleaned up (and also making a new patch to decouple a section of the migration rollout that does not need to be coupled)14:47
clarkbmordred: in any case we merged the migration fixes from yesterday and expect to generate new migration data today. Also we should make sure that ansible lint syntax check runs over it though that may be slow in CI14:47
fungimnaser: isolated to one region in ovh, or multiple?14:47
*** hongbin has joined #openstack-infra14:47
mnaserfungi every case i've seen fail is ovh14:47
mordredclarkb: yup. totally agree14:47
fungimnaser: sure, but more than one ovh region? (gra1, bhs1...)14:48
mnaserhttp://logs.openstack.org/12/507512/6/check/gate-puppet-octavia-puppet-syntax-4-centos-7/22578c6/console.html - ovh-gra1 failure14:48
mnaserall failure have been gra114:48
fungiokay, so coming across the atlantic14:48
funginot from canada14:49
fungicould be a backbone peering issue, or could be something local to their environment i suppose. either way we can temporarily remove them from nodepool if retries aren't working around it sufficiently14:50
*** trown is now known as trown|brb14:50
fungithough it looks like no retries there. i guess 282099 never actually landed14:50
fungizuul v3 will make the need for that obsolete anyway14:51
*** ralonsoh_ is now known as ralonsoh14:52
*** lbragstad has joined #openstack-infra14:54
*** eharney has quit IRC14:56
*** jamesmcarthur has quit IRC14:56
openstackgerritMonty Taylor proposed openstack-infra/project-config master: Make project-config gating in zuul v3  https://review.openstack.org/50721914:56
openstackgerritMonty Taylor proposed openstack-infra/project-config master: Migrate zuul v2 project pipelines to v3  https://review.openstack.org/50717914:56
openstackgerritMonty Taylor proposed openstack-infra/project-config master: Revert "Revert "Add the rest of the projects to Zuul v3""  https://review.openstack.org/50756414:56
openstackgerritMonty Taylor proposed openstack-infra/openstack-zuul-jobs master: Migrate jobs from zuulv2 to zuulv3  https://review.openstack.org/50717614:56
*** liujiong has quit IRC14:59
jeblairo/15:01
openstackgerritMonty Taylor proposed openstack-infra/openstack-zuul-jobs master: Add jobs for project-config  https://review.openstack.org/50755115:02
clarkbmordred: is ^ the split out change you were talking about?15:03
*** ganso has joined #openstack-infra15:04
Shrewsmordred: i'm trying to figure out the "no such file or directory" error from 507544, but the script runs perfectly on my system  :(15:05
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Rename outfiles to zuul- to match rollout  https://review.openstack.org/50753515:05
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Sort list of required projects  https://review.openstack.org/50753615:05
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Set a max on timeout  https://review.openstack.org/50753715:05
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Sort fields in post playbook  https://review.openstack.org/50753815:05
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Add local file copying steps  https://review.openstack.org/50754415:05
mordredclarkb: yes15:06
mordredShrews: oh - looking15:06
*** kgiusti has quit IRC15:06
mordredShrews: I see it15:07
inc0hey guys, how is zuulv3 implementation going? Anything I can help with?15:07
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Add local file copying steps  https://review.openstack.org/50754415:07
mordredinc0: it's going well! we hit a couple of small issues yesterday then ran out of time before it was EOD - once we're awake we'll be taking another stab at it15:08
gansohello guys. I am stacking my devstack again for the first time since a long time. I don't see my logs in a screen anymore. I see that the services are running through systemd now, but I can't seem to find the screens with the logs. Is this a known bug or we are not doing that anymore? is there a doc entry  to get the screens back up?15:08
mordredinc0: and I think we're good- but thanks!15:08
mordredganso: there is a doc about devstack and systemd ... one sec15:09
Shrewsgah15:09
mordredganso: https://docs.openstack.org/devstack/latest/systemd.html15:09
gansomordred: thanks! :)15:09
mordredganso: sure thing!15:09
fungiganso: also you might consider asking future devstack questions on #openstack-qa, devstack is not an infra project15:09
mordredinfra-root: I have updated the etherpad with a new list of changes to land before we go through rollout again - and have also updated the rollout and rollback steps15:10
fungi(granted, we do happen to know a fair amount about it and a lot of the devstack authors/maintainers also hang out in this channel)15:10
gansofungi: oh sorry, will ask devstack-related questions on the QA channel from now on, thanks!15:10
*** slaweq_ has joined #openstack-infra15:10
jeblairmordred: reviewing changes now15:10
fungiganso: no apology necessary, it's probably a confusing division of labor there15:10
mordredjeblair: thanks!15:10
*** eharney has joined #openstack-infra15:11
*** jamesmcarthur has joined #openstack-infra15:12
*** felipemonteiro has joined #openstack-infra15:15
mordredinfra-root: tl;dr on a couple of them - 507551 and 507219 are a manual creation/migration of the project-config jobs so that the steps "make v3 gating on project-config" don't need to depend on landing the larger set of new jobs in OZJ (that needs a better acronym)15:15
jeblairmordred: -1 on 50754415:15
*** rnoriega_ is now known as rnoriega15:16
mordredjeblair: yah - just saw - fixing15:16
*** felipemonteiro_ has joined #openstack-infra15:16
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Add local file copying steps  https://review.openstack.org/50754415:17
*** srobert has joined #openstack-infra15:17
*** trown|brb is now known as trown15:18
*** andreas_s has quit IRC15:18
openstackgerritGage Hugo proposed openstack-infra/project-config master: Skip non-doc jobs for keystone libraries  https://review.openstack.org/50424315:19
*** felipemonteiro has quit IRC15:20
*** felipemonteiro_ has quit IRC15:21
*** tonyb has quit IRC15:22
mordredShrews: wanna re: +3 https://review.openstack.org/#/c/507544 ?15:23
ShrewsNO15:23
Shrewsi mean15:23
ShrewsDONE15:23
mordredheh15:23
*** tonyb has joined #openstack-infra15:24
jeblairwhat does 'build-specs-site' do?15:24
jeblairmordred: ^15:25
fungiis that the one that regenerates the site index?15:25
mordredjeblair: that runs the tox environment 'specs' to generate the site index - but does not publish it15:26
*** ganso has left #openstack-infra15:26
jeblairthat's for the root of the specs site, or the infra root?15:26
mordredpublish-openstack-specs-site does the publication - I suppose we should maybe call build-specs-site build-opnestack-specs-site ...15:26
jeblairthere is no infra root15:27
jeblairso i guess that's the answer to my question :)15:27
mordred:)15:27
mordredjeblair: shall I rename thatto build-openstack-specs-site real quick?15:27
jeblairmordred: i'm writing descriptions for all those jobs you're adding in 55115:27
jeblairmordred: that sounds good to me15:28
*** e0ne has quit IRC15:28
mordredjeblair: ok. one sec15:28
jeblairmordred: oh15:28
jeblairmordred: left a -1 on that too; maybe fix in same update15:28
mordredkk15:28
*** hashar has quit IRC15:30
openstackgerritJames E. Blair proposed openstack-infra/openstack-zuul-jobs master: Add some missing job descriptions  https://review.openstack.org/50756615:30
clarkbmordred: looks like https://review.openstack.org/#/c/507535/3 is still running py35 job and failing (does that imply it does break py35?)15:31
jeblairgoing forward, let's try not to add jobs to zuul-jobs, openstack-zuul-jobs, or project-config without descriptions15:31
jeblairi think we'll want a gate job to force that eventually15:31
*** sree has joined #openstack-infra15:32
openstackgerritMonty Taylor proposed openstack-infra/openstack-zuul-jobs master: Add jobs for project-config  https://review.openstack.org/50755115:32
*** psachin has quit IRC15:32
mordredjeblair: ^^ thereyougo15:33
mordredjeblair: and yes - I totally agree15:33
openstackgerritMonty Taylor proposed openstack-infra/openstack-zuul-jobs master: Add some missing job descriptions  https://review.openstack.org/50756615:33
mordredjeblair: ^^ rebased yours15:33
jeblairmordred: double thanxs15:33
openstackgerritMonty Taylor proposed openstack-infra/openstack-zuul-jobs master: Migrate jobs from zuulv2 to zuulv3  https://review.openstack.org/50717615:34
*** dhajare has quit IRC15:34
jeblairclarkb: i don't think those py35 errors are related; i think they're running on a very slow node15:35
openstackgerritMonty Taylor proposed openstack-infra/project-config master: Make project-config gating in zuul v3  https://review.openstack.org/50721915:35
mordredjeblair: yah - but I'd added in a ... oh, there'sa .zuul.yaml change there so irrelevant files didn't exclude the job15:36
jeblairmordred: yep.  i think that's probably okay.  :)15:36
mordredyah15:36
*** sree has quit IRC15:37
clarkbbecuaes we aren't speculatively applying that change?15:38
*** armax has quit IRC15:38
jeblairclarkb: we are speculatively applying it, but the change includes a change to .zuul.yaml, which isn't on the irrelevant files list (which makes it a *relevant* file).  so we run the test.15:41
clarkbgotcha15:41
jeblairmordred: zuul -1 on 50721915:41
openstackgerritMonty Taylor proposed openstack-infra/project-config master: Make project-config gating in zuul v3  https://review.openstack.org/50721915:42
mordredjeblair: blerg15:42
*** felipemonteiro has joined #openstack-infra15:42
mordredclarkb: feel like a +3 on https://review.openstack.org/#/c/507551 ?15:43
clarkbmordred: re changing the afs package location will we have to update puppet where we consume that?15:44
mordredclarkb: ah- yah - we should probably do that, yes15:44
clarkbsynchronize pull implies delegate to I guess15:45
openstackgerritDavid Moreau Simard proposed openstack-infra/openstack-zuul-jobs master: Add integration tests for multi-node-hosts-file  https://review.openstack.org/50578915:46
openstackgerritMonty Taylor proposed openstack-infra/puppet-openafs master: Pull centos packages from project-config subdir  https://review.openstack.org/50761415:46
openstackgerritDavid Moreau Simard proposed openstack-infra/openstack-zuul-jobs master: Add integration tests for multi-node-firewall  https://review.openstack.org/50478815:47
mordredclarkb: https://review.openstack.org/50761415:47
openstackgerritDavid Moreau Simard proposed openstack-infra/openstack-zuul-jobs master: Add integration tests for multi-node-bridge role  https://review.openstack.org/50478915:47
mordredclarkb: yah - synchronize is special in general15:47
clarkbmordred: ty15:47
*** jcoufal has quit IRC15:48
*** jcoufal_ has joined #openstack-infra15:48
*** ykarel is now known as ykarel|afk15:48
*** LindaWang has quit IRC15:49
openstackgerritDavid Moreau Simard proposed openstack-infra/openstack-zuul-jobs master: Add multinode integration jobs and integration tests for known_hosts  https://review.openstack.org/50478715:50
openstackgerritDavid Moreau Simard proposed openstack-infra/openstack-zuul-jobs master: Add integration tests for multi-node-hosts-file  https://review.openstack.org/50578915:50
openstackgerritDavid Moreau Simard proposed openstack-infra/openstack-zuul-jobs master: Add integration tests for multi-node-firewall  https://review.openstack.org/50478815:50
openstackgerritDavid Moreau Simard proposed openstack-infra/openstack-zuul-jobs master: Add integration tests for multi-node-bridge role  https://review.openstack.org/50478915:50
dmsimardsorry for the rebase spam15:50
ShrewsFYI, not sure how close we are to pulling the zuulv3 trigger again, but will need to step out in about 30m or so.15:51
mordredShrews: I think once the pre-req patches all land we'll be ready to pull the trigger (basically, once we've got project-config transitioned to gating in v3, I think we're ready to go)15:51
*** ccamacho has quit IRC15:51
mordredjeblair: mind hitting https://review.openstack.org/#/c/507614/ ?15:52
openstackgerritDavid Moreau Simard proposed openstack-infra/openstack-zuul-jobs master: Add multinode integration jobs and integration tests for known_hosts  https://review.openstack.org/50478715:52
openstackgerritDavid Moreau Simard proposed openstack-infra/openstack-zuul-jobs master: Add integration tests for multi-node-hosts-file  https://review.openstack.org/50578915:52
openstackgerritDavid Moreau Simard proposed openstack-infra/openstack-zuul-jobs master: Add integration tests for multi-node-firewall  https://review.openstack.org/50478815:52
openstackgerritDavid Moreau Simard proposed openstack-infra/openstack-zuul-jobs master: Add integration tests for multi-node-bridge role  https://review.openstack.org/50478915:52
dmsimardmordred: I think that should do it for integration tests ^ looking at the actual roles now..15:52
mordredShrews: how long you gonna be afk? (no worries either way, just curious)15:53
dmsimardmordred: looks like the roles from zuul-jobs are clear from rebase conflicts (I guess cause it's new stuff.)15:53
*** kiennt26 has joined #openstack-infra15:54
openstackgerritOpenStack Proposal Bot proposed openstack/os-testr master: Updated from global requirements  https://review.openstack.org/50364515:54
Shrewsmordred: 1.5h or so? got a chiro appt and meeting someone for lunch15:54
mordredcool15:57
jeblairmordred: done.  looks like everything except 219 is approved and gating15:57
jeblairmordred: another zuul -1 on 21915:57
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Rename outfiles to zuul- to match rollout  https://review.openstack.org/50753515:58
*** srobert has quit IRC15:58
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Sort list of required projects  https://review.openstack.org/50753615:58
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Set a max on timeout  https://review.openstack.org/50753715:58
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Sort fields in post playbook  https://review.openstack.org/50753815:58
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Add local file copying steps  https://review.openstack.org/50754415:59
*** srobert has joined #openstack-infra15:59
*** egonzalez has quit IRC16:00
*** sbezverk has quit IRC16:00
openstackgerritMonty Taylor proposed openstack-infra/project-config master: Make project-config gating in zuul v3  https://review.openstack.org/50721916:00
*** jistr is now known as jistr|mtg16:01
clarkbmy parents are in town and this is the day my brothers have off work, which means I'm getting dragged to the coast with everyone. I expect I'll be able to run the meeting and keep an eye on zuulv3 things (and help out where I can) via a cell phone tether. But in case things don't work out as expected I've asked fungi to run the meeting (assuming we don't cancel due to the transition)16:01
pabelangerclarkb: ack16:02
fungiyeah, also be forewarned my power may go out, so it wouldn't hurt for us to get a backup backup volunteer chair as well16:02
*** dprince has quit IRC16:02
* fungi is getting a visit from maria this afternoon16:02
jeblairmordred: KeyError: 'bindep-fallback'16:03
jeblairmordred: that's the unknown config error.  i'll go work on a patch to zuul to make that a known config error.16:03
*** ykarel|afk is now known as ykarel16:04
mordredjeblair: isn't the issue actually that it wants just '- bindep-fallback' and not '- name: bindep-fallback' ?16:04
*** dprince has joined #openstack-infra16:05
*** rcernin has quit IRC16:05
jeblairmordred: you fixed that in ps8.  the new error is on ps8.16:05
mordredoh! silly me16:06
mordredjeblair: that's the error we expect at this point becaue the ozj patch hasn't landed and project-config is a config repo yah?16:06
*** udesale has quit IRC16:08
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Consume project templates correctly in projects  https://review.openstack.org/50761616:08
mordredjeblair, Shrews, clarkb: ^^ that fixes that in the migration script16:08
openstackgerritMerged openstack-infra/tripleo-ci master: Enable Debug: True for all OVB jobs  https://review.openstack.org/50724116:08
*** Apoorva has joined #openstack-infra16:09
*** dizquierdo has quit IRC16:09
tinwoodasettle, have you got a moment please for me to ask a question about https://review.openstack.org/#/c/507268/1 (Add link to OpenStack Charms Deployment Guide) -- it's tangential, but I can't work out how the charm-deployment-guide has ended up where it is in docs.openstack.org tree !?16:11
asettletinwood: hellooooo16:11
tinwoodhi16:11
Shrewsmordred: you sure it isn't project['template'] ??? it's 'template' in the yaml16:11
asettletinwood: what's confusing you about the placement?16:12
tinwoodasettle, I asked over in #openstack-docs but it seemed kind of quiet there.  Do you want to move there, or stay here?16:12
mordredShrews: it's template in v2 - it's templates in v316:12
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Handle unknown project-template error  https://review.openstack.org/50761716:12
asettletinwood: sorry, that's probably my fault. I admittedly am not always doing a great job of monitoring that chan16:12
asettleLet's switch there16:12
tinwoodasettle, okay16:12
Shrewsmordred: right, but you change it to 'templates' on line 1388 (where it should be the v2 project)16:12
Shrewsmordred: or i'm wrong about that...16:13
Shrewsoh, no, that's v316:13
openstackgerritDavid Moreau Simard proposed openstack-infra/openstack-zuul-jobs master: Add integration tests for multi-node-firewall  https://review.openstack.org/50478816:13
openstackgerritDavid Moreau Simard proposed openstack-infra/openstack-zuul-jobs master: Add integration tests for multi-node-bridge role  https://review.openstack.org/50478916:13
asettletinwood: heh #openstack-doc (singular) that's why it was quiet ;)16:13
mordredShrews: hrm. no - there's definitely an issue there ...16:13
tinwoodasettle, ahhh16:13
* tinwood doh16:13
tinwoodI'll go to the *right* one!16:14
* Shrews needs to pull up the entire script..16:14
mgagnelooking at this bug https://bugs.launchpad.net/openstack-gate/+bug/161730916:15
openstackLaunchpad bug 1617309 in OpenStack-Gate "internap-mtl01 network connectivity issues" [Undecided,New]16:15
*** armax has joined #openstack-infra16:15
mgagneI found that according to logstash, it's not isolated to mtl0116:15
mordredShrews: I'm adding a comment about which thing is which thing16:16
Shrewsmordred: i think that's correct, since it gets passed the new project16:16
Shrewsjust had to wrap my mind around what thing is what version16:17
*** jistr|mtg is now known as jistr16:18
mordredShrews: it's still not fully working - I'm winding up with both templates and template sections :(16:20
*** armax has quit IRC16:20
Shrewsmordred: oops16:20
jeblairmordred: i only see templates: in the output16:20
jeblairhttp://logs.openstack.org/16/507616/1/check/zuul-migrate/d862fec/openstack-zuul-jobs/zuul.d/projects.yaml16:20
jeblairwhere do you see template?16:20
mordredlocally - one sec, lemme push up ...16:21
mordredOH - I know why16:21
mordredsorry - operator error locally16:22
*** jpich has quit IRC16:22
Shrewshah. ok, reapplying my +2 then16:22
Shrewsgtg now. bbs16:23
mordred(the git reset in the script was competing with me having checked out the prior version so that it wouldn't merge with itself)16:23
*** trown is now known as trown|lunch16:23
mordredShrews: cool! have fun16:23
fungimgagne: that's the problem with trying to bend a system geared for tracking bugs in software to also track short-term incidents in infrastructure16:24
fungimgagne: note that bug is well over a year old now so almost certainly no longer relevant (at least in its current form)16:24
openstackgerritMonty Taylor proposed openstack-infra/project-config master: Revert "Revert "Add the rest of the projects to Zuul v3""  https://review.openstack.org/50756416:24
openstackgerritMonty Taylor proposed openstack-infra/project-config master: Migrate zuul v2 project pipelines to v3  https://review.openstack.org/50717916:24
fungimgagne: probably just needs to be closed out and any related queries removed from elastic-recheck (if there still are any)16:25
fungiin most intermittent connectivity cases that we never have a clear resolution, the situation just silently solves itself16:26
mgagnethere are still occurrences but it looks to be specific to an other provider: http://logstash.openstack.org/#/dashboard/file/logstash.json?query=message:%5C%22errno%3DNetwork%20is%20unreachable%5C%22%20AND%20tags:console%20AND%20voting:1&from=864000s16:26
fungisometimes one of us has the time to traceroute bidirectionally and then go hunting on nanog for discussions of backbone peering problems or the like16:27
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Add jobs for project-config  https://review.openstack.org/50755116:27
*** r-daneel has joined #openstack-infra16:27
fungimgagne: yeah, that just looks like the query in e-r was less specific than the bug which was opened16:28
jeblairmordred: you said 507219 depends on something?16:28
mordredjeblair: that thing has now merged16:28
clarkbis 616 good now?16:28
mordredjeblair: https://review.openstack.org/507551 was what it depended on16:28
*** jamesmcarthur has quit IRC16:28
clarkbseems like lots of back and forth on that one16:29
mordredclarkb: yah16:29
mordredclarkb: I had a local derp - the patch was good16:29
fungimgagne: also, nobody in infra ever looks at openstack-gate bugs as far as i know. that repo exists solely so people who are building elastic-recheck queries can have a point of reference for them16:29
clarkbmordred: gotcha16:29
jeblairmordred: what adds the bindep-fallback template?16:29
mgagneok, just wanted to make sure there was no pending issue related to inap-mtl0116:29
fungimgagne: not that i know of, and certainly not one which has persisted for over a year since the bug was opened16:30
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Consume project templates correctly in projects  https://review.openstack.org/50761616:31
mordredjeblair: oh crapbasket. that's in the big job dump16:31
*** tosky has quit IRC16:35
*** kiennt26 has quit IRC16:35
mordredjeblair: well - I was mostly trying to decouple flipping project-config gating from the bigger ozj patch ... but maybe we're gonna spin too many wheels on that at this point16:35
mordredjeblair: I mean, we can make the bindep fallback jobs real quick of course, but that was a better idea to me when I was doing it this morning before people woke up16:36
jeblairmordred: whatever gets us running faster.  :)16:36
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Don't emit yaml suffixes in run and post-run lines  https://review.openstack.org/50762416:37
mordredjeblair: how do you feel about cheating just a little bit?16:38
jeblairmordred: not good16:39
mordredthe suggestion I was going to make was to put an bindep-fallback project-template into ozj so that we can land the project-config patch -  I doubt we're going to land bindep-fallback changes in the next hour - then replace the empty template with the real one in the large jobs patch16:40
jeblairmordred: what does decoupling project-config get us?16:40
mordredjeblair: just let's use have that landed and not part of the roll-forward/roll-back sequencing16:40
mordrednot decouping also obviously works16:41
mordredand we just nee to swap the main.yaml patch and the project-config-gating patch16:41
jeblairmordred: i think we should re-couple and move forward.  if we roll-back, we'll have another opportunity to try decoupling.16:41
*** jamesmcarthur has joined #openstack-infra16:41
*** jamesmcarthur has quit IRC16:41
mordredok. cool. lemme change those stacks real quick and we'll be good to go16:41
*** jamesmcarthur has joined #openstack-infra16:41
*** kjackal_ has quit IRC16:42
*** dtantsur is now known as dtantsur|afk16:42
*** jascott1 has joined #openstack-infra16:43
openstackgerritmelanie witt proposed openstack-infra/devstack-gate master: WIP Add mysqladmin -v extended-status processlist  https://review.openstack.org/50762616:43
jeblairi've let the release team know we're about to start work again and to please not make releases16:45
jeblairthis job looks really unhappy: http://zuulv3.openstack.org/static/stream.html?uuid=fd6fc295e9764d2d9e9e335481d7c2b6&logfile=console.log16:47
jeblairlooks like each playbook takes about 10m to timeout16:47
mordredjeblair: I think it's having ssh connection problems during fact gathering16:48
pabelangeragree16:49
jeblairmordred: ya; that must happen to also return exit code 416:49
*** SumitNaiksatam has quit IRC16:49
mordredyah. so our log header of "parse error" may be a bit erroneous16:49
jeblairit's unfortunate it takes so long to error out16:50
jeblairthat's been holding up the gate pipeline for an hour.16:50
*** ykarel is now known as ykarel|away16:50
mordredagree16:51
pabelangerSSH connects but handshake is failing it seems16:51
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Handle unknown project-template error  https://review.openstack.org/50761716:51
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Don't emit yaml suffixes in run and post-run lines  https://review.openstack.org/50762416:51
jeblairi guess the only way we could detect that is by parsing job-output.json and checknig for unreachable?16:52
*** claudiub has quit IRC16:52
*** derekh has quit IRC16:54
jeblairmordred: i saved job-output.json from that job; anything else i should save before it terminates?16:54
mordredjeblair: no16:54
jeblair(and yes, exit code is 4)16:55
mordredawesome16:55
pabelangerokay, I am in the node16:55
pabelangerI used a different cipher16:55
jeblairpabelanger: ?16:56
pabelangerand I think it was just killed16:56
openstackgerritMonty Taylor proposed openstack-infra/project-config master: Revert "Revert "Add the rest of the projects to Zuul v3""  https://review.openstack.org/50756416:56
openstackgerritMonty Taylor proposed openstack-infra/project-config master: Make project-config gating in zuul v3  https://review.openstack.org/50721916:56
openstackgerritMonty Taylor proposed openstack-infra/project-config master: Migrate zuul v2 project pipelines to v3  https://review.openstack.org/50717916:56
jeblairpabelanger: yes, job just finished16:56
pabelangerssh -v -c aes128-ctr -oConnectTimeout=5 root@46.254.11.12616:56
pabelangerthat worked16:56
openstackgerritMonty Taylor proposed openstack-infra/openstack-zuul-jobs master: Migrate jobs from zuulv2 to zuulv3  https://review.openstack.org/50717616:56
pabelangerso, maybe something was wrong with our hostkeys16:56
jeblairpabelanger: ya, maybe next time we spot this, we'll have time to examine them on host16:57
jeblairpabelanger: also, your use of connecttimeout makes me wonder if we can change an ansible setting to make this fail faster than 15 minutes16:57
mordred++16:58
pabelangerjeblair: possible, when I didn't have -c aes128-ctr, it did hang (greater then 5 seconds)16:58
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Add some missing job descriptions  https://review.openstack.org/50756616:58
jeblairpabelanger: oh, so if there is an option, that's not the one :/16:58
mordredinfra-root: I've updated the etherpad rollout steps and have rebased the patches16:59
pabelangerya, hopefully it happens again, and we can properly debug16:59
pabelangerHmm17:00
pabelangerI am seeing a number of failures in citycloud-sto2 around SSH connections17:01
pabelangerI wonder if something is up17:01
pabelangerthat is on nodepool.o.o17:01
*** baoli has quit IRC17:01
mordredmaybe we should turn down citycloud-sto2 for now so issues there don't make us think there are v3 issue?17:02
jeblairpabelanger: http://zuulv3.openstack.org/static/stream.html?uuid=08282a21fa2b45eead4eeeb5069efdc1&logfile=console.log17:02
*** slaweq_ has quit IRC17:02
jeblairthat job is seeing it too, but is also about to finish17:02
*** baoli has joined #openstack-infra17:02
jeblairpabelanger: are those both sto2 nodes?17:03
mordredthat is citycloud-sto2 and I was able to ssh in17:03
pabelangerjeblair: yes17:03
pabelangerokay on node again17:03
jeblairi was also able to ssh17:03
pabelangerokay, lost terminal17:03
pabelangerwhen I ran dmesg17:04
jeblairboth errors were on ze0817:04
fungiconnectivity issues in sto2, or do we know?17:04
jeblairi logged in to the node as zuul user from ze0817:05
pabelangerYa, I think we are having networking issues in sto2. Every time I try to read a file, I stop getting SSH traffic from the node17:05
fungiwe were getting reports of connectivity issues in ovh-gra1 earlier too, so wondering if there's some transatlantic routing problem today17:05
jeblairgathering facts is like reading a big file17:05
pabelangerya17:05
fungithough pabelanger's symptom sounds like a pmtud blackhole or something17:05
pabelangerso, no surprise it is failing, I am seeing the same issue17:05
jeblairmordred, pabelanger, jlk: is there a simple "ansible ..." cli i can use to just grab facts from a host?17:05
mordredjeblair: yah - ansible -m setup17:06
pabelangeryup17:06
mordredjeblair: or, 'ansible hostname -m setup' - obvs17:06
openstackgerritMonty Taylor proposed openstack-infra/project-config master: Shift nodepool quota from v2 to v3  https://review.openstack.org/50722217:06
openstackgerritMonty Taylor proposed openstack-infra/project-config master: Move tripleo-test-cloud-rh1 into nl01  https://review.openstack.org/50727517:06
mordredrebased those two17:06
pabelangerI'd be okay with dropping sto2 down to 0 for now, and keep working on zuulv3 roll out17:07
pabelangerhttp://grafana.openstack.org/dashboard/db/nodepool-city-cloud show an upwards trend in building nodes for sto217:08
jeblairzuul@ze08:/var/lib/zuul/builds/08282a21fa2b45eead4eeeb5069efdc1/ansible$ ANSIBLE_CONFIG=playbook_0/ansible.cfg ansible ubuntu-xenial -m setup17:08
jeblairthat actually fails really fast17:08
jeblairzuul@ze08:/var/lib/zuul/builds/08282a21fa2b45eead4eeeb5069efdc1/ansible$ ansible -i inventory.yaml ubuntu-xenial -m setup --private-key /var/lib/zuul/ssh/nodepool_id_rsa  -vvv17:11
jeblairthat is hanging17:11
*** rcernin has joined #openstack-infra17:11
*** panda is now known as panda|off17:11
jeblairi think the controlpersist is having an impact17:12
jeblairyeah17:12
mordredjeblair: oh - so control persist making the failure take longer17:13
jeblairyep17:13
jeblairso maybe the network problem has tanked the controlpersist connection?17:13
mordredyah - leading to things getting weird subsequently17:13
jeblairexactly17:13
pabelangerI'm still surprised we get exit code: 417:14
pabelangerI would have expected exit code: 317:14
pabelangeror 217:14
*** dhajare has joined #openstack-infra17:15
pabelangeroh, https://github.com/ansible/ansible/issues/19720 could be related to exit code 417:15
mordredjeblair: ssh -O check 46.254.11.12817:15
mordredjeblair: should check the status of the control persist if I'm reading correctly17:15
jeblairssh -o ControlPath=/home/zuul/.ansible/cp/cacf3fef3b -O check 46.254.11.12817:16
jeblairMaster running (pid=26185)17:16
mordredok. well, that didn't help us :)17:17
*** jamesmcarthur has quit IRC17:17
jeblairpabelanger: i see the same behavior if i run dmesg17:18
jeblairheck, i can't even cat .ssh/authorized_keys without seeing problems17:19
pabelangerso, I am thinking out handling of exit code 4 is not correct in this case, reading https://github.com/ansible/ansible/issues/19720 I don't think we can assume it is a 'parse error' for yaml17:20
pabelangerin the case of now, zuul should be doing what we do for exit code: 3, return self.RESULT_UNREACHABLE and retry the job17:21
*** ykarel|away has quit IRC17:21
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Print cloud location information in job header  https://review.openstack.org/50763117:21
*** SumitNaiksatam has joined #openstack-infra17:21
jeblairpabelanger: but how do we detect this case?  ideally, in less than 15 minutes?17:21
jeblair(but 15m is still preferable to 1hour)17:22
*** Swami has joined #openstack-infra17:23
pabelangerjeblair: I think we need to still wait for exit code from ansible-playbook, then validate if UNREACHABLE is logged.  I'm still looking to see where the 15m timeout is happening, possible an ansible.cfg setting17:24
mordredjeblair: we could run "ansible '*' -m setup" - but with a much shorter watchdog timer17:24
*** trown|lunch is now known as trown17:24
*** kjackal_ has joined #openstack-infra17:24
mordredwould also like to know where the 15 minute timeout is coming from thoguh17:24
jeblaircould be ssh17:25
pabelangerserver side?17:25
jeblairdoubt it; i was thinking controlpersist17:25
jeblairmordred: i think that's the best idea so far17:26
pabelangerwouldn't that mean we always gather facts?17:27
mordredpabelanger: we already always gather facts - but we have fact caching set up so it only happens once17:28
odyssey4medo you really always want to gather all facts - it's pretty slow and onerous17:28
mordredpabelanger: now - I'm not 100% sure if running ansible foo -m setup will write to the fact cache or not17:28
pabelangermordred: well, we do set gather_facts: no for logs.o.o17:28
odyssey4memordred it does, we use that in OSA17:28
pabelangerso we don't hammer it when we run 1000 jobs17:28
jeblairpabelanger: and i believe mordred is suggesting an explicit hard-coded sanity check on the job nodes in the executor before starting the job17:28
mordredpabelanger: right. that's different and wouldn't be affected17:28
mordredodyssey4me: awesome17:28
pabelangerokay cool17:28
mordredso it's not any EXTRA things we're not already doing17:29
odyssey4memordred https://github.com/openstack/openstack-ansible/blob/master/scripts/gate-check-commit.sh#L18417:29
clarkbjeblair: something aimilar is what I wish jenkins had when it had ssh trouble17:29
clarkba simpl ia tcp + ssh passing traffic check17:29
jeblairmordred: and as far as runtime, we'll then have the facts cached before our first "real" pre-playbook, so we're mostly just shifting when that happens, yeah?17:29
pabelangermordred: also, we add-host for logs.o.o, so that would be ignored at the start17:29
mordredjeblair: yes, that's right17:30
odyssey4mewe also limit the facts gathered, 'cos that cuts time taken down considerably17:30
mordredpabelanger: yes17:30
pabelangerodyssey4me: ya, we've been do gather_subset=!all, seems to be a good balance17:30
*** eranrom has joined #openstack-infra17:31
jeblairclarkb: yeah.  nodepool is giving us that now in v2.  the idea was that the first pre-playbook would give us the same thing in zuul v3, it just seems that we've hit an edge case where we can't detect the issue.17:31
odyssey4mepabelanger yup, that works if that's enough facts... and if you're not doing bash which doesn't like !17:31
pabelangerodyssey4me: and onetime cost now that we cache17:31
odyssey4mewell, it interprets  it17:31
mordredjeblair: do you thing we should implement the setup check now?17:32
odyssey4meare you caching on file, or in zookeeper or something?17:32
mordredodyssey4me: file17:32
odyssey4mefile gets super slow when you have a really big inventory, and will actually hang17:32
mordredodyssey4me: http://git.openstack.org/cgit/openstack-infra/zuul/tree/zuul/executor/server.py?h=feature/zuulv3#n144117:32
mordredodyssey4me: hopefully we won't have jobs with super big inventories :)17:32
*** jpena is now known as jpena|off17:32
odyssey4meja, I guess the inventory is only the number of nodes in a job - so you should be good17:32
jeblairmordred: i think we need it soon.  we can either do it now or shut-down sto2 and put it on the backlog.17:33
mordredjeblair: I think we should shut down sto2 and put it on the backlog - it might take us a few iterations to get it right and also to verify to our satisfaction that it works17:33
mordredjeblair: and I think it's a thing we should get right and not a thing we should feel rushed about implementing17:34
odyssey4meit might also make sense to up the timeout for the cached facts?17:34
jeblairmordred: ack; adding to etherpad17:34
mordredodyssey4me: that's probably a great idea - especially since we know what job timeouts are :)17:34
pabelangerfact_caching_timeout = 86400 is the current default17:35
pabelangerand we run a fact cache per job17:35
mordredwell - our top job timeout is 10800 - so we should be covered there17:35
odyssey4mealright, there should also be an ability to pipeline ssh and also to retry on connection failure IIRC, which may be useful17:36
odyssey4meit has been for our CI jobs17:36
mordredodyssey4me: yah - we have controlpersist set up and all that17:36
odyssey4mesome of these may be handy: https://github.com/openstack/openstack-ansible/blob/master/group_vars/all/all.yml#L107-L11717:37
odyssey4mewe found those useful in OSIC17:37
*** tesseract has quit IRC17:37
openstackgerritJames E. Blair proposed openstack-infra/project-config master: Zuul v3: Stop using citycloud-sto2  https://review.openstack.org/50763617:38
jeblairi'll update the nodepool quota move patch too17:39
mordredjeblair: cool17:39
mordredjeblair, clarkb, pabelanger: ^^ we don't have tcp keepalive or ServerAlive settings currently - maybe we should consider those when we work on the setup task in the backlog?17:39
jeblairmordred: i'm going to rebase the project-config stack on my sto2 change17:40
mordredjeblair: ++17:40
mordredodyssey4me: I'm curious about you turning compression off - I'm assuming that was too much cpu spent dealing with compression?17:40
pabelangermordred: sure, or even seeing about exposing the setting some how to make tuning easier17:40
odyssey4memordred yep, ultimately it just slowed things down17:40
odyssey4menot really about CPU utilisation - more about connectivity speed and reliability17:41
mordredpabelanger: k. I made a note about that17:41
mordredodyssey4me: oh really? that's interesting17:42
openstackgerritJames E. Blair proposed openstack-infra/project-config master: Revert "Revert "Add the rest of the projects to Zuul v3""  https://review.openstack.org/50756417:42
openstackgerritJames E. Blair proposed openstack-infra/project-config master: Make project-config gating in zuul v3  https://review.openstack.org/50721917:42
openstackgerritJames E. Blair proposed openstack-infra/project-config master: Shift nodepool quota from v2 to v3  https://review.openstack.org/50722217:42
odyssey4memordred IIRC cloudnull spent around a week testing and tuning things to figure out the optimal bits17:42
cloudnullo/17:42
odyssey4meWB from holiday cloudnull :)17:43
cloudnullthanks!17:43
cloudnullit was a good time away17:43
jeblairmordred, pabelanger, clarkb: https://review.openstack.org/50763617:43
mordredjeblair: +217:44
mordredcloudnull: welcome back just in time for the zuul v3 happy fun times!17:44
cloudnullwoot!~17:44
jeblairmordred, pabelanger: you both +2d.  i'll +317:44
mordredjeblair, pabelanger, clarkb: I have added a few more notes to the etherpad from the discussion with odyssey4me17:44
mordredpabelanger: +2 race :)17:44
cloudnullI'm looking forward to that17:44
jeblairmordred: i need to afk for 30m; can you continue pushing changes through up to 507176 ?17:45
mordredcloudnull: enjoy https://review.openstack.org/#/c/507176/ - and the fact that you might get to see that gerrit has a patch-loading-progress-bar17:45
mordredjeblair: yup17:45
*** jamesmcarthur has joined #openstack-infra17:45
odyssey4memordred one more thing that may be handy: https://github.com/ansible/ansible/blob/devel/lib/ansible/plugins/connection/ssh.py#L74-L8517:46
mordredjeblair: as in - you're fine with going ahead and landing the add-projects-to-main.yaml patch?17:46
cloudnull++17:46
odyssey4meanyway, enough of that - lemme get out of all y'all's hair17:47
*** lbragstad has quit IRC17:47
jeblairmordred: yes17:47
mordredjeblair: kk17:47
mordredinfra-root: y'all ready to get this show re-started? I'm going to land the add-projects patch so that we're ready to rock-and-roll when jeblair gets back17:48
pabelangermordred: I am ready17:49
mordredwoot17:49
* fungi puts on böc's "r.u. ready 2 rock"17:50
pabelangeryes I am17:51
*** ralonsoh has quit IRC17:51
*** lbragstad has joined #openstack-infra17:51
mordrednow it's my favorite part - the part where we wait for patches to land17:51
fungijust like any other day17:52
*** ijw has joined #openstack-infra17:53
rm_workhey, getting some WEIRD build issues with octavia pep8 check gate17:54
rm_workhttp://logs.openstack.org/75/504175/12/check/gate-octavia-pep8-ubuntu-xenial/f9afa68/console.html#_2017-09-26_05_05_47_13088517:54
rm_work./octavia/tests/unit/certificates/manager/test_barbican.py:19:1: F401 'six' imported but unused17:54
rm_workexcept ... https://review.openstack.org/#/c/504175/12/octavia/tests/unit/certificates/manager/test_barbican.py17:54
rm_workit's like it isn't testing the patch17:54
*** ijw has quit IRC17:55
rm_worklike it's testing MASTER instead of the CR17:55
rm_work(we merged changes to our pep8 checks and now they fail17:55
rm_work)17:55
rm_workerr sorry, actually not master, but like ... an older patchset17:55
rm_workor something. i'm really not sure wtf (since master had "six" but also used it, and current code doesn't use it, but also removed the import17:56
mordredrm_work: 2017-09-26 05:01:50.340663 | INFO:zuul.Cloner:Prepared openstack/octavia repo with commit 612b309f98f6af8978218ee5c609be60bc953db2 is the line where it's checking out octavia repo17:57
rm_worki'm really unable to figure out what it could possibly be looking at where that line is there in that position17:58
rm_workummmmmm17:58
*** ijw has joined #openstack-infra17:58
rm_workok well that isn't the commit of this patchset17:58
*** ijw has joined #openstack-infra17:58
mordredwhich is the zuul commit: http://logs.openstack.org/75/504175/12/check/gate-octavia-pep8-ubuntu-xenial/f9afa68/_zuul_ansible/vars.yaml17:59
rm_workuhhh wut17:59
mordred(mostly just walking back through to try to figure out what might be happening)17:59
rm_workyeah huh, i see that, how the heck did that get set17:59
*** bh526r has quit IRC17:59
mordredthat's how zuul works :)18:00
rm_worki can't even find that commit hash anywhere18:00
mordredit's merging these: openstack/octavia:master:refs/changes/75/504175/1218:00
mordredright. it's a temporary speculative hash for the purposes of testing that change18:00
rm_workoh, right. k18:00
*** baoli has quit IRC18:00
fungirm_work: it's a spontaneously created commit merging your change onto master for the purpose of testing that it's viable with the target branch tip18:00
mordredbut all of te things look correct- it should have merged that change with master, then zuul-cloner should have cloned it18:01
*** baoli has joined #openstack-infra18:01
mordredand z-c is reporting that it cloned the ref that zuul think it should have cloned -so that's good at least -18:01
mordredthen tox is reporting that it installed octavia:18:01
mordred2017-09-26 05:05:54.990605 | octavia==1.0.0.0rc2.dev60  # git sha 612b30918:01
mordredat that same commit18:02
rm_workso that's good also i assume18:02
*** Apoorva_ has joined #openstack-infra18:02
rm_workyeah uhhh ok wtf18:03
rm_workso it's not a zuul problem it's a git problem18:03
rm_worki guess?18:03
rm_worki just cherry-picked that commit onto master18:03
rm_workand it ... does not generate the file that i'd expect18:03
*** jkilpatr has quit IRC18:03
mordredyah - git merging that commit on to master winds up with six in that file18:03
*** rbrndt has quit IRC18:04
mordredit may be worth rebasing at this point18:04
rm_workeugh18:04
fungiif there are repetitive sections, git can sometimes get confused when merging18:04
*** jkilpatr has joined #openstack-infra18:04
rm_workthis is the first time i've ever seen this much of an issue18:04
mordredyah - I just did a rebase and wound up with six in that file18:04
rm_workor really ANY issue18:04
mordredso you can at least fix it that way :)18:04
rm_workyeah, ugh i wonder what all it did18:04
mordredrm_work: git merging is perfect and magical ... except when it fails horribly18:05
fungiit is definitely magical, and sometimes magic backfires18:05
rm_worklol literally it just added six magically18:05
pabelangerya18:05
pabelangerI just reproduced the issue18:05
mordredrm_work: maybe it knows something you don't ...18:05
mordredrm_work: and maybe you really _do_ want six in that file :)18:06
rm_worklol18:06
rm_workwell i just "fixed" that18:06
pabelangergit-review -d504175; git merge origin/master; tox -epep8; fails18:06
fungitrust the git. the git is your friend18:06
rm_worki wonder if it added any other random lines to my files18:06
*** Apoorva has quit IRC18:06
rm_workffff now i'm ALWAYS going to wonder that18:06
rm_workgreat, thanks, now i don't actually trust git anymore T_T18:06
*** pvaneck has joined #openstack-infra18:06
fungirm_work: i wonder that just about constantly. a healthy dose of paranoia never hurts18:06
fungithis is one of the reasons we couple revision control with tons of testing and review18:07
pabelanger++18:07
*** leong has joined #openstack-infra18:08
fungiand even if a new patchset claims to be a trivial rebase, that doesn't stop me skimming it again18:08
*** Apoorva_ has quit IRC18:08
*** srobert_ has joined #openstack-infra18:08
*** Apoorva has joined #openstack-infra18:08
*** hashar has joined #openstack-infra18:09
odyssey4meoh yeah, mordred - one more thing regarding ssh connections - if you're using delegation much, then be aware of the issue described in https://docs.openstack.org/openstack-ansible/pike/admin/advanced-config.html#ansible-forks18:09
odyssey4mewell, delegation and forks18:10
openstackgerritMerged openstack-infra/project-config master: Zuul v3: Stop using citycloud-sto2  https://review.openstack.org/50763618:10
leongjust a small favor if anyone from infra has time: to remove a maillist: https://review.openstack.org/#/c/48225118:11
*** dave-mcc_ is now known as dave-mccowan18:12
*** srobert has quit IRC18:12
amoralejwe are hitting networking issues today in ovh18:13
amoralejpabelanger, ^ has it been reported?18:13
amoralejexample http://logs.openstack.org/55/507555/1/check/gate-puppet-openstack-integration-4-scenario003-tempest-centos-7/37b111d/console.html18:15
clarkbamoralej: yes18:15
*** kgiusti has joined #openstack-infra18:15
amoralejok18:15
clarkbmordred: so we are just waiting for stuff to merge now?18:15
openstackgerritMerged openstack-infra/puppet-openafs master: Pull centos packages from project-config subdir  https://review.openstack.org/50761418:16
mordredclarkb: yup. our ffavorite game18:17
jeblairmordred: o/18:18
mordredjeblair: you have missed literally no things18:18
jeblairsto2 backout merged!18:18
mordredwell - yah18:18
mordredthta happend18:18
jeblaircentos afs thingy merged!18:19
mordredjeblair: ok. you missed a couple of things18:19
Shrewsooh, back just in time to watch merging18:19
jeblairthere are no open changes in the changes section of etherpad!18:19
clarkbleong: we dont typically remove mailing lists instead the list admin can disable new membership, that way the archives stay up18:19
clarkbfungi: ^ that sound right to you?18:19
jeblairlooks like 564 is on the cusp of merging18:20
mordredyah - it's been on the cusp for a little bit now ...18:20
jeblairhrm18:21
mordredthe job looks finished18:21
openstackgerritMerged openstack-infra/project-config master: Revert "Revert "Add the rest of the projects to Zuul v3""  https://review.openstack.org/50756418:21
jeblairthere it goes :)18:21
*** amoralej is now known as amoralej|off18:21
mordredwoot18:21
leongclarkb: ok thanks.18:21
jeblairshall i send out a status notice?18:21
mordredyah18:22
mordredI'm going to kick pippet to accelerate v3 picking that up18:22
jeblairstatus notice The infra team is continuing work to bring Zuul v3 online; expect service disruptions and please see https://docs.openstack.org/infra/manual/zuulv3.html for more information.18:22
mordredit's almost like we know what we're doing with this step18:22
openstackgerritHongbin Lu proposed openstack-infra/project-config master: [Zun] Remove the tempest plugin testing job  https://review.openstack.org/50688618:22
jeblairhow's that ^18:22
mordredjeblair: ++ wfm18:22
jeblair#status notice The infra team is continuing work to bring Zuul v3 online; expect service disruptions and please see https://docs.openstack.org/infra/manual/zuulv3.html for more information.18:23
openstackstatusjeblair: sending notice18:23
*** dprince has quit IRC18:23
pabelangerhere we go18:23
mordredjeblair: v3 should be doing the config dance now18:23
mordredit's nice starting this at 18:23 - we have so many hours in which we can do things!18:24
*** xyang1 has joined #openstack-infra18:24
-openstackstatus- NOTICE: The infra team is continuing work to bring Zuul v3 online; expect service disruptions and please see https://docs.openstack.org/infra/manual/zuulv3.html for more information.18:24
andreafgood luck with the migration!18:24
mordredandreaf: thanks!!!18:24
jeblair2017-09-26 18:22:58,875 DEBUG zuul.Scheduler: Performing reconfiguration18:24
*** electrofelix has quit IRC18:25
fungiclarkb: leong hah, i was just in the process of adding that comment to the proposed change. cleaning it up from the puppet manifest seems fine, but they18:25
openstackstatusjeblair: finished sending notice18:26
fungithey'll also want to set options to reject new posts and hide the list from the mailman index18:26
andreafjeblair: it looks like zuul does retry of post plays only in case post failures, right? so we need post jobs to be idempotent always18:26
leongfungi: i just abandon that change18:27
leongfungi: clarkb: what is your suggestion?18:27
fungileong: yep, i discovered that when it refused to accept my +2 w/ comments a moment ago18:27
*** slaweq_ has joined #openstack-infra18:27
*** xyang1 has quit IRC18:27
mordredandreaf: post playbooks shouldn't get retried- but all of the post playbooks should get run even if one of the previous ones fails18:27
leongbtw, also the admin email is no longer valid as carol has left the company18:27
mordredandreaf: so a post playbook shouldn't assume success on the part of other post playbooks18:28
jeblairandreaf: it shouldn't retry a post playbook; it runs all the post playbooks as deep as it has run pre-playbooks.  if any one fails, it marks the build as POST_FAILURE, but continues to run subsequent post playbooks.18:28
mordredyah. that18:28
leongfungi: or should i "restore"18:28
*** e0ne has joined #openstack-infra18:28
fungileong: set the necessary options through the listadmin interface to hide it from the main mailman index, disallow new subscriptions, reject posts and remove all existing subscribers. removing it from the puppet manifest like you did in your proposed change seems fine as additional cleanup, but is not strictly needed18:28
openstackgerritDavid Moreau Simard proposed openstack-infra/openstack-zuul-jobs master: Add integration tests for multi-node-firewall  https://review.openstack.org/50478818:28
openstackgerritDavid Moreau Simard proposed openstack-infra/openstack-zuul-jobs master: Add integration tests for multi-node-bridge role  https://review.openstack.org/50478918:29
leongi am not the admin to that maillist now...18:29
fungileong: i can add you (or someone) as a new listadmin to be able to shut down the list18:29
EmilienMFYI stderr: 'fatal: unable to access 'http://zm01.openstack.org/p/openstack/python-tripleoclient/': Failed to connect to zm01.openstack.org port 80: Connection timed out'18:29
EmilienMjust in case18:29
jeblairandreaf: for example, in https://docs.openstack.org/infra/manual/zuulv3.html#job-inheritance if tox pre-run fails, it won't run any tox-py27 playbooks, and it won't run tox post-run, but will run unittests and base post-run.  and it will run both of those regardless of any other failures.18:30
pabelangerEmilienM: ovh?18:30
jeblair2017-09-26 18:28:04,161 DEBUG zuul.Scheduler: Reconfiguration complete18:30
pabelangerYay18:30
EmilienMpabelanger: yes ubuntu-xenial-ovh-gra1-1114723018:30
mordredwoot!18:30
* mordred rechecks ozj patch18:30
pabelangerEmilienM: expected, we are having networking issues today18:30
EmilienMperfect18:30
leongfungi: that will be great18:30
andreafmordred, jeblair: that doesn't seem to be what happens in my case - I get RETRY_LIMIT when my post change fails https://review.openstack.org/#/c/506835/ and the rest of post is not executed since I don't get logs18:30
andreafmordred, jeblair: I could see in the live log that the failure happens during my post-tempest job18:31
Shrewsooh, lots of requests in nodepool18:31
jeblairandreaf: i'll check the logs for that18:31
pabelangermordred: ansible-playbook --syntax-check is likely going to take some time on the legacy playbooks. Not sure we want to run it long term on legacy things18:31
pabelangeror we setup parallel18:32
andreafjeblair: heh there are no logs - at least according to the gerrit comment18:32
mordredpabelanger: I agree - in fact, I think running it exactly this one time seems like the right amount of times to run it18:32
fungileong: you want me to set it to your gmail address or a different one?18:32
leongdo you have my intel address?18:32
jeblairandreaf: i mean on the executor; we can sometimes learn a bit about catastrophic failures like that from zuul logs18:32
mordredthis might wind up being the longest linter job ever :)18:33
fungileong: probably, just a sec and i'll double-check18:33
*** Apoorva has quit IRC18:33
leongfungi: thanks18:33
pabelangerindeed18:33
andreafjeblair: ah ok18:33
*** Apoorva has joined #openstack-infra18:33
jeblairandreaf: http://paste.openstack.org/show/621965/18:33
pabelangerwe are linting now18:33
jeblairandreaf: i think there may be some zuul bugs in there.  we should check with mordred and dmsimard on that when we have a break in the migration work18:33
mordredyah. wow. this isn't going to be quick18:34
pabelangermy testing with parallel got it down to about 8mins on my laptop18:34
pabelangerwith 8 instances of ansible-playbook running18:34
pabelangerour default timeout on tox-linters is 30mins, so I'm now wondering if we'll not finish in time18:35
mordredpabelanger: if we have to iterate on this one again, we probably want to add that18:35
jeblairandreaf, mordred, dmsimard: in short, it looks like zuul_json may be having issues with something...18:35
jeblairdmsimard: did your include_role fixes get merged?18:35
mordredjeblair: well - I think the real issue is: An exception occurred during task execution. To see the full traceback, use -vvv. The error was: IOError: [Errno 13] Permission denied: '/opt/stack/tempest/etc/tempest_conf.txt.gz'"18:35
jeblairandreaf: are you using include_role?18:35
andreafjeblair: I am indeed18:35
dmsimardjeblair: nope, I mostly disappeared between last friday and today18:35
dmsimardjeblair: so I didn't get the chance to pair up with a friend :(18:36
andreafjeblair: it works fine when I run it locally18:36
dmsimardandreaf: there is a known issue that we are trying to resolve18:36
fungileong: the one listed at https://wiki.openstack.org/wiki/Enterprise_Working_Group is still correct for you?18:36
mordredandreaf: yah - there'sa bug in our log processing related to include_role18:36
dmsimardandreaf: https://review.openstack.org/#/c/504238/18:36
andreafmordred: yes that's the real issue in my code, but zuul should handle that a bit more nicely :)18:36
*** e0ne has quit IRC18:36
jeblairmordred: that's an error, but i don't think that's what prevented the log uploads from happening; this is looking a lot like the zuul_json thing18:37
jeblairso hopefully we can get it squared away asap18:37
mordredandreaf: indeed - I believe the include_role issue is the thing that's causing our attempt to tell you about it to fail18:37
mordredjeblair: yah18:37
*** srobert_ has quit IRC18:37
mordreddmsimard: the balls all look green on that patch - were you still working on getting the failure to reproduce in test caes? or is that ready for review?18:37
dmsimardandreaf: making very 'core' bits more resilient to failure is a continuous and iterative process that we're working on18:37
dmsimardmordred: the zuul_json patch ? nope, the balls are not green, see the finger url in the last job run18:38
clarkbok beach tether is currently working18:38
mordreddmsimard: oh ! yes, you are totally right18:38
andreafmordred, jeblair, dmsimard: it looks like I'm doing good testing of zuul v3 ;)18:38
dmsimardmordred: I need someone who has access to executor logs to help because I am unable to reproduce the issue using your tools/test-logs.sh script18:38
mordredandreaf: yes indeed!18:39
dmsimardI've been meaning to stand up a local zuul v3 instance but haven't got around to it yet18:39
mordredpabelanger: how hard would it be to put your parallel stuff into the tox.ini?18:39
mordredpabelanger: cause I agree, I doubt we'll finish within 30 minutes - and we get to run the linters a second time once we approve18:40
fungiclarkb: i vote we keep today's meeting short at least18:40
clarkbindeed18:40
fungi(and would not be against cancelling due to inclement zuul)18:40
clarkbya maybe take the tempurature in 10 minutes then decide if we want a meeting or focus on zuul18:41
mordredI mean - right  now focusing on zuul means "watch job run and wait for it to possibly timeout"18:41
openstackgerritDavid Moreau Simard proposed openstack-infra/openstack-zuul-jobs master: Add integration tests for multi-node-firewall  https://review.openstack.org/50478818:42
openstackgerritDavid Moreau Simard proposed openstack-infra/openstack-zuul-jobs master: Add integration tests for multi-node-bridge role  https://review.openstack.org/50478918:42
andreafdmsimard: do you think it makes sense for me to try and rebase my patch on top of yours and see what happens? will zuul be able to actually test a change to itself and to a job live?18:43
clarkbmordred: this is the ansible lint syntax check of doom ya?18:44
mordredyah18:44
jeblairandreaf: no, the component with the issue is very low level and needs the installation of zuul we're running updated in order to change18:44
leongfungi: yes.18:44
andreafjeblair: right, I thought so18:44
dmsimardandreaf: that wouldn't help because my patch doesn't completely fix the issue. It's missing something and we can only tell what it is by looking at the logs directly on the zuul node.18:46
clarkbmordred: does it make sense to just have someone run it locally on a fast machine and if that goes green call it good?18:46
dmsimardandreaf: we are able to reproduce the issue in-gate with a "special" integration test job18:46
clarkbat this point probably doesn't make sense to stop the existing job though18:46
*** slaweq_ has quit IRC18:46
mordredclarkb, pabelanger: http://paste.openstack.org/show/621967/ how does that look?18:47
clarkbmordred: xargs can do it too18:47
*** slaweq_ has joined #openstack-infra18:47
mordredclarkb: well - considering we hav eto run the check again in the gate - it may be worth considering restarting with a faster version of the job, or having someone check it out of band as you suggested18:47
clarkb-P18:47
mordredclarkb: xargs does parallel operation?18:47
mordredah-TIL18:47
clarkbmordred: yup18:47
pabelangerdidn't know that18:47
mordredso bash -c "cd playbooks; find . -type f -regex '.*.y[a]?ml' -print0 | xargs -t -P -n1 -0 ansible-playbook --syntax-check -i {toxinidir}/tests/inventory > /dev/null"18:48
fungileong: it should hopefully have e-mailed you the new password i reset for that ml just now18:48
clarkbmordred: you have to pass the number of processes to -P18:48
mordredclarkb: anywho - do you think we should restart with -P ? or just have someone do local testing of playbook syntax and restart this with legacy playbooks excluded?18:49
fungileong: use that at http://lists.openstack.org/cgi-bin/mailman/admin/enterprise-wg/ and you can get to all the list settings18:49
clarkb-P $(nproc)18:49
openstackgerritAndrea Frittoli proposed openstack-infra/devstack-gate master: Add post step to Tempest base job  https://review.openstack.org/50683518:49
clarkbmordred: I'm thinking might be quicket to have someone run with -P $(nproc) locally and verify the result18:49
clarkbI'm guessing its predominantly cpu bound and local cpus likely to go pretty quick18:49
mordredclarkb: kk. I don't have a fast computer locally available - anybody else got a good one?18:50
clarkb(assuming we have decent local cpus I want to say my laptop cpu is roughly the speed of a gate VM)18:50
pabelangerI have local testing going too with parallel, but -P for xargs is +118:50
andreafjeblair: the permission issue is because I was trying to compress tempest files directly in their folder but the user running post does not have write access there18:50
openstackgerritBrian Haley proposed openstack-infra/project-config master: Make neutron-multinode job the default  https://review.openstack.org/48360018:51
*** slaweq_ has quit IRC18:52
*** gouthamr has joined #openstack-infra18:52
*** dprince has joined #openstack-infra18:53
pabelangerclarkb: mordred: local testing shows now errors18:53
pabelangerbut also testing -P ($nproc) now18:53
pabelangermuch faster :)18:53
pabelangermordred: clarkb: http://paste.openstack.org/show/621969/18:54
pabelangerwe should use that next patch18:54
mordredpabelanger: ++18:56
*** gouthamr has quit IRC18:57
leongfungi: got it. thanks!18:57
mordredpabelanger: so - you tested locally with that patch and it did not show ansible-playbook syntax errors?18:57
*** baoli has quit IRC18:57
mordredpabelanger: how long did it take to run?18:57
pabelangermordred: ya, I used parallel first, things worked. Just waiting for 2nd pass using xargs18:57
mordredkk18:57
pabelangermordred: ~6mins to run first time18:58
clarkbok we are tminus one minute or so, do we want to bother with a meeting?18:58
clarkb(I'm good to run it if we do want one)18:58
*** rbrndt has joined #openstack-infra18:58
clarkbI do think it will be relatively quick too since the thing to tlak about is what we are deploying18:58
mordredI don't have much to report other than v3 going18:59
mordredso happy to do meeting  - or to skip meeting18:59
pabelangersame18:59
*** baoli has joined #openstack-infra18:59
fungisame here. if we do have the meeting, maybe acknowledge that people are probably paying more attention to the cut-over18:59
pabelanger  linters: commands succeeded19:00
clarkbI think I'll start it and note we are busy with zuul and go straight ot open discussion if anyone has anything19:00
clarkbrathe rthan a normal structured meeting19:00
* clarkb does this19:00
mordredpabelanger: so you have run it twice locally in the time we're still waiting on the first iteration to run in the gate19:00
pabelangeryup19:00
pabelangerrunning again with time19:00
mordredpabelanger: I vote you amend that commit with the -P $(nproc)  - because even if we're almost done with the current run - we still have the gate run19:00
fungiclarkb: thanks, wfm19:01
pabelangermordred: sure19:01
mordredclarkb, fungi, jeblair: ^^ y'all have opinion on that ?19:01
*** ianw|pto is now known as ianw19:01
clarkbmordred: sounds reasonable19:02
clarkbbut don't we also have t ofix the errors pabelanger ran into?19:02
clarkbor did I misparse?19:02
fungiyeah, and if we're really concerned about duration for running twice we could enqueue to the gate once pushed?19:02
mordredclarkb: pabelanger did not run in to any issues19:02
pabelangerclarkb: no, I don't have errors locally19:02
clarkbah ok I misparsed then19:02
pabelangerpush up?19:02
clarkbfungi: ++19:02
*** Goneri has quit IRC19:03
jeblair++19:03
*** ijw has quit IRC19:03
mordredah - it just passed19:04
pabelangerkk19:04
mordrednope. it just finished19:04
mordredand timed out19:04
mordredpabelanger: so yes - please push up :)19:04
openstackgerritPaul Belanger proposed openstack-infra/openstack-zuul-jobs master: Migrate jobs from zuulv2 to zuulv3  https://review.openstack.org/50717619:04
pabelangerdone19:04
mordredwoot19:04
mordredfinger crossed the gate node is at least as fast as pabelanger's computer19:05
*** hashar has quit IRC19:06
pabelangerreal 6m52.422s19:07
pabelangertime to beat19:07
mordredShrews, pabelanger: I'm getting no log from http://zuulv3.openstack.org/static/stream.html?uuid=2042402c144c44edaf21d52540ff54c0&logfile=console.log19:07
pabelangersame19:07
*** dhajare has quit IRC19:08
mordredsocket.gaierror: [Errno -2] Name or service not known19:08
mordredhave we had another hostname issue?19:08
Shrewsummm19:08
*** jaosorior has quit IRC19:09
jeblairmordred: where's that error?19:09
pabelangerwhere did you see that?19:09
mordredjeblair: web-debug.log on zuulv3.o.o19:09
jeblair2017-09-26 19:08:53,352 DEBUG zuul.web.LogStreamingHandler: Connecting to finger server ze01:7919:09
jeblairthe replacement ze01 does not know its hostname appropriately19:09
mordredI just checked and hostname --fqdn reports the rightthing19:10
mordredbut hostname does not19:10
jeblairyeah, it's the only executor without a fqdn for /etc/hostname19:10
mordredI agree19:11
mordredwe just built it yesterday - so that makes me think we have a bug in launch-node perhaps?19:11
fungithat's... a strange change in behavior for sure19:11
jeblairi feel like it's always changing :(19:11
pabelanger2017-09-25 20:05:11,942 - util.py[DEBUG]: Running command ['hostname', 'ze01'] with allowed return codes [0] (shell=False, capture=True)19:12
pabelangercloud-init again19:12
mordredUGH19:12
Shrewsgeez, clouds are hard19:12
mordredok.. seriously. as soon as zuulv3 has settled I would like to go on a vendetta march and erradicate cloud-init from our lives as it brings nothing but sorry and misery19:12
mordredof course, that's hard and there are more important things ... so I'll probably do other things instead19:13
fungiyeah, the difference seems to be in /etc/hostname having only the shortname on ze01 vs an fqdn on the other 719:13
mordredyah19:13
fungioh, jeblair already found that19:14
pabelangerI guess /etc/cloud/cloud-init.disabled didn't work19:14
jeblairlaunch-node sets the hostname using an ansible playbook19:14
fungipabelanger: or didn't kick in soon enough?19:14
jeblairanyway, what do we want to do?  manually fix and restart?19:16
jeblairi added this to the non-blocking bugs section19:16
fungiyeah, i can edit /etc/hostname by hand and reboot the server... any safety precautions around running jobs there?19:16
mordredwell - let's wait until the linters job finished19:17
mordredoh! it finished19:17
jeblairmordred: it's not running on v3?19:17
jeblairi need to get food19:17
mordredjeblair: it's totally running on v3 - but it's finished19:17
mordred(this is the tox-linters job for openstack-zuul-jobs)19:17
pabelangersuccess in 8m 25s19:17
jeblairwell it was running :)19:17
jeblairbut it is not running now :)19:18
mordred:)19:18
fungii don't see any ansible processes on ze0119:18
jeblairi'm fine with manually fixing and rebooting ze0119:18
mordredfungi: you can just do service zuul-executor stop19:18
jeblairi'll be back in maybe 20m19:18
mordredfungi: to stop the executor19:18
mordredjeblair: kk19:18
pabelangerif we reboot, won't cloud-init reset hostname?19:18
*** xarses has quit IRC19:18
mordredI am, of course, fine with deleting cloud-init from the servers19:19
fungimordred: sure, and done now. was more pointing out that there weren't running jobs on it (i don't think)19:19
mordredfungi: ++19:19
mordredfungi: I agree with your assessment19:19
pabelangeror even disabling cloud-init services19:19
mordredinfra-root: https://review.openstack.org/#/c/507176/ is ready for +2/+319:20
pabelangerbut ya, after zuulv3 I'm totally working on DIB control plane :D19:20
fungishould we be uninstalling cloud-init packages?19:20
mordredfungi: I tested doing that on a server the other day and it worked fine19:20
fungiit seems to be installed on the other executors19:20
fungiand they aren't exhibiting this behavior19:20
fungiwhich leads me to suspect it won't revert on ze01 once fixed either19:21
Shrewsmordred: there's one +219:21
fungii'm testing that theory now, fwiw19:21
mordredfungi: good test19:21
fungihostname corrected, server rebooted, cloud-init not uninstalled19:21
*** srobert has joined #openstack-infra19:22
*** xarses has joined #openstack-infra19:22
fungihostname reports correct fqdn now19:22
*** srobert has quit IRC19:22
fungiso we possibly have a race of some sort in our launch tooling19:22
mordredk. that'll be good to sort out19:22
*** hashar has joined #openstack-infra19:23
* mordred believes in pabelanger and getting DIB control plane sorted :)19:23
*** srobert has joined #openstack-infra19:23
fungibut anyway, should be safe to proceed. i've started zuul-executor again19:23
mordredwoot19:23
fungier, except it isn't running19:23
mordredfungi: how you did start it?19:24
fungisudo service zuul-executor start19:25
fungithere's no stale pid file19:25
*** andreww has joined #openstack-infra19:25
fungino new entries in its debug log19:25
mordredfungi: that all seems correct to me19:25
fungibut no process in the process table19:25
mordredof course, sometimes systemd gets confused- try systemctl stop zuul-executor19:25
*** sree has joined #openstack-infra19:25
mordredthen starting it again19:25
fungino dice19:26
mordredclarkb: you wanna pull the trigger on https://review.openstack.org/#/c/507176/ ?19:26
pabelangerservice zuul-executor status19:26
pabelangersystemd things it is running19:26
*** xarses has quit IRC19:26
openstackgerritNick Stones-Havas proposed openstack/python-jenkins master: QuietDown requires a POST request  https://review.openstack.org/50765619:27
fungicould there be something about the hostname changing which is causing it to silently bomb?19:27
mordredfungi: mind if I poke for a sec?19:28
clarkbmordred: we are ready for +W?19:28
mordredclarkb: yup!19:28
clarkbok here goes19:28
pabelangerYay19:28
*** ijw has joined #openstack-infra19:28
fungimordred: please poke, my next step was resorting to starting in fireground19:28
fungiforeground19:28
fungithough the fireground might work better19:29
clarkbnot in oregon please19:29
fungitouché19:30
mordredit's filesystem19:30
mordredthe /var/lib/zuul volume is mounted on /opt19:30
*** sree has quit IRC19:30
*** slaweq_ has joined #openstack-infra19:31
funginot that i can tell... did you fix it already?19:31
mordredI don't fully know how to consistently find the traceback - but I happened to see http://paste.openstack.org/show/621975/ when I ran service zuul-executor status19:32
openstackgerritDavid Moreau Simard proposed openstack-infra/zuul-jobs master: Multi-node: Set up connectivity with default OVS bridge  https://review.openstack.org/50455419:32
openstackgerritDavid Moreau Simard proposed openstack-infra/zuul-jobs master: Multi-node: Streamline multi-node-known-hosts  https://review.openstack.org/50462919:32
mordredI ahve fixed it - and have also fixed the /etc/fstab file19:32
openstackgerritDavid Moreau Simard proposed openstack-infra/zuul-jobs master: Append /sbin and /usr/sbin to $PATH for 'ip' commands  https://review.openstack.org/50474319:32
mordredwe should fix that file on all of the executors19:32
*** ijw has quit IRC19:33
*** chenying has quit IRC19:33
pabelangerOh19:33
pabelangerthat would explain it19:33
mordredansible 'ze0*' -m shell -a "sed -i 's/\/opt/\/var\/lib\/zuul/' /etc/fstab"19:34
mordredI have done that - so fstab should be correct now19:34
openstackgerritmelanie witt proposed openstack-infra/devstack-gate master: WIP Add mysqladmin -v extended-status processlist  https://review.openstack.org/50762619:34
*** chenying has joined #openstack-infra19:34
*** rcernin has quit IRC19:34
fungiyeah, i didn't think to go manually set up filesystems after launching19:35
fungithough i wonder why it was working previously19:35
mordredfungi: we had done the filesystem /opt to /var/lib/zuul swap after booting19:35
*** ijw has joined #openstack-infra19:35
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Migrate jobs from zuulv2 to zuulv3  https://review.openstack.org/50717619:35
mordredfungi: we just hadn't persisted that into /etc/fstab19:35
pabelanger\o/19:35
mordredzomg19:36
Shrewsand merged. let's all take a week off starting.... now!19:36
mordredI have rechecked the project-config change to make project-config gate on v319:36
pabelangernext up 50721919:36
fungimordred: thanks, that 'splains it19:36
*** chenying_ has joined #openstack-infra19:37
Shrewsoh right. still have that one19:37
fungimordred: should i go ahead and start the service on it now? still isn't running19:37
jeblairback19:37
mordredShrews, tobiash, tristanC: http://paste.openstack.org/show/621978/ is the traceback that happened with ze01 was misconfigured. although it shouldn't happen again - maybe we should trap that error and send something to the websocket console that says like "couldn't connect to ze01" or something19:38
mordredfungi: oh - yah - safe to start now19:38
fungistarted19:38
fungiand running correctly this time, it seems19:38
mordredinfra-root: ok - journalctl -t zuul-executor would have shown the traceback error (and still does)19:39
mordredwelcome back jeblair19:39
fungibut not systemctl status19:39
fungiodd19:39
*** leong has quit IRC19:39
*** chenying has quit IRC19:40
*** ijw has quit IRC19:40
*** d0ugal has joined #openstack-infra19:41
*** d0ugal has quit IRC19:41
*** d0ugal has joined #openstack-infra19:41
*** armax has joined #openstack-infra19:43
*** Apoorva has quit IRC19:44
*** jamesmcarthur has quit IRC19:46
openstackgerritDavid Shrewsbury proposed openstack-infra/zuul feature/zuulv3: Return stream log message on finger client failure  https://review.openstack.org/50766319:48
Shrewsmordred: ^^^19:49
openstackgerritDavid Shrewsbury proposed openstack-infra/zuul feature/zuulv3: Return stream log message on finger client failure  https://review.openstack.org/50766319:49
Shrewspep8 fail19:49
*** jkilpatr has quit IRC19:50
mordredShrews: does % e work - I thought you had to do str(e)19:50
mordredShrews: neat! it does - TIL19:50
Shrewsmordred: yeah, the %s forces the conversion19:50
mordredgotcha19:51
fungigets coerced to a text type by the formatting19:51
fungithough sometimes you want more direct control over what it's coerced to, it usually "just works" for most data types19:51
* mordred is enjoying watching the job running that's checked the jjb output19:52
jeblairugh, we missed a "dump all the file content into the log" debug line19:52
pabelangerhttps://review.openstack.org/507219/ +1 from jenkins19:53
*** jamesmcarthur has joined #openstack-infra19:53
mordredwoot!19:53
openstackgerritIhar Hrachyshka proposed openstack-infra/devstack-gate master: Switch from lib/neutron-legacy to lib/neutron  https://review.openstack.org/43679819:53
mordredjeblair: you wanna do the honors on https://review.openstack.org/#/c/507219/ ?19:53
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Don't output cat file contents during reconfig  https://review.openstack.org/50766519:54
jeblairmordred: gimme a sec, i'm trying to determine zuul's current state which is difficult with all that output in the logs19:54
mordredjeblair: kk19:55
*** jamesmcarthur has quit IRC19:57
jeblairnot bad19:58
jeblairzuul took 12 seconds to reconfigure after the project-config change with all the content landed19:58
*** jamesmcarthur has joined #openstack-infra19:58
fungiwow!19:58
jeblairthat *includes* the rpc call to fetch the project-config data19:58
jlkhells ya19:58
clarkbwow not bad19:59
jeblairand, um, that also includes logging it in its entirety to the debug log.  :|19:59
fungiapparently19:59
clarkball 200k lines?20:00
jlklines are cheap!20:00
jeblairclarkb: it's efficient! it logged it in a single line!20:00
*** baoli has quit IRC20:00
Shrewsno one should be forced to wait 12 whole seconds20:00
clarkbnice20:00
mnaserhi folks20:00
jeblairmordred: +3 50721920:01
* mnaser just got to ottawa for the openstack meetup20:01
jeblairmnaser: wave to pabelanger for me!20:01
pabelangernice, I'm making my way over in about 50mins20:01
mnasercame to learn about life of an upstreamer :D20:01
pabelangergoing to stream zuulv3 roll out for talk :D20:02
mordredpabelanger: :)20:02
fungithat's be worth seeing!20:02
fungioh, wait...20:02
jeblairmnaser: wow that sounds like some kind of post-apocalyptic gang20:02
mordredpabelanger: sounds like you'll be in a position to be the first to present zuulv3 as a live thing20:02
jeblairlook at all the red!20:02
pabelangermordred: no pressure20:02
mnaseri have an hour or so before this starts20:03
mnaserso um i can try to unbreak puppet20:03
fungi"and here's the new version of the upstream ci system; it's been live for ... *looks at watch* 20 minutes"20:03
mordredmnaser: dude. the puppet jobs are just going to work20:03
mnasermordred lets take bets :D20:03
mordredit's odyssey4me's ansible jobs that I'm worried about :)20:04
*** baoli has joined #openstack-infra20:04
*** pcaruana has quit IRC20:05
jlkthose crazy people wanting to run Ansible things in CI20:07
Shrewsi can't wait to tell folks that we've *finally* replaced jenkins with hudson20:08
jlkಠ_ಠ20:08
mordredinfra-root: so - once this patch lands, I think we should run kick.sh on zuul.o.o since there is a layout removal and we don't really want double-gating lurking in the weeds20:08
fungisounds sane20:09
jeblair++20:09
pabelanger++20:09
jeblaireta 3min20:09
*** dave-mcc_ has joined #openstack-infra20:09
pabelangerwhen we shutdown zuulv2, are we going to abort running jobs? To release nodes back to nodepool.o.o?20:10
openstackgerritMerged openstack-infra/project-config master: Make project-config gating in zuul v3  https://review.openstack.org/50721920:11
prometheanfireno way to drain?20:11
mordred\o/20:11
pabelangerprometheanfire: ya but takes upwards of 3 hours (with tripleo jobs)20:11
* mordred runs kick20:11
prometheanfireheh, true...20:11
*** dave-mccowan has quit IRC20:12
jeblairpabelanger: won't shutting down zuul v2 scheduler stop all running jobs?20:12
pabelangerjeblair: I cannot remember to be honest20:12
prometheanfirejust kill (imo), gertty tag all and recheck is easy20:12
mordredok. zuul v2 kick has run20:13
mordredso the layout change should be applied20:13
mordredrechecking the nodepool patch to make sure20:13
fungithough i suppose the alternative question is whether we could export the check/gate pipelines and then reenqueue on zuulv320:13
*** rlandy is now known as rlandy|afk20:14
pabelangerpost jobs for project-config running now :)20:14
pabelangerhttp://zuulv3.openstack.org/20:14
jeblairfungi: i'll do the export, though we may need some translation to re-enqueue20:14
mordredand check jobs20:14
fungishould we do a #status alert (or notice)?20:14
pabelangerya20:14
fungii guess there was the one a while back20:15
pabelangerwe have a failure20:15
mordredmight still be nice (or just fun) to do a "hold on to your hats, we're flipping the main lever"20:15
mordredBLAST20:15
pabelangerhttp://paste.openstack.org/show/621982/20:15
fungiwas around two hours ago, so probably recent enough20:16
clarkbwe're about to hit 88mph20:16
jeblairmordred: blast for what pabelanger wrote or something else?20:17
pabelangerprimary hostvar doesn't work for non multinode jobs20:17
mordredoh. yah. pabelanger thing20:17
mordredyah- we havea bug in that pre playbook20:17
*** Apoorva has joined #openstack-infra20:18
mordredit'll work for devstack jobs20:18
mordreddo we should we write that primary file even if there is not a primary node defined if there's only one?20:19
jeblairit seems unlikely that any non-devstack non-multinode job would use it20:19
mordredor should we just skip it and switch people that need it to the devstack-singlenode20:20
mordredyah20:20
mordredjust gonna put in a when for now then20:20
openstackgerritMonty Taylor proposed openstack-infra/openstack-zuul-jobs master: Don't write out nodepool primary file on non-primary node  https://review.openstack.org/50766720:20
mordredjeblair, pabelanger: ^^20:20
mordredbah20:21
pabelangermordred: can't we just use {{ nodepool.private_ip }} ?20:21
pabelangererr20:22
mordredoh- you're right20:22
pabelangernodepool.private_ipv420:22
*** florianf has quit IRC20:22
mordredwell - no20:22
mordredwe can't do that20:22
mordredbceaues that won't work for the devstackjobs20:22
mordredwe could do it if primary isn't defined20:22
pabelangerAh right20:22
pabelangerbecause multinode20:22
mordredyah20:22
openstackgerritMonty Taylor proposed openstack-infra/openstack-zuul-jobs master: Don't write out nodepool primary file on non-primary node  https://review.openstack.org/50766720:23
openstackgerritMonty Taylor proposed openstack-infra/openstack-zuul-jobs master: Write private ip of non-primary node  https://review.openstack.org/50766920:23
mordredthere- I did the otherthing as a follow on - so we can add it if we want to20:23
*** mat128 has quit IRC20:23
*** dave-mccowan has joined #openstack-infra20:24
mordredooh. this one is new: http://logs.openstack.org/22/507222/3/check/legacy-bindep-fallback-fedora-26/c6183a9/20:25
*** dave-mcc_ has quit IRC20:25
clarkbwait if we write tge same file with the same content in both places why cant we write it wuth one task?20:26
*** jkilpatr has joined #openstack-infra20:26
jlkit's not the same content20:26
mordredclarkb: becaue the source from which we get the data is different20:26
mordredyah. what jlk said20:26
mordredpabelanger, jeblair: potentially larger problem - we're publishing things to logs.o.o with dir permissions of 70020:27
mordredmordred@static:~$ ls -ltra /srv/static/logs/22/507222/3/check/legacy-bindep-fallback-fedora-26/20:27
mordreddrwx------  5 jenkins jenkins 4096 Sep 26 20:21 c6183a920:27
jlkFORBIDDEN20:27
jeblairmordred: all 3 of runs of those jobs have that20:27
jeblairyeah20:27
pabelangerHmm20:27
*** jamesmcarthur has quit IRC20:27
clarkbwell the filename is the same in both places20:27
clarkbshould have the same content20:27
mordredclarkb: nope. it's not the same content20:27
mordredbecaue in multi-node jobs it's writing theip of the primary host on all of the hosts whetherthey are primary or not20:28
clarkbok then why are both primary_node_private?20:28
jlkin multi-node there is a "primary" node, that's the one for multinode setups. But if there is just one node, ther eis no "primary"20:28
clarkbright we dont write primary node if not multinide20:28
clarkbwe write nothing20:28
mordredgreat. so we should abandon that patch20:28
jlkI see what you're suggesting20:28
mordred(that's why I put it up separate)20:28
clarkbgotcha ok20:28
mordredabandoned20:29
pabelangermordred: I think this is because /home/zuul on fedora is 070020:29
jeblairpretty sure nodepool writes it anyway20:29
pabelangerand we preserve the permissions20:29
pabelanger2017-09-26 20:18:48.655308 | TASK [Copy files from /home/zuul/ on node]20:29
jeblairbut like i said earlier, i'm fine attempting the regression20:29
mordredawesome.20:29
clarkbI thought nodepool only wrote thos files in multinide setup20:29
clarkbit writes some other files always20:30
jeblairi wonder why zuul v2.5 is not hitting this?20:30
pabelangermordred: http://logs.openstack.org/22/507222/3/check/legacy-bindep-fallback-ubuntu-trusty/00eb779/job-output.txt.gz#_2017-09-26_20_23_40_37786920:30
mordredjeblair: because workspace shifted20:30
pabelangerI'm surprised we are syncing /home/zuul to executor20:30
mordredjeblair: in 2.5 workspace and home are not the same thing20:30
mordredin v3 they are the same thing20:30
jeblairclarkb: http://git.openstack.org/cgit/openstack-infra/nodepool/tree/nodepool/nodepool.py#n64820:30
jeblairmordred: gotcha20:30
mordredpabelanger: we're not actually syncing all of /home/zuul - there are rsync filters in place20:31
pabelangerhttp://git.openstack.org/cgit/openstack-infra/openstack-zuul-jobs/tree/playbooks/legacy/bindep-fallback-fedora-26/post.yaml#n4 is the issue20:31
mordredpabelanger: it's part of the migration/translation of the old-style jjb publishers20:31
pabelangerI think we are preserving the permissions on /home/zuul20:31
*** ltomasbo has quit IRC20:31
jlkgrabbing all of it would be... amusing.20:31
clarkbah so it doesnt write subnodes but does the primary always20:31
jlkso whatever creates the logs directory/files should maybe set perms?20:31
*** jpena|off has quit IRC20:32
*** Goneri has joined #openstack-infra20:32
openstackgerritDavid Shrewsbury proposed openstack-infra/zuul feature/zuulv3: We should await ws.send_str()  https://review.openstack.org/50767120:32
*** amoralej|off has quit IRC20:32
mordredyah - I think having the log publisher do it is more sane - we don't really have any valid use cases for publishing logs that people can't read20:32
jeblairwe could 1) fix this in the migration script by adding an extra step after the rsync.  or 2) fix it in the v3 post-logs playbook so all logs are always published world-readable20:32
jeblairjlk, mordred: i think you just favored #2 ^?20:33
mordredwe could also put a chmod 755 /home/zuul in the pre-playbook before we copy git repos over even20:33
mordredjeblair: yes - I favor #220:33
pabelanger#2 seems good step20:33
mordredbecause something else could cause a similar issue20:33
pabelangerya20:33
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Return stream log message on finger client failure  https://review.openstack.org/50766320:33
jlkwhere is this set up?20:34
jeblairerror in log: http://paste.openstack.org/show/621984/20:34
jeblairjlk: project-config/playbooks/base/post-logs20:34
jlkah20:34
*** ltomasbo has joined #openstack-infra20:35
clarkbfeiw this logs publishing perms issue existed in 2.5 as well20:35
jeblairjlk: at least, that's where we would implement #2; you might have been asking something else...20:35
*** baoli_ has joined #openstack-infra20:35
mordredwhich uses roles/upload-logs/ from zuul-jobs20:35
clarkbwe mostly told jobs to chmod first20:35
*** jpena|off has joined #openstack-infra20:35
*** amoralej has joined #openstack-infra20:35
clarkbbut handling that in zuul might be nice20:35
clarkb(at least in shared role)20:35
mordredclarkb: yah- the difference here is that we shifted what WORKSPACE is20:35
jlkwell20:35
pabelangerI thought we removed afs_publisher_target20:35
mordredclarkb: so ratherthan WORKSPACE eing a dir that was created - it's thezuul user's homedir20:35
jeblairi'm going to track down the final job error; assuming someone else will work on logs20:36
jlkI was thinking that whatever is initially creating the directory and files on the Fedora host where the logs go is what should set the permissions20:36
*** ijw has joined #openstack-infra20:36
*** felipemonteiro has quit IRC20:36
*** baoli has quit IRC20:36
pabelangerjlk: that might not work, because we are pulling from /home/zuul, which is 070020:36
mordredyah. although - we could also do a chmod on that very early in the base pre-playbook20:37
jlkare the logs not going into a subdir? just directly in the WORKSPACE?20:38
jeblairjlk: they are in a subdir, but the rsync is rooted at WORKSPACE20:38
jlksure20:38
jlkI'm not seeing where that rsync is that's pulling it from the node20:39
jeblairjlk: pabelanger said it was here: http://git.openstack.org/cgit/openstack-infra/openstack-zuul-jobs/tree/playbooks/legacy/bindep-fallback-fedora-26/post.yaml#n420:39
jlkI see it pushing from the executor, from zuul.executor.log_root20:39
jlkoh20:40
jlkthat's... odd20:40
jeblairjlk: yeah, if you're looking where i told you earlier, that's not it.  i apparently misunderstood your question.20:40
mordredjlk: ansible file module - for making contents of a directory readable -20:40
pabelangerthe more I look at http://git.openstack.org/cgit/openstack-infra/openstack-zuul-jobs/tree/playbooks/legacy/bindep-fallback-fedora-26/post.yaml the more I see an issue20:40
pabelanger.tox/*/log will not be in /home/zuul20:41
jeblairmordred: http://git.openstack.org/cgit/openstack-infra/zuul/tree/.zuul.yaml?h=feature/zuulv3#n80 that sure is a modification of a final job20:41
pabelangerit will be in /home/zuul/src/...20:41
pabelangerso, I think we might have an issue on publishers20:41
mordredpabelanger: yah - that rsync invocation will actually do the correct thing20:41
mordredpabelanger: there is a big comment about it in th e2.5 source and also in the migration script source - it's using some advanced rsync magic from jeblair from the 2.5 days20:42
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Don't output cat file contents during reconfig  https://review.openstack.org/50766520:42
pabelangerokay20:42
mordredjlk: is it smart enough to be able to do files +r and dirs +rx?20:42
jlkmordred: ¯\_(ツ)_/¯  I'd have to read the code.20:42
mordredpabelanger: http://git.openstack.org/cgit/openstack-infra/zuul/tree/zuul/cmd/migrate.py?h=feature/zuulv3#n56620:42
jeblairpabelanger: http://logs.openstack.org/22/507222/3/check/legacy-bindep-fallback-fedora-26/c6183a9/20:42
mordredjlk: butyou know everything!20:42
pabelangerI would just expect http://git.openstack.org/cgit/openstack-infra/openstack-zuul-jobs/tree/playbooks/legacy/bindep-fallback-fedora-26/post.yaml#n58 to be src/{{ zuul.project.canonical_name }}20:43
jeblairpabelanger: check that again, i made it world-readable20:43
jlkI just know where to look :)20:43
jeblairpabelanger: you can verify whether it copies what you expect20:43
*** ijw has quit IRC20:43
jlkIf i understand this job, does anything explicitly in ansible create these directories we're pulling from?20:43
jlkand also, why is it up to this particular job to gather the logs like this? Wasn't a design consideration of zuul that jobs can just plunk content into a known path and it'll automatically get scooped up and published?20:44
mordredaha! I have learned a thing today: u=rwX,g=rX,o=rX20:44
pabelangerhttp://logs.openstack.org/22/507222/3/check/legacy-bindep-fallback-fedora-26/c6183a9/ara/result/d45f3f0f-b093-467f-8860-2eef073e89b1/20:44
pabelangerI don't believe that will get our tox logs20:44
prometheanfirepabelanger: still needed or can be abandoned? https://review.openstack.org/50695120:44
pabelangerbut, happy to wait until we runt he job again20:44
mordredprometheanfire: right. bit you can see that it does20:44
mordredgah20:44
mordredpabelanger:20:44
fungiyeah, X is awesome in newer chmod anyway20:44
prometheanfirelol20:44
mordredoh! I lied - they are not there20:45
pabelangerright20:45
jeblairhttp://logs.openstack.org/22/507222/3/check/legacy-bindep-fallback-fedora-26/c6183a9/tox/ is empty20:45
mordredwell- let's get the perms thing done real quick20:45
pabelangerlets wait until 507667 lands and recheck for tox things20:45
jlkmordred: with that finding, yes file module can handle different perms for files and dirs20:46
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Ensure logs are readable before uploading them  https://review.openstack.org/50767320:46
mordredjlk, jeblair, pabelanger: ^^20:46
jlk(assuming a new enough chmod on the target)20:46
*** jamesmcarthur has joined #openstack-infra20:46
pabelangerI have to relocation to mnaser location, but will be taking laptop with me20:46
mnaserwe have wifi :>20:46
pabelangeryay20:47
jlkIs this legacy job copying content around just a hold over until the job is re-designed to put logs into the expected path?20:48
mordredjlk: yes20:48
jeblairwell20:48
*** jcoufal_ has quit IRC20:49
jeblairto be clear, in our use of v3 jobs still need to fetch logs from workers and put them on the executor for them to be archived.20:49
jlkwhy does it copy things to the executor itself, rather than just to where the expected path is on the worker node so that the standard "scoop and publish" routine does its thing?20:49
jeblairso the "plop it into a location" location is on the executor20:49
jeblairnot on the worker20:49
jlkah, there is no base job that scoops?20:49
*** armax has quit IRC20:50
mordredit does - from the executor20:50
jeblairjlk: nope, only publishes from executor.  no automatic scooping from node.20:50
jlkright.20:50
*** armax has joined #openstack-infra20:50
jlkokay20:50
jeblairyou could *totally* write a v3 base playbook to do that, but we didn't.20:50
jlkin bonny land, we felt teaching folks about the executor was a bit of a stretch, so we were planning to have known paths for artifact upload20:50
*** armax has quit IRC20:50
jlkand making everybody write their own rsync tasks would have been rude :)20:51
jeblairthat would be a really good thing to agree on so that things in zuul-jobs implement the same interface.20:51
*** armax has joined #openstack-infra20:51
mordredwe talked in denver a little bit about trying to rework that some - similar to that ^^ - but I'm not sure we got to consensus on that yet20:51
mordredbutyes - I agree with jeblair20:51
jlknod20:51
*** armax has quit IRC20:51
mordredjeblair: I'm concerned about the tox dir being empty in that job though20:51
jeblairi think andreaf and i (or maybe just andreaf because i've been busy and haven't looked!) will end up with a handy role to collect logs.20:51
*** armax has joined #openstack-infra20:52
*** armax has quit IRC20:52
*** jamesmcarthur has quit IRC20:52
jeblairjlk: at the end of the day though "copy this from a to b" vs "copy this from a to c" isn't that different :)20:52
*** armax has joined #openstack-infra20:52
jlkwell, it's a lot easier to tell a task to redirect it's output to path/over/there20:53
*** armax has quit IRC20:53
jlkthan to rsync things around20:53
jlkIf you can document "anything placed in directory X will be published to logs", it makes it easy to design jobs around that20:53
rm_workerg, anyone else run into issues with cert validation for https://bootstrap.pypa.io/ from xenial machines?20:54
*** ijw has joined #openstack-infra20:54
jlk(at least, that's my hypothesis, I have no data to back it up)20:54
rm_workseems the ca-certificates installed do not validate the cert they're using now20:54
jeblairjlk: that is true, but only in some limited cases (like "run this test runner").  much harder with "archive this service log".  and then you end up with "copy from a to b" followed by "copy from b to c".20:54
*** camunoz has quit IRC20:54
jlkjeblair: where one is exposed to the end user (copy from A to B on the ephemeral host), and the other is a hidden implementation detail of the service provider20:55
*** trown is now known as trown|outtypewww20:55
jlkan end user can easily use some shell to copy desired files into the standard path. Less easily write a proper rsync task in Ansible20:56
mordredjeblair: so- http://logs.openstack.org/36/507636/1/gate/gate-project-config-layout/9ad4622/_zuul_ansible/post_playbook is an old 2.5 generated post playbook and http://git.openstack.org/cgit/openstack-infra/openstack-zuul-jobs/tree/playbooks/legacy/bindep-fallback-fedora-26/post.yaml is the legacy-bindep-fallback job20:56
jlk(Ansible use is another implementation detail that I guess many consumers would not care about)20:56
jeblairjlk: i will categorically say that evaluating this decision in the light of the auto-migrated jobs is not something we should do.20:56
jeblairjlk: if we reopen this conversation, it should only be in the context of actual native v3 jobs.  :)20:56
rm_worknevermind, my VM's clock is off (FML)20:56
jlk100% agreement20:56
jeblairmordred: looking20:57
jeblairmordred: i guess we're looking at the last one, line 56 in the new playbook?20:58
mordredSOGH20:58
mordredred herring20:58
mordredthat job does not use tox20:58
mordredthe test-results macro from jjb is used on that job, which attempts to collect a pile of things20:59
mordredjeblair: ^^20:59
jeblairokay, does anyone know what this job is supposed to output?20:59
mordredit just runs /usr/local/jenkins/slave_scripts/install-distro-packages.sh21:00
mordredso it should just have console output21:00
*** hemna__ has quit IRC21:00
jeblairand it does have console output.21:00
mordredhttp://logs.openstack.org/22/507222/3/check/legacy-bindep-fallback-fedora-26/c6183a9/job-output.txt.gz21:00
jeblairi mean, it's missing the bulk of the job because of the primary thing, but that's in progress21:00
mordredjeblair: http://logs.openstack.org/22/507222/3/check/project-config-nodepool/4b8c33b/21:01
jeblairtox-linters timed out on that21:01
jeblairhttp://logs.openstack.org/67/507667/2/check/tox-linters/773f7e0/21:01
jeblairmordred: what am i looking at there?21:01
mordredoh- nevermind- I was thining that was oneusing migrated content- please ignore link21:02
jeblair2017-09-26 20:26:57.628776 | TASK [tox-siblings : Install any sibling python packages]21:02
jeblair2017-09-26 20:26:58.515040 | ubuntu-xenial | MODULE FAILURE21:02
jeblairmordred: it does include that21:02
*** ihrachys has quit IRC21:03
jlkOne other approach we could take with permissions is to use filesystem ACLs so that anything placed in the executor log path gets the ACLs we want (group/world readable), and use the right flag when syncing from executor to fileserver so that ACLs are preserved. Would save needing an explicit chmod -R call prior to sync.21:03
openstackgerritMerged openstack-infra/zuul-jobs master: Ensure logs are readable before uploading them  https://review.openstack.org/50767321:04
jeblairjlk: worth looking into21:05
jeblairmordred, clarkb: what's the deal with tox-linters on ozj?21:05
jeblairit looks like it's still taking 30+ minutes to run21:05
*** dprince has quit IRC21:06
jeblairi guess we *are* running in parallel, it's just slow?21:07
*** baoli_ has quit IRC21:07
SpamapSjlk: rsync 3.1 has some stuff to do that in-flight I think21:08
*** panda|off has quit IRC21:09
jlvillalSo is there a better place for us to download ipmitool from? Our Ironic job downloads it from sourceforge. But I am seeing this message.21:10
mordredjeblair: ok - I see the bug in the project-config-nodepool job21:10
jlvillalWe're sorry -- the Sourceforge site is currently in Disaster Recovery mode, and currently requires21:10
jlvillalthe use of javascript to function.  Please check back later.21:10
*** camunoz has joined #openstack-infra21:10
jeblairmordred: sorry i didn't know there was a bug21:10
jlvillalCan we store the ipmitool file on some openstack infra site?21:10
mordredjeblair: the module failure you pasted above21:10
mordredjeblair: it's a bug specific to project-config :)21:10
jeblairmordred: ok21:10
jeblairmordred: is there a reason we only do one file at a time with ansible-playbook --syntax check?21:11
jeblairit accepts multiple playbooks at once21:11
jlkjlvillal: why isn't it packaged in a distro?21:11
mordredjeblair: I do not believe there is a reason for that no - also - the earlier run took 6 or 8 minutes iirc- so I do also think we got a very slow node21:11
jlvillaljlk: Our distro is tinycore linux21:12
*** srobert has quit IRC21:12
*** armax has joined #openstack-infra21:12
jlkinfra has methods to mirror the package repo content of many distros, which is typically the way it provides resiliency against failures like this. Random file mirrors aren't necessarily a thing it likes to do21:13
jlvillaljlk: Is there a solution in that answer? Is there something we can do to fix this?21:14
*** slaweq_ has quit IRC21:14
jlksorry I'm not really a part of the infra decision makers, I'm just sharing observations from witnessing a similar request at PTG21:14
jlvillaljlk: Ah, okay. Thanks.21:15
jlkwhich ended with reluctant mirroring of a file, but only with an explicit promise to get the file into a package for a distro21:15
jlvillalGood to know21:16
*** thorst has quit IRC21:16
*** thorst has joined #openstack-infra21:16
*** pabelanger1 has joined #openstack-infra21:16
pabelanger1o/21:16
pabelanger1unable to connect to my IRC proxy from location :(21:17
SpamapSjlvillal: perhaps you should build it from source21:17
*** hashar has quit IRC21:17
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Handle projets without setup.py or valid setup.cfg  https://review.openstack.org/50767821:17
mordredjeblair: ok. that ^^ is the module failure issue from above21:18
jlvillalSpamapS: That is what we are doing...  https://github.com/openstack/ironic-python-agent/blob/master/imagebuild/tinyipa/build-tinyipa.sh#L68-L7021:18
mordredjeblair: (project-config is special and does not have a setup.py file)21:18
jlvillalSpamapS: https://github.com/openstack/ironic-python-agent/blob/master/imagebuild/tinyipa/build-tinyipa.sh#L148-L15321:19
openstackgerritJames E. Blair proposed openstack-infra/openstack-zuul-jobs master: Check all playbooks simultaneously  https://review.openstack.org/50768021:19
mordredjeblair: also - yay - it seems the readable logs thing helped21:19
openstackgerritJames E. Blair proposed openstack-infra/openstack-zuul-jobs master: Run ansible-list on all playbooks simultaneously  https://review.openstack.org/50768121:20
jlvillalAs an FYI, our failure getting ipmitool tarball: http://logs.openstack.org/72/507272/3/check/gate-tempest-dsvm-ironic-lib-wholedisk-agent_ipmitool-ubuntu-xenial/6d8f17d/logs/devstacklog.txt.gz#_2017-09-26_18_01_22_73821:20
jeblairmordred, pabelanger1: 507680 works locally.  i haven't tested 50768121:20
mordredjeblair: I believe if we're using -P for parallel we need to use -n or -L or else it'll all just be sent to one of the parallel processes21:21
*** thorst has quit IRC21:21
*** tosky has joined #openstack-infra21:21
jeblairmordred: are you sure it doesn't divide it up appropriately?21:21
mordredjeblair: the man page says you're supposed to use -n or -L ...21:22
jeblairmordred: okay, i'll drop -P21:22
mordred"Use the -n option or the -L option with -P; otherwise chances are that only one exec will be  done."21:22
*** wolverineav has quit IRC21:22
mordredjeblair: kk21:22
jeblairmordred: well, i get the same behavior in both cases21:24
openstackgerritTin Lam proposed openstack-infra/project-config master: Add nfs test gate to OpenStack-Helm  https://review.openstack.org/50768221:24
jeblairit basically makes -P a noop.  so it's not an error, it's just not useful.21:24
mordredyah21:24
mordredjeblair: but if the all-in-one makes it happy - that's great by me!21:24
jlkSpamapS: ah, so  you would have an option of --chmod=ugo=rwX in the synchronize task rsync flags21:25
openstackgerritJames E. Blair proposed openstack-infra/openstack-zuul-jobs master: Check all playbooks simultaneously  https://review.openstack.org/50768021:25
openstackgerritJames E. Blair proposed openstack-infra/openstack-zuul-jobs master: Run ansible-list on all playbooks simultaneously  https://review.openstack.org/50768121:25
jeblairmordred: there are some post failures on 50767821:26
jeblairmordred: perhaps the chmod thing was broken21:26
mordredthe chmod thing worked for other patches - I'll investigate21:27
jeblairhad it run yet? i thought it only recently landed21:29
mordredyah - may have been race condition21:30
mordredworth rechecking for sure21:30
*** hashar has joined #openstack-infra21:30
jeblairthere are lots of post_21:30
jeblairfailures on the page21:30
mordredjeblair: nope. still getting failure21:30
jeblair2017-09-26 21:30:28.432104 | logs.openstack.org |   "msg": "There was an issue creating /var/lib/zuul as requested: [Errno 13] Permission denied: '/var/lib/zuul'",21:30
jeblair2017-09-26 21:30:28.432242 | logs.openstack.org |   "path": "/var/lib/zuul/builds/7d566c9729284f4b89522bbf8fedb3f5/work/logs/",21:30
*** eharney has quit IRC21:30
*** panda has joined #openstack-infra21:30
jeblairmordred: task running on wrong host21:30
mordredwell that certainly doesn't seem right21:30
jlkwait21:30
mordredAH21:31
jlkyou want ansible-lint to run on all the playbooks at once, right?21:31
jlkinstead of serially one after the other21:31
jlkor do you want multiple runs of ansible-lint to happen, one per playbook found?21:31
jeblairjlk: to be honest, i want the job to finish quickly and don't care.  :)21:31
jlkjeblair: your change removed the -n1, but didn't tell xargs to do anything in parallel, so I'm confused21:32
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: chmod the files on the executor  https://review.openstack.org/50768621:32
jeblairjlk: the job was previously serialized, and very very slow, so folks made it run $nproc at once, at which point it was only slow.  i'm trying to do all at once and have it be fast.21:32
jlkoh wait.21:32
jeblairjlk: that will cause all the files to be put on the commandline together21:32
jlkI see what the default is, stack them all into a command21:32
jeblairso, very long command.  so long, xargs actually has to run it twice21:32
jeblairmordred: we'll need to force-merge that21:33
mordredjeblair: ^^ that patch - and I apologize we will need to force-merge that21:33
jlknod21:33
mordredjeblair: jinx21:33
jeblairmordred: i'll push the buttons21:33
jlkyou can skip xargs all together though21:33
mordredjeblair: cool. thank you21:33
jeblairjlk: even better!21:33
jlkfind -exec <whatever> {} +21:33
jlkthe + groups all the output, but I wonder if that will fail if it's too many things21:34
mordredjlk: will that run exec more than once if the argument list would be too long?21:34
openstackgerritMerged openstack-infra/zuul-jobs master: chmod the files on the executor  https://review.openstack.org/50768621:34
jlkthat's what I"m about to check21:34
mordredneat21:34
mordredwe're gonna have this lint job OPTIMIZED21:34
jeblairmordred: well, it's giving us plenty of time in which to do so :|21:34
jlkyes it appears it will21:35
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Handle projects without setup.py or valid setup.cfg  https://review.openstack.org/50767821:35
jeblairjlk: so we need xargs to split the long line?21:35
jlkno, the man page says it builds the command(s) like xarg does, presumably breaking it into multiple commands21:35
jlkif the arg list grows beyond bounds21:36
jeblairjlk: oh, heh, i thought you were saying it will fail.  but you're saying it will work.21:36
jlkhuh, "This variant of -exec21:36
jlk              always returns true."21:36
jlkthat may not be what we want :/21:36
jlknow I'm confused21:37
*** aeng has joined #openstack-infra21:37
jlkthat paragraph is confusing. If the exec command fails, you get a non-zero exit, which is what we want21:38
mordredjeblair: 678 is trending green this time21:39
mordredand 680 is WAY quicker21:40
mordredjeblair: woot. 680 finished in 2 minutes21:41
mordredjeblair: I find that an acceptable timeframe :)21:41
jeblairmordred: ++21:41
fungithat _is_ way quicker21:41
jlkhttp://paste.openstack.org/show/621989/21:42
jlk^^ that should do what you want, avoiding xargs21:42
mordredfungi: mind +Aing https://review.openstack.org/#/c/507680 ?21:42
clarkbwhat is/was problem with xargs?21:42
jlkit's... not necessary?21:42
clarkbah21:42
*** r-daneel has quit IRC21:43
jlkoriginally it's use was improperly limiting things to one execution per file21:43
jeblairjlk: cool, want to convert that into a patch?21:43
jlksure can!21:43
jeblairjlk: it's self testing ;)21:43
*** armax has quit IRC21:43
jeblairthough, obviously, not the bit about always returning 021:43
jlkheh21:43
jlkyeah I'll do that once 680 merges21:43
jlkor make mine depend on it21:44
*** d0ugal has quit IRC21:44
*** claudiub has joined #openstack-infra21:44
openstackgerritNick Stones-Havas proposed openstack/python-jenkins master: QuietDown requires a POST request  https://review.openstack.org/50768821:44
jeblairmordred: if your stack is clear, should we look at the final problem?  (/me evades glare from conan doyle)21:45
mordredfungi, clarkb: https://review.openstack.org/#/c/507681 also when you get a sec (it's trending green and I expect it to pas)21:45
mordredjeblair: yes! what's the final problem?21:45
jeblairmordred: http://paste.openstack.org/show/621984/21:45
jeblairmordred: caused by this definition: http://git.openstack.org/cgit/openstack-infra/zuul/tree/.zuul.yaml?h=feature/zuulv3#n8021:45
jeblairpabelanger1: ^ of interest to you21:46
mordredjeblair: ah! that. so - I think we can just remove that var21:46
jeblairmordred: var was for earlier version of job?21:47
mordredjeblair: it is no longer necessary - publish-openstack-python-docs-infra publishes to infra/{short_project_name} already21:47
*** hashar has quit IRC21:47
mordredyah21:47
jeblairmordred: cool, i'll nab it21:47
pabelanger1reading21:47
mordredpabelanger1: it's the docs-infra job passing afs_publisher_target thing ... we can just remove the var21:48
pabelanger1k21:48
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Remove var override for docs publish job  https://review.openstack.org/50769021:48
mordredjeblair: good - I was worried we'd found an deeper issue :)21:48
mordredjeblair: why did that only just now trigger?21:48
pabelanger1+221:48
jeblairmordred: nope, we just exercised a thing that was only exercised in tests21:48
*** ijw has quit IRC21:49
jeblairmordred: probably didn't just trigger now, probably just went unnoticed21:49
jeblairi'm trolling logs for issues21:49
mordredah- gotcha21:49
mordred*phew*21:49
mordredI'm WAY more ok with that21:49
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Check all playbooks simultaneously  https://review.openstack.org/50768021:49
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Don't write out nodepool primary file on non-primary node  https://review.openstack.org/50766721:49
jeblairbtw, i am keeping the etherpad up to date21:50
mordredcool - thank you21:50
*** ijw has joined #openstack-infra21:50
*** pabelanger1 has quit IRC21:50
*** pabelanger1 has joined #openstack-infra21:50
openstackgerritJesse Keating proposed openstack-infra/openstack-zuul-jobs master: Remove spurrious use of xargs  https://review.openstack.org/50769121:50
*** pabelanger has quit IRC21:50
*** pabelanger1 is now known as pabelanger21:50
Shrewsmordred: while we wait, this is cleanup from the finger thing: https://review.openstack.org/50767121:51
mordredjeblair: incidentally- zuul is still running check jobs for 507686 even though it has merged - it's an edge case, since we normally shouldn't be merging things out from under zuul - but thought I'd point it out21:51
*** pabelanger_ has joined #openstack-infra21:51
mordredShrews: +221:51
jeblairmordred: ya, that's something that should be fixed (it likely represents a minor bug with pipeline requirements)21:52
mordredShrews, clarkb, fungi: can haz +3 on https://review.openstack.org/#/c/507678/ please?21:52
*** kjackal_ has quit IRC21:52
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Run ansible-list on all playbooks simultaneously  https://review.openstack.org/50768121:53
jeblairmordred: oh, is the module failure blocking the nodepool change?21:53
Shrewsmordred: i like the existential comment in that one21:53
mordredjeblair: yes21:54
mordredjeblair: because of the oddness of tox and project-config21:54
jeblairgotcha21:54
mordredjeblair: we got ALL the good edge conditions with this batch21:54
jeblairmordred: promise?21:54
fungimordred: it's okay that we're skipping that for projects which may have a setup.py but aren't pbr/distutils2-based?21:55
*** yamahata has joined #openstack-infra21:55
fungi(and so no setup.cfg)21:55
mordredfungi: yah - it's a new fancy feature in v3 - if someone in our zuul isn't using pbr, then they don't get the shiny feature21:55
pabelangerHmm, error with bindep fallback21:55
fungik21:56
fungipabelanger: i believe that's what 507678 was trying to address21:56
mordredfungi: skipping just reverts back to the way tox jobs have worked all this time so far21:56
pabelangerfungi: looking21:56
mordredyes. it is21:56
fungimordred: thanks, makes sense21:56
pabelangeris bindep-fallback even valid in zuulv3 world?21:57
openstackgerritNick Stones-Havas proposed openstack/python-jenkins master: QuietDown requires a POST request  https://review.openstack.org/50768821:57
mordredpabelanger: the job? it needs to be rewritten for sure21:58
mordredpabelanger: rather than calling that script, it should call the bindep role pointing to the contents21:58
*** Goneri has quit IRC21:58
pabelangermordred: right21:58
*** esberglu has quit IRC21:58
mordredpabelanger: but we just went with the auto-migrated version of it for today - which is good - we found some bugs!21:59
openstackgerritJesse Keating proposed openstack-infra/openstack-zuul-jobs master: Remove spurrious use of xargs  https://review.openstack.org/50769122:00
pabelangermordred: ya, I'm trying to see if http://git.openstack.org/cgit/openstack-infra/openstack-zuul-jobs/tree/playbooks/legacy/bindep-fallback-debian-jessie/run.yaml#n46 exists any more22:00
pabelangermordred: atleast for the zuul user22:00
fungitricky bit about the bindep fallback job, is that it should probably have been two jobs22:01
pabelangerOh, I think i see the issue22:02
pabelangerfungi: ya, I think you are right22:02
pabelangerbecause, project-config path to jenkins/data is not correct now22:02
fungione to test that proposed changes to the fallback list work with the version of bindep installed on various images, the other to test that changes to bindep work with the version of the fallback list baked into all our images22:02
pabelangerexport PACKAGES=~zuul/src/git.openstack.org/openstack-infra/project-conifg/jenkins/data/bindep-fallback.txt22:03
*** jklare has quit IRC22:03
pabelangerthat is the fix22:03
mordredpabelanger: no - it's fine22:03
pabelangerokay22:03
mordredpabelanger: it's an auto-migrated job - so it's running zuul-cloner at the top - which is cloning project-config into .22:03
mordredsee: http://git.openstack.org/cgit/openstack-infra/openstack-zuul-jobs/tree/playbooks/legacy/bindep-fallback-debian-jessie/run.yaml#n2622:04
*** bobh has quit IRC22:04
pabelangermordred: oh, so legacy jobs don't have ~/src/git.o.o22:04
mordredthey DO22:04
mordredbut they don't know anything about that yet22:04
fungiso the one overloaded job optionally either tested by installing the provided bindep source via zuul-cloner and calling it against the default fallback list present on the node, or by not installing bindep and invoking the preinstalled bindep against the zuul-cloner checked out copy of the fallback list22:04
pabelangerokay, I'll stand down for now22:04
mordredso the zuul-cloner script we install clones from ~/src/git.o.o to wherever the job thinks its source repos should be22:05
mordredpabelanger: I mean - keep looking for issues for sure :)22:05
mordredpabelanger: this is a big beastie after all22:05
fungii'd be fine if we just declare the fallback list and bindep frozen for a little bit while we rework the job22:05
fungithat one at least is on us, since it22:06
fungi's infra projects on all sides22:06
andreafjeblair, mordred: I made a handy role called save_file which does something like what save_file would do on d-g https://review.openstack.org/#/c/506835/7/roles/save-file/tasks/main.yaml22:06
jlkwtaf22:07
jlktox seems to be eating my {} argument22:07
mordredfungi: yah- also - it would be great to make bindep-fallback go away too - so freezing the fallback list is certainly fine by me22:07
andreafjeblair, mordred: however to use it I need to use include_role e.g. https://review.openstack.org/#/c/506835/7/roles/fetch-tempest-logs/tasks/main.yaml22:07
pabelangerI'm going to autohold a bindep-falback job to see inspect node22:07
mordredUGH22:07
jlk{} is turning into :22:07
mordredhit race condition bug in zuul on 50767822:07
mordred:(22:07
jeblairmordred: ya; just rechecked it22:08
andreafjeblair, mordred: which breaks the ansible callback zuul_json22:08
jeblairandreaf: why is include_role needed for that?22:08
mordredandreaf: also - I don't think you can use with_items with include_role either22:08
pabelangerShrews: how can I list autohold jobs?22:09
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Remove var override for docs publish job  https://review.openstack.org/50769022:09
Shrewspabelanger: you can't right now22:09
pabelangerShrews: kk22:09
jeblairandreaf: oh, because you want to do that from within a role i guess?  (as opposed to within the tempest post playbook directly?)22:09
andreafmordred: yes you can - it works fine (at least on my local machine)22:09
mordredandreaf: oh neat22:10
andreafjeblair: yeah I wanted to have a role that encapulates that22:10
openstackgerritJesse Keating proposed openstack-infra/openstack-zuul-jobs master: Remove spurrious use of xargs  https://review.openstack.org/50769122:10
jeblairandreaf: makes sense22:10
andreafjeblair: but I guess I could move it to the play22:10
andreafjeblair: at least until zuul can handle it22:10
jeblairShrews: autohold list would be super helpful if you have a sec22:10
*** jklare has joined #openstack-infra22:11
andreafif I don't use the include role though, I cannot use the with_items form which I though was quite nice and readable... but ok that's just a minor thing22:11
*** esberglu has joined #openstack-infra22:12
jeblairandreaf: yeah, i think what you have there is a good thing once we fix zuul22:12
mordredandreaf: yah- we do want to fix the include_role bug22:12
*** sdague has quit IRC22:13
*** camunoz has quit IRC22:14
*** jaypipes has quit IRC22:16
andreafjeblair: I wonder if the latest ps on  https://review.openstack.org/#/c/506835 still triggers the permission issue on the archive - do you have time to grab the zuul log for that for me again? or is there a way I can see that log otherwise22:16
*** esberglu has quit IRC22:16
pabelangerokay, I have a bindep-fallback node held22:17
jlkhaha, my version seems to shave 6 seconds off the run.yaml bit. 49 to 43 seconds. I think that's well within the "noise" range.22:17
mordredpabelanger: ok. I stand corrected - that is definitely failing22:17
mordredpabelanger: which node you have? can I look too?22:17
pabelanger199.204.45.8122:17
pabelangerwe cloned into /home/zuul/project-config, but script expects /home/zuul22:18
mordredoh goodie22:18
pabelangerhttp://logs.openstack.org/22/507222/3/check/legacy-bindep-fallback-ubuntu-trusty/0db3741/job-output.txt.gz#_2017-09-26_22_15_30_36276022:18
pabelangerI think that is why22:18
pabelangerwe need training slash?22:18
jeblairandreaf: i only see the zuul_json issues for that build; no permissions errors.22:19
pabelangermordred: http://git.openstack.org/cgit/openstack-infra/openstack-zuul-jobs/tree/playbooks/legacy/bindep-fallback-ubuntu-trusty/run.yaml#n24 is clonemap22:19
pabelangerso I _think_ trailing slash might do it22:19
mordredwe can't do that though22:19
pabelangerya, might get messy22:20
mordredthis is auto-migrated content - we need to fix the zuul-cloner shim if it's not doing what the previous macros expect22:20
pabelangerya22:20
mordredjeblair: ^^ have a sec to turn your eyes to this?22:20
pabelangerwhere is our zuul-cloner shim?22:21
jeblairShrews: ^ also22:21
jeblairpabelanger: in zuul22:21
Shrewspabelanger: zuul-jobs22:21
jeblairer that22:21
pabelangerty22:21
mordredpabelanger: http://logs.openstack.org/03/506803/1/check/gate-bindep-bindep-fallback-ubuntu-trusty/69c2538/_zuul_ansible/scripts/02-e3e05dc31f7d466fbb9f8aa7dd60d582.sh22:21
mordredis the v2.5 script22:22
pabelangermordred: thanks22:22
mordredwhich did this: "openstack-infra/bindep -> /home/jenkins/workspace/gate-bindep-bindep-fallback-ubuntu-trusty"22:22
mordredhttp://logs.openstack.org/03/506803/1/check/gate-bindep-bindep-fallback-ubuntu-trusty/69c2538/console.html#_2017-09-22_23_05_06_03163322:23
Shrewspabelanger: are you looking at the cloner issue? i can after dinner things22:23
pabelangerwait, /home/jenkins?22:23
pabelangerShrews: I'm looking, but learning how it works22:23
mordredyah- the v2.5 generated versions of the same thing22:24
mordredrun this playbook: http://logs.openstack.org/03/506803/1/check/gate-bindep-bindep-fallback-ubuntu-trusty/69c2538/_zuul_ansible/playbook22:24
pabelangerOh, ya22:24
pabelangerthat makes sense22:24
mordredwhich sets WORKSPACE to /home/jenkins/workspace/gate-bindep-bindep-fallback-ubuntu-trusty and then chdir's to that before each script22:24
pabelanger2017-09-26 22:15:30.362760 | ubuntu-trusty | cp -al /home/zuul/src/git.openstack.org/openstack-infra/project-config /home/zuul22:24
pabelangerthat is what we did22:24
pabelangerwhich creates /home/zuul/project-config22:24
*** Sukhdev has joined #openstack-infra22:24
mordredShrews: so - tl;dr - I believe dest: . needs to clone into the current dir instead of one dir down22:24
pabelangerfor zuulv322:24
mordredpabelanger: yah22:24
*** mat128 has joined #openstack-infra22:25
openstackgerritMerged openstack-infra/zuul-jobs master: Handle projects without setup.py or valid setup.cfg  https://review.openstack.org/50767822:25
mordredoh good22:25
mordredinfra-root: I believe this is a showstopper bug - and it's late in the day22:26
jeblairi suspect this may be a difference between "cp" and "git clone"22:26
mordredor, it's late in the day for me I suppose22:26
jeblairso the fix may not be in clone mapper expansion, but rather in constructing the cp command22:26
mordredyah22:26
pabelangerya22:27
mnaseri know this isn't the best time to ask but would we be okay with disabling ovh-gra1 till git.openstack.org timeouts disappear?  we've had a gate blocking puppet change fail to pass the gate 4 times today because one of the jobs land in ovh-gra122:27
jeblairmordred: whatcha wanna do?  rollback?  leave it and hope no one starts using it?  leave it and shut it down?22:28
mordredjeblair: just get drunk and pass out?22:28
andreafjeblair: cool thanks - so if I move it to the play it should be fine22:29
mordredjeblair: honestly, I think we should just leave it - other than current inability to respond to mnaser's request22:29
*** mat128 has quit IRC22:29
mordredjeblair: the jobs themselves are landed, so people can't add .zuul.yaml files wiht conflicting job names22:29
*** jascott1 has quit IRC22:29
*** kgiusti has left #openstack-infra22:30
jeblairmordred: ok.  do you want to send today's status update email?22:30
mordredjeblair: but - I tihnk we could also rollback the gate-project-config-on-v3 work depending on what people think about the chances that we'll get that zuul-cloner bug fixed in a reasonable enough time to be able to land nodepool quota changes as needed22:30
*** tpsilva has quit IRC22:31
mordredjeblair: you're online for longer than I am today - what do you think about the current project-config state22:31
mordredand sure - I can do the status update email22:31
*** jascott1 has joined #openstack-infra22:32
jeblairmordred: we *might* have zuul-cloner fixed by my eod?22:32
Shrewsso, if i read correctly, should z-c just "cp $SRC/* ." when destination is "." instead of "cp $SRC ." ?  because that is a very easy fix22:32
pabelangerwell, as long as no project has a src directory in tree, otherwise, we already have a src folder in /home/zuul22:32
pabelangermaybe we should create a workspace directory in /home/zuul/workspace for legacy?22:33
Shrewsoh, the clonemapper stuff might make that more difficult though22:33
jeblairShrews: put simply, the "cp" command should do whatever the "git clone" command did before.22:34
jeblairShrews: i'm not sure that quite matches up with anything else said so far.22:35
Shrewsok, well i'm not sure what the difference is offhand, and i'd like to finish dinner. so maybe best to hold off until tomorrow, unless someone else has a handle on it22:35
openstackgerritJesse Keating proposed openstack-infra/openstack-zuul-jobs master: Remove spurious use of xargs  https://review.openstack.org/50769122:35
jeblairShrews: ya, go finish dinner.  i'm trying to get a handle on it.  i'll let you know how far i get.  :)22:36
Shrewsk22:36
fungithere are two patches to revert at this stage if we want to rollback and try again tomorrow?22:36
*** dave-mccowan has quit IRC22:36
jeblairfungi: 3 i think?22:36
jeblairfungi: though mordred is arguing that if we can accept project-config being broken we can leave it.22:37
pabelangerI am dropping for the next 90mins, I'll be able to come back after that22:37
*** Guest61942 has joined #openstack-infra22:37
openstackgerritJesse Keating proposed openstack-infra/openstack-zuul-jobs master: Remove gratuitous use of xargs  https://review.openstack.org/50769122:37
mordredinfra-root: https://etherpad.openstack.org/p/3ep1A1uNfz there's current draft of status email22:39
jeblairmordred: lgtm22:40
mordredfungi: the biggest issue currently is that it's not possible to land patches to nodepool's config - but the projects being registered with v3 and the jobs landed in openstack-zuul-jobs aren't really destructure and should be able to be left in place22:40
*** lbragstad has quit IRC22:40
mordredfungi: I think if we do choose to roll anything back it would want to be the make-project-config-voting-in-v3 piece22:41
*** pabelanger has quit IRC22:41
mordredwhich might actually need to be force-merged anyway since the issue is that we can't merge patchesto project-config22:41
mordredoh - WELL22:41
mordredjeblair, fungi: we could force-merge a patch to project-config to remove the legacy-bindep-fallback patches from the projet pipeline config - that way if the decision is made to roll back v3 gating it's just a simple revert and then a kick.sh to let zuulv2 know to take over - and we won't be stuck on landing patches22:43
fungiso can nodepool config adjustments be force-merged at least, if we decide we need to turn off a particular provider/region?22:43
mordredbut if we do that we should not roll anything forward without re-adding those jobs and verifying that they work22:43
mordredfungi: yah- totally22:43
mordredso - there's a few options on thetable depending on how people are feeling as they need to EOD22:44
fungiwhile it's not great, i'm okay with the supposition that manually gerrit-submitting very trivial emergency changes to project-config is the workaround22:45
fungiif it allows us to get further tomorrow without having to catch back up to the bits we unwound otherwise22:45
mordredyah- I thnk it sets us up to be further along in the morning to take the next steps22:46
*** threestrands has joined #openstack-infra22:54
*** threestrands has quit IRC22:54
*** threestrands has joined #openstack-infra22:54
openstackgerritAndrea Frittoli proposed openstack-infra/devstack-gate master: Add post step to Tempest base job  https://review.openstack.org/50683522:55
andreafjeblair: moved save-file invocation to the post-tempest play ^^^ hopefully it will pass CI this time22:56
openstackgerritMonty Taylor proposed openstack-infra/project-config master: Migrate zuul v2 project pipelines to v3  https://review.openstack.org/50717922:56
mordredthat's just rebased - also marked it WIP ^^22:56
*** claudiub has quit IRC22:58
mordredinfra-root: I'm going to send status email now unless anyone has any objections22:58
fungilgtm22:59
*** rbrndt has quit IRC23:01
*** rbrndt has joined #openstack-infra23:01
*** rbrndt has quit IRC23:01
*** thorst has joined #openstack-infra23:04
*** yamamoto has joined #openstack-infra23:05
* mordred afks for a bit - will check back in in a couple of hours23:06
openstackgerritJames E. Blair proposed openstack-infra/zuul-jobs master: Always create dest dir in zuul-cloner shim  https://review.openstack.org/50770423:07
*** jascott1 has quit IRC23:07
jeblairShrews: ^ hopefully the commit message explains my state of mind.  in the tests i could think of to run locally, that makes z-c and z-c-s do the same things.23:08
*** thorst has quit IRC23:08
mordredjeblair: cool23:09
* mordred hasn't QUITE afk'd yet23:09
mordredjeblair, Shrews: pabelanger_ brought up the possibility of a conflict if someone has a src dir in their repo - I think we should handle those as a great opportunity to get someone migrated off of legacy jobs23:10
mordredthe vast majority of openstack python repos should not have src/ dirs23:11
mordredok. NOW I afk23:11
*** pabelanger has joined #openstack-infra23:16
*** hongbin has quit IRC23:22
*** jascott1 has joined #openstack-infra23:25
*** bobh has joined #openstack-infra23:26
*** sree has joined #openstack-infra23:26
*** jamesmcarthur has joined #openstack-infra23:27
*** jascott1 has quit IRC23:29
*** sree has quit IRC23:30
*** jcoufal has joined #openstack-infra23:31
*** jamesmcarthur has quit IRC23:32
*** rlandy|afk is now known as rlandy23:33
*** andreww has quit IRC23:33
*** tosky has quit IRC23:35
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Fix race in test_in_repo_branch  https://review.openstack.org/50770823:36
jeblairmordred: that *might* fix that race we keep hitting; i'm running it locally in a loop23:36
clarkbis projct config broken due to the zuul cloner thing?23:37
clarkbI'll review the two changes above shortl23:39
jeblairclarkb: yes; current thinking is force-merge patches if necessary23:39
*** jcoufal has quit IRC23:40
jeblair#status log project-config is unable to merge changes due to problems found during zuul v3 migration.  for the time being, if any emergency changes are needed (eg, nodepool config), please discuss in #openstack-infra and force-merge them.23:40
openstackstatusjeblair: finished logging23:40
jeblairinfra-root: ^23:40
jeblairclarkb: but yeah, if the zuul-cloner patch is correct, we might be able to just fix that23:41
*** bobh has quit IRC23:46
*** Swami has quit IRC23:46
*** jdandrea_ has joined #openstack-infra23:50
clarkbjeblair: the commit message on the z-c shim fix makes esnse to me23:50
fungitaking a look too23:51
*** pvaneck has quit IRC23:51
clarkbseparately concerned that blindly using cp -l might fail across fs boundaries23:53
clarkb(does that gracefully degreade to proper copy or failure?)23:53
*** jascott1 has joined #openstack-infra23:56
bkeromordred: Here you go :) https://www.facebook.com/notes/lcdfans/special-machine-t70-cpu-i7-7700hqwith-ips/1882070528726717/?qid=6433062313884965485&mf_story_key=361676640162146357423:56
clarkbcp: cannot create hard link '/tank/data/foo' to 'foo': Invalid cross-device link <- ya its an error23:57
clarkbso we may need to fix that too23:57
*** SumitNaiksatam has quit IRC23:58
clarkboh right the shim happens after any caching so should be on the same device23:59

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