Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2016-11-29

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

All times shown according to UTC.

Time Nick Message
00:09 ducttape_ joined #puppet-openstack
00:34 openstackgerrit gecong proposed openstack/puppet-magnum: Fix a typo  https://review.openstack.org/403957
00:35 michapma_alt joined #puppet-openstack
00:35 jtriley joined #puppet-openstack
00:36 openstackgerrit zhongshengping proposed openstack/puppet-neutron: Add release note for updating fwaas configuration  https://review.openstack.org/403509
00:41 openstackgerrit zhongshengping proposed openstack/puppet-zaqar: Add support for Debian system  https://review.openstack.org/399980
00:50 jtriley joined #puppet-openstack
01:02 jtriley joined #puppet-openstack
01:16 dgurtner joined #puppet-openstack
01:16 dgurtner joined #puppet-openstack
01:22 openstackgerrit zhongshengping proposed openstack/puppet-ceilometer: Add hooks for external install & svc management  https://review.openstack.org/403964
01:42 openstackgerrit zhongshengping proposed openstack/puppet-designate: Deprecate host and port options  https://review.openstack.org/403967
01:51 openstackgerrit zhongshengping proposed openstack/puppet-ceilometer: Add hooks for external install & svc management  https://review.openstack.org/403964
02:25 openstackgerrit zhongshengping proposed openstack/puppet-ceilometer: Add hooks for external install & svc management  https://review.openstack.org/403964
02:48 ilbot3 joined #puppet-openstack
02:48 Topic for #puppet-openstack is now CI status: green - Puppet OpenStack modules http://docs.openstack.org/deve​loper/puppet-openstack-guide/
02:50 openstackgerrit zhongshengping proposed openstack/puppet-openstack-integration: Fix to listen on for designate  https://review.openstack.org/403985
03:22 openstackgerrit Xing Zhou proposed openstack/puppet-murano: Fix default murano client package name error for RedHat platforms  https://review.openstack.org/403999
03:25 jtriley joined #puppet-openstack
03:26 openstackgerrit zhongshengping proposed openstack/puppet-ceilometer: Add hooks for external install & svc management  https://review.openstack.org/403964
03:30 openstackgerrit Xing Zhou proposed openstack/puppet-murano: Fix default murano client package name error for RedHat platforms  https://review.openstack.org/404002
03:38 jtriley_ joined #puppet-openstack
03:58 openstackgerrit zhongshengping proposed openstack/puppet-manila: Fix the annotation  https://review.openstack.org/404007
04:33 openstackgerrit zhongshengping proposed openstack/puppet-openstack-integration: Fix to listen on for designate  https://review.openstack.org/403985
04:42 openstackgerrit Keith Schincke proposed openstack/puppet-ceph: WIP Add support for installing Ceph rbd-mirror  https://review.openstack.org/401416
04:49 snarwade joined #puppet-openstack
04:49 llua joined #puppet-openstack
04:53 openstackgerrit zhongshengping proposed openstack/puppet-zaqar: Add support for Debian system  https://review.openstack.org/399980
05:22 _ody joined #puppet-openstack
05:28 _ody joined #puppet-openstack
06:22 jaosorior joined #puppet-openstack
06:45 fxpester joined #puppet-openstack
07:08 jaosorior joined #puppet-openstack
07:26 rasca joined #puppet-openstack
07:43 chem joined #puppet-openstack
07:47 chem joined #puppet-openstack
07:58 tdb joined #puppet-openstack
08:07 shardy joined #puppet-openstack
08:16 rmart04 joined #puppet-openstack
08:16 jpich joined #puppet-openstack
08:23 fxpester joined #puppet-openstack
08:31 beddari joined #puppet-openstack
08:38 tdb joined #puppet-openstack
08:53 paramite joined #puppet-openstack
08:56 rcernin joined #puppet-openstack
09:18 derekh joined #puppet-openstack
09:19 danpawlik joined #puppet-openstack
09:29 openstackgerrit zhongshengping proposed openstack/puppet-congress: Add server.pp for congress  https://review.openstack.org/404110
09:29 gfidente joined #puppet-openstack
09:32 openstackgerrit zhongshengping proposed openstack/puppet-ceilometer: Add hooks for external install & svc management  https://review.openstack.org/403964
09:35 openstackgerrit Alejandro Andreu proposed openstack/puppet-tripleo: Add new tripleo::network::midonet::nsdb class  https://review.openstack.org/400869
09:37 mkoderer joined #puppet-openstack
09:38 openstackgerrit zhongshengping proposed openstack/puppet-cinder: Fix lint issue  https://review.openstack.org/404113
09:45 openstackgerrit Alejandro Andreu proposed openstack/puppet-tripleo: Add new tripleo::network::midonet::nsdb class  https://review.openstack.org/400869
09:47 openstackgerrit zhongshengping proposed openstack/puppet-congress: Add server.pp for congress  https://review.openstack.org/404110
09:49 openstackgerrit joined #puppet-openstack
09:56 tdb joined #puppet-openstack
10:02 openstackgerrit Merged openstack/puppet-barbican: Add hooks for external install & svc management  https://review.openstack.org/402317
10:26 openstackgerrit Merged openstack/puppet-murano: Fix default murano client package name error for RedHat platforms  https://review.openstack.org/403999
10:32 openstackgerrit Merged openstack/puppet-murano: Fix default murano client package name error for RedHat platforms  https://review.openstack.org/404002
10:55 openstackgerrit Xing Zhou proposed openstack/puppet-murano: Fix default murano-cfapi package and service name error for RedHat platforms  https://review.openstack.org/404145
11:00 michapma_alt joined #puppet-openstack
11:02 openstackgerrit Christian Schwede proposed openstack/puppet-tripleo: Enable object-expirer on Swift proxy profile  https://review.openstack.org/404149
11:12 richm joined #puppet-openstack
11:13 openstackgerrit Alejandro Andreu proposed openstack/puppet-tripleo: Adds new tripleo::network::midonet::config class  https://review.openstack.org/401342
11:26 jtomasek joined #puppet-openstack
11:31 ntpttr joined #puppet-openstack
11:32 openstackgerrit Alejandro Andreu proposed openstack/puppet-tripleo: Add new tripleo::network::midonet::nsdb class  https://review.openstack.org/400869
12:23 openstackgerrit Mykyta Karpin proposed openstack/puppet-murano: Switch puppet-murano to rspec-puppet-facts  https://review.openstack.org/403774
12:34 EmilienM hello
12:34 mkarpin hi
12:54 dprince joined #puppet-openstack
13:01 johnmilton joined #puppet-openstack
13:06 fultonj joined #puppet-openstack
13:14 openstackgerrit Mykyta Karpin proposed openstack/puppet-ceph: Use facts from puppet-openstack_spec_helper  https://review.openstack.org/404207
13:15 EmilienM social: can you please backport https://review.openstack.org/#/c/403263/ ?
13:16 EmilienM social: same for https://review.openstack.org/#/c/403262/ once it's merged
13:25 iurygregory good morning
13:25 openstackgerrit Lukas Bezdicka proposed openstack/puppet-oslo: Switch ensure_packages to ensure_resources for python-memcache  https://review.openstack.org/404210
13:28 EmilienM mwhahaha, dmsimard: this weekend I had an idea which is probably crazy but what would you think of redoing poi/run_tests.sh in ansible, so we can use ARA to improve our CI debug, etc
13:29 iurygregory wow
13:40 jtomasek joined #puppet-openstack
13:41 rasca joined #puppet-openstack
13:46 jtriley joined #puppet-openstack
13:57 dmsimard EmilienM: it's not a crazy idea, I thought about it a long time ago -- not particularly in the context of ARA but more about the upcoming Zuul v3 ansible features
13:57 EmilienM dmsimard: yeah also
13:57 dmsimard What's crazy is for people to be able to overcome the fact that we're using Ansible to do puppet things, it's just perception .. it's like coke vs pepsi
13:58 EmilienM actually, infra is already running puppet from ansible
13:58 dmsimard Yeah but infra is not a puppet project :p
13:58 EmilienM dmsimard: anyway, shouldn't be that hard and long to make it
13:59 openstackgerrit Alejandro Andreu proposed openstack/puppet-tripleo: Adds new tripleo::network::midonet::config class  https://review.openstack.org/401342
14:00 dmsimard EmilienM: I'll take a stab at it
14:00 fultonj joined #puppet-openstack
14:01 jtriley joined #puppet-openstack
14:01 EmilienM cool
14:04 openstackgerrit Merged openstack/puppet-horizon: Switch ensure_packages to ensure_resources for python-memcache  https://review.openstack.org/403262
14:04 rmart04_ joined #puppet-openstack
14:07 dmsimard Murano test count further lowered from 61 to 40 (total from 108 to 40) \o/
14:08 dmsimard Radio silence from the neutron/lbaas guys to lower from 109 to 30 :(
14:08 iurygregory nice =)
14:09 dmsimard amoralej|lunch: fyi ^
14:11 dmsimard EmilienM, iurygregory: I mentioned this in Barcelona.. but the more it goes, the more the differences between UCA and RDO become obvious and problematic. I don't know how yet, but we should start managing this properly instead of trying to manage each and every exception.
14:11 dmsimard (Just thinking out loud) perhaps use hiera to include the right classes in the fixtures based on operating system.. or something
14:12 EmilienM dmsimard: in tripleo meeting, bbl
14:14 iurygregory dmsimard, yeah for each problem most time we need to disable something to have our CI green =(
14:14 jperry joined #puppet-openstack
14:18 fxpester joined #puppet-openstack
14:20 fxpester|2 joined #puppet-openstack
14:22 zhongshengping joined #puppet-openstack
14:22 fxpester|3 joined #puppet-openstack
14:24 agrebennikov joined #puppet-openstack
14:27 ducttape_ joined #puppet-openstack
14:32 fxpester joined #puppet-openstack
14:32 zhongshengping joined #puppet-openstack
14:42 openstackgerrit Alex Schultz proposed openstack/puppet-openstack-integration: Switch to default_transport_url  https://review.openstack.org/395707
14:44 alee joined #puppet-openstack
14:45 bklei joined #puppet-openstack
14:51 openstackgerrit Merged openstack/puppet-tripleo: Include local CA in haproxy PEM  https://review.openstack.org/394793
14:52 rmart04 joined #puppet-openstack
14:53 fxpester|2 joined #puppet-openstack
14:55 ducttape_ joined #puppet-openstack
14:57 EmilienM dmsimard: we already have conditionals, why hiera would help?
14:57 dmsimard Have a hiera file per distro instead of managing conditionals everywhere.
14:57 dmsimard i.e, we no longer manage exceptions
14:57 dmsimard we manage what is tested on each distro
14:57 zhongshengping joined #puppet-openstack
14:57 dmsimard it's sort of a nuance
14:59 dmsimard Trying to make everything work for every distro is sort of dishonest because we know there is no parity between RDO, UCA and eventually Debian
14:59 EmilienM I would rather invest our little time into translating our bash scripts into ansible
14:59 EmilienM right now, I see very low value of using hiera to deploy classes, versus conditionals in the manifests.
15:02 dmsimard not claiming otherwise
15:03 iurygregory meeting time \o/
15:03 dmsimard it's just that we're about to merge yet another thing that is supported by RDO but not by Ubuntu
15:05 fxpester joined #puppet-openstack
15:08 jaosorior joined #puppet-openstack
15:09 jtriley joined #puppet-openstack
15:12 EmilienM dmsimard: you never join meetings? :(
15:12 EmilienM dmsimard: we're talking about you in puppet meeting
15:13 dmsimard what channel, sorry, I ought to add it to my agenda ..
15:13 EmilienM dmsimard: openstack-meeting-4
15:14 jtriley joined #puppet-openstack
15:14 fxpester|2 joined #puppet-openstack
15:16 b00tcat joined #puppet-openstack
15:17 fxpester|3 joined #puppet-openstack
15:18 b00tcat Hi, what do I have to modify to prevent the cloning of `puppet-openstack-integration` when I run the unit tests?
15:20 fxpester joined #puppet-openstack
15:20 EmilienM b00tcat: you can't, why do you want to prevent that?
15:22 b00tcat because some of my tests are breaking due to the usage of the duritong/puppet-sysctl puppet module, and wanted to test if it would happen the same with the newer model https://github.com/thias/puppet-sysctl
15:22 b00tcat s/model/module/
15:24 b00tcat EmilienM: ^
15:25 EmilienM b00tcat: ah, I think you can hack something to clone the modules that you wany
15:25 EmilienM want*
15:26 b00tcat EmilienM: ah ok, I'll try to overwrite it in the Puppetfile and see if rake doesn't complain - is that what you meant?
15:26 EmilienM b00tcat: probably
15:27 alee EmilienM, Hey! I commented on https://review.openstack.org/#/c/396789/ .  Do you have other questions/concerns?
15:27 openstackgerrit Iury Gregory Melo Ferreira proposed openstack/puppet-openstack-guide: Add meeting #99  https://review.openstack.org/404276
15:28 openstackgerrit Saverio Proto proposed openstack/puppet-openstacklib: Create multiple databases with same db username.  https://review.openstack.org/404277
15:29 EmilienM alee: will look shortly
15:29 alee EmilienM, great - thanks
15:30 dmsimard mwhahaha: that puppet-horizon issue should be resolved now
15:30 dmsimard can you try a recheck ?
15:30 mwhahaha Yea let me dig it up
15:32 EmilienM alee: done
15:34 fxpester|2 joined #puppet-openstack
15:35 mwhahaha dmsimard: oh it was already merged so yea it's fixed
15:35 jtriley joined #puppet-openstack
15:36 openstackgerrit Merged openstack/puppet-oslo: Switch ensure_packages to ensure_resources for python-memcache  https://review.openstack.org/404210
15:36 dmsimard mwhahaha: how awesome is that
15:36 dmsimard mwhahaha: thanks for finding an issue in RDO packaging good sir
15:37 rasca joined #puppet-openstack
15:38 fxpester joined #puppet-openstack
15:41 openstackgerrit David Moreau Simard proposed openstack/puppet-openstack-integration: Re-enable Designate on CentOS7  https://review.openstack.org/393738
15:43 openstackgerrit Alejandro Andreu proposed openstack/puppet-tripleo: Adds tripleo::network::midonet::insights class  https://review.openstack.org/402062
15:44 danpawlik mwhahaha: about my PR https://review.openstack.org/#/c/399189/ should I update and use os_transport_url or just leave it how now it is and your script will fix it ?
15:44 mwhahaha danpawlik: please fix it
15:44 danpawlik ok
15:46 fxpester|2 joined #puppet-openstack
15:48 fxpester|3 joined #puppet-openstack
15:49 bnemec joined #puppet-openstack
15:50 jtriley joined #puppet-openstack
15:56 xarses joined #puppet-openstack
15:57 alee EmilienM, looking ..
15:57 xarses_ joined #puppet-openstack
15:59 alee EmilienM, I can remove config and params.  I do need ipaclient though -- https://review.openstack.org/#/c/397836/8
16:00 _ducttape_ joined #puppet-openstack
16:08 StraubTW joined #puppet-openstack
16:09 mwhahaha EmilienM: https://review.openstack.org/#/c/399108/
16:10 EmilienM mwhahaha: +A
16:12 ducttape_ joined #puppet-openstack
16:22 openstackgerrit Merged openstack/puppet-watcher: Revert "Fix centos7 ci"  https://review.openstack.org/399108
16:23 openstackgerrit Alejandro Andreu proposed openstack/puppet-tripleo: Adds class tripleo::network::midonet::gateway  https://review.openstack.org/401983
16:30 dmsimard designate is broken now, commented https://review.openstack.org/#/c/393738/4 :(
16:30 openstackgerrit Merged openstack/puppet-murano: Switch puppet-murano to rspec-puppet-facts  https://review.openstack.org/403774
16:35 openstackgerrit Keith Schincke proposed openstack/puppet-ceph: WIP Add support for installing Ceph rbd-mirror  https://review.openstack.org/401416
16:39 ducttape_ joined #puppet-openstack
16:41 _ducttape_ joined #puppet-openstack
16:47 jtriley joined #puppet-openstack
16:50 openstackgerrit Keith Schincke proposed openstack/puppet-ceph: Unify parameter arguments between classes.  https://review.openstack.org/403820
16:51 rasca joined #puppet-openstack
17:08 jtriley joined #puppet-openstack
17:14 openstackgerrit Merged openstack/puppet-horizon: Switch ensure_packages to ensure_resources for python-memcache  https://review.openstack.org/404261
17:14 openstackgerrit Merged openstack/puppet-designate: Deprecate host and port options  https://review.openstack.org/403967
17:44 xarses joined #puppet-openstack
17:45 xarses joined #puppet-openstack
17:50 markvoelker joined #puppet-openstack
17:50 dprince joined #puppet-openstack
17:59 jperry joined #puppet-openstack
18:22 dmsimard The deprecated notices from puppet with the 15 line long stack traces are freaking obnoxious
18:23 dmsimard This is *one* warning stack trace: http://paste.openstack.org/raw/590865/
18:23 dmsimard Hunner: ^ I know we've told you about this but just venting a bit of frustration here, sorry :)
18:23 dmsimard When you have about 200 of those in your output, it's really awesome (not)
18:26 dprince joined #puppet-openstack
18:27 Hunner Yup
18:28 crinkle joined #puppet-openstack
18:30 dmsimard colonwq: I'm trying to get scenario004 to run outside the gate and running into those errors (which is probably why image uploads are failing) http://paste.openstack.org/show/590866/ any idea ?
18:32 danpawlik_ joined #puppet-openstack
18:33 danpawlik_ dmsimard: About the bug http://logs.openstack.org/89/399189/18/check/gate​-puppet-openstack-integration-4-scenario004-tempe​st-centos-7/85c02dd/logs/watcher/applier.txt.gz  -> it should be handled by puppet-watcher module.
18:34 dmsimard danpawlik_: ok so all the errors are because of an issue in puppet-watcher ?
18:35 danpawlik_ dmsimard: hmm, I check other log files
18:36 danpawlik_ in this one is only deprecated parameter, but I will check other files if there is some other problems
18:45 openstackgerrit Daniel Pawlik proposed openstack/puppet-openstack-integration: Add puppet-watcher and enable it in scenario004  https://review.openstack.org/399189
18:53 colonwq dmsimard, I'm looking now.
18:54 colonwq dmsimard, what Ceph package version are you running? 10.2.2 is missinve IPv6 support in civetweb. It is fixed in 10.2.3
18:56 dmsimard I guess whatever comes provided from the CentOS storage sig, let me look
18:57 dmsimard yeah 10.2.2
18:57 colonwq that's why I did 004 with only ipv4 for rgw initially. It was a surprising issue to find.
18:58 dmsimard but I'm running whatever scenario004 runs, do we run with ipv6 in gate ?
18:59 colonwq can  you try with $enable_sig false?
18:59 colonwq let me check.
19:00 dmsimard I can't really do that, I just run scenario004 as-is without modifying anything
19:01 colonwq $ipv6 is false on line 23 of scenario004 in my local copy. I pulled it down yesterday
19:01 mwhahaha 4 was merged with ipv6 on for all
19:02 colonwq arg.
19:02 mwhahaha It caused problems when ubuntu broke some how on ipv6
19:02 dmsimard huh so it works in gate but not outside, maybe there's something else going on
19:04 xarses joined #puppet-openstack
19:04 colonwq dmsimard, my patch this morning used 10.2.3 in CI.
19:05 xarses_ joined #puppet-openstack
19:05 xarses_ joined #puppet-openstack
19:09 colonwq 10.2.3 is available here: http://buildlogs.centos.org/cent​os/7/storage/x86_64/ceph-jewel/
19:13 dmsimard hm, buildlogs but not stable release yet
19:13 dmsimard huh, oh, yes it is
19:22 jtriley joined #puppet-openstack
19:43 paramite joined #puppet-openstack
19:52 dmsimard mwhahaha: beat you to it
19:53 mwhahaha :o
19:53 openstackgerrit Ryan Bak proposed openstack/puppet-monasca: Add adminRole option to api config  https://review.openstack.org/400959
19:53 dmsimard deal with it /me puts on sunglasses
19:53 * mwhahaha throws broken dependencies at dmsimard
20:00 xarses joined #puppet-openstack
20:00 openstackgerrit Ryan Bak proposed openstack/puppet-monasca: Add adminRole option to api config  https://review.openstack.org/400959
20:01 xarses joined #puppet-openstack
20:06 openstackgerrit Merged openstack/puppet-openstack-integration: Update scenario004 for updates to puppet-ceph  https://review.openstack.org/403865
20:28 jtriley joined #puppet-openstack
20:28 richm left #puppet-openstack
20:29 richm joined #puppet-openstack
20:34 iurygregory dmsimard, https://review.openstack.org/#/c/399189/
20:34 colonwq EmilienM, xarses do I need to do trusty support for new code in puppet-ceph?
20:35 EmilienM no
20:35 EmilienM only xenial
20:35 colonwq thanks. My patch got easier. :)
20:35 xarses =)
20:41 dgurtner joined #puppet-openstack
20:41 dgurtner joined #puppet-openstack
21:16 jtriley joined #puppet-openstack
21:26 EmilienM _ody, Hunner: how can I know from which class / define a provider is called?
21:26 EmilienM in the catalog
21:28 mwhahaha so why don't we just use sosreport for the log capture?
21:28 mwhahaha i guess for our case we'd loose the password stuff but that's about it
21:29 mwhahaha we should work on getting more of the openstack services integrated tho so i guess it wouldn't work for puppet-openstack
21:29 mwhahaha since they only have the main ones
21:29 dmsimard does sosreport even exist on non-rhel ?
21:29 mwhahaha yea
21:29 mwhahaha it's a project
21:30 mwhahaha http://packages.ubuntu.com/xenial/sosreport
21:30 dmsimard TIL
21:30 mwhahaha i was pushing for it in fuel too but everyone wanted to continue to write their own
21:31 mwhahaha it even does password scrubbing out of configs
21:31 mwhahaha which for our CI is not necessarily useful but useful for projects
21:36 openstackgerrit David Moreau Simard proposed openstack/puppet-openstack-integration: No longer modprobe br_netfilter on Ubuntu  https://review.openstack.org/404427
21:55 mwhahaha so that's interesting to see a 1hour timeout on both centos and ubuntu for the same scenario, https://review.openstack.org/#/c/399189/
21:58 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: pacemaker: create Mysql_user once Galera is ready (puppet4)  https://review.openstack.org/404437
22:03 openstackgerrit Keith Schincke proposed openstack/puppet-ceph: WIP Add support for installing Ceph rbd-mirror  https://review.openstack.org/401416
22:22 EmilienM danpawlik: congrats on adding watcher in poi, really cool work!
22:22 EmilienM our testing coverage is really amazing
22:25 jtriley joined #puppet-openstack
22:31 rcernin joined #puppet-openstack
22:47 dmsimard EmilienM: started hacking on ansible
22:47 dmsimard can I assume we're not going to test anything other than xenial/centos7 and puppet4 in >= master ?
22:47 dmsimard all those conditions about puppet3 and trusty vs xenial hurts my brain
22:47 EmilienM dmsimard: yes
22:48 EmilienM we won't support stable branches I guess
22:48 EmilienM don't hurt your brain too much
22:48 dmsimard no way we're backporting that
22:50 EmilienM :)
22:52 dgurtner joined #puppet-openstack
22:52 dgurtner joined #puppet-openstack
22:54 Hunner EmilienM: probably by resource.parent.name or something? Is that what you're looking for?
22:54 Hunner It looks like for classes it's just the name of the class. I didn't try for defines
22:55 Hunner There is also resource.path which may be helpful, depending on what you want
22:55 EmilienM Hunner: nice, thanks
23:04 rcernin joined #puppet-openstack
23:04 rcernin joined #puppet-openstack
23:06 openstackgerrit Marcus Furlong proposed openstack/puppet-swift: memcache_servers support in object-expirer and container-reconciler  https://review.openstack.org/377304
23:07 duritong joined #puppet-openstack
23:08 rochaporto joined #puppet-openstack
23:14 ducttape_ joined #puppet-openstack
23:16 openstackgerrit Marcus Furlong proposed openstack/puppet-swift: memcache_servers support in object-expirer and container-reconciler  https://review.openstack.org/377304
23:17 openstackgerrit Marcus Furlong proposed openstack/puppet-swift: memcache_servers support in object-expirer and container-reconciler  https://review.openstack.org/377304
23:22 openstackgerrit David Moreau Simard proposed openstack/puppet-openstack-integration: WIP: Translate run_test.sh framework to Ansible  https://review.openstack.org/404471
23:22 dmsimard EmilienM: ^
23:22 EmilienM it was slow
23:22 EmilienM :P
23:23 mwhahaha why is 'setup' a role
23:23 dmsimard I don't know yet
23:23 dmsimard just logically organizing things
23:23 * mwhahaha adds to his list of things he's not a fan of with ansible
23:23 dmsimard There's a bit of setup all over the place right now, some in run_tests.sh, some in functions, some in install_modules
23:24 dmsimard mwhahaha: what I'm saying is that I'm not saying (!?) the setup role will stay
23:24 mwhahaha yea i get that they are all setup actions, just trying to understand why a role
23:24 dmsimard I'm just putting it there right now to collect all the setup related things there cause it's such a mess everywhere
23:24 mwhahaha i've also struggled with this when trying to make quality ansible playbooks
23:24 mwhahaha just a commentary
23:24 dmsimard roles are typically re-useable pieces of code
23:25 mwhahaha but they aren't
23:25 mwhahaha they never are :(
23:25 mwhahaha tasks are reusable peices of code
23:25 dmsimard Why not ? For example: setup -> scenario001 -> test.. or setup -> scenario002 -> test.. or whatever
23:25 mwhahaha roles are organizations of those peices of code
23:25 dmsimard I guess
23:26 dmsimard Ansible does not have strong nomenclature around that, there's just one thing and it's roles
23:26 mwhahaha yea i know, and i hate it
23:26 dmsimard ¯\_(ツ)_/¯
23:27 dmsimard You can also organize it in task files but then they might become unmanageable due to amount/size
23:27 dmsimard I don't know
23:27 mwhahaha so for me, the role is CI host
23:27 mwhahaha the tasks may fall under setup, execution, post
23:27 mwhahaha or something to that effect
23:28 dmsimard yeah, I mean, we could do that too. Just one role called p-o-i or something with everything in it from end to end
23:29 dmsimard I don't have a particularly strong opinion, I've always personally split/exploded things before they became too big
23:29 mwhahaha yea just talking through it. its just one of those things that in my mind it hurts when trying to organize ansible playbooks
23:30 openstackgerrit Merged openstack/puppet-openstack-guide: Add meeting #99  https://review.openstack.org/404276
23:30 mwhahaha litterally it hurts my brain because it doesn't have a clear cut pattern and people end up making roles that are just a grouping of random tasks and not something you'd actually consider a role
23:31 dmsimard Yup.
23:34 openstackgerrit Merged openstack/puppet-cloudkitty: Introduce cloudkitty::deps and cloudkitty::storage  https://review.openstack.org/394018
23:34 openstackgerrit Merged openstack/puppet-cloudkitty: Add support for configuring common component  https://review.openstack.org/394028
23:40 dmsimard mwhahaha: anyway, how things are organized is an implementation detail for me and not really important for the time being. I'll get things working as a poc and then we can decide how best to structure/architecture things.
23:40 mwhahaha dmsimard: sure just keep it in mind :)
23:40 * dmsimard nods
23:41 * mwhahaha goes back to hating other yaml based DSLs
23:41 openstackgerrit Merged openstack/puppet-openstack-integration: No longer modprobe br_netfilter on Ubuntu  https://review.openstack.org/404427
23:42 openstackgerrit Merged openstack/puppet-manila: Fix the annotation  https://review.openstack.org/404007
23:45 alee joined #puppet-openstack
23:45 openstackgerrit David Moreau Simard proposed openstack/puppet-openstack-integration: WIP: Translate run_test.sh framework to Ansible  https://review.openstack.org/404471
23:48 openstackgerrit David Moreau Simard proposed openstack/puppet-openstack-integration: WIP: Translate run_test.sh framework to Ansible  https://review.openstack.org/404471
23:48 dmsimard mwhahaha: such as ? lol
23:49 dmsimard oh. heat. :P
23:49 mwhahaha mistral, heat
23:49 mwhahaha i think i'll just give up for the day
23:49 mwhahaha come back when i'm not pulling my hair out at yaml and python :D
23:49 * mwhahaha wanders off
23:53 openstackgerrit David Moreau Simard proposed openstack/puppet-openstack-integration: WIP: Translate run_test.sh framework to Ansible  https://review.openstack.org/404471

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