Tuesday, 2017-10-17

*** morazi has quit IRC00:08
*** markvoelker_ has quit IRC00:17
*** morazi has joined #puppet-openstack00:19
*** markvoelker has joined #puppet-openstack00:25
*** markvoelker has quit IRC00:29
*** markvoelker has joined #puppet-openstack00:34
*** markvoelker has quit IRC00:39
mnaserEmilienM 6 years later but better than ever.  i believe i converted the integration jobs only (those are the ones that merged).  the remaining legacy *should* be working .. so they have to be converted, however, they aren't blocking anything00:41
EmilienMlol00:41
EmilienMexactly00:41
* mnaser has been putting fires out all day :(00:41
EmilienMcool00:41
EmilienMmnaser: everything is fixed?00:41
mnaserif there are any zuul v3 jobs failing please ping me00:41
mnaserEmilienM well, we were pretty much running on a single core router which is terrifying to me.  so we managed to source another router and get it up and running as a backup00:42
mnaserbut pretty much the facility had a cooling issue, the inlet temps on our *most important rack* went from 16C to 25C blown into it00:42
mnaserthat was enough to makes things nice and hot.  most important core routers are top of rack, heat rises so the top of the rack got nice and toasty, managedo to kill the psu (a slow death too)00:43
*** markvoelker has joined #puppet-openstack00:43
mnaserin some sadistic way it was kinda fascinating because it was half working as it was dying00:43
*** markvoelker has quit IRC00:48
*** markvoelker has joined #puppet-openstack00:52
*** markvoelker has quit IRC00:57
EmilienMmnaser: good luck01:00
*** markvoelker has joined #puppet-openstack01:01
*** vnogin has joined #puppet-openstack01:03
*** markvoelker has quit IRC01:06
*** vnogin has quit IRC01:07
*** markvoelker has joined #puppet-openstack01:11
*** markvoelker has quit IRC01:15
*** markvoelker has joined #puppet-openstack01:20
*** ilbot3 has quit IRC01:46
*** markvoelker has quit IRC01:53
*** ilbot3 has joined #puppet-openstack01:58
*** markvoelker has joined #puppet-openstack01:59
*** morazi has quit IRC02:02
*** markvoelker has quit IRC02:03
*** markvoelker has joined #puppet-openstack02:08
*** markvoelker has quit IRC02:12
*** markvoelker has joined #puppet-openstack02:17
*** markvoelker has quit IRC02:21
*** markvoelker has joined #puppet-openstack02:26
*** markvoelker has quit IRC02:30
*** markvoelker has joined #puppet-openstack02:35
*** tonyb has quit IRC02:36
*** tonyb has joined #puppet-openstack02:37
*** oyrogerg has quit IRC02:38
*** oyrogerg has joined #puppet-openstack02:39
*** markvoelker has quit IRC02:40
*** markvoelker has joined #puppet-openstack02:44
*** markvoelker has quit IRC02:49
*** skramaja has joined #puppet-openstack02:50
*** markvoelker has joined #puppet-openstack02:53
*** udesale has joined #puppet-openstack03:08
*** markvoelker has quit IRC03:27
*** mtreinish has quit IRC03:42
*** mtreinish has joined #puppet-openstack03:42
*** udesale has quit IRC03:47
*** udesale has joined #puppet-openstack03:50
*** udesale has quit IRC03:52
*** udesale has joined #puppet-openstack03:53
*** ykarel|away has joined #puppet-openstack04:13
*** ykarel|away is now known as ykarel04:13
*** markvoelker has joined #puppet-openstack04:17
*** markvoelker has quit IRC04:22
*** markvoelker has joined #puppet-openstack04:27
*** markvoelker has quit IRC04:33
*** markvoelker has joined #puppet-openstack04:34
openstackgerritMerged openstack/puppet-trove master: Switch trove auth provider to keystone v3  https://review.openstack.org/51231404:39
*** yprokule has joined #puppet-openstack04:52
*** jaganathan has joined #puppet-openstack05:23
*** ccamacho has quit IRC05:28
*** markvoelker has quit IRC05:54
*** karthiks has joined #puppet-openstack06:02
*** agurenko has joined #puppet-openstack06:03
*** bcafarel has quit IRC06:14
*** bcafarel has joined #puppet-openstack06:16
*** jtomasek has joined #puppet-openstack06:34
*** jtomasek has quit IRC06:34
*** jtomasek has joined #puppet-openstack06:35
*** markvoelker has joined #puppet-openstack06:50
*** ykarel_ has joined #puppet-openstack06:57
*** indistylo has joined #puppet-openstack06:59
*** ykarel has quit IRC06:59
*** shardy has joined #puppet-openstack07:08
*** tesseract has joined #puppet-openstack07:16
*** indistylo has quit IRC07:28
*** ccamacho has joined #puppet-openstack07:30
*** chem has joined #puppet-openstack07:37
*** jpena|off is now known as jpena07:43
*** amoralej|off is now known as amoralej07:49
*** skramaja has quit IRC08:07
*** mkarpin has quit IRC08:09
*** vnogin has joined #puppet-openstack08:14
*** mkarpin has joined #puppet-openstack08:24
*** ykarel_ is now known as ykarel08:32
*** derekh has joined #puppet-openstack08:34
*** jaganathan has quit IRC08:41
*** dtantsur|afk is now known as dtantsur08:43
*** salmankhan has joined #puppet-openstack08:56
*** mkarpin has quit IRC08:58
*** markvoelker has quit IRC09:03
*** markvoelker has joined #puppet-openstack09:04
*** salmankhan has quit IRC09:05
*** mkarpin has joined #puppet-openstack09:09
*** salmankhan has joined #puppet-openstack09:09
*** gfidente has joined #puppet-openstack09:11
*** gfidente has quit IRC09:11
*** gfidente has joined #puppet-openstack09:11
*** ccamacho has quit IRC09:11
*** salmankhan1 has joined #puppet-openstack09:15
*** salmankhan has quit IRC09:15
*** salmankhan1 is now known as salmankhan09:15
*** udesale__ has joined #puppet-openstack09:19
*** udesale has quit IRC09:19
*** udesale__ has quit IRC09:22
*** udesale has joined #puppet-openstack09:22
*** shardy has quit IRC09:32
*** chem has quit IRC09:32
*** shardy has joined #puppet-openstack09:45
*** ykarel is now known as ykarel|lunch09:47
*** shardy has quit IRC10:01
*** shardy has joined #puppet-openstack10:04
*** ccamacho has joined #puppet-openstack10:24
*** openstackgerrit has quit IRC10:33
*** openstackgerrit has joined #puppet-openstack10:39
openstackgerritDmitry Tantsur proposed openstack/puppet-ironic master: Turn on DHCP logging in ironic-inspector when debug is true  https://review.openstack.org/51257810:39
*** dprince has joined #puppet-openstack10:42
*** dprince has quit IRC10:42
*** tesseract has quit IRC10:43
*** tesseract has joined #puppet-openstack10:43
*** indistylo has joined #puppet-openstack10:46
*** ansmith has quit IRC10:53
*** ykarel|lunch is now known as ykarel10:58
*** atoth has joined #puppet-openstack11:14
*** shardy is now known as shardy_lunch11:18
*** dtantsur is now known as dtantsur|brb11:28
*** udesale has quit IRC11:38
*** udesale has joined #puppet-openstack11:44
*** salmankhan has quit IRC11:49
*** yprokule has quit IRC11:49
*** amoralej is now known as amoralej|lunch11:57
*** salmankhan has joined #puppet-openstack11:58
*** udesale has quit IRC12:01
*** shardy_lunch is now known as shardy12:02
*** jpena is now known as jpena|lunch12:04
openstackgerritKeith Schincke proposed openstack/puppet-ceph master: WIP: Set file ACLs for Ceph keyrings  https://review.openstack.org/50902112:04
openstackgerritMerged openstack/puppet-barbican master: apache wsgi: Exchange defaults for workers and threads  https://review.openstack.org/51171812:14
*** dprince has joined #puppet-openstack12:17
*** vnogin_ has joined #puppet-openstack12:31
*** alee has quit IRC12:32
*** trown|outtypewww is now known as trown12:33
*** vnogin has quit IRC12:34
*** ykarel is now known as ykarel|away12:34
*** ansmith has joined #puppet-openstack12:35
*** mkarpin has quit IRC12:42
*** mkarpin has joined #puppet-openstack12:49
*** jpena|lunch is now known as jpena12:51
*** pabelanger has quit IRC13:01
*** pabelanger has joined #puppet-openstack13:01
*** yprokule has joined #puppet-openstack13:02
*** vnogin_ has quit IRC13:05
*** vnogin has joined #puppet-openstack13:06
*** amoralej|lunch is now known as amoralej13:07
*** chem has joined #puppet-openstack13:09
*** dtantsur|brb is now known as dtantsur13:21
*** yprokule has quit IRC13:37
*** yprokule has joined #puppet-openstack13:37
*** alee has joined #puppet-openstack13:40
*** trown is now known as trown|brb13:53
*** trown|brb is now known as trown14:02
*** vnogin_ has joined #puppet-openstack14:04
*** vnogin has quit IRC14:04
mnasermwhahaha i think "Unable to associate floating IP 172.24.5.13 to fixed IP 10.100.0.3 for instance 66791bfb-bcc1-46d9-9fe4-3144acc8dd09. Error: Request to https://127.0.0.1:9696/v2.0/floatingips/bc11c5eb-6999-4669-aa51-39888db15624 timed out" is an official failure in xenail now14:34
mwhahahathat's unfortunate14:35
mwhahahai know we've seen like xenial bugs cause something similar in the past14:35
mnaserthat's quite a fundamental failure with their packages and its blocking our gate :\14:35
mwhahahais it all scenarios?14:35
*** vnogin has joined #puppet-openstack14:36
*** mkarpin has quit IRC14:36
mwhahahacause i know in the past it was like a network issue (ie ovs vs linux bridges)14:36
*** mkarpin has joined #puppet-openstack14:36
mnaseri mean ive def seen in scenario1 a few times14:36
mnaserhttp://logs.openstack.org/08/511708/2/check/puppet-openstack-integration-4-scenario001-tempest-ubuntu-xenial/ab9e376/logs/neutron/neutron-l3-agent.txt.gz#_2017-10-17_11_29_13_85814:37
mwhahahaneutron.agent.linux.utils [-] Exit code: 2; Stdin: ; Stdout: ; Stderr: bind: Cannot assign requested address14:37
mwhahahathat's an interesting error message14:37
*** vnogin_ has quit IRC14:37
mnaserhttp://logs.openstack.org/08/511708/2/check/puppet-openstack-integration-4-scenario001-tempest-ubuntu-xenial/ab9e376/logs/openvswitch/ovs-vswitchd.txt.gz -- i wonder if the "No such device" is related14:38
mwhahahaprobably14:38
mnaserhttps://bugs.launchpad.net/neutron/+bug/150707814:39
openstackLaunchpad bug 1507078 in neutron "arping for floating IPs fail on newer kernels" [Medium,Fix released] - Assigned to Brian Haley (brian-haley)14:39
mwhahahayou can poke corycb or jamespage on #ubuntu-server if you need specific fixes in the UCA packages14:40
mwhahahacoreycb rather14:40
mnaserhttps://ihrachyshka.com/2017/05/17/gratuitous-arp-for-openstack-neutron/14:40
*** andreww has joined #puppet-openstack14:40
mnaserctrl+f bind: Cannot assign requested address14:40
*** andreww has quit IRC14:44
mnaserEmilienM mwhahaha so given our gates are blocked because of this issue, is it okay for us to make ubuntu jobs non voting for now (I feel llike its a bit of fu for ubuntu users.. but i have no idea when they'll get back to us) .. what do you think?14:46
*** xarses has joined #puppet-openstack14:46
EmilienMremove ubuntu jobs14:46
EmilienMlike I said 1000x times :D14:46
mwhahahamake them non-voting14:46
* mwhahaha is less axey than EmilienM 14:46
*** karthiks has quit IRC14:47
mnaserEmilienM mwhahaha ill move the ubuntu jobs to nv for now till we figure out what to do because i dont want our gate to be even more backed up14:49
mnaserand now we can do that without infra14:49
mnaserTHANKS ZUUL V314:50
*** indistylo has quit IRC14:50
EmilienMyeah14:51
EmilienM:D14:51
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration master: Set Ubuntu Xenial to non-voting  https://review.openstack.org/51265214:53
* mnaser watches status14:53
mnaseryep they're all non voting in status, have a look EmilienM mwhahaha ^14:54
*** jtomasek has quit IRC14:54
*** jtomasek has joined #puppet-openstack14:55
*** jtomasek has quit IRC14:58
*** jtomasek has joined #puppet-openstack14:58
*** agurenko has quit IRC15:14
*** ccamacho has quit IRC15:14
*** markvoelker has quit IRC15:17
mwhahahai will say yay v3 on this one15:18
mwhahahamakes that way easier15:18
*** markvoelker has joined #puppet-openstack15:18
EmilienMmwhahaha: that's why I asked dmsimard on #tripleo if we could do the same for tripleo jobs15:19
EmilienMmwhahaha: import the layout in, let's say, tripleo-ci repo15:19
EmilienMit should be a copy-pasta15:19
*** markvoelker has quit IRC15:22
openstackgerritDan Radez proposed openstack/puppet-ironic master: Conditionalize syslinux requirement  https://review.openstack.org/51239315:24
*** vnogin has quit IRC15:26
EmilienMdmsimard: why puppet-nova repo doesn't have zuul config? where do we tell puppet-nova to get zuul layout from p-o-i?15:28
mnaserEmilienM that hasn't done yet, its getting it from openstack-zuul-jobs in the zuul-legacy-projects.yaml file (or maybe it was in project-config)15:28
EmilienMah ok15:28
EmilienMI see now15:28
EmilienMmnaser: can we tell puppet-nova to get layout from p-o-i without any duplication?15:29
dmsimardEmilienM: see my reply on #tripleo, no15:29
mnaserEmilienM unfortunately no15:29
dmsimardEmilienM: the jobs, templates and nodesets are shared between projects, but projects can only set their own layout15:29
dmsimardproject-config is a 'core' trusted project which can set layouts/pipelines for every project15:30
openstackgerritDan Radez proposed openstack/puppet-ironic master: Adding support for UEFI client arch type 11 (EFI aarch64)  https://review.openstack.org/51187715:31
EmilienMok15:32
EmilienMI see now15:33
*** vnogin has joined #puppet-openstack15:49
*** vnogin has quit IRC15:54
*** owalsh_ is now known as owalsh15:59
*** trown is now known as trown|lunch16:05
*** mwynne has quit IRC16:10
*** ntpttr_laptop has joined #puppet-openstack16:11
*** yprokule has quit IRC16:16
*** vnogin has joined #puppet-openstack16:23
*** vnogin has quit IRC16:23
*** Lokesh_Jain__ has joined #puppet-openstack16:27
*** gfidente is now known as gfidente|afk16:35
*** ykarel|away has quit IRC16:38
*** tesseract has quit IRC17:02
*** trown|lunch is now known as trown17:08
*** derekh has quit IRC17:09
mnaserso ubuntu folks say that they ran tempest against a pike deployment and it has no problems17:14
mnaserso i guess we're doing something wrong, somewher17:14
mnaseri also noticed its always scenario1 failing17:14
mnaserthe only differences i can see is.. vitrage? (but that shouldn't be too related)17:15
mnaser"om rpc" is amqp117:15
mnaserrelated?17:15
*** dtantsur is now known as dtantsur|afk17:19
*** jpena is now known as jpena|off17:19
EmilienMmnaser: what's the logs trace?17:20
mnaserhttp://logs.openstack.org/08/511708/2/check/puppet-openstack-integration-4-scenario001-tempest-ubuntu-xenial/ab9e376/logs/neutron/neutron-l3-agent.txt.gz#_2017-10-17_11_29_13_858 -- maybe a kernel issue17:22
EmilienMouch17:30
EmilienMmnaser: yeah, not rpm I thikn17:30
EmilienMthink17:30
openstackgerritWojciech Dec proposed openstack/puppet-neutron master: Add default parameters to cisco vts ml2 plugin manifest  https://review.openstack.org/51275917:37
*** rwsu has quit IRC17:41
mnaserwth17:56
mnaserit just passed in gate17:56
mnaserbut canonical folks pointed me to this https://bugs.launchpad.net/neutron/+bug/171730217:57
openstackLaunchpad bug 1717302 in neutron "Tempest floatingip scenario tests failing on DVR Multinode setup with HA" [High,Confirmed]17:57
EmilienMmnaser: magic!18:00
openstackgerritKeith Schincke proposed openstack/puppet-ceph master: WIP: Set file ACLs for Ceph keyrings  https://review.openstack.org/50902118:01
*** vnogin has joined #puppet-openstack18:01
*** rwsu has joined #puppet-openstack18:02
*** vnogin has quit IRC18:03
openstackgerritMerged openstack/puppet-openstack-integration master: Set Ubuntu Xenial to non-voting  https://review.openstack.org/51265218:03
*** salmankhan has quit IRC18:19
*** vnogin has joined #puppet-openstack18:28
mnaserpuppet cores - https://review.openstack.org/#/c/509568/ and https://review.openstack.org/#/c/509569/ - to fix stuff in stable branch18:30
EmilienMmnaser: ack18:31
EmilienMmnaser: ah18:31
EmilienMit won't merge18:31
EmilienMbecause the p-o-i aren't merged18:31
mnaserEmilienM p-o-i doesnt run for this18:32
EmilienMmnaser: you have it in Depends-On18:32
EmilienMso the patch won't merge18:32
*** rwsu has quit IRC18:32
mnaseroh18:32
mnaseroh18:32
mnasera recheck should fix that i spoke with infra let me test18:33
*** amoralej is now known as amoralej|off18:35
*** rwsu has joined #puppet-openstack18:45
*** vnogin has quit IRC18:56
*** gfidente|afk is now known as gfidente18:56
*** vnogin has joined #puppet-openstack18:58
*** markvoelker has joined #puppet-openstack19:26
*** hjensas has quit IRC19:42
openstackgerritBenedikt Trefzer proposed openstack/puppet-horizon master: add parameter to overwrite/add wsgi process options  https://review.openstack.org/51281019:58
*** jtomasek has quit IRC20:00
openstackgerritBenedikt Trefzer proposed openstack/puppet-horizon master: add parameter to overwrite/add wsgi process options  https://review.openstack.org/51281020:04
*** jtomasek has joined #puppet-openstack20:07
*** markvoelker_ has joined #puppet-openstack20:15
*** markvoelker has quit IRC20:18
openstackgerritAlex Schultz proposed openstack/puppet-aodh master: Prepare for Queens Milestone 1  https://review.openstack.org/51281820:26
openstackgerritAlex Schultz proposed openstack/puppet-barbican master: Prepare for Queens Milestone 1  https://review.openstack.org/51281920:26
openstackgerritAlex Schultz proposed openstack/puppet-ceilometer master: Prepare for Queens Milestone 1  https://review.openstack.org/51282020:26
openstackgerritAlex Schultz proposed openstack/puppet-congress master: Prepare for Queens Milestone 1  https://review.openstack.org/51282120:27
openstackgerritAlex Schultz proposed openstack/puppet-cinder master: Prepare for Queens Milestone 1  https://review.openstack.org/51282220:27
openstackgerritAlex Schultz proposed openstack/puppet-designate master: Prepare for Queens Milestone 1  https://review.openstack.org/51282320:27
openstackgerritAlex Schultz proposed openstack/puppet-ec2api master: Prepare for Queens Milestone 1  https://review.openstack.org/51282420:27
openstackgerritAlex Schultz proposed openstack/puppet-glance master: Prepare for Queens Milestone 1  https://review.openstack.org/51282520:27
openstackgerritAlex Schultz proposed openstack/puppet-gnocchi master: Prepare for Queens Milestone 1  https://review.openstack.org/51282620:27
openstackgerritAlex Schultz proposed openstack/puppet-heat master: Prepare for Queens Milestone 1  https://review.openstack.org/51282720:27
openstackgerritAlex Schultz proposed openstack/puppet-horizon master: Prepare for Queens Milestone 1  https://review.openstack.org/51282820:27
openstackgerritAlex Schultz proposed openstack/puppet-ironic master: Prepare for Queens Milestone 1  https://review.openstack.org/51282920:27
openstackgerritAlex Schultz proposed openstack/puppet-keystone master: Prepare for Queens Milestone 1  https://review.openstack.org/51283020:27
openstackgerritAlex Schultz proposed openstack/puppet-magnum master: Prepare for Queens Milestone 1  https://review.openstack.org/51283120:27
openstackgerritAlex Schultz proposed openstack/puppet-manila master: Prepare for Queens Milestone 1  https://review.openstack.org/51283220:27
openstackgerritAlex Schultz proposed openstack/puppet-mistral master: Prepare for Queens Milestone 1  https://review.openstack.org/51283320:28
openstackgerritAlex Schultz proposed openstack/puppet-murano master: Prepare for Queens Milestone 1  https://review.openstack.org/51283420:28
*** jtomasek has quit IRC20:28
openstackgerritAlex Schultz proposed openstack/puppet-neutron master: Prepare for Queens Milestone 1  https://review.openstack.org/51283520:28
openstackgerritAlex Schultz proposed openstack/puppet-nova master: Prepare for Queens Milestone 1  https://review.openstack.org/51283620:28
openstackgerritAlex Schultz proposed openstack/puppet-octavia master: Prepare for Queens Milestone 1  https://review.openstack.org/51283720:28
openstackgerritAlex Schultz proposed openstack/puppet-openstack_extras master: Prepare for Queens Milestone 1  https://review.openstack.org/51283820:28
openstackgerritAlex Schultz proposed openstack/puppet-openstacklib master: Prepare for Queens Milestone 1  https://review.openstack.org/51283920:28
openstackgerritAlex Schultz proposed openstack/puppet-oslo master: Prepare for Queens Milestone 1  https://review.openstack.org/51284020:28
openstackgerritAlex Schultz proposed openstack/puppet-ovn master: Prepare for Queens Milestone 1  https://review.openstack.org/51284120:28
openstackgerritAlex Schultz proposed openstack/puppet-panko master: Prepare for Queens Milestone 1  https://review.openstack.org/51284220:28
openstackgerritAlex Schultz proposed openstack/puppet-sahara master: Prepare for Queens Milestone 1  https://review.openstack.org/51284320:28
openstackgerritAlex Schultz proposed openstack/puppet-swift master: Prepare for Queens Milestone 1  https://review.openstack.org/51284420:28
openstackgerritAlex Schultz proposed openstack/puppet-tacker master: Prepare for Queens Milestone 1  https://review.openstack.org/51284520:29
openstackgerritAlex Schultz proposed openstack/puppet-tempest master: Prepare for Queens Milestone 1  https://review.openstack.org/51284620:29
openstackgerritAlex Schultz proposed openstack/puppet-trove master: Prepare for Queens Milestone 1  https://review.openstack.org/51284720:29
openstackgerritAlex Schultz proposed openstack/puppet-vswitch master: Prepare for Queens Milestone 1  https://review.openstack.org/51284820:29
openstackgerritAlex Schultz proposed openstack/puppet-watcher master: Prepare for Queens Milestone 1  https://review.openstack.org/51284920:29
openstackgerritAlex Schultz proposed openstack/puppet-zaqar master: Prepare for Queens Milestone 1  https://review.openstack.org/51285020:29
mwhahahamnaser: not sure if we should release puppet-vitrage for milestone 1? since we're still trying to release it for pike20:31
mwhahahai could add it to the list and bump it to 1.1.0 with no changes20:31
*** ntpttr_laptop has quit IRC20:32
*** xarses has quit IRC20:42
*** ansmith has quit IRC20:43
*** xarses has joined #puppet-openstack20:53
*** trown is now known as trown|outtypewww21:07
mwhahahameh the build job is broken http://logs.openstack.org/21/512821/1/check/puppet-openstack-module-build/88f5cc6/job-output.txt.gz#_2017-10-17_20_33_45_48013121:11
mwhahahaalso we seem to be running beaker/unit tests on just metadata.json changes which I think is a regression21:12
mwhahahait hink we only ran the lint/syntax/buld previously21:13
EmilienMsounds like something related to zuul v3 changes21:13
mwhahahayea21:14
*** xarses has quit IRC21:14
* mwhahaha converts run-build-tests.yaml to proper ansible21:17
*** dprince has quit IRC21:19
*** ntpttr_laptop has joined #puppet-openstack21:19
*** gfidente has quit IRC21:20
openstackgerritAlex Schultz proposed openstack/puppet-openstack-integration master: Update module build job to ansible  https://review.openstack.org/51286621:22
openstackgerritAlex Schultz proposed openstack/puppet-aodh master: Prepare for Queens Milestone 1  https://review.openstack.org/51281821:25
*** alee has quit IRC21:27
*** xarses has joined #puppet-openstack21:28
*** ansmith has joined #puppet-openstack21:35
*** ntpttr_laptop__ has joined #puppet-openstack21:36
*** ntpttr_laptop has quit IRC21:36
mwhahahadmsimard: are vars defined in a child job available to the parent job?22:11
openstackgerritAlex Schultz proposed openstack/puppet-openstack-integration master: Update module build job to ansible  https://review.openstack.org/51286622:25
*** vnogin has quit IRC22:33
dmsimardmwhahaha: the variables are merged, childs taking precedence.22:35
mwhahahainteresting22:35
dmsimardVariables defined in a job are available to all playbooks: they're declared in the inventory22:36
*** xarses has quit IRC22:36
*** atoth has quit IRC22:43
*** alee has joined #puppet-openstack22:44
openstackgerritAlex Schultz proposed openstack/puppet-openstack-integration master: Update module build job to ansible  https://review.openstack.org/51286622:55
mwhahahahttp://my1.fr/files/emilien-right-now.jpg22:55
dmsimardI see you have EmilienM's bookmark too23:08
mnaserhi i can help unbreak what i broke23:09
openstackgerritKeith Schincke proposed openstack/puppet-ceph master: WIP: Set file ACLs for Ceph keyrings  https://review.openstack.org/50902123:09
dmsimardmwhahaha: wow man23:10
dmsimardmwhahaha: that review is something else23:10
mwhahahawat23:10
dmsimardhang on, let me comment23:11
mwhahahayes plz23:11
mnaserhttps://review.openstack.org/#/c/509204/ + https://review.openstack.org/#/c/509202/ + https://review.openstack.org/#/c/509205/23:11
mnaserstable branch job fixes23:11
mwhahahamnaser: the build job might not work, but we'll see23:12
mnaseri dont know if that block: thing works23:12
mwhahahawhy not?23:13
mnaser.. because i've never used it but that's not a very valid reason, haha23:13
mwhahaha:o23:13
mnaserso i was going to find out if it was a thing today23:13
mnaser:p23:13
*** ntpttr_laptop__ has quit IRC23:13
mwhahahaevidently we use it in tripleo-quickstart23:13
mwhahahai'm not sure of the internals of the apt/yum modules so i don't know if they clear the caches23:14
mnaserafaik they do and they even apt-get update if-need-be23:14
* mnaser plays elevator music while we wait for job to run23:15
mnasermwhahaha: i'll be up late today, let me know when you're getting off .. i'll work on the patch some more if needed where the gate is nice and quiet at night :p23:15
mwhahahawilldo23:16
dmsimardmwhahaha: added a comment23:24
dmsimardsorry23:24
mwhahahaew23:24
mwhahahathe cache cleaning should be part of ansible imho23:25
mwhahahabut i guess you don't know when you're installing a repo deb/rpm23:25
mwhahahalet me see if this works prior to refactoring23:26
dmsimardmwhahaha: does infra not reverse proxy/mirror puppetlabs repo ?23:27
mwhahahai don't believe so23:27
dmsimardThe vars/ thing from ansible is the equivalent pattern from ::params in puppet to handle differences in distro23:30
mwhahahayea i know23:30
mwhahahajust it's so much extra stuff to accomplish something this23:30
mwhahaha9 basic lines of shell script replacing with 3 files and 40 lines of yaml23:32
mwhahahaseems excessive23:32
* mwhahaha shrugs23:32
mwhahahaat that point i'm half tempted to just stick it back as a shell task23:33
*** vnogin has joined #puppet-openstack23:33
mwhahahahey it worked23:34
mwhahahaoh no it didn't23:35
mwhahaha:/23:35
mwhahahadmsimard: for v3 do we put roles/ in the playbooks/ folder or is it in the root?23:37
*** vnogin has quit IRC23:38
dmsimardmwhahaha: if you want to share roles with other repos, it needs to be at the root of the repo in roles/23:39
mwhahahameh23:39
mwhahahanot sure i like that structure but ok23:39
dmsimard<repository_root>/roles is included as a role path by default23:39
mnaseritll be a problem in our case mwhahaha23:39
dmsimardYeah, there's been some comments around how to make it more flexible23:40
mwhahahamnaser: how so?23:40
mnaserdmsimard because the job is in p-o-i and the $project will be puppet-foo23:40
dmsimardRight now that's how it works but it might be improved in the future23:40
mnaserso puppet-foo/roles will be in the path, but i dont think p-o-i/roles will be23:40
mnaserok23:40
mnaseri think im wrong23:40
mwhahahaok back to shell script then23:40
dmsimardmnaser: you need to set roles: directive in the job23:40
mwhahahai tried23:40
dmsimardmnaser: roles: - openstack/puppet-openstack-integration23:41
dmsimard(on my phone..)23:41
mnaserdmsimard ah yes good point23:41
mnaseri think the hardest part is fixing jobs while making ./run_tests.sh still consumeable23:41
mnaseroh thats a nicely though out role dmsimard23:42
dmsimardmnaser: and ideally not breaking external usage :(23:42
mnaseri think yeah as we move we'll slowly role-things-out .. ha get it23:43
dmsimardWhat, agent thing ?23:43
mnaserthe install-puppet-agent role23:43
openstackgerritAlex Schultz proposed openstack/puppet-openstack-integration master: Fix puppet-agent install  https://review.openstack.org/51286623:43
dmsimardBah I've written so much that it just comes out on it's own23:43
* mwhahaha gives roles the finger for now23:43
dmsimardI'm "fluent" in ansible the same way I was fluent in puppet like 3 years ago23:44
mwhahahajust cause you're fluent doesn't mean it's a good idea :D23:44
mnaserdmsimard now you have bolt so you have both ha-ha-ha23:45
dmsimardmwhahaha: Established patterns and best practices ? :(23:45
dmsimardmnaser: 5 years later23:45
mnaserlets get the release out / fix jobs and yeah we can slowly iterate on refactoring things into roles (once we figure out the best approach not to break consumers)23:45
mnaseri still think it's really stupid that the orchestration stuff is part of PE23:46
dmsimardI don't have a lot of bandwidth to drive actual writing stuff but please add me to reviews and I can help23:46
mwhahahadmsimard: i think it's more an issue of how we're wedging ansible in here and the structure required to do $trivial thing than if we were an anisible thing and this is what you do23:46
mwhahahahaving ansible roles in a puppet module seems completely not a set of established patterns and best practices23:47
dmsimardmwhahaha: I don't disagree23:47
dmsimardPlease write a puppet bolt driver for zuul v423:47
dmsimard:D23:47
mnaserlmao23:47
mwhahahapfft bolt23:47
mwhahahaglorified mcollective23:47
* mwhahaha wants to stop having to rewrite everything every other year23:47
dmsimardmwhahaha: I'll point out that your puppet modules are pip installable23:48
dmsimard(╯°□°)╯︵ ┻━┻23:48
mwhahahathanks reno23:48
dmsimardThey need to be in rubygems and galaxy too23:48
* mwhahaha blames upstream openstack assumptions for all of these things23:48
dmsimardThen we're in business23:48
mnaserlol23:48
mwhahahaaaaaanyway23:49
mwhahahai'm not looking forward to tripleo ci v323:49
mnasermwhahaha forgot to recheck the puppet-aodh change23:49
mnaser?23:49
dmsimardIt's an opportunity of looking at 5 years of legacy jobs and bash and do it right ?23:50
mwhahahadmsimard: we haven't moved off of tripleo-ci yet23:50
mwhahahadmsimard: what makes you think any of that is going away23:50
dmsimardlol23:50
mwhahahaand don't get me started on what we moved to23:50
* mwhahaha sighs23:50
mwhahahanew and shiney for the win?23:51
mwhahahaif new and shiney is 60% of what we had with no tests23:51
mwhahahaWOOOO23:51
dmsimardI'll... go get some alcohol now.23:51
mwhahahayes plz23:51
mwhahahaspeaking of which, are there ways to write unit tests for ansible roles?23:52
* mwhahaha hasn't looked lately23:52
dmsimardUnit tests not so much, but functional/integration tests23:52
dmsimardHang on23:52
dmsimardmwhahaha: example: https://github.com/openstack-infra/openstack-zuul-jobs/blob/master/tests/multi-node-firewall.yaml23:53
mnaseri always thought unit tests were kinda weird for something like puppet or ansible23:54
dmsimardThe "assert" module23:54
mwhahahathey are if you're just doing 1:123:54
mwhahahathey aren't if you have logic in the module23:54
mwhahahaso testing all the when conditions and items for ansible should be a thing23:54
dmsimardmwhahaha: the upstream integration tests for the different modules are here https://github.com/ansible/ansible/tree/devel/test/integration/targets23:55
mwhahahayea those are for modules which i would assume is a thing23:55
mwhahahabut the actual developed roles should have unit testing23:55
* mwhahaha shrugs23:55
dmsimardLike https://github.com/ansible/ansible/blob/devel/test/integration/targets/include_vars/tasks/main.yml23:55
dmsimardMy first link was an integration test for a role23:55
mwhahahai guess you could write a bunch of integration tests with your various conditions23:56
mwhahahaanyway i'm going to run and go eat some dinner23:57
mwhahahai may or maynot be back later to check on that build task23:57
mwhahahamnaser: if it needs fixing, feel free to take over23:57
mnaserack23:57
mnasermwhahaha bon appetit23:57
mnaserretry_limit23:57
mnaserya broke it23:57
mnaser:P23:57
mwhahahai win23:57
mnaserenjoy23:58
openstackgerritAlex Schultz proposed openstack/puppet-openstack-integration master: Fix puppet-agent install  https://review.openstack.org/51286623:58
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration master: Fix puppet-agent install  https://review.openstack.org/51286623:59
mwhahaha:D i beat ya to it23:59
mnaseroh23:59
mnaserhaha23:59
mnaseri was so confused for a second23:59
mnaser:p23:59

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