Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2017-10-17

| Channels | #puppet-openstack index | Today | | Search | Google Search | Plain-Text | summary

All times shown according to UTC.

Time Nick Message
00:19 morazi joined #puppet-openstack
00:25 markvoelker joined #puppet-openstack
00:34 markvoelker joined #puppet-openstack
00:41 mnaser EmilienM 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 anything
00:41 EmilienM lol
00:41 EmilienM exactly
00:41 * mnaser has been putting fires out all day :(
00:41 EmilienM cool
00:41 EmilienM mnaser: everything is fixed?
00:41 mnaser if there are any zuul v3 jobs failing please ping me
00:42 mnaser EmilienM 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 backup
00:42 mnaser but pretty much the facility had a cooling issue, the inlet temps on our *most important rack* went from 16C to 25C blown into it
00:43 mnaser that 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 joined #puppet-openstack
00:43 mnaser in some sadistic way it was kinda fascinating because it was half working as it was dying
00:52 markvoelker joined #puppet-openstack
01:00 EmilienM mnaser: good luck
01:01 markvoelker joined #puppet-openstack
01:03 vnogin joined #puppet-openstack
01:11 markvoelker joined #puppet-openstack
01:20 markvoelker joined #puppet-openstack
01:58 ilbot3 joined #puppet-openstack
01:58 Topic for #puppet-openstack is now CI Status: Green | Puppet OpenStack modules https://docs.openstack.org/puppet-openstack-guide/latest/
01:59 markvoelker joined #puppet-openstack
02:08 markvoelker joined #puppet-openstack
02:17 markvoelker joined #puppet-openstack
02:26 markvoelker joined #puppet-openstack
02:35 markvoelker joined #puppet-openstack
02:37 tonyb joined #puppet-openstack
02:39 oyrogerg joined #puppet-openstack
02:44 markvoelker joined #puppet-openstack
02:50 skramaja joined #puppet-openstack
02:53 markvoelker joined #puppet-openstack
03:08 udesale joined #puppet-openstack
03:42 mtreinish joined #puppet-openstack
03:50 udesale joined #puppet-openstack
03:52 udesale joined #puppet-openstack
04:13 ykarel|away joined #puppet-openstack
04:17 markvoelker joined #puppet-openstack
04:27 markvoelker joined #puppet-openstack
04:34 markvoelker joined #puppet-openstack
04:39 openstackgerrit Merged openstack/puppet-trove master: Switch trove auth provider to keystone v3  https://review.openstack.org/512314
04:52 yprokule joined #puppet-openstack
05:23 jaganathan joined #puppet-openstack
06:02 karthiks joined #puppet-openstack
06:03 agurenko joined #puppet-openstack
06:16 bcafarel joined #puppet-openstack
06:34 jtomasek joined #puppet-openstack
06:35 jtomasek joined #puppet-openstack
06:50 markvoelker joined #puppet-openstack
06:57 ykarel_ joined #puppet-openstack
06:59 indistylo joined #puppet-openstack
07:08 shardy joined #puppet-openstack
07:16 tesseract joined #puppet-openstack
07:30 ccamacho joined #puppet-openstack
07:37 chem joined #puppet-openstack
08:14 vnogin joined #puppet-openstack
08:24 mkarpin joined #puppet-openstack
08:34 derekh joined #puppet-openstack
08:56 salmankhan joined #puppet-openstack
09:04 markvoelker joined #puppet-openstack
09:09 mkarpin joined #puppet-openstack
09:09 salmankhan joined #puppet-openstack
09:11 gfidente joined #puppet-openstack
09:11 gfidente joined #puppet-openstack
09:15 salmankhan1 joined #puppet-openstack
09:19 udesale__ joined #puppet-openstack
09:22 udesale joined #puppet-openstack
09:45 shardy joined #puppet-openstack
10:04 shardy joined #puppet-openstack
10:24 ccamacho joined #puppet-openstack
10:39 openstackgerrit joined #puppet-openstack
10:39 openstackgerrit Dmitry Tantsur proposed openstack/puppet-ironic master: Turn on DHCP logging in ironic-inspector when debug is true  https://review.openstack.org/512578
10:42 dprince joined #puppet-openstack
10:43 tesseract joined #puppet-openstack
10:46 indistylo joined #puppet-openstack
11:14 atoth joined #puppet-openstack
11:44 udesale joined #puppet-openstack
11:58 salmankhan joined #puppet-openstack
12:04 openstackgerrit Keith Schincke proposed openstack/puppet-ceph master: WIP: Set file ACLs for Ceph keyrings  https://review.openstack.org/509021
12:14 openstackgerrit Merged openstack/puppet-barbican master: apache wsgi: Exchange defaults for workers and threads  https://review.openstack.org/511718
12:17 dprince joined #puppet-openstack
12:31 vnogin_ joined #puppet-openstack
12:35 ansmith joined #puppet-openstack
12:49 mkarpin joined #puppet-openstack
13:01 pabelanger joined #puppet-openstack
13:02 yprokule joined #puppet-openstack
13:06 vnogin joined #puppet-openstack
13:09 chem joined #puppet-openstack
13:37 yprokule joined #puppet-openstack
13:40 alee joined #puppet-openstack
14:04 vnogin_ joined #puppet-openstack
14:34 mnaser mwhahaha 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 now
14:35 mwhahaha that's unfortunate
14:35 mwhahaha i know we've seen like xenial bugs cause something similar in the past
14:35 mnaser that's quite a fundamental failure with their packages and its blocking our gate :\
14:35 mwhahaha is it all scenarios?
14:36 vnogin joined #puppet-openstack
14:36 mwhahaha cause i know in the past it was like a network issue (ie ovs vs linux bridges)
14:36 mkarpin joined #puppet-openstack
14:36 mnaser i mean ive def seen in scenario1 a few times
14:37 mnaser http://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
14:37 mwhahaha neutron.agent.linux.utils [-] Exit code: 2; Stdin: ; Stdout: ; Stderr: bind: Cannot assign requested address
14:37 mwhahaha that's an interesting error message
14:38 mnaser http://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 related
14:38 mwhahaha probably
14:39 mnaser https://bugs.launchpad.net/neutron/+bug/1507078
14:39 openstack Launchpad bug 1507078 in neutron "arping for floating IPs fail on newer kernels" [Medium,Fix released] - Assigned to Brian Haley (brian-haley)
14:40 mwhahaha you can poke corycb or jamespage on #ubuntu-server if you need specific fixes in the UCA packages
14:40 mwhahaha coreycb rather
14:40 mnaser https://ihrachyshka.com/2017/05/17/gratuitous-arp-for-openstack-neutron/
14:40 andreww joined #puppet-openstack
14:40 mnaser ctrl+f bind: Cannot assign requested address
14:46 mnaser EmilienM 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 joined #puppet-openstack
14:46 EmilienM remove ubuntu jobs
14:46 EmilienM like I said 1000x times :D
14:46 mwhahaha make them non-voting
14:46 * mwhahaha is less axey than EmilienM
14:49 mnaser EmilienM 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 up
14:49 mnaser and now we can do that without infra
14:50 mnaser THANKS ZUUL V3
14:51 EmilienM yeah
14:51 EmilienM :D
14:53 openstackgerrit Mohammed Naser proposed openstack/puppet-openstack-integration master: Set Ubuntu Xenial to non-voting  https://review.openstack.org/512652
14:53 * mnaser watches status
14:54 mnaser yep they're all non voting in status, have a look EmilienM mwhahaha ^
14:55 jtomasek joined #puppet-openstack
14:58 jtomasek joined #puppet-openstack
15:18 mwhahaha i will say yay v3 on this one
15:18 mwhahaha makes that way easier
15:18 markvoelker joined #puppet-openstack
15:19 EmilienM mwhahaha: that's why I asked dmsimard on #tripleo if we could do the same for tripleo jobs
15:19 EmilienM mwhahaha: import the layout in, let's say, tripleo-ci repo
15:19 EmilienM it should be a copy-pasta
15:24 openstackgerrit Dan Radez proposed openstack/puppet-ironic master: Conditionalize syslinux requirement  https://review.openstack.org/512393
15:28 EmilienM dmsimard: 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 mnaser EmilienM 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 EmilienM ah ok
15:28 EmilienM I see now
15:29 EmilienM mnaser: can we tell puppet-nova to get layout from p-o-i without any duplication?
15:29 dmsimard EmilienM: see my reply on #tripleo, no
15:29 mnaser EmilienM unfortunately no
15:29 dmsimard EmilienM: the jobs, templates and nodesets are shared between projects, but projects can only set their own layout
15:30 dmsimard project-config is a 'core' trusted project which can set layouts/pipelines for every project
15:31 openstackgerrit Dan Radez proposed openstack/puppet-ironic master: Adding support for UEFI client arch type 11 (EFI aarch64)  https://review.openstack.org/511877
15:32 EmilienM ok
15:33 EmilienM I see now
15:49 vnogin joined #puppet-openstack
16:11 ntpttr_laptop joined #puppet-openstack
16:23 vnogin joined #puppet-openstack
16:27 Lokesh_Jain__ joined #puppet-openstack
17:14 mnaser so ubuntu folks say that they ran tempest against a pike deployment and it has no problems
17:14 mnaser so i guess we're doing something wrong, somewher
17:14 mnaser i also noticed its always scenario1 failing
17:15 mnaser the only differences i can see is.. vitrage? (but that shouldn't be too related)
17:15 mnaser "om rpc" is amqp1
17:15 mnaser related?
17:20 EmilienM mnaser: what's the logs trace?
17:22 mnaser http://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 issue
17:30 EmilienM ouch
17:30 EmilienM mnaser: yeah, not rpm I thikn
17:30 EmilienM think
17:37 openstackgerrit Wojciech Dec proposed openstack/puppet-neutron master: Add default parameters to cisco vts ml2 plugin manifest  https://review.openstack.org/512759
17:56 mnaser wth
17:56 mnaser it just passed in gate
17:57 mnaser but canonical folks pointed me to this https://bugs.launchpad.net/neutron/+bug/1717302
17:57 openstack Launchpad bug 1717302 in neutron "Tempest floatingip scenario tests failing on DVR Multinode setup with HA" [High,Confirmed]
18:00 EmilienM mnaser: magic!
18:01 openstackgerrit Keith Schincke proposed openstack/puppet-ceph master: WIP: Set file ACLs for Ceph keyrings  https://review.openstack.org/509021
18:01 vnogin joined #puppet-openstack
18:02 rwsu joined #puppet-openstack
18:03 openstackgerrit Merged openstack/puppet-openstack-integration master: Set Ubuntu Xenial to non-voting  https://review.openstack.org/512652
18:28 vnogin joined #puppet-openstack
18:30 mnaser puppet cores - https://review.openstack.org/#/c/509568/ and https://review.openstack.org/#/c/509569/ - to fix stuff in stable branch
18:31 EmilienM mnaser: ack
18:31 EmilienM mnaser: ah
18:31 EmilienM it won't merge
18:31 EmilienM because the p-o-i aren't merged
18:32 mnaser EmilienM p-o-i doesnt run for this
18:32 EmilienM mnaser: you have it in Depends-On
18:32 EmilienM so the patch won't merge
18:32 mnaser oh
18:32 mnaser oh
18:33 mnaser a recheck should fix that i spoke with infra let me test
18:45 rwsu joined #puppet-openstack
18:58 vnogin joined #puppet-openstack
19:26 markvoelker joined #puppet-openstack
19:58 openstackgerrit Benedikt Trefzer proposed openstack/puppet-horizon master: add parameter to overwrite/add wsgi process options  https://review.openstack.org/512810
20:04 openstackgerrit Benedikt Trefzer proposed openstack/puppet-horizon master: add parameter to overwrite/add wsgi process options  https://review.openstack.org/512810
20:07 jtomasek joined #puppet-openstack
20:15 markvoelker_ joined #puppet-openstack
20:26 openstackgerrit Alex Schultz proposed openstack/puppet-aodh master: Prepare for Queens Milestone 1  https://review.openstack.org/512818
20:26 openstackgerrit Alex Schultz proposed openstack/puppet-barbican master: Prepare for Queens Milestone 1  https://review.openstack.org/512819
20:26 openstackgerrit Alex Schultz proposed openstack/puppet-ceilometer master: Prepare for Queens Milestone 1  https://review.openstack.org/512820
20:27 openstackgerrit Alex Schultz proposed openstack/puppet-congress master: Prepare for Queens Milestone 1  https://review.openstack.org/512821
20:27 openstackgerrit Alex Schultz proposed openstack/puppet-cinder master: Prepare for Queens Milestone 1  https://review.openstack.org/512822
20:27 openstackgerrit Alex Schultz proposed openstack/puppet-designate master: Prepare for Queens Milestone 1  https://review.openstack.org/512823
20:27 openstackgerrit Alex Schultz proposed openstack/puppet-ec2api master: Prepare for Queens Milestone 1  https://review.openstack.org/512824
20:27 openstackgerrit Alex Schultz proposed openstack/puppet-glance master: Prepare for Queens Milestone 1  https://review.openstack.org/512825
20:27 openstackgerrit Alex Schultz proposed openstack/puppet-gnocchi master: Prepare for Queens Milestone 1  https://review.openstack.org/512826
20:27 openstackgerrit Alex Schultz proposed openstack/puppet-heat master: Prepare for Queens Milestone 1  https://review.openstack.org/512827
20:27 openstackgerrit Alex Schultz proposed openstack/puppet-horizon master: Prepare for Queens Milestone 1  https://review.openstack.org/512828
20:27 openstackgerrit Alex Schultz proposed openstack/puppet-ironic master: Prepare for Queens Milestone 1  https://review.openstack.org/512829
20:27 openstackgerrit Alex Schultz proposed openstack/puppet-keystone master: Prepare for Queens Milestone 1  https://review.openstack.org/512830
20:27 openstackgerrit Alex Schultz proposed openstack/puppet-magnum master: Prepare for Queens Milestone 1  https://review.openstack.org/512831
20:27 openstackgerrit Alex Schultz proposed openstack/puppet-manila master: Prepare for Queens Milestone 1  https://review.openstack.org/512832
20:28 openstackgerrit Alex Schultz proposed openstack/puppet-mistral master: Prepare for Queens Milestone 1  https://review.openstack.org/512833
20:28 openstackgerrit Alex Schultz proposed openstack/puppet-murano master: Prepare for Queens Milestone 1  https://review.openstack.org/512834
20:28 openstackgerrit Alex Schultz proposed openstack/puppet-neutron master: Prepare for Queens Milestone 1  https://review.openstack.org/512835
20:28 openstackgerrit Alex Schultz proposed openstack/puppet-nova master: Prepare for Queens Milestone 1  https://review.openstack.org/512836
20:28 openstackgerrit Alex Schultz proposed openstack/puppet-octavia master: Prepare for Queens Milestone 1  https://review.openstack.org/512837
20:28 openstackgerrit Alex Schultz proposed openstack/puppet-openstack_extras master: Prepare for Queens Milestone 1  https://review.openstack.org/512838
20:28 openstackgerrit Alex Schultz proposed openstack/puppet-openstacklib master: Prepare for Queens Milestone 1  https://review.openstack.org/512839
20:28 openstackgerrit Alex Schultz proposed openstack/puppet-oslo master: Prepare for Queens Milestone 1  https://review.openstack.org/512840
20:28 openstackgerrit Alex Schultz proposed openstack/puppet-ovn master: Prepare for Queens Milestone 1  https://review.openstack.org/512841
20:28 openstackgerrit Alex Schultz proposed openstack/puppet-panko master: Prepare for Queens Milestone 1  https://review.openstack.org/512842
20:28 openstackgerrit Alex Schultz proposed openstack/puppet-sahara master: Prepare for Queens Milestone 1  https://review.openstack.org/512843
20:28 openstackgerrit Alex Schultz proposed openstack/puppet-swift master: Prepare for Queens Milestone 1  https://review.openstack.org/512844
20:29 openstackgerrit Alex Schultz proposed openstack/puppet-tacker master: Prepare for Queens Milestone 1  https://review.openstack.org/512845
20:29 openstackgerrit Alex Schultz proposed openstack/puppet-tempest master: Prepare for Queens Milestone 1  https://review.openstack.org/512846
20:29 openstackgerrit Alex Schultz proposed openstack/puppet-trove master: Prepare for Queens Milestone 1  https://review.openstack.org/512847
20:29 openstackgerrit Alex Schultz proposed openstack/puppet-vswitch master: Prepare for Queens Milestone 1  https://review.openstack.org/512848
20:29 openstackgerrit Alex Schultz proposed openstack/puppet-watcher master: Prepare for Queens Milestone 1  https://review.openstack.org/512849
20:29 openstackgerrit Alex Schultz proposed openstack/puppet-zaqar master: Prepare for Queens Milestone 1  https://review.openstack.org/512850
20:31 mwhahaha mnaser: not sure if we should release puppet-vitrage for milestone 1? since we're still trying to release it for pike
20:31 mwhahaha i could add it to the list and bump it to 1.1.0 with no changes
20:53 xarses joined #puppet-openstack
21:11 mwhahaha meh 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_480131
21:12 mwhahaha also we seem to be running beaker/unit tests on just metadata.json changes which I think is a regression
21:13 mwhahaha it hink we only ran the lint/syntax/buld previously
21:13 EmilienM sounds like something related to zuul v3 changes
21:14 mwhahaha yea
21:17 * mwhahaha converts run-build-tests.yaml to proper ansible
21:19 ntpttr_laptop joined #puppet-openstack
21:22 openstackgerrit Alex Schultz proposed openstack/puppet-openstack-integration master: Update module build job to ansible  https://review.openstack.org/512866
21:25 openstackgerrit Alex Schultz proposed openstack/puppet-aodh master: Prepare for Queens Milestone 1  https://review.openstack.org/512818
21:28 xarses joined #puppet-openstack
21:35 ansmith joined #puppet-openstack
21:36 ntpttr_laptop__ joined #puppet-openstack
22:11 mwhahaha dmsimard: are vars defined in a child job available to the parent job?
22:25 openstackgerrit Alex Schultz proposed openstack/puppet-openstack-integration master: Update module build job to ansible  https://review.openstack.org/512866
22:35 dmsimard mwhahaha: the variables are merged, childs taking precedence.
22:35 mwhahaha interesting
22:36 dmsimard Variables defined in a job are available to all playbooks: they're declared in the inventory
22:44 alee joined #puppet-openstack
22:55 openstackgerrit Alex Schultz proposed openstack/puppet-openstack-integration master: Update module build job to ansible  https://review.openstack.org/512866
22:55 mwhahaha http://my1.fr/files/emilien-right-now.jpg
23:08 dmsimard I see you have EmilienM's bookmark too
23:09 mnaser hi i can help unbreak what i broke
23:09 openstackgerrit Keith Schincke proposed openstack/puppet-ceph master: WIP: Set file ACLs for Ceph keyrings  https://review.openstack.org/509021
23:10 dmsimard mwhahaha: wow man
23:10 dmsimard mwhahaha: that review is something else
23:10 mwhahaha wat
23:11 dmsimard hang on, let me comment
23:11 mwhahaha yes plz
23:11 mnaser https://review.openstack.org/#/c/509204/ + https://review.openstack.org/#/c/509202/ + https://review.openstack.org/#/c/509205/
23:11 mnaser stable branch job fixes
23:12 mwhahaha mnaser: the build job might not work, but we'll see
23:12 mnaser i dont know if that block: thing works
23:13 mwhahaha why not?
23:13 mnaser .. because i've never used it but that's not a very valid reason, haha
23:13 mwhahaha :o
23:13 mnaser so i was going to find out if it was a thing today
23:13 mnaser :p
23:13 mwhahaha evidently we use it in tripleo-quickstart
23:14 mwhahaha i'm not sure of the internals of the apt/yum modules so i don't know if they clear the caches
23:14 mnaser afaik they do and they even apt-get update if-need-be
23:15 * mnaser plays elevator music while we wait for job to run
23:15 mnaser mwhahaha: 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 :p
23:16 mwhahaha willdo
23:24 dmsimard mwhahaha: added a comment
23:24 dmsimard sorry
23:24 mwhahaha ew
23:25 mwhahaha the cache cleaning should be part of ansible imho
23:25 mwhahaha but i guess you don't know when you're installing a repo deb/rpm
23:26 mwhahaha let me see if this works prior to refactoring
23:27 dmsimard mwhahaha: does infra not reverse proxy/mirror puppetlabs repo ?
23:27 mwhahaha i don't believe so
23:30 dmsimard The vars/ thing from ansible is the equivalent pattern from ::params in puppet to handle differences in distro
23:30 mwhahaha yea i know
23:30 mwhahaha just it's so much extra stuff to accomplish something this
23:32 mwhahaha 9 basic lines of shell script replacing with 3 files and 40 lines of yaml
23:32 mwhahaha seems excessive
23:32 * mwhahaha shrugs
23:33 mwhahaha at that point i'm half tempted to just stick it back as a shell task
23:33 vnogin joined #puppet-openstack
23:34 mwhahaha hey it worked
23:35 mwhahaha oh no it didn't
23:35 mwhahaha :/
23:37 mwhahaha dmsimard: for v3 do we put roles/ in the playbooks/ folder or is it in the root?
23:39 dmsimard mwhahaha: if you want to share roles with other repos, it needs to be at the root of the repo in roles/
23:39 mwhahaha meh
23:39 mwhahaha not sure i like that structure but ok
23:39 dmsimard <repository_root>/roles is included as a role path by default
23:39 mnaser itll be a problem in our case mwhahaha
23:40 dmsimard Yeah, there's been some comments around how to make it more flexible
23:40 mwhahaha mnaser: how so?
23:40 mnaser dmsimard because the job is in p-o-i and the $project will be puppet-foo
23:40 dmsimard Right now that's how it works but it might be improved in the future
23:40 mnaser so puppet-foo/roles will be in the path, but i dont think p-o-i/roles will be
23:40 mnaser ok
23:40 mnaser i think im wrong
23:40 mwhahaha ok back to shell script then
23:40 dmsimard mnaser: you need to set roles: directive in the job
23:40 mwhahaha i tried
23:41 dmsimard mnaser: roles: - openstack/puppet-openstack-integration
23:41 dmsimard (on my phone..)
23:41 mnaser dmsimard ah yes good point
23:41 mnaser i think the hardest part is fixing jobs while making ./run_tests.sh still consumeable
23:42 mnaser oh thats a nicely though out role dmsimard
23:42 dmsimard mnaser: and ideally not breaking external usage :(
23:43 mnaser i think yeah as we move we'll slowly role-things-out .. ha get it
23:43 dmsimard What, agent thing ?
23:43 mnaser the install-puppet-agent role
23:43 openstackgerrit Alex Schultz proposed openstack/puppet-openstack-integration master: Fix puppet-agent install  https://review.openstack.org/512866
23:43 dmsimard Bah I've written so much that it just comes out on it's own
23:43 * mwhahaha gives roles the finger for now
23:44 dmsimard I'm "fluent" in ansible the same way I was fluent in puppet like 3 years ago
23:44 mwhahaha just cause you're fluent doesn't mean it's a good idea :D
23:45 mnaser dmsimard now you have bolt so you have both ha-ha-ha
23:45 dmsimard mwhahaha: Established patterns and best practices ? :(
23:45 dmsimard mnaser: 5 years later
23:45 mnaser lets 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:46 mnaser i still think it's really stupid that the orchestration stuff is part of PE
23:46 dmsimard I don't have a lot of bandwidth to drive actual writing stuff but please add me to reviews and I can help
23:46 mwhahaha dmsimard: 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 do
23:47 mwhahaha having ansible roles in a puppet module seems completely not a set of established patterns and best practices
23:47 dmsimard mwhahaha: I don't disagree
23:47 dmsimard Please write a puppet bolt driver for zuul v4
23:47 dmsimard :D
23:47 mnaser lmao
23:47 mwhahaha pfft bolt
23:47 mwhahaha glorified mcollective
23:47 * mwhahaha wants to stop having to rewrite everything every other year
23:48 dmsimard mwhahaha: I'll point out that your puppet modules are pip installable
23:48 dmsimard (╯°□°)╯︵ ┻━┻
23:48 mwhahaha thanks reno
23:48 dmsimard They need to be in rubygems and galaxy too
23:48 * mwhahaha blames upstream openstack assumptions for all of these things
23:48 dmsimard Then we're in business
23:48 mnaser lol
23:49 mwhahaha aaaaanyway
23:49 mwhahaha i'm not looking forward to tripleo ci v3
23:49 mnaser mwhahaha forgot to recheck the puppet-aodh change
23:49 mnaser ?
23:50 dmsimard It's an opportunity of looking at 5 years of legacy jobs and bash and do it right ?
23:50 mwhahaha dmsimard: we haven't moved off of tripleo-ci yet
23:50 mwhahaha dmsimard: what makes you think any of that is going away
23:50 dmsimard lol
23:50 mwhahaha and don't get me started on what we moved to
23:50 * mwhahaha sighs
23:51 mwhahaha new and shiney for the win?
23:51 mwhahaha if new and shiney is 60% of what we had with no tests
23:51 mwhahaha WOOOO
23:51 dmsimard I'll... go get some alcohol now.
23:51 mwhahaha yes plz
23:52 mwhahaha speaking of which, are there ways to write unit tests for ansible roles?
23:52 * mwhahaha hasn't looked lately
23:52 dmsimard Unit tests not so much, but functional/integration tests
23:52 dmsimard Hang on
23:53 dmsimard mwhahaha: example: https://github.com/openstack-infra/openstack-zuul-jobs/blob/master/tests/multi-node-firewall.yaml
23:54 mnaser i always thought unit tests were kinda weird for something like puppet or ansible
23:54 dmsimard The "assert" module
23:54 mwhahaha they are if you're just doing 1:1
23:54 mwhahaha they aren't if you have logic in the module
23:54 mwhahaha so testing all the when conditions and items for ansible should be a thing
23:55 dmsimard mwhahaha: the upstream integration tests for the different modules are here https://github.com/ansible/ansible/tree/devel/test/integration/targets
23:55 mwhahaha yea those are for modules which i would assume is a thing
23:55 mwhahaha but the actual developed roles should have unit testing
23:55 * mwhahaha shrugs
23:55 dmsimard Like https://github.com/ansible/ansible/blob/devel/test/integration/targets/include_vars/tasks/main.yml
23:55 dmsimard My first link was an integration test for a role
23:56 mwhahaha i guess you could write a bunch of integration tests with your various conditions
23:57 mwhahaha anyway i'm going to run and go eat some dinner
23:57 mwhahaha i may or maynot be back later to check on that build task
23:57 mwhahaha mnaser: if it needs fixing, feel free to take over
23:57 mnaser ack
23:57 mnaser mwhahaha bon appetit
23:57 mnaser retry_limit
23:57 mnaser ya broke it
23:57 mnaser :P
23:57 mwhahaha i win
23:58 mnaser enjoy
23:58 openstackgerrit Alex Schultz proposed openstack/puppet-openstack-integration master: Fix puppet-agent install  https://review.openstack.org/512866
23:59 openstackgerrit Mohammed Naser proposed openstack/puppet-openstack-integration master: Fix puppet-agent install  https://review.openstack.org/512866
23:59 mwhahaha :D i beat ya to it
23:59 mnaser oh
23:59 mnaser haha
23:59 mnaser i was so confused for a second
23:59 mnaser :p

| Channels | #puppet-openstack index | Today | | Search | Google Search | Plain-Text | summary