Saturday, 2017-10-21

*** slaweq has quit IRC00:00
fungimnaser: yup, thanks00:00
*** Apoorva_ has joined #openstack-infra00:00
*** hongbin has quit IRC00:01
*** jamesmcarthur has joined #openstack-infra00:02
*** Apoorva has quit IRC00:03
*** Apoorva_ has quit IRC00:04
*** slaweq has joined #openstack-infra00:05
*** xarses has joined #openstack-infra00:07
*** jamesmcarthur has quit IRC00:07
clarkbbah getting the python version right with pip and doing pyyaml installs is more painful across all the distros than I was hoping00:12
fungidown to 27 executor processes on ze0100:12
clarkbfungi: make sure they aren't all just ssh agents00:12
clarkbyou can kill thsose00:12
clarkb(once zuul isn't running anymore)00:13
fungioh, cool checking. thanks00:13
pabelangerfungi: I wonder if they are no longer aborting, and just finishing / timing out per usually00:13
*** salv-orlando has joined #openstack-infra00:13
*** bobh has joined #openstack-infra00:13
fungibut yeah, the main zuul-executor process (and a fork i think) are still running00:13
fungii'll kill the ssh sessions and see if everything wraps up00:14
clarkbya there are still bwrap'd jobs going00:14
fungiyeah, 20 still with ssh in the process string00:14
fungiit's safe to just kill those? or not until the parent terminates?00:15
clarkbI've waited on those00:15
fungik00:15
clarkbbut if executor sticks around after everything else but ssh-agent is gone I've killed the executors00:15
funginoted00:16
*** salv-orlando has quit IRC00:18
clarkbbah its like whackamole00:18
clarkbfix one thing with pip 10 and another thing falls out00:18
clarkbI'm calling it, I'll be around to answer ze questions but closing laptop00:18
fungithanks! i've probably got it from this point anywa00:19
fungiy00:19
*** xarses has quit IRC00:20
*** mriedem1 has joined #openstack-infra00:21
*** mriedem has quit IRC00:22
*** gyee has quit IRC00:36
*** slaweq has quit IRC00:38
*** markvoelker_ has quit IRC00:38
*** bobh has quit IRC00:38
*** mriedem1 is now known as mriedem00:40
*** andreas_s has joined #openstack-infra00:43
jeblairclarkb, fungi: template test failure is a missing git add, sorry00:45
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Ignore duplicate project-template definitions  https://review.openstack.org/51385300:45
*** andreas_s has quit IRC00:47
*** slaweq has joined #openstack-infra00:47
openstackgerritMohammed Naser proposed openstack-infra/zuul feature/zuulv3: Refactor Zuul job to use lightweight Vue components  https://review.openstack.org/51388500:48
openstackgerritMohammed Naser proposed openstack-infra/zuul feature/zuulv3: Switch change list to use Vue.js  https://review.openstack.org/51388600:48
fungistill 4 bwrap processes on ze0100:49
fungiguess i'll give them a little more time00:49
*** thorst has joined #openstack-infra00:49
*** thorst has quit IRC00:51
*** gmann_afk is now known as gmann00:54
*** ijw has quit IRC01:01
*** dhajare has joined #openstack-infra01:05
*** armax has quit IRC01:11
*** edmondsw has joined #openstack-infra01:12
*** salv-orlando has joined #openstack-infra01:14
*** owalsh_ has joined #openstack-infra01:15
*** edmondsw has quit IRC01:16
*** salv-orlando has quit IRC01:19
*** owalsh has quit IRC01:19
*** slaweq has quit IRC01:19
fungithey finally wrapped up01:22
fungii had to kill the executor process and ssh processes once there were no bwrap processes left01:22
*** markvoelker has joined #openstack-infra01:23
fungilooks like it's starting to pick up work, so i'll move on to the rest01:23
*** markvoelker has quit IRC01:27
*** bobh has joined #openstack-infra01:28
*** armax has joined #openstack-infra01:29
*** slaweq has joined #openstack-infra01:30
*** markvoelker has joined #openstack-infra01:32
*** bobh has quit IRC01:32
*** dhajare has quit IRC01:35
*** markvoelker has quit IRC01:36
*** ihrachys has quit IRC01:37
*** fanzhang_ has joined #openstack-infra01:40
*** markvoelker has joined #openstack-infra01:41
*** slaweq has quit IRC01:59
*** rosmaita has quit IRC02:08
*** slaweq has joined #openstack-infra02:08
*** sshnaidm|off has quit IRC02:10
*** dhinesh has quit IRC02:10
*** xarses has joined #openstack-infra02:11
*** markvoelker has quit IRC02:14
*** salv-orlando has joined #openstack-infra02:15
*** salv-orlando has quit IRC02:19
pabelangerinfra-root: we may also want to consider stopping nodepool processes on nodepool.o.o, we are still running since last sundays zuulv3 rollout.02:20
pabelangermostly for log file clean up02:20
*** xinliang has joined #openstack-infra02:24
openstackgerritEmilien Macchi proposed openstack-infra/tripleo-ci master: DEBUG: downgrade iptables  https://review.openstack.org/51387302:31
fungi#status log all zuul executors restarted to pick up the /usr/share/ca-certificates addition02:33
openstackstatusfungi: finished logging02:33
openstackgerritEmilien Macchi proposed openstack-infra/tripleo-ci master: DEBUG: downgrade iptables  https://review.openstack.org/51387302:33
*** bstinson has quit IRC02:34
*** Kevin_Zheng has joined #openstack-infra02:35
*** slaweq has quit IRC02:38
*** bstinson has joined #openstack-infra02:40
*** slaweq has joined #openstack-infra02:44
*** thorst has joined #openstack-infra02:52
*** dave-mccowan has quit IRC02:54
*** thorst has quit IRC02:57
kazshHi Infra team, could you kindly add myself `kaz_shinohara` to heat-dashboard-core team ?  Need it to move forward our journey, https://review.openstack.org/#/c/504166/02:58
EmilienMclarkb: weshay|ruck and I are still debugging the iptables thing02:59
EmilienMclarkb: we tried to reproduce it in our env and failed, we think it has something to do with the rules, maybe not saved03:00
EmilienMare nodes restarted before running jobs?03:00
jeblairEmilienM: no, it's the first boot03:00
EmilienMok03:00
EmilienMthx03:00
*** markvoelker has joined #openstack-infra03:05
openstackgerritwes hayutin proposed openstack-infra/tripleo-ci master: DEBUG: downgrade iptables  https://review.openstack.org/51389103:09
*** markvoelker has quit IRC03:10
clarkbEmilienM: my patch to zuul will help us in debugging it03:11
clarkbI should make sure tests passed03:11
EmilienMclarkb: which patch please?03:11
EmilienMweshay|ruck: I was wondering, could we reproduce the same problem in RDO CI, if we used the same nodepool image?03:12
EmilienMso we can see if it's something with zuul v303:12
EmilienMclarkb: I remember you told me that you don't publish the images anymore03:12
weshay|ruckEmilienM, ya.. let me see how those jobs are running03:12
EmilienMand that I have to use DIB myself03:12
clarkbwe've never published the images03:13
EmilienMweshay|ruck: maybe we can just run a multinode job with latest centos7 (and even run yum -y update before anything) in RDO CI03:13
EmilienMclarkb: on nodepool.o.o ?03:13
weshay|ruckya03:13
clarkbEmilienM: ehat you can do is run the overlay on the nodes I held for you03:13
EmilienMclarkb: ok03:13
EmilienMclarkb: but did zuul v3 prepared the nodes before?03:14
*** markvoelker has joined #openstack-infra03:14
clarkbEmilienM: I'm not sure how far it got before the job was aborted03:15
weshay|ruckEmilienM, rdo jobs are hitting the same issue03:15
EmilienMweshay|ruck: awesome, so we can more easily debug03:15
EmilienMclarkb: any change to revert to the previous image?03:15
EmilienMall our gate is stuck now03:15
*** salv-orlando has joined #openstack-infra03:16
EmilienMif we can debug in RDO CI with new image in the meantime03:16
weshay|ruckhttps://logs.rdoproject.org/openstack-periodic-4hr/periodic-tripleo-ci-centos-7-multinode-1ctlr-featureset005-master/3c6f838/console.txt.gz#_2017-10-20_20_12_03_23803:16
clarkbEmilienM: are we sure it us the image?03:16
EmilienMweshay|ruck: perfect03:16
EmilienMclarkb: probably the same packages03:16
clarkbwe only have yesterdays image03:16
EmilienMperfect03:16
clarkbso unsure if that goes back far enough03:16
EmilienMit broke this morning03:16
weshay|ruckthey build the upstream image in rdo and use it.. same thing03:16
clarkb(I havent checked)03:17
weshay|ruckya.. it broke today for sure03:17
clarkbweshay|ruck: that doesnt mean it is the image though03:17
EmilienMthis morning, around 11am PST03:17
clarkbis my point03:17
EmilienMright03:17
EmilienMin our investigation, we saw iptables updated03:17
EmilienMand a new package: iptables-services03:17
weshay|ruckclarkb, right.. I can go find where they build the image, it's the upstream dib03:17
EmilienMso maybe we miss something here03:17
EmilienMweshay|ruck: RDO CI image uses iptables-1.4.21-18.2.el7_4.x86_64 as well03:18
EmilienMlet's compare all rpms03:18
weshay|ruckiptables-services.x86_64    1.4.21-18.2.el7_4                          @quickstart-centos-updates03:19
EmilienMhttps://www.diffchecker.com/8Dd11LlG03:19
*** ianychoi_ has joined #openstack-infra03:19
EmilienMlook the diff ^03:19
EmilienMI don't see rpm that could be break networking03:19
EmilienMso either it's something in zuul v3 playbooks or it's something in the image03:20
*** salv-orlando has quit IRC03:20
clarkbor something i  tripleo :)03:20
*** fanzhang_ has quit IRC03:20
EmilienMclarkb: yeah03:21
EmilienMclarkb: but it broke this morning suddenly03:21
EmilienMclarkb: and tbh we didn't merge stuffs today I think03:21
EmilienMlet me check that as well03:21
*** ianychoi has quit IRC03:21
weshay|rucknot much has merged03:22
EmilienMyeah03:22
EmilienMand fyi it also affect stable branches03:22
EmilienMand we didn't merge backports today03:22
clarkbkeep in mind if it is new packages reverting the image wont help as we install those new packages03:25
EmilienMyum update03:26
EmilienMok03:26
EmilienMclarkb: what's your patch in zuul you mentionned?03:26
clarkb513850, the tests are failing03:27
*** lamt has quit IRC03:27
clarkbok I've got to go family calls03:28
jeblairclarkb, EmilienM: it's the weekend -- you could probably just tell zuul to hold 20 nodes and see what it catches03:28
EmilienMclarkb: no worries, family first03:32
EmilienMjeblair: that would be great03:32
EmilienMI think it's iptables or systemd03:32
EmilienMboth upgrades03:32
EmilienMupgraded*03:32
EmilienMI'm reading git diff now for both pkg upgrades03:32
jeblairEmilienM: what job and project?03:33
EmilienMjeblair: all tripleo multinode jobs03:33
*** lamt has joined #openstack-infra03:33
EmilienMjeblair: tripleo-heat-templates, tripleo-ci, every project03:33
jeblairlooks like clarkb did ('openstack', 'git.openstack.org/openstack/tripleo-heat-templates', 'legacy-tripleo-ci-centos-7-nonha-multinode-oooq')  before03:34
openstackgerritwes hayutin proposed openstack-infra/tripleo-ci master: DEBUG: downgrade iptables  https://review.openstack.org/51389103:34
EmilienMlegacy-tripleo-ci-centos-7-nonha-multinode-oooq for example03:34
jeblairi have to pick one -- it doesn't regex03:34
jeblairEmilienM: should i just do that again?03:34
EmilienMyeah good03:34
*** lamt is now known as Guest9041603:34
EmilienMor wait03:34
EmilienMjeblair: openstack-infra/tripleo-ci03:34
EmilienMjeblair: it's better so we can make change in this repo to debug stuffs with our framework03:35
EmilienMthanks03:35
jeblairEmilienM, clarkb: http://paste.openstack.org/show/624239/03:36
EmilienMperfect03:36
EmilienMjeblair: ok so I can just send patches now, and when it fail just SSH?03:37
EmilienM(my key is missing)03:37
jeblairso the next 20 runs of that job on that project should get held03:37
jeblairEmilienM: i guess an infra-root will have to add your ssh key03:38
EmilienMok03:38
EmilienMthis change in iptables worries me https://git.centos.org/blobdiff/rpms!iptables.git/4d0bb22ecdc68576e7a4a7afd966207c50b664d2/SPECS!iptables.spec03:44
*** markvoelker has quit IRC03:48
*** nicolasbock has quit IRC03:59
*** jamesmcarthur has joined #openstack-infra04:03
*** mriedem has quit IRC04:04
openstackgerritwes hayutin proposed openstack-infra/tripleo-ci master: DEBUG: downgrade iptables  https://review.openstack.org/51389104:06
*** jamesmcarthur has quit IRC04:07
SamYapleEmilienM: how so?04:09
EmilienMSamYaple: we're debugging multinode centos7 failures04:09
EmilienMSamYaple: the latest centos7 image seems not working in the current state of things04:09
EmilienMsubnode-2 isn't reachable from main node04:10
*** salv-orlando has joined #openstack-infra04:17
*** slaweq has quit IRC04:17
*** salv-orlando has quit IRC04:21
SamYapleoh i see. that wasnt just an acedemic concern. youre looking for the cause04:23
SamYaplethanks EmilienM04:23
SamYaplegood info to know04:23
EmilienMSamYaple: yeah, tripleo jobs are all failing now04:25
EmilienMSamYaple: do you run multinode on centos7,04:25
EmilienM?04:25
EmilienMin OSA04:25
SamYapleEmilienM: i dont know. i dont work on OSA04:26
SamYaplepretty sure kolla does though04:26
SamYaplebut they are just setting up those gates04:26
EmilienMoh right, kolla04:26
*** slaweq has joined #openstack-infra04:26
EmilienMsorry, late04:26
*** bhavik1 has joined #openstack-infra04:26
SamYaplethe confusion is i used to work at RAX ;)04:26
EmilienMSamYaple: oh where are you now?04:27
SamYaple1099 for Verizon Digital Media Services04:27
EmilienMwouhou, nice !04:28
EmilienMhuge clouds as well :D04:28
*** bhavik1 has quit IRC04:32
SamYaplenot the same Verizon, its a CDN formerly know as Edgecast04:34
*** claudiub|2 has joined #openstack-infra04:38
*** markvoelker has joined #openstack-infra04:38
*** markvoelker has quit IRC04:45
*** Kevin_Zheng has quit IRC04:45
*** edmondsw has joined #openstack-infra04:48
openstackgerritwes hayutin proposed openstack-infra/tripleo-ci master: DNM, testing fix for vxlan ping failure  https://review.openstack.org/51389604:48
*** bobh has joined #openstack-infra04:49
*** markvoelker has joined #openstack-infra04:50
*** markvoelker has quit IRC04:51
*** edmondsw has quit IRC04:52
*** thorst has joined #openstack-infra04:53
*** armax has quit IRC04:59
*** armax has joined #openstack-infra04:59
*** armax has quit IRC04:59
*** thorst has quit IRC05:00
*** armax has joined #openstack-infra05:00
*** armax has quit IRC05:00
*** armax has joined #openstack-infra05:01
*** armax has quit IRC05:01
*** armax has joined #openstack-infra05:02
*** armax has quit IRC05:02
*** armax has joined #openstack-infra05:02
*** armax has quit IRC05:03
*** florianf has quit IRC05:07
*** florianf has joined #openstack-infra05:08
*** bobh has quit IRC05:09
openstackgerritwes hayutin proposed openstack-infra/tripleo-ci master: DEBUG: downgrade iptables  https://review.openstack.org/51389105:16
*** salv-orlando has joined #openstack-infra05:17
*** florianf has quit IRC05:18
*** florianf has joined #openstack-infra05:18
*** salv-orlando has quit IRC05:21
*** gouthamr has joined #openstack-infra05:22
*** slaweq has quit IRC05:31
*** slaweq has joined #openstack-infra05:35
*** armaan has joined #openstack-infra05:42
*** armaan has quit IRC05:49
*** armaan has joined #openstack-infra05:50
*** armaan has quit IRC05:51
*** gouthamr has quit IRC05:53
*** florianf has quit IRC06:05
*** florianf has joined #openstack-infra06:05
openstackgerritOpenStack Proposal Bot proposed openstack-infra/project-config master: Normalize projects.yaml  https://review.openstack.org/51389906:07
*** slaweq has quit IRC06:08
openstackgerritAnastasia Kravets proposed openstack-infra/project-config master: [ec2-api] Increase timeout for rally job  https://review.openstack.org/51380206:09
*** xarses has quit IRC06:14
*** slaweq has joined #openstack-infra06:15
openstackgerritwes hayutin proposed openstack-infra/tripleo-ci master: DEBUG: downgrade iptables  https://review.openstack.org/51389106:15
*** salv-orlando has joined #openstack-infra06:18
*** salv-orlando has quit IRC06:22
openstackgerritAndreas Florath proposed openstack/diskimage-builder master: mk_build_dir: fix and refactor  https://review.openstack.org/51034406:23
*** makowals has quit IRC06:25
*** claudiub|2 has quit IRC06:29
*** edmondsw has joined #openstack-infra06:36
*** edmondsw has quit IRC06:40
*** slaweq has quit IRC06:47
*** ijw has joined #openstack-infra06:48
*** yamamoto has joined #openstack-infra06:50
*** claudiub|2 has joined #openstack-infra06:51
*** ijw has quit IRC06:52
*** armaan has joined #openstack-infra06:55
*** yamamoto has quit IRC06:55
*** yamamoto has joined #openstack-infra06:56
*** thorst has joined #openstack-infra06:56
*** baoli has joined #openstack-infra06:58
*** yamamoto has quit IRC07:00
*** thorst has quit IRC07:01
*** slaweq has joined #openstack-infra07:01
*** armaan has quit IRC07:02
*** baoli has quit IRC07:03
*** salv-orlando has joined #openstack-infra07:19
*** Guest90416 is now known as lamt___07:21
*** salv-orlando has quit IRC07:23
*** claudiub|2 has quit IRC07:34
*** claudiub|2 has joined #openstack-infra07:35
*** slaweq has quit IRC07:39
*** florianf has quit IRC07:41
*** slaweq has joined #openstack-infra07:42
*** priteau has joined #openstack-infra07:43
*** florianf has joined #openstack-infra07:43
*** priteau has quit IRC07:47
*** thorre has quit IRC07:57
AJaegerinfra-root, first time a proposal job running with v3: https://review.openstack.org/#/c/513899/ . Woot!08:00
*** thorre has joined #openstack-infra08:03
*** jamesmcarthur has joined #openstack-infra08:03
*** hashar has joined #openstack-infra08:04
*** jamesmcarthur has quit IRC08:08
SamYaplenice!08:13
openstackgerritAndreas Jaeger proposed openstack-infra/project-config master: Remove now redundant git review setup  https://review.openstack.org/51390808:13
*** armaan has joined #openstack-infra08:13
openstackgerritMerged openstack-infra/project-config master: Normalize projects.yaml  https://review.openstack.org/51389908:14
AJaegerand we have first translation proposal job - https://review.openstack.org/#/c/513903/ !08:16
*** psachin has joined #openstack-infra08:16
*** salv-orlando has joined #openstack-infra08:19
*** hashar is now known as hasharAway08:20
openstackgerritAntoine Musso proposed openstack/diskimage-builder master: dib-lint: test elements have README.rst file  https://review.openstack.org/17783208:23
openstackgerritAntoine Musso proposed openstack/diskimage-builder master: dib-lint: test elements have README.rst file  https://review.openstack.org/17783208:23
AJaegerinfra-root, we now have all periodic translation jobs running in parallel - and get "[ERROR] Operation failed: operation returned 429 (null): <html><head><title>Error</title></head><body>Too many concurrent requests for client API key (maximum is 6)</body></html>"08:24
*** edmondsw has joined #openstack-infra08:24
*** salv-orlando has quit IRC08:24
*** slaweq has quit IRC08:25
AJaeger#status log increased Zanata limit for concurrent requests to 20 using Zanata UI08:26
openstackstatusAJaeger: finished logging08:26
*** edmondsw has quit IRC08:28
*** slaweq has joined #openstack-infra08:28
*** slaweq has quit IRC08:29
*** yolanda has quit IRC08:33
*** sshnaidm has joined #openstack-infra08:35
openstackgerritAndreas Jaeger proposed openstack-infra/project-config master: Remove zuul v2 and jjb content  https://review.openstack.org/50718008:37
openstackgerritAndreas Jaeger proposed openstack-infra/project-config master: Remove jenkins Frozen tests  https://review.openstack.org/51391008:37
AJaegermordred: rebased and added a patch to allow it to merge ^08:37
*** slaweq has joined #openstack-infra08:52
openstackgerritAndreas Jaeger proposed openstack-infra/project-config master: Rename oslo.messaging legacy jobs  https://review.openstack.org/51391108:55
openstackgerritAndreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Remove legacy oslo.messaging jobs  https://review.openstack.org/51391208:55
*** thorst has joined #openstack-infra08:57
*** thorst has quit IRC09:02
*** armaan has quit IRC09:05
openstackgerritAndreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Remove Ceilometer legacy jobs  https://review.openstack.org/51072009:13
mordredAJaeger: morning!09:15
*** jtomasek has joined #openstack-infra09:15
mordredAJaeger: awesome - have we talked about when we want to merge that?09:16
*** salv-orlando has joined #openstack-infra09:20
mordredAJaeger: yay on proposal bot!!!09:23
*** salv-orlando has quit IRC09:25
*** yamamoto has joined #openstack-infra09:27
*** slaweq has quit IRC09:28
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Remove Ceilometer legacy jobs  https://review.openstack.org/51072009:29
*** tosky has joined #openstack-infra09:31
*** claudiub has joined #openstack-infra09:35
*** yamamoto has quit IRC09:36
*** claudiub|2 has quit IRC09:37
openstackgerritMerged openstack-infra/project-config master: Rename oslo.messaging legacy jobs  https://review.openstack.org/51391109:44
*** slaweq has joined #openstack-infra09:46
*** yamamoto has joined #openstack-infra09:47
*** armaan has joined #openstack-infra09:55
*** dizquierdo has joined #openstack-infra09:56
*** claudiub has quit IRC09:57
*** claudiub has joined #openstack-infra09:57
*** jtomasek has quit IRC10:04
*** slaweq has quit IRC10:06
openstackgerritMonty Taylor proposed openstack-infra/shade master: Add unittest tips jobs  https://review.openstack.org/51391310:12
openstackgerritMonty Taylor proposed openstack-infra/shade master: Add support for configured NAT source variable  https://review.openstack.org/51391410:12
openstackgerritMonty Taylor proposed openstack-infra/shade master: Add unittest tips jobs  https://review.openstack.org/51391310:15
openstackgerritMonty Taylor proposed openstack-infra/shade master: Add support for configured NAT source variable  https://review.openstack.org/51391410:15
*** salv-orlando has joined #openstack-infra10:21
*** yamamoto has quit IRC10:25
*** salv-orlando has quit IRC10:25
AJaegermordred: morning! We discussed at last infra team meeting "next week", so I expect the next few days...10:28
*** claudiub has quit IRC10:30
*** claudiub has joined #openstack-infra10:30
*** pbourke has quit IRC10:33
*** armaan has quit IRC10:34
*** pbourke has joined #openstack-infra10:34
*** claudiub|2 has joined #openstack-infra10:35
*** yamamoto has joined #openstack-infra10:35
mordredAJaeger: sweet10:36
*** claudiub has quit IRC10:36
*** claudiub has joined #openstack-infra10:37
AJaegermordred: remaing publish docs publishing job with problems - that I know of - is placement-api, care to review the stack starting at https://review.openstack.org/513822 ?10:37
*** dizquierdo has quit IRC10:38
AJaegeranother open issue is still the propose-update job, our grep fails - I've put something in the etherpad and haven't had time to dig deeper10:38
openstackgerritMonty Taylor proposed openstack-infra/shade master: Add unittest tips jobs  https://review.openstack.org/51391310:39
openstackgerritMonty Taylor proposed openstack-infra/shade master: Add support for configured NAT source variable  https://review.openstack.org/51391410:39
openstackgerritMonty Taylor proposed openstack/os-client-config master: Added nat_source flag for networks.  https://review.openstack.org/51375110:39
openstackgerritMonty Taylor proposed openstack/os-client-config master: Fix doc typo  https://review.openstack.org/51383610:39
openstackgerritMonty Taylor proposed openstack/os-client-config master: Add shade-tox-tips jobs  https://review.openstack.org/51391510:39
*** claudiub|2 has quit IRC10:41
*** claudiub|2 has joined #openstack-infra10:42
*** claudiub has quit IRC10:46
openstackgerritAndreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: WIP: Remove some unused jobs and templates  https://review.openstack.org/51391610:49
openstackgerritAndreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: WIP: Remove some unused jobs and templates  https://review.openstack.org/51391610:52
*** ijw has joined #openstack-infra10:53
*** yamamoto has quit IRC10:56
openstackgerritMonty Taylor proposed openstack-infra/shade master: Add devstack jobs for zuul v3  https://review.openstack.org/50036510:57
*** ijw has quit IRC10:57
*** thorst has joined #openstack-infra10:58
*** thorst has quit IRC11:02
openstackgerritMonty Taylor proposed openstack-infra/openstack-zuul-jobs master: Use fetch-cover-output role in openstack-tox-cover  https://review.openstack.org/51391711:04
mordredAJaeger: ^^ just noticed - we fixed tox-cover but not openstack-tox-cover11:04
openstackgerritMonty Taylor proposed openstack-infra/openstack-zuul-jobs master: Set success-url on coverage jobs to cover/  https://review.openstack.org/51391811:15
*** smcginnis has quit IRC11:16
*** thorst has joined #openstack-infra11:21
*** thorst has quit IRC11:21
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Support upper-constraints in tox-siblings  https://review.openstack.org/51319911:21
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Change tox-siblings disablement to allow opt-in  https://review.openstack.org/51391911:21
*** smcginnis has joined #openstack-infra11:22
*** salv-orlando has joined #openstack-infra11:22
mordredinfra-root: ^^ if anyone is around on a saturday, https://review.openstack.org/513919 should keep tox-siblings disabled until such a time as we're ready to re-enable it - but also let jobs that need it opt-in to the behavior in the meantime11:24
*** salv-orlando has quit IRC11:26
openstackgerritMonty Taylor proposed openstack-infra/shade master: Add unittest tips jobs  https://review.openstack.org/51391311:27
openstackgerritMonty Taylor proposed openstack-infra/shade master: Add devstack jobs for zuul v3  https://review.openstack.org/50036511:27
mordredsuch as https://review.openstack.org/51391311:27
*** armaan has joined #openstack-infra11:27
openstackgerritMonty Taylor proposed openstack-infra/shade master: Add unittest tips jobs  https://review.openstack.org/51391311:28
openstackgerritMonty Taylor proposed openstack-infra/shade master: Add devstack jobs for zuul v3  https://review.openstack.org/50036511:28
*** armaan has quit IRC11:31
*** dave-mccowan has joined #openstack-infra11:35
openstackgerritMonty Taylor proposed openstack-infra/shade master: Add support for configured NAT source variable  https://review.openstack.org/51391411:37
mordredproject-config-core: https://review.openstack.org/#/c/513913/ shows https://review.openstack.org/513919 and https://review.openstack.org/513918 both working11:38
openstackgerritMonty Taylor proposed openstack/os-client-config master: Add shade-tox-tips jobs  https://review.openstack.org/51391511:43
openstackgerritMonty Taylor proposed openstack/os-client-config master: Added nat_source flag for networks.  https://review.openstack.org/51375111:43
openstackgerritMonty Taylor proposed openstack/os-client-config master: Fix doc typo  https://review.openstack.org/51383611:43
openstackgerritMonty Taylor proposed openstack-infra/shade master: Add support for configured NAT source variable  https://review.openstack.org/51391411:50
*** yamamoto has joined #openstack-infra11:56
*** priteau has joined #openstack-infra12:01
*** armaan has joined #openstack-infra12:04
*** yamamoto has quit IRC12:06
*** nicolasbock has joined #openstack-infra12:17
*** salv-orlando has joined #openstack-infra12:23
*** yamamoto has joined #openstack-infra12:27
*** salv-orlando has quit IRC12:27
*** yamamoto has quit IRC12:29
*** bhavik1 has joined #openstack-infra12:31
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Make build-python-release job  https://review.openstack.org/51392512:36
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Remove old python-sdist job  https://review.openstack.org/51392612:36
openstackgerritMonty Taylor proposed openstack-infra/openstack-zuul-jobs master: Stop using tox to build python tarballs  https://review.openstack.org/51392712:36
openstackgerritMonty Taylor proposed openstack-infra/project-config master: Stop using tox for releasing python tarballs  https://review.openstack.org/51392812:36
*** bhavik1 has quit IRC12:39
*** wolverineav has joined #openstack-infra12:52
openstackgerritDavid Moreau Simard proposed openstack-infra/openstack-zuul-jobs master: Add missing required-projects for puppet-openstackci  https://review.openstack.org/51393013:17
pabelangermordred: +2 all around13:18
openstackgerritAndreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: WIP: Remove some unused jobs and templates  https://review.openstack.org/51391613:18
openstackgerritDavid Moreau Simard proposed openstack-infra/puppet-openstackci master: Add support for installing ARA wsgi middleware for sqlite databases  https://review.openstack.org/51386613:18
*** yamamoto has joined #openstack-infra13:19
dmsimardhttps://review.openstack.org/513930 fixes the broken gate for puppet-openstackci13:19
AJaegermordred: see https://review.openstack.org/#/c/513737/2 and compare with https://review.openstack.org/51391713:20
*** psachin has quit IRC13:21
dmsimardmordred: why am I seeing messages about Ansible 2.4 in gate jobs ?13:21
dmsimardhttp://logs.openstack.org/68/513868/1/check/legacy-infra-puppet-apply-3-ubuntu-trusty/c2828a7/job-output.txt.gz#_2017-10-20_23_07_56_50733013:21
mordredlookie there - it's a pabelanger and a dmsimard13:22
*** thorst has joined #openstack-infra13:22
mordredAJaeger: looking13:22
dmsimardmostly trying to make progress on ara which I've been neglecting (in favor of other important things, but still)13:23
mordredAJaeger: let's give https://review.openstack.org/#/c/513737 a try13:23
AJaegercould you review the stack starting with https://review.openstack.org/513822 some review, please?13:23
*** salv-orlando has joined #openstack-infra13:24
dmsimardOh, so it looks like system-config doesn't pin Ansible and ends up with 2.4: tools/prep-apply.sh:/tmp/apply-ansible-env/bin/pip install ansible13:24
mordredyup. on it13:24
mordredAJaeger: look great13:26
*** thorst has quit IRC13:27
AJaegerthanks13:27
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: make openstack-tox-cover inherit from tox-cover  https://review.openstack.org/51373713:27
*** salv-orlando has quit IRC13:28
openstackgerritMonty Taylor proposed openstack-infra/openstack-zuul-jobs master: Set success-url on coverage jobs to cover/  https://review.openstack.org/51391813:28
mordredAJaeger: while you're here - what do you think about https://review.openstack.org/#/c/513919/ ?13:30
openstackgerritAndreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: WIP: Remove some unused jobs and templates  https://review.openstack.org/51391613:30
mordred(that'll unblock a big stack of shade and occ changes)13:30
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Add build-placement-api-ref  https://review.openstack.org/51382213:30
mordredbtw - new-style v3 devstack jobs for shade are green and ready to land :)13:31
AJaeger\o/13:31
mordred(the legacy job is bong - but dtroyer and I need to make a new version of that that installs old openstack anyway - so I'm not particuarly interested in fixing the old version)13:31
dmsimardAJaeger: typos in https://review.openstack.org/#/c/513824/13:31
AJaegermordred: +3A13:32
mordredAJaeger: thanks!13:32
openstackgerritAndreas Jaeger proposed openstack-infra/project-config master: Fix placement-api jobs  https://review.openstack.org/51382413:33
openstackgerritDavid Moreau Simard proposed openstack-infra/project-config master: Add Zuul merge-check template.  https://review.openstack.org/51133913:33
AJaegerdmsimard: quickly fixed - thanks13:33
AJaegerdmsimard: merge-check? We removed that one...13:33
dmsimardAJaeger: it's just a rebase13:34
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Remove networking-ovn legacy jobs  https://review.openstack.org/51367513:34
AJaegerdmsimard: I'll fix...13:34
dmsimardAJaeger: sure13:34
openstackgerritAndreas Jaeger proposed openstack-infra/project-config master: Add Zuul system-required template  https://review.openstack.org/51133913:35
openstackgerritMonty Taylor proposed openstack-infra/zuul feature/zuulv3: Fix doc typo  https://review.openstack.org/51393213:35
openstackgerritwes hayutin proposed openstack-infra/tripleo-ci master: DEBUG: downgrade iptables  https://review.openstack.org/51389113:36
AJaegercleanup change for review: https://review.openstack.org/#/c/51143213:37
dmsimardAJaeger: lol openstack/tatu doesn't even exist ?13:38
openstackgerritMerged openstack-infra/project-config master: Remove manila legacy jobs  https://review.openstack.org/51307613:38
dmsimardAJaeger: ah: https://review.openstack.org/#/c/511335/13:38
dmsimardSSH as a service ?13:38
dmsimardಠ_ಠ13:39
*** mriedem has joined #openstack-infra13:39
AJaegerdmsimard: ah ;)13:39
openstackgerritDavid Moreau Simard proposed openstack-infra/project-config master: Add Zuul system-required template  https://review.openstack.org/51133913:39
openstackgerritMerged openstack-infra/project-config master: Fix neutron-lbaas-dashboard to use -horizon jobs  https://review.openstack.org/51356113:39
openstackgerritMerged openstack-infra/project-config master: [ec2-api] Increase timeout for rally job  https://review.openstack.org/51380213:39
openstackgerritMerged openstack-infra/project-config master: Fix placement-api jobs  https://review.openstack.org/51382413:40
openstackgerritAndreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: WIP: Remove some unused jobs and templates  https://review.openstack.org/51391613:41
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Set success-url on coverage jobs to cover/  https://review.openstack.org/51391813:43
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Remove legacy jobs in Karbor  https://review.openstack.org/51143213:44
*** priteau has quit IRC13:45
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Change domain in test fixture to example.com  https://review.openstack.org/51344813:45
dmsimardAJaeger: how are you finding unusued jobs/playbooks ?13:45
openstackgerritMerged openstack-infra/zuul-jobs master: Change tox-siblings disablement to allow opt-in  https://review.openstack.org/51391913:45
dmsimardAJaeger: or do you just know ?13:45
AJaegerdmsimard: for i in `echo playbooks/legacy/*` ; do j=`basename $i`;echo $j ;grep $j ../project-config/zuul.d/projects.yaml ; done13:46
AJaegerdmsimard: called from openstack-zuul-jobs and then verifying ...13:46
mordreddmsimard: I always just assume AJaeger has a perfect mental map of all jobs in his head13:46
AJaegerthere might be better ways...13:46
dmsimardAJaeger: maybe we should add a job that checks for orphaned content13:46
AJaegerdmsimard: when should it run? Could be non-voting... Or just a better script13:47
dmsimardAJaeger: I was thinking about this when I was writing my "role/playbook" finder, Zuul probably knows all of this in real time. For example when you add a job that tries to use a playbook that doesn't exist, it'll yield a syntax error iirc13:49
dmsimardThen again, it might be okay to have a playbook or a role that isn't used -- it *could* be used in the future13:50
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Specify duplicate project defn behavior  https://review.openstack.org/51102513:50
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Ignore duplicate project-template definitions  https://review.openstack.org/51385313:50
openstackgerritMerged openstack-infra/zuul feature/zuulv3: Add support for result data in child jobs  https://review.openstack.org/50480813:52
*** slaweq has joined #openstack-infra13:53
openstackgerritMerged openstack-infra/shade master: Add unittest tips jobs  https://review.openstack.org/51391313:58
*** ramishra has joined #openstack-infra13:59
AJaegerand one more cleanup: https://review.openstack.org/#/c/510431/14:00
openstackgerritAndreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Remove some unused jobs and templates  https://review.openstack.org/51391614:01
weshay|ruckdmsimard, k.. issue was iptables. This is passing tripleo-ci https://review.openstack.org/#/c/513891/14:01
mordredjeblair: I just clicked approve on adding new-style devstack jobs to shade woot14:01
mordredweshay|ruck: excellent!14:02
weshay|ruck:)14:02
openstackgerritDavid Moreau Simard proposed openstack-infra/puppet-openstackci master: Add support for installing ARA wsgi middleware for sqlite databases  https://review.openstack.org/51386614:02
mordredweshay|ruck: I mena, not great that centos released an iptables update that broke things ... but good that you have a fix for your things ... AND that they actually caught a real error in the world14:02
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Remove legacy oslo.messaging jobs  https://review.openstack.org/51391214:03
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Remove now unused legacy placement jobs  https://review.openstack.org/51382314:03
weshay|ruckya14:03
weshay|rucksuch is life14:03
dmsimardweshay|ruck: ok so now we need to understand why this is happening14:03
mordredyup14:03
dmsimardweshay|ruck: excluding a base package like that sucks a lot14:03
dmsimarddo we have a clue ?14:04
weshay|ruckhttps://git.centos.org/tag/rpms!iptables.git/imports!c7!iptables-1.4.21-18.2.el7_414:04
weshay|ruckoh weird. that link redirects14:05
*** Jeffrey4l has quit IRC14:05
dmsimardweshay|ruck: so https://git.centos.org/blobdiff/rpms!iptables.git/4d0bb22ecdc68576e7a4a7afd966207c50b664d2/SPECS!iptables.spec I guess14:05
openstackgerritAndreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Remove ironic legacy jobs  https://review.openstack.org/51126414:06
*** ramishra has quit IRC14:06
*** Jeffrey4l has joined #openstack-infra14:06
weshay|ruckya.. the two things that EmilienM and I were discussing last night.. was line 21914:06
weshay|ruckand 16514:06
openstackgerritAndreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Remove legacy OSA role tests  https://review.openstack.org/51263514:07
AJaeger511264 is rebased, let's get it in, please14:07
dmsimardweshay|ruck: so what's really happening ? Something around restarts or iptables+ip6tables running simultaneously ?14:08
*** slaweq has quit IRC14:08
AJaegeroh, the ironic and OSA changes need some more work - then let's get https://review.openstack.org/513916 in, please14:08
dmsimardI've seen iptables and ip6tables running simultaneously, is that really an issue ? :/14:08
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Remove legacy jobs in Vitrage  https://review.openstack.org/51043114:08
* AJaeger will rebase the other two once that is in14:09
mordredAJaeger: oh wow - that's an exciting patch :)14:09
dmsimardAJaeger: that's a bit hard to review, I'll take some glances but I have faith in yo14:09
dmsimardyou*14:09
weshay|ruckdmsimard, the only thing I've been able to 100% determine so far, is w/ the newer version of the rpm we see iptables dropping/rejecting connections during the ci run14:09
AJaegerdmsimard: good that zuul is self-testing ;)14:10
mordredAJaeger: yah - zuul syntax testing itself is pretty amazeballs14:10
dmsimardweshay|ruck: we can try and reproduce this outside of a tripleo context, do you know what kind of connections are being dropped ? some ports ?14:10
AJaegernow we need to improve ansible linting...14:10
dmsimardweshay|ruck: we have a job that is able to test multinode setup14:10
AJaegerpabelanger: want to get your linting change updated?14:11
weshay|ruckdmsimard, which job are you referring to?14:11
dmsimardweshay|ruck: let me submit an example that you can take over if you want, hang on14:12
weshay|ruckk.. thanks14:12
dmsimardweshay|ruck: what's the launchpad id for your bug ?14:13
jeblairAJaeger, jlk, dmsimard, mordred, pabelanger: current release of ansible-lint should have jlk's fix for the 'cmd' issue, so we should be able to use it again now.14:14
openstackgerritMonty Taylor proposed openstack-infra/openstack-zuul-jobs master: Stop using tox to build python tarballs  https://review.openstack.org/51392714:14
openstackgerritMonty Taylor proposed openstack-infra/openstack-zuul-jobs master: Enable tox_install_siblings for tox-linters  https://review.openstack.org/51393314:14
mordredjeblair: woot14:15
jeblairhttps://github.com/willthames/ansible-lint/commit/0cedf2ebf25295bd678d74933e58e3932e59b291 is the merged pr for that and is in the 3.4.16 release14:15
jeblairsorry that was the commit, the pr is https://github.com/willthames/ansible-lint/pull/28414:15
jeblairi like the last comment: "I actually think the cmd argument should be the better practice, tempted to add a rule for ansible-review to enforce it (but not ansible-lint, as not using it isn't a terrible thing)"14:16
jeblair(i very much agree)14:16
dmsimardyeah, the shell and command modules are a bit weird14:16
dmsimardand then you have the 'args' argument for other parameters14:16
dmsimardeverything should be at the first "depth" and 'cmd' is the actual command to run14:16
dmsimardthose two modules are probably the oldest ones in Ansible and never grew out of that design14:17
*** wolverineav has quit IRC14:18
dmsimardimagine the outrage if they deprecated that usage :)14:18
jeblairya.  would be nice to have a new version and put the old ones on a long deprecation plan.14:18
jeblairmeh, it's easy to deprecate things when you can just add new modules.14:19
jeblairi mean, after you decide on a name it's easy14:19
dmsimardweshay|ruck: nevermind, found it https://bugs.launchpad.net/tripleo/+bug/172545114:19
openstackLaunchpad bug 1725451 in tripleo "CI: new centos7 image contains a regression that prevents multinode networking setup to work" [Critical,Triaged] - Assigned to Emilien Macchi (emilienm)14:19
weshay|ruckdmsimard, ya.. in alerts on #tripleo14:20
weshay|ruckI'm going to update the commit message and doc on this14:20
dmsimardweshay|ruck: hmm... I *think* I found a flaw in our multinode firewall set up14:22
dmsimardweshay|ruck: my test will confirm it14:22
weshay|ruckk14:22
*** salv-orlando has joined #openstack-infra14:24
openstackgerritwes hayutin proposed openstack-infra/tripleo-ci master: downgrade iptables to iptables-services-1.4.21-18.0.1.el7  https://review.openstack.org/51389114:25
weshay|ruckdmsimard, ok I'm going to go feed my kids, but I'll be back later. Please let me know what you found14:25
openstackgerritwes hayutin proposed openstack-infra/tripleo-ci master: downgrade iptables to iptables-services-1.4.21-18.0.1.el7  https://review.openstack.org/51389114:26
dmsimardweshay|ruck: so we're not adding the bridge IPs on iptables, but I'm not sure if that matters14:26
dmsimardweshay|ruck: we have a test that checks if we can ping the bridge addresses and it works so... anyway, we'll see.14:27
weshay|ruckk.. :) back in a bit, thanks for poking at it14:27
*** weshay|ruck is now known as weshay|bbiab14:27
openstackgerritAndreas Florath proposed openstack/diskimage-builder master: [WIP] disk-usage refactoring  https://review.openstack.org/51208514:27
openstackgerritDavid Moreau Simard proposed openstack-infra/openstack-zuul-jobs master: Do not merge: Try to reproduce #1725451  https://review.openstack.org/51393414:27
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Remove some unused jobs and templates  https://review.openstack.org/51391614:28
*** salv-orlando has quit IRC14:29
dmsimardweshay|bbiab: wow, jlk created the iptables distgit repo for iptables in fedora :)14:30
dmsimardsmall world14:30
*** dave-mccowan has quit IRC14:32
dmsimardman, rhel and centos packages... iptables version 1.4.21 BUT release 18.214:32
dmsimardlike kernel packages.... 2.6.32-103714:33
dmsimardEmilienM, weshay|bbiab: ohhhhhhhhhhhhh https://bugzilla.redhat.com/show_bug.cgi?id=150464714:36
openstackbugzilla.redhat.com bug 1504647 in iptables "Fix invalid syntax in ip6tables.service systemd unit file" [High,Assigned] - Assigned to psutter14:36
openstackgerritJames E. Blair proposed openstack-infra/project-config master: Grafana: scale zuul executor load correctly  https://review.openstack.org/51393614:38
openstackgerritAndreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Remove ironic legacy jobs  https://review.openstack.org/51126414:39
openstackgerritAndreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Remove ironic legacy jobs  https://review.openstack.org/51126414:43
openstackgerritAndreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Remove ironic legacy jobs  https://review.openstack.org/51126414:45
dmsimardhm, persisting our iptables rules will be tricky, it's different on every distro :/14:46
dmsimardAJaeger: do you know for sure how to persist iptables rules on suse ? seeing different answers on stackoverflow :)14:50
*** priteau has joined #openstack-infra14:54
openstackgerritAndreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Removal of kolla legacy jobs  https://review.openstack.org/50895014:58
AJaegerdmsimard: not really. Maybe dirk does?14:58
dmsimardAJaeger: I'll spin up an opensuse instances and try to figure it out14:59
AJaegerdmsimard: I'm running opensuse here as well - do you have something for me to check?15:01
dmsimardAJaeger: mine is spawning right now, was mostly interested in the content of /etc/sysconfig/SuSEfirewall215:02
openstackgerritAndreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Remove manila legacy jobs  https://review.openstack.org/51307515:03
*** priteau has quit IRC15:03
dirkdmsimard there is a way to do custom rules with susefirewall15:03
dirkNewer distros of opensuse use firewalld15:03
dirkdmsimard afk15:04
dmsimardAJaeger: rhel/centos/fedora use /etc/sysconfig/iptables and /etc/sysconfig/ip6tables (output from iptables-save), debian/ubuntu use iptables-persistent which use /etc/iptables/rules.v4 and /etc/iptables/rules.v6 (content from iptables-save)15:04
dmsimarddirk: is there a way to do like the above ^ on suse ?15:04
AJaegerthis one is ready for review now https://review.openstack.org/50895015:04
AJaegerdmsimard: could you +2A https://review.openstack.org/#/c/513278 , please?15:05
dmsimardoh man SuseFirewall2 is another format entirely15:06
dmsimardlooks more like firewalld kind of thing15:06
cmurphythis is how we persist iptables rules with susefirewall2 http://git.openstack.org/cgit/openstack-infra/project-config/tree/nodepool/elements/nodepool-base/install.d/20-iptables#n9015:08
dmsimardcmurphy: oh cool so same thing as rhel/centos then, let me try that15:10
openstackgerritMerged openstack-infra/openstack-zuul-jobs master: Remove legacy-cross-*-* jobs related to global requirements  https://review.openstack.org/51327815:14
dmsimardcmurphy: do you know what loads iptables in the first place ? I'm able to list rules (i.e, iptables -vnL) but closest service I'm finding is "SuSEfirewall2.service" but it's disabled.15:14
AJaegerand https://review.openstack.org/#/c/513075/2 is ready for review as well15:14
AJaegerdmsimard: there's also SuSEfirewall2_init.service15:15
dmsimardAJaeger: but that's also disabled15:15
dmsimardthere's no 'iptables' service as far as I can tell either15:15
openstackgerritAndreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Remove legacy OSA role tests  https://review.openstack.org/51263515:16
dirkdmsimard: that's correct15:17
mordredjeblair: looking at your scale zuul load average patch for grafana got me looking at the zuul status grafana dashboard15:17
cmurphydmsimard: SuSEfirewall2 is the service, it should load the rules on start when you have that FW_CUSTOMRULES setting and loading function like at the bottom of the file i linked15:17
dirkUnfortunately there isn't anything equivalently simple on opensuse. Either Susefirewall or firewalld (starting with the upcoming release)15:18
mordredjeblair: "Zuul Jobs Launched (per Hour)" and "Gerrit Events (per hour)" both stop having data after 16 Oct15:18
mordredjeblair: is that expected?15:18
dmsimardcmurphy: ok, at least on my test instance that service is disabled and not running, I'll fiddle with it a bit..15:19
dirkdmsimard: that's nodepool image?15:19
dmsimarddirk: an opensuse leap image, closest I've got :/15:19
openstackgerritMerged openstack/os-client-config master: Update make_rest_client to work with version discovery  https://review.openstack.org/51301315:19
dmsimarddirk: maybe there's some stuff handling that in the nodepool image15:20
AJaegermordred, could you +2 https://review.openstack.org/508950 , please?15:20
dirkdmsimard: it is possible that we forgot to enable the service15:20
dirkcmurphy: congratulations, btw!15:20
cmurphydirk: ty15:20
dmsimarddirk: I'll blindly try cmurphy's suggestion on the integration job which runs on the nodepool image and we'll see what happens :)15:21
AJaegercmurphy: congratulations from me as well15:21
mordred++15:21
dmsimardoh, yeah, congrats cmurphy :)15:21
dirkdmsimard: on suse services normally don't get enabled just by installing, they need to be sustemctl enable'd explicitly. That's a difference to most other distros15:22
dmsimarddirk: pretty sure debian and red hat does that a lot too, but probably not universally15:23
dmsimardlike yum install httpd; systemctl enable --now httpd :)15:23
*** thorst has joined #openstack-infra15:23
*** claudiub|2 has quit IRC15:23
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Add stats for executor and merger count  https://review.openstack.org/51394015:25
*** salv-orlando has joined #openstack-infra15:25
jeblairmordred: yeah, it's not in a great state is it? :)15:25
*** fultonj has joined #openstack-infra15:27
*** fultonj has left #openstack-infra15:27
*** thorst has quit IRC15:27
openstackgerritAndreas Jaeger proposed openstack-infra/openstack-zuul-jobs master: Remove unused legacy templates  https://review.openstack.org/51394115:29
jeblairmordred: i *think* we're probably emitting the right stats for that still, but just with a different key.  i haven't looked it up yet.15:29
*** salv-orlando has quit IRC15:30
openstackgerritAndreas Jaeger proposed openstack-infra/project-config master: Use write-root-marker role  https://review.openstack.org/51394215:32
openstackgerritDavid Moreau Simard proposed openstack-infra/zuul-jobs master: Persist iptables rules  https://review.openstack.org/51394315:32
openstackgerritDavid Moreau Simard proposed openstack-infra/openstack-zuul-jobs master: Do not merge: Try to reproduce #1725451  https://review.openstack.org/51393415:33
mordredjeblair: ok. cool- so it's a thing that's worth investigating, rather than a thing we stopped emitting on purpose and should clean up the graphs15:33
jeblairmordred: yup15:33
dmsimardAJaeger, dirk, cmurphy: I think https://review.openstack.org/#/c/513943/1/roles/multi-node-firewall/tasks/persist/Suse.yaml should do it15:33
openstackgerritDavid Moreau Simard proposed openstack-infra/zuul-jobs master: Persist iptables rules  https://review.openstack.org/51394315:34
openstackgerritDavid Moreau Simard proposed openstack-infra/zuul-jobs master: Persist iptables rules  https://review.openstack.org/51394315:37
openstackgerritAndreas Jaeger proposed openstack-infra/project-config master: Revert "Fix branch check for proposal script"  https://review.openstack.org/51394415:37
AJaegermordred, jeblair, dmsimard the above should fix our proposal script ^15:37
dmsimardok signing off for now, see ya later15:38
AJaegerthanks, dmsimard ! Enjoy the weekend15:38
*** fultonj has joined #openstack-infra15:39
mordredAJaeger: +215:43
*** armax has joined #openstack-infra15:46
*** armax_ has joined #openstack-infra15:49
mordredinfra-root, project-config-root: I have a VERY small sample size so far - but I've had 2 devstack runs fail shade functional tests in the last 4 hours in a very weird way (timeout waiting for a VM to boot) - both were in OVH15:50
mordredgah. project-config-core I meant15:50
*** armax has quit IRC15:50
*** armax_ is now known as armax15:50
mordredI mention it just so folks can keep an eye out to see if there is a real something or if I just got unluckyu15:51
openstackgerritAndreas Jaeger proposed openstack-infra/project-config master: Fix wheel/release  https://review.openstack.org/51394515:55
mordredAJaeger: ^^ whoops :)15:55
*** caphrim007 has joined #openstack-infra15:56
EmilienMdmsimard: yeah15:56
EmilienMdmsimard: I said it was iptables :D15:56
EmilienMdmsimard: https://review.openstack.org/#/c/513891/15:57
openstackgerritJames E. Blair proposed openstack-infra/project-config master: Grafana: fix zuul job launch graph  https://review.openstack.org/51394615:58
*** fultonj has quit IRC15:58
EmilienMweshay|bbiab: https://review.openstack.org/51394315:59
*** fultonj has joined #openstack-infra15:59
dmsimardEmilienM: I updated the launchpad15:59
AJaegermordred: currently going through jlk's https://review.openstack.org/#/c/503751 and looking for nuggets ;)15:59
AJaegerwe need ansible-linter15:59
openstackgerritJames E. Blair proposed openstack-infra/project-config master: Grafana: fix gerrit event graph  https://review.openstack.org/51394715:59
*** armax has quit IRC16:00
EmilienMdmsimard: ok thanks16:01
clarkbmordred: are you disabling kvm and using qemu?16:01
*** fultonj has quit IRC16:01
clarkbmordred: devstack-gate does this in the legacy jobs16:02
*** fultonj has joined #openstack-infra16:02
clarkb(for this reason, weird vm boot failures and crashes)16:02
mordredclarkb: I'll look - I don't know if the new devstack basejobs do thator not16:03
*** nhandler has quit IRC16:04
mordredclarkb: booyah. great call! https://review.openstack.org/513948 Disable nested virt in base devstack job16:05
jeblairlooks like we need to add LIBVIRT_TYPE: qemu16:05
jeblairwhich is what that patch does!  +216:06
mordredalso - in case anyone is wondering - the failure rate is approx 1/8 of the time ... so it IS still a good idea that we do it :)16:06
*** nhandler has joined #openstack-infra16:06
*** armax has joined #openstack-infra16:08
*** claudiub|2 has joined #openstack-infra16:10
*** yamamoto has quit IRC16:10
*** caphrim007 has quit IRC16:10
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Report initial pipeline changes under correct key  https://review.openstack.org/51394916:10
openstackgerritJames E. Blair proposed openstack-infra/project-config master: Grafana: fix zuul pipeline gauges  https://review.openstack.org/51395016:12
*** reed has quit IRC16:12
EmilienMdmsimard: if the workaround works, https://review.openstack.org/#/c/513891/ - we'll merge it until we can use your patch16:12
openstackgerritMonty Taylor proposed openstack-infra/shade master: Re-enable shade-functional-devstack-legacy  https://review.openstack.org/51395116:12
jeblairheh, i wondered why those were working -- turns out we were only emitting them on reconfiguration.  which happens a lot.  so...16:12
*** reed has joined #openstack-infra16:13
*** abishop has joined #openstack-infra16:13
*** fultonj has quit IRC16:13
dmsimardEmilienM: I'm not sure my patch fixes your issue, but it's something we need to be doing regardless16:14
dmsimardEmilienM: I'm trying to do a reproducer outside of tripleo and understand what is going on16:15
EmilienMdmsimard: are you able to reproduce it outside of tripleo? we did reproduce the same problem in RDO CI16:16
openstackgerritMerged openstack-infra/project-config master: Grafana: scale zuul executor load correctly  https://review.openstack.org/51393616:16
dmsimardEmilienM: haven't been able yet, but I found the persist issue so I'm taking care of that first16:17
EmilienMdmsimard: wes's patch confirms it's iptables anyway16:18
EmilienMdmsimard: and looking at the git diff in iptables, I confirm the persistence changed16:18
EmilienMhttps://git.centos.org/blob/rpms!iptables.git/4d0bb22ecdc68576e7a4a7afd966207c50b664d2/SPECS!iptables.spec#L27816:18
dmsimardEmilienM: look at the bugzilla I linked in the launchpad, there are some issue with unit files too16:19
EmilienMsorry this link: https://git.centos.org/commitdiff/rpms!iptables.git/4d0bb22ecdc68576e7a4a7afd966207c50b664d216:19
EmilienMdmsimard: yeah indeed :/16:19
openstackgerritAndreas Jaeger proposed openstack-infra/project-config master: Fix flake8  https://review.openstack.org/51395216:19
openstackgerritAndreas Jaeger proposed openstack-infra/project-config master: Run pep8 everywhere  https://review.openstack.org/51395316:19
openstackgerritMerged openstack-infra/project-config master: Grafana: fix zuul job launch graph  https://review.openstack.org/51394616:20
openstackgerritMerged openstack-infra/project-config master: Grafana: fix gerrit event graph  https://review.openstack.org/51394716:20
openstackgerritMerged openstack-infra/project-config master: Grafana: fix zuul pipeline gauges  https://review.openstack.org/51395016:20
*** slaweq has joined #openstack-infra16:22
*** slaweq has quit IRC16:23
mordredAJaeger: don't know if you were aware - but you can totally do depends-on with patches in the same repo - so if you wanted you could depends-on that second pep8 patch on the v2 removal patch (also, it's totally not necessary)16:23
*** armax has quit IRC16:23
*** salv-orlando has joined #openstack-infra16:26
AJaegermordred: didn#t think about it - good idea...16:26
openstackgerritAndreas Jaeger proposed openstack-infra/project-config master: Run pep8 everywhere  https://review.openstack.org/51395316:27
AJaegermordred: added dependency...16:28
mordredAJaeger: +2 from me16:29
*** salv-orlando has quit IRC16:30
openstackgerritAndreas Jaeger proposed openstack-infra/project-config master: Fix flake8  https://review.openstack.org/51395216:31
AJaegermordred: thanks. Need to fix the first one - now dib linter complains...16:32
*** fultonj has joined #openstack-infra16:32
*** nhandler has quit IRC16:35
openstackgerritAndreas Jaeger proposed openstack-infra/project-config master: Fix flake8  https://review.openstack.org/51395216:37
AJaegerwhat is this: http://logs.openstack.org/ec/ec22e46a850af1871cbb331c6a679474a3bbf965/post/publish-openstack-python-branch-tarball/dbf3ac0/ara/result/246a9359-8641-4a1b-a831-cc40d797a59d/  - Operatoin not permitted for the publish job?16:38
*** nhandler has joined #openstack-infra16:40
*** eharney has quit IRC16:41
mordredAJaeger: looking16:41
mordredAJaeger: that file (/srv/static/tarballs/trove/images/README) is owned by root16:43
jlkdmsimard: I created every Fedora dist-git repo (as of 2010 ish I think).  I did the migration from cvs to git for all of Fedora (and RHEL).16:44
mordredclarkb: if you swing back through, feel like +Ain https://review.openstack.org/#/c/513948/ ? (It's the nested virt patch)16:45
SpamapSmmmmmm csv to git16:45
SpamapSthat's a happy day16:45
mordredbzr to git has nothing on cvs to git16:46
* SpamapS once took a company from {nothing} to csv to svn to perforce to bzr over a 6 year period...16:46
SpamapScsv? cvs.. thanks16:46
*** dave-mccowan has joined #openstack-infra16:46
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Add stats for executor and merger count  https://review.openstack.org/51394016:49
openstackgerritJames E. Blair proposed openstack-infra/zuul feature/zuulv3: Report initial pipeline changes under correct key  https://review.openstack.org/51394916:49
jeblairSpamapS: assumed you just meant they were programming in excel16:49
openstackgerritAndreas Jaeger proposed openstack-infra/project-config master: Run pep8 everywhere  https://review.openstack.org/51395316:50
openstackgerritDavid Moreau Simard proposed openstack-infra/zuul-jobs master: Persist iptables rules  https://review.openstack.org/51394316:53
SpamapSjeblair: that excel to svn migration was hell.16:55
SpamapSHad to write it all in VBscript16:55
*** mriedem has quit IRC16:56
AJaegermordred: and we should not overwrite it I guess - why is it pushed?16:56
openstackgerritAndreas Jaeger proposed openstack-infra/project-config master: Enable ansible syntax test  https://review.openstack.org/51395516:56
AJaegeror change permissions to allow pushing?16:56
*** Adri2000 has joined #openstack-infra16:57
*** Adri2000 has joined #openstack-infra16:57
openstackgerritJames E. Blair proposed openstack-infra/project-config master: Ignore some more pep8 errors  https://review.openstack.org/51395617:01
openstackgerritJames E. Blair proposed openstack-infra/project-config master: Ignore some more pep8 errors  https://review.openstack.org/51395617:03
jeblairAJaeger: some of the things you fixed are things we typically ignore ^17:03
jeblair'fixed' should have been in quotes17:03
jeblair(the 'fixes' are fine, after all, they're ignored :)17:04
openstackgerritDavid Moreau Simard proposed openstack-infra/project-config master: Add multinode-integration jobs to zuul-jobs  https://review.openstack.org/51395717:05
dmsimardproject-config-core: ^ can we get that in, they're not running on z-j see https://review.openstack.org/#/c/513943/17:06
mordredAJaeger: I'm not sure which things are right17:07
*** yamamoto has joined #openstack-infra17:11
*** gmann is now known as gmann_afk17:14
*** yamamoto has quit IRC17:19
pabelangerAJaeger: yah, I'll push up a change this afternoon17:20
AJaegerjeblair: ok, understood...17:26
openstackgerritAndreas Jaeger proposed openstack-infra/project-config master: Enable ansible syntax test  https://review.openstack.org/51395517:26
openstackgerritAndreas Jaeger proposed openstack-infra/project-config master: Enable ansible syntax test  https://review.openstack.org/51395517:32
*** bhavik1 has joined #openstack-infra17:33
*** bhavik1 has quit IRC17:34
openstackgerritPaul Belanger proposed openstack-infra/project-config master: Create infra-zuul-jobs-linters job  https://review.openstack.org/51395817:35
pabelangerAJaeger: ^suggestions for job name welcome17:35
AJaegerpabelanger: I'll abandon 513955 ;)17:37
openstackgerritDavid Moreau Simard proposed openstack-infra/zuul-jobs master: Persist iptables rules  https://review.openstack.org/51394317:37
AJaegerpabelanger: care to +2A https://review.openstack.org/513952  , please?17:37
*** salv-orlando has joined #openstack-infra17:37
openstackgerritPaul Belanger proposed openstack-infra/project-config master: Create infra-zuul-jobs-linters job  https://review.openstack.org/51395817:38
mordredpabelanger: does tox even give us any value in that job? I kinda wonder if we shouldn't just have playbook/role that does that the linters env in each repo does - but targetting all three repos content too17:39
mordredpabelanger: (I think that patch looks great - just wondering out loud)17:39
AJaegerpabelanger: what about adding a new job for now - ansible-lint, have that non-voting and once we make it voting, merge it with linters?17:39
pabelangermordred: yah, we discussed dropping tox and writing playbooks directly17:40
AJaegerI don't want to have regression come in for pep8 or bashate17:40
mordredpabelanger: kk. maybe we do tha as a follow up17:40
pabelangerso, if we want to do that, we can iterate in ozj first, then promote to project-config (because main.yaml sort things)17:40
AJaegerpabelanger: leave the job name, just call tox differently...17:41
pabelangermordred: AJaeger: k17:41
*** salv-orl_ has joined #openstack-infra17:41
mordredpabelanger: well - if we keep tox-linters in each repo for pep8 and main.yaml sort - we could do infra-zuul-jobs-linters in ozj and leave it there - or does that seem like too much?17:41
*** salv-orl_ has quit IRC17:42
pabelangermordred: yah, that would be fine I think17:42
AJaegeryes, we could also do it first for ozj and zuul-jobs - and once those two are voting, add to project-config and fix jobs at the same time17:42
pabelangerright, just keep in mind, we can start in ozj, but no other project will be able to run the job, due to sort ording in zuul-jobs. But once it is working, we can put it back into project-config17:43
*** salv-orlando has quit IRC17:44
pabelangerI'll iterate on 51271517:44
pabelangerand abandon the other17:44
mordredpabelanger: I'm not following what you mean about sort ordering?17:44
pabelangermordred: project-config, wont see the job in ozj17:44
pabelangermordred: same with zuul-jobs17:44
pabelangerbecause ozj is listed after those projects in main.yaml17:45
mordredthat's only for parent/child relations17:45
mordredpabelanger: we can totally add it to projet pipelines of all three projects17:45
pabelangerOh, yes, you are correct17:45
pabelangernevermind then :D17:45
openstackgerritDavid Moreau Simard proposed openstack-infra/openstack-zuul-jobs master: Add integration test coverage for iptables persistence  https://review.openstack.org/51393417:45
mordredwoot! I was worried for a sec that jetlag was messing with me :)17:45
pabelangerno, just my brain not working again :)17:46
mordredpabelanger: it's been a long few weeks :)17:46
mordredpabelanger: congrats (or condolences) btw!17:46
AJaegerpabelanger: congratulations !17:47
AJaegermordred: let's talk about condolences after a few weeks ;)17:47
pabelangermordred: AJaeger: Thanks. Lots of new stuff to learn17:47
*** jamesmcarthur has joined #openstack-infra17:48
AJaeger;)17:48
*** nicolasbock has quit IRC17:49
AJaegerpabelanger: did your tests catch https://review.openstack.org/513945 ? jlk found that with ansible-lint, I just pushed it forward17:49
*** dave-mccowan has quit IRC17:50
openstackgerritMerged openstack-infra/project-config master: Fix flake8  https://review.openstack.org/51395217:50
openstackgerritMerged openstack-infra/project-config master: Ignore some more pep8 errors  https://review.openstack.org/51395617:50
pabelangerAJaeger: no, the job did run on project-config yet17:52
pabelangerbut would have17:52
* AJaeger signs off for today, have a great Saturday!17:54
*** fultonj has quit IRC17:57
*** jamesmcarthur has quit IRC18:04
*** thorst has joined #openstack-infra18:05
*** thorst has quit IRC18:06
openstackgerritMerged openstack-infra/shade master: Add devstack jobs for zuul v3  https://review.openstack.org/50036518:22
smcginnisAm I right thinking we'll need this to land before we try another lib update release? https://review.openstack.org/#/c/513944/18:23
openstackgerritPaul Belanger proposed openstack-infra/project-config master: Use new targets for Test Nodes panel  https://review.openstack.org/51395918:25
pabelangeranybody know why we have 16 nodes held? http://grafana.openstack.org/dashboard/db/nodepool18:26
pabelangerseems to be climbing too18:26
smcginnispabelanger: Hopefully not leftover from when we were debugging release failures.18:26
smcginnispabelanger: What's the time range on that data?18:27
clarkb pabelanger I think jeblair set a hold 20 for EmilienM to debug multinode overlay in tripleo18:27
clarkbbecause it is the weekend18:27
pabelangerah18:27
pabelangeryah, I see tripleo iptables debug18:27
pabelangerkewl18:27
pabelangersmcginnis: top right shows the default, which is 6 hours18:28
* smcginnis finally notices the 6 hour message in the top corner18:28
EmilienMhttps://review.openstack.org/#/c/513891/18:28
smcginnis:)18:28
mordred\o/18:28
mordrednative devstack jobs for shade landed18:28
EmilienMthis is a workaround for now - we needed to make sure it was IPtables.18:28
pabelangermordred: Nice18:28
EmilienMpabelanger: and dmsimard is working on https://review.openstack.org/51394318:29
smcginnisI believe this one also needs to land before we can get the proposal-bot working: https://review.openstack.org/#/c/513864/18:29
pabelangerEmilienM: wait, you are using wget to fetch an RPM?18:29
pabelangerEmilienM: doesn't the old version exist in the repo?18:29
*** yamahata has joined #openstack-infra18:29
openstackgerritMerged openstack-infra/project-config master: Revert "Fix branch check for proposal script"  https://review.openstack.org/51394418:30
EmilienMpabelanger: I tried in my env, and no I didn't have the old pkg18:30
*** xarses has joined #openstack-infra18:30
pabelangerhttp://mirror.dfw.rax.openstack.org/centos/7/os/x86_64/Packages/iptables-1.4.21-18.0.1.el7.centos.i686.rpm18:31
pabelangerwe still mirror it18:31
EmilienMfor how long18:31
EmilienMbut ok18:31
EmilienMweshay|bbiab: ^18:31
pabelangerso, you could just be able to yum install==<that version18:31
EmilienMright18:31
pabelangerwell, it should be there until centos purges it18:31
pabelangerI don't know how often that is18:31
pabelangerwait18:32
pabelangerthat will be there forever18:32
pabelangerthat is the base OS package18:32
weshay|bbiabwe call yum update, etc.. so we need to handle that too18:32
pabelangerso, that was the release of 7.4 then18:32
weshay|bbiabI assume this will be fixed in an upcoming release of iptables18:32
EmilienMpabelanger: no, it was iptables upgrade 2 days ago18:32
SamYaplemordred: nice!18:33
pabelangeryah, so just uninstall iptable from updates repo18:33
pabelangeryou'll fall back to base OS version of iptables18:33
EmilienMhttps://git.centos.org/blob/rpms!iptables.git/4d0bb22ecdc68576e7a4a7afd966207c50b664d2/SPECS!iptables.spec#L27818:33
EmilienMdmsimard also found a BZ https://bugzilla.redhat.com/show_bug.cgi?id=150464718:33
openstackbugzilla.redhat.com bug 1504647 in iptables "Fix invalid syntax in ip6tables.service systemd unit file" [High,Assigned] - Assigned to psutter18:33
*** andreww has joined #openstack-infra18:33
pabelangerEmilienM: right, if you install centos 7.4 with out any updates / security iptable will work. It's only after you install package from updates repo, so you can uninstall and blacklist the updates version, and by defualt it should install iptables from OS (7.4 release version)18:34
weshay|bbiabya.. the bug indicates it should not have passed tests18:34
SamYapleif its just a service file issue, you could override that, even before iptables update is installed18:35
SamYaplein the short term18:35
*** xarses has quit IRC18:35
weshay|bbiabodly I noticed this didn't do the trick sudo yum-config-manager --save --setopt=exclude=iptables*18:35
weshay|bbiabanyone see something I did wrong there18:36
clarkbyou shouldn't require people to depend on an unpatched platform to use your software. Ideally the fix would be to the updated packages18:36
weshay|bbiabya.. agree18:36
pabelangerclarkb: yah, that too18:37
pabelangerwhen did centos-7 break?18:41
pabelangerEmilienM:18:42
EmilienMyesterday morning 11am18:42
pabelangeryah, we could have rolled the image back18:43
EmilienMFYI i'm totally unresponsive today, doing customer support all day and stuck on that.18:43
EmilienMtalk with weshay|bbiab  please18:43
pabelangerbut, looks like we've already upload images for today18:43
EmilienMpabelanger: I asked roll back18:43
pabelangerso, we've deleted the good images18:43
weshay|bbiabpabelanger, so you are suggesting "yum downgrade --allowerasing  iptables iptables-services"18:45
pabelangerweshay|bbiab: yah, if you downgrade to working iptables, everything should be fine right?18:45
mordredyah. I think that's the workaround for now18:45
weshay|bbiabpabelanger, as long as we save the rules ya18:46
*** rwsu has quit IRC18:46
mordredand should allow breathing room to figure out what the right way to deal with the new packages is18:46
pabelangeryah, I don't think you need to wget rpms from the internet, since this seems to be the first update for iptable in 7.4.18:46
pabelangerand os package works as expected18:47
weshay|bbiabk k.. so just remove the wget18:47
pabelangerdoes centos-7 on devstack fail too?18:47
pabelangerI would assume so18:47
openstackgerritwes hayutin proposed openstack-infra/tripleo-ci master: downgrade iptables to iptables-services-1.4.21-18.0.1.el7  https://review.openstack.org/51389118:50
weshay|bbiabhrm.. I may need remove .rpm18:51
mordredpabelanger: I think the devstack and tripleo-ci iptables rules and setup are different - I believe dmsimard started looking to see what was different between them, but the focus has been on stopping the bleeding first18:52
pabelangermordred: Yah, bleeding for sure. I'll wait to see what dmsimard finds out18:52
openstackgerritwes hayutin proposed openstack-infra/tripleo-ci master: downgrade iptables to iptables-services-1.4.21-18.0.1.el7  https://review.openstack.org/51389118:53
weshay|bbiabpabelanger, does that look right to you now?18:54
clarkbjeblair: in trying to fix my tests for 513850 I think at least some of the zuul tests are buggy in how they use getJobFromHistory18:55
openstackgerritAndreas Florath proposed openstack/diskimage-builder master: [WIP] disk-usage refactoring  https://review.openstack.org/51208518:55
clarkbbasically we don't reset the history when checking subsequent jobs so you can match against the original results18:56
mordredweshay|bbiab: looks sane to me ... fwiw, you don't need those ; at the ends of the lines - but they also don't hurt anything18:57
clarkbtest_merger_repack attempts to reset that history but I think it is operating on the wrong object18:57
clarkb(should be self.executor_server)18:57
weshay|bbiabmordred, thanks.. I think it's just for good luck :)18:57
mordredweshay|bbiab: ++ :)18:57
pabelangermordred: weshay|bbiab yah, looks correct18:57
weshay|bbiabthanks guys18:57
mordredweshay|bbiab: fingers crossed!18:58
*** rwsu has joined #openstack-infra19:00
openstackgerritMonty Taylor proposed openstack-infra/shade master: Avoid tox_install.sh for constraints support  https://review.openstack.org/50806119:01
openstackgerritClark Boylan proposed openstack-infra/zuul feature/zuulv3: Only autohold failed builds  https://review.openstack.org/51385019:01
mordredtonyb: ^^ I rebased that and removed the WIP comment - I'm using both constraints and tox-siblings in shade, so it should be a decent guinea pig for the non-neutron case19:02
clarkbjeblair: I'll probably dig into fixing those tests on monday (for now have jsut fixed the tests on my change)19:03
openstackgerritMonty Taylor proposed openstack/os-client-config master: Add shade-tox-tips jobs  https://review.openstack.org/51391519:04
openstackgerritMonty Taylor proposed openstack/os-client-config master: Added nat_source flag for networks.  https://review.openstack.org/51375119:04
openstackgerritMonty Taylor proposed openstack/os-client-config master: Fix doc typo  https://review.openstack.org/51383619:04
*** bobh has joined #openstack-infra19:13
*** bobh has quit IRC19:14
*** bobh has joined #openstack-infra19:16
*** e0ne has joined #openstack-infra19:16
jlkAJaeger: I don't think I had caught that19:21
AJaegerjlk: https://review.openstack.org/503751 - I split it off to move forward...19:22
openstackgerritMerged openstack-infra/project-config master: Fix wheel/release  https://review.openstack.org/51394519:26
*** spectr has joined #openstack-infra19:29
*** slaweq has joined #openstack-infra19:31
*** bobh has quit IRC19:32
fungii have ~500 lines of scrollback... anything which needs my urgent attention while i have a few minutes between weekend chores?19:34
fungifwiw, if the tripleo failures are due to some distro releasing new broken packages i don't think image rollbacks are really an appropriate workaround, considering the message to users in general then is to not upgrade their packages. if jobs need workarounds, they should either shift focus to help fix those broken packages or be employing the same solutions as all those poor schmucks whose clouds19:41
fungijust broke because of lax qa on distro packages19:41
clarkbfungi: I don't think there is anything urgent19:43
clarkbwere there changes to statsd reporting in zuul recently? I don't think http://logs.openstack.org/50/513850/3/check/tox-py35/4f8140d/job-output.txt.gz#_2017-10-21_19_10_05_267890 is related to my change19:46
clarkbI;ve rechecked it to see if it is consistent but need to get back to the weekend19:48
dmsimardclarkb: if you're still around, could you +A https://review.openstack.org/#/c/513957/ ? Otherwise we don't have multinode coverage on zuul-jobs20:03
*** jamesmcarthur has joined #openstack-infra20:05
*** thorst has joined #openstack-infra20:07
*** jamesmcarthur has quit IRC20:09
*** thorst has quit IRC20:11
*** e0ne has quit IRC20:12
*** nicolasbock has joined #openstack-infra20:22
*** rhallisey has quit IRC20:23
*** d0ugal has quit IRC20:25
*** spectr has quit IRC20:37
*** d0ugal has joined #openstack-infra20:37
openstackgerritMerged openstack-infra/project-config master: Add multinode-integration jobs to zuul-jobs  https://review.openstack.org/51395720:55
*** hasharAway has quit IRC20:58
*** edmondsw has joined #openstack-infra21:01
*** edmondsw has quit IRC21:05
*** hongbin has joined #openstack-infra21:12
smcginnisIf anyone's around or checks in, would love to see https://review.openstack.org/#/c/513864/ go through. I would then try another release to see if we've cleared everything up.21:13
*** felipemonteiro has joined #openstack-infra21:26
openstackgerritMerged openstack-infra/project-config master: move git configuration for release jobs to ansible tasks  https://review.openstack.org/51386421:32
openstackgerritMerged openstack-infra/project-config master: Remove now redundant git review setup  https://review.openstack.org/51390821:32
tonybmordred: Thanks.  I'll check out the tox-siblings work21:41
*** nicolasbock has quit IRC21:46
*** hongbin_ has joined #openstack-infra21:47
*** hongbin__ has joined #openstack-infra21:49
*** hongbin has quit IRC21:49
*** hongbin_ has quit IRC21:51
*** leyal has quit IRC22:03
*** leyal has joined #openstack-infra22:03
*** dimak has quit IRC22:04
*** lihi has quit IRC22:04
*** markvoelker has joined #openstack-infra22:04
*** dimak has joined #openstack-infra22:04
*** lihi has joined #openstack-infra22:04
openstackgerritMerged openstack/os-client-config master: Add shade-tox-tips jobs  https://review.openstack.org/51391522:04
openstackgerritMerged openstack/os-client-config master: Added nat_source flag for networks.  https://review.openstack.org/51375122:04
openstackgerritMerged openstack/os-client-config master: Fix doc typo  https://review.openstack.org/51383622:04
*** thorst has joined #openstack-infra22:08
*** felipemonteiro has quit IRC22:08
smcginnisRelease-test job was a success. Released, announced, and proposed to requirements.22:12
*** thorst has quit IRC22:12
*** yamahata has quit IRC22:13
*** jbadiapa has quit IRC22:15
tonybsmcginnis: \o/22:16
*** jbadiapa has joined #openstack-infra22:31
*** slaweq has quit IRC22:37
*** hongbin__ has quit IRC22:39
*** markvoelker has quit IRC22:48
*** edmondsw has joined #openstack-infra22:48
*** markvoelker has joined #openstack-infra22:53
*** edmondsw has quit IRC22:53
*** jamesmcarthur has joined #openstack-infra23:05
*** markvoelker has quit IRC23:06
*** thorst has joined #openstack-infra23:08
*** thorst has quit IRC23:09
*** jamesmcarthur has quit IRC23:09
*** slaweq has joined #openstack-infra23:10
mordredsmcginnis, tonyb: woot23:11
*** armaan has quit IRC23:12
inc0hey guys, I might need some help with debugging jobs23:14
inc0https://review.openstack.org/#/c/512779/ <- this thing - galera can't build cluster because messages times out23:15
inc0I used these multinode playbooks that's supposed tu build tunnels, but didn't help23:15
mordredinc0: I don't see multinode in that one - is that in a previous patchset?23:20
mordredah - I see whatyou're doing - one sec23:20
inc0ceph one should be multinode23:20
inc0thanks mordred23:20
SamYapleinc0: but you arent using the tunnels23:23
SamYapleinc0: the logs show you trying to connect to the public ip address23:23
inc0172.24.4.0/24 is tunnel23:24
inc0actually it's 172.24.4.0/23 I believe23:24
inc0https://github.com/openstack-infra/zuul-jobs/blob/master/roles/multi-node-bridge/defaults/main.yaml#L623:24
SamYapleah i see your point23:25
SamYaplelets see what configs youre dropping23:25
inc0http://logs.openstack.org/79/512779/13/check/kolla-ansible-ubuntu-source-ceph/d609bd6/secondary1/logs/kolla_configs/mariadb/galera.cnf23:26
inc0fyi without tunnels I had same issue - that's why I wanted to do tunnels in the first place23:26
SamYaplewell you should definetely be doing the tunnels, i agree23:27
inc0and tunnels I do23:27
inc0now when I look at it, tunnel mtu is 1450, which can be issue if nodepool uses vxlan too23:28
inc0let's try lowering it to 140023:28
SamYapleinc0: http://logs.openstack.org/79/512779/13/check/kolla-ansible-ubuntu-source-ceph/d609bd6/primary/logs/kolla/mariadb/mariadb.txt.gz23:28
SamYaplethats showing your issue23:28
SamYaplelooks like multinode mariadb bootstrap might be wonky23:29
SamYapleany recent changes?23:29
inc0not really23:29
inc0to me it looks like it can't build cluster because everything times out23:29
SamYaplewell thats just it though, the *first* node should be started just fine23:30
SamYaplebecause it doesnt connect to anyhtign else23:30
SamYaplebut thats failing because its trying to connect to other things23:30
inc0well it tries to connect to 2 and 3....but you're right it shouldn23:30
SamYapleinc0: you know in the #kolla channel there were alot of complaints about getting past mariadb for 5.0.0 release multinode23:32
SamYapleinc0: are you sure its not busted?23:32
inc0well I deployed 5.0 many times now... but idk really23:32
inc0we had multinode before zuulv3 and it worked23:32
inc0it was 2 node, but well23:33
SamYapleok well im jsut saying, its popped up alot in the channel23:33
SamYapleyou guys did a massive rework of the mariadb bootstrap stuff right before 5.0.023:33
inc0yeah, I know, although I've seen rabbitmq more23:33
*** tosky has quit IRC23:34
inc0not really...I mean besides become23:34
inc0but we run become here on all tasks23:34
SamYapleinc0: i was talkign about this https://github.com/openstack/kolla-ansible/commit/5cd55bf236adc4c271418bda7306ec2336fa881623:35
SamYaplethat was a few days before 5.0.023:35
inc0hmm, well let's try to rever it then23:37
SamYaplelooks like youd have to revert 3 patches in total to rollback that far23:39
SamYapleinc0: when was the last time you did a from-scratch 3 node deploy?23:39
SamYaple(by hand)23:39
inc0can't remember, few weeks ago, but I'll try it23:39
*** slaweq has quit IRC23:41
* mordred hands SamYaple a pie23:43
SamYaplemordred: fun fact, i am known as sampie around my office because of my love of pies23:44
mordredwell! I handed you the right thing then23:45
SamYaple:)23:45
*** slaweq has joined #openstack-infra23:45
inc0not sure tho, I mean we did have multinode gates:/23:46
* inc0 sad23:46
inc0anyway, I kicked off fresh local master deploy23:46
SamYapleinc0: that patch might be fine, but maybe this 3 node gate is exposing a bug that your users have been hitting. eitherway, its good news (i think)23:46
inc0well I'll try and see if I can reproduce it locally23:47
SamYapleyea good first step. agreed23:47
*** bobh has joined #openstack-infra23:49
*** bobh has quit IRC23:55
pabelangerinteresting23:56
pabelangerI had a type in a playbook, for the wrong host, zuul couldn't run the playbook, and my job passed: http://logs.openstack.org/62/513962/3/check/ansible-role-shade-ubuntu-xenial/2128b00/job-output.txt.gz#_2017-10-21_20_20_29_56191523:56
pabelangertypo23:56
pabelangerwouldn't have expected -1 on that run23:56
pabelangerwill check executor why23:57
pabelanger2017-10-21 20:20:29,572 DEBUG zuul.AnsibleJob: [build: 2128b003ba1847d4b4b3b89f3a5baa5d] Ansible output: b'skipping: no hosts matched'23:57
pabelangeransible exit code 023:58
clarkbyes we abuse that behavior in d-g23:58
pabelangerah23:58
clarkbI think there is a new flag to make it error that zuul can use23:58
*** bobh has joined #openstack-infra23:58
pabelangercool23:58

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