Tuesday, 2015-12-15

*** davideagle has quit IRC00:14
*** mgrosz has joined #puppet-openstack00:20
*** ducttape_ has joined #puppet-openstack00:20
*** mgrosz has quit IRC00:21
*** aimon has quit IRC00:23
*** xarses has joined #puppet-openstack00:24
*** aimon has joined #puppet-openstack00:25
*** gildub has quit IRC00:25
*** ducttape_ has quit IRC00:33
*** yottatsa has joined #puppet-openstack00:38
*** baker has joined #puppet-openstack00:42
*** ducttape_ has joined #puppet-openstack00:43
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: all-in-one: install wget  https://review.openstack.org/25763400:48
*** aimon has quit IRC00:49
*** yottatsa has quit IRC00:51
*** yottatsa has joined #puppet-openstack00:53
*** baker has quit IRC00:53
*** xarses has quit IRC00:53
*** yottatsa has quit IRC00:54
*** markvoelker has joined #puppet-openstack00:56
*** karimb has quit IRC00:57
*** markvoelker has quit IRC01:01
*** openstack has joined #puppet-openstack01:09
*** mgrosz has joined #puppet-openstack01:14
*** mgrosz has quit IRC01:15
*** aimon has joined #puppet-openstack01:26
*** xarses has joined #puppet-openstack01:28
*** markvoelker has joined #puppet-openstack01:37
*** ducttape_ has quit IRC01:40
openstackgerritAdam Vinsh proposed openstack/puppet-swift: swiftinit provider check init/systemd file content  https://review.openstack.org/25764401:50
*** cwolferh has quit IRC01:56
*** ducttape_ has joined #puppet-openstack01:56
*** Guest44309 is now known as saneax_AFK01:57
*** baker has joined #puppet-openstack02:04
*** ducttape_ has quit IRC02:06
*** mgrosz has joined #puppet-openstack02:08
*** mgrosz has quit IRC02:09
*** mtreinish has quit IRC02:20
*** mtreinish has joined #puppet-openstack02:20
*** iurygregory has quit IRC02:27
*** ericksonsantos has quit IRC02:28
*** ericksonsantos has joined #puppet-openstack02:29
*** iurygregory has joined #puppet-openstack02:29
*** baker has quit IRC02:31
*** bklei has quit IRC02:35
*** bklei has joined #puppet-openstack02:35
*** bklei has quit IRC02:40
*** richm has quit IRC02:47
*** gildub has joined #puppet-openstack02:57
*** baker has joined #puppet-openstack02:57
*** mgrosz has joined #puppet-openstack03:02
*** mgrosz has quit IRC03:04
*** ducttape_ has joined #puppet-openstack03:29
*** markvoelker has quit IRC03:52
*** mgrosz has joined #puppet-openstack03:56
*** mgrosz has quit IRC03:58
*** ducttape_ has quit IRC04:04
*** baker has quit IRC04:07
*** openstackstatus has quit IRC04:24
*** openstack has joined #puppet-openstack04:26
*** openstackstatus has joined #puppet-openstack04:26
*** ChanServ sets mode: +v openstackstatus04:26
*** ducttape_ has joined #puppet-openstack04:29
*** ducttape_ has quit IRC04:39
*** a7ndrew has joined #puppet-openstack04:44
*** mgrosz has joined #puppet-openstack04:51
*** mgrosz has quit IRC04:52
*** markvoelker has joined #puppet-openstack04:53
*** markvoelker has quit IRC04:57
*** ducttape_ has joined #puppet-openstack05:01
*** ducttape_ has quit IRC05:09
openstackgerritGilles Dubreuil proposed openstack/puppet-tripleo: Adds IPv6 support for interface_for_ip function  https://review.openstack.org/24564305:23
*** mgrosz has joined #puppet-openstack05:42
*** mgrosz has quit IRC05:44
*** saneax_AFK is now known as saneax05:44
*** saneax is now known as Guest9971705:45
*** mgrosz has joined #puppet-openstack05:48
*** mgrosz has quit IRC05:49
openstackgerritGilles Dubreuil proposed openstack/puppet-tripleo: Adds IPv6 support for interface_for_ip function  https://review.openstack.org/24564306:09
gildubxarses, hi - would like to chat about https://review.openstack.org/#/c/253317/06:52
xarsesgildub: sure, It's quite late so we might not have too much time06:54
*** markvoelker has joined #puppet-openstack06:54
gildubxarses, no worries, I wanted to ping you earlier today but got caught. Just not sure about your comments, I mean I understand but I thought adding a param to control the timeout is better than hardcoded, we need it anyway because on slow hardware (or tests) exec needs more time.06:55
gildubxarses, anyway I'm responding to your comments, let's take it from there.06:57
xarsesthe commands that I called out, like creating the key, I've never seen take more than 1-5 sec (5 would be abnormal) much longer than that, and it's been due to the cluster being in a very bad way06:58
*** markvoelker has quit IRC06:58
xarsesat the same time. I very much agree, the osd create timeout is way too small and needs to be increased06:59
gildubxarses, okay let me remove the timeout for the fast ones, and let's keep it for the heavier ones then07:00
gildubxarses, thaks07:01
gildubxarses, thanks07:01
xarsesif we really have one that is a problem, please point it out. It's just not been my experience, so doubling seems like a wait that we dont need07:01
xarses(the light ones)07:01
xarsesgildub: no problem, we all want the same thing, perfect deployments =)07:02
xarsesgildub: keep in mind that CI was brokenish today, so you might see it fail CI07:03
gildubxarses, I don't know which ones are fast, this is why I initially blindly applied the timeout to the exec commands that were doing cluster things, I have to ask the person who reported the bug which one created problem07:04
gildubxarses, ok re CI07:04
xarsesgildub: it was most likely the osd activate, osd prepare, and what ever the first command is after the mon comes up07:06
gildubxarses, ok, I'll do a new pass and we take it from there. Thanks again!07:07
xarseshmm, thinking of we might not have the same command forced to run after the mon comes up, so we might want to make one that is intented to wait for the quorum07:07
xarsesI'll have to look that up07:07
gildubxarses, ok07:07
xarsesgildub: thanks, ttyl07:08
gildubxarses, cheers07:09
openstackgerritAndrew Spiers proposed openstack/puppet-designate: Adding psych on Gemfile explicitly  https://review.openstack.org/25771007:43
*** marfx000 has joined #puppet-openstack07:55
*** marfx000 has quit IRC07:56
*** chem has joined #puppet-openstack07:58
openstackgerritGilles Dubreuil proposed openstack/puppet-ceph: Added default timeout for exec resources  https://review.openstack.org/25331708:00
*** chem has quit IRC08:15
*** rleander has joined #puppet-openstack08:29
*** admin0 has joined #puppet-openstack08:29
*** dgurtner has joined #puppet-openstack08:34
*** dgurtner has quit IRC08:34
*** dgurtner has joined #puppet-openstack08:34
*** jpena has joined #puppet-openstack08:40
*** dgurtner has quit IRC08:40
*** karimb has joined #puppet-openstack08:47
*** morazi has joined #puppet-openstack08:50
*** rochaporto has joined #puppet-openstack08:53
*** markvoelker has joined #puppet-openstack08:55
*** darkxploit has quit IRC08:55
*** jistr has joined #puppet-openstack08:56
*** markvoelker has quit IRC08:59
*** dgurtner has joined #puppet-openstack09:01
*** dgurtner has quit IRC09:01
*** dgurtner has joined #puppet-openstack09:01
*** darkxploit has joined #puppet-openstack09:08
*** admin0 has quit IRC09:09
*** karimb_ has joined #puppet-openstack09:16
*** karimb has quit IRC09:16
*** admin0 has joined #puppet-openstack09:20
*** skolekonov has quit IRC09:21
*** openstackgerrit has quit IRC09:47
*** openstackgerrit has joined #puppet-openstack09:48
*** jtomasek has joined #puppet-openstack09:51
*** IvanBerezovskiy has joined #puppet-openstack09:52
*** skolekonov has joined #puppet-openstack09:54
*** derekh has joined #puppet-openstack10:01
*** astupnikov has joined #puppet-openstack10:06
*** dgurtner has quit IRC10:07
*** davideagle has joined #puppet-openstack10:08
*** pma has joined #puppet-openstack10:13
pmaCould someone please review one-line code https://review.openstack.org/#/c/251490/10:14
*** foxhunt_away is now known as foxhunt7210:16
*** admin0 has left #puppet-openstack10:18
openstackgerritMerged openstack/puppet-heat: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/24571510:30
*** openstackgerrit has quit IRC10:32
*** openstackgerrit has joined #puppet-openstack10:33
*** skolekonov has quit IRC10:38
*** davideagle has quit IRC10:41
*** davideagle has joined #puppet-openstack10:41
*** gildub has quit IRC10:46
openstackgerritKyrylo Galanov proposed openstack/puppet-swift: Create ring builders under swift user  https://review.openstack.org/24815010:48
*** paramite has joined #puppet-openstack10:50
*** markvoelker has joined #puppet-openstack10:55
*** dgurtner has joined #puppet-openstack10:58
*** dgurtner has quit IRC10:58
*** dgurtner has joined #puppet-openstack10:58
*** markvoelker has quit IRC11:00
*** bodepd has quit IRC11:03
*** skolekonov has joined #puppet-openstack11:04
*** admin0 has joined #puppet-openstack11:12
*** dgurtner has quit IRC11:15
*** admin0 has left #puppet-openstack11:25
*** paramite is now known as paramite|afk11:33
*** paramite|afk is now known as paramite11:35
*** gfidente has joined #puppet-openstack11:35
*** gfidente has quit IRC11:35
*** gfidente has joined #puppet-openstack11:35
*** guimaluf has joined #puppet-openstack11:38
*** trown|outttypeww is now known as trown11:40
openstackgerritAnshu Prateek proposed openstack/puppet-openstacklib: Remove references to puppet-openstack from README  https://review.openstack.org/25784811:43
*** mgrosz has joined #puppet-openstack11:44
*** paramite is now known as paramite|afk11:52
*** paramite|afk is now known as paramite11:55
*** skolekonov has quit IRC12:00
*** skolekonov has joined #puppet-openstack12:01
openstackgerritMartin Mágr proposed openstack/puppet-keystone: Preparation for default endpoint names rename  https://review.openstack.org/23479812:05
*** mgrosz has quit IRC12:08
*** mgrosz has joined #puppet-openstack12:09
*** dprince has joined #puppet-openstack12:26
EmilienMgood morning12:28
openstackgerritDenis Egorenko proposed openstack/puppet-sahara: DNM - TEST COMMIT  https://review.openstack.org/25774712:32
*** guimaluf has quit IRC12:33
*** guimaluf has joined #puppet-openstack12:35
*** jpena is now known as jpena|lunch12:40
*** markvoelker has joined #puppet-openstack12:41
*** skolekonov has quit IRC12:44
openstackgerritMartin Mágr proposed openstack/puppet-neutron: LBaaSv2 support  https://review.openstack.org/23472812:44
*** markvoelker has quit IRC12:46
mgrosz@EmilienM: good Morning!12:49
*** admin0 has joined #puppet-openstack12:52
*** ducttape_ has joined #puppet-openstack12:52
EmilienMparamite: I'm looking at https://review.openstack.org/#/c/234728/ and it overlaps a bit with neutron::agents::lbaas, isn't ?12:54
*** skolekonov has joined #puppet-openstack12:56
*** davideagle has quit IRC13:00
*** davideagle has joined #puppet-openstack13:00
*** markvoelker has joined #puppet-openstack13:04
*** ducttape_ has quit IRC13:06
*** mgrosz has quit IRC13:06
*** mgrosz has joined #puppet-openstack13:07
*** mgrosz has quit IRC13:10
*** mgrosz has joined #puppet-openstack13:10
*** jayg|g0n3 is now known as jayg13:15
*** davideagle has quit IRC13:18
*** trown is now known as trown|brb13:21
openstackgerritSebastien Badia proposed openstack/puppet-modulesync-configs: Add new managed modules  https://review.openstack.org/25735713:25
*** slagle has quit IRC13:25
*** davideagle has joined #puppet-openstack13:26
*** ducttape_ has joined #puppet-openstack13:30
*** trown|brb is now known as trown13:31
openstackgerritAnshu Prateek proposed openstack/puppet-magnum: Remove references to puppet-openstack from README  https://review.openstack.org/25789713:34
*** dgurtner has joined #puppet-openstack13:35
*** dgurtner has quit IRC13:35
*** dgurtner has joined #puppet-openstack13:35
iurygregorymorning =)13:38
iurygregorydegorenko, EmilienM  when  you guys have time https://review.openstack.org/#/c/256570/13:38
openstackgerritGael Chamoulaud proposed openstack/puppet-trove: Put all the logging related parameters to the logging class  https://review.openstack.org/24239513:39
*** dgurtner has quit IRC13:40
openstackgerritMerged openstack/puppet-tripleo: Modify cassandra dependency  https://review.openstack.org/25131913:43
*** ducttape_ has quit IRC13:49
*** slagle has joined #puppet-openstack13:52
openstackgerritJaume Devesa proposed openstack/puppet-tripleo: Modify cassandra dependency  https://review.openstack.org/25790813:56
EmilienMiurygregory: ack13:56
EmilienMspredzy: maybe you can also look ^ ( https://review.openstack.org/#/c/256570/ ) - is it safe for you?13:57
openstackgerritMerged openstack/puppet-openstacklib: Fix unit tests failing against Puppet 4.3.0  https://review.openstack.org/25645213:58
openstackgerritMerged openstack/puppet-neutron: Fix dependencies for Nuage plugin configuration  https://review.openstack.org/24723913:58
*** rcernin has joined #puppet-openstack14:02
*** jpena|lunch is now known as jpena14:02
openstackgerritOleksiy Molchanov proposed openstack/puppet-ceph: Fix upstart service declaration for rgw  https://review.openstack.org/25791414:04
spredzyEmilienM, The patch in itself will work ok, but this will drop the support for non-hiera installation. The idea of real was to have those values possibly defined at two level (the init.pp - non hiera) and the db.pp - hiera14:04
openstackgerritOleksiy Molchanov proposed openstack/puppet-ceph: Fix upstart service declaration for rgw  https://review.openstack.org/25791414:05
EmilienMspredzy: that's true, so I don't think we want that14:05
spredzyIf this is merged, a user will need to explicitly call the X::db class and specified the parameters14:05
spredzyIf we want to keep supporting the non-hiera way - no not yet14:05
*** richm has joined #puppet-openstack14:09
*** chem has joined #puppet-openstack14:12
gchamoulhi Guys, is there someone able to look at this one https://review.openstack.org/#/c/242395/ , please? will appreciate.14:15
*** skolekonov has quit IRC14:18
*** skolekonov has joined #puppet-openstack14:25
*** mgrosz has quit IRC14:29
*** bklei has joined #puppet-openstack14:32
*** ducttape_ has joined #puppet-openstack14:36
*** dslevin_ has quit IRC14:42
EmilienMweekly meeting in 15 min! :)14:44
openstackgerritEmilien Macchi proposed openstack/puppet-cinder: implement class and defined type for cinder GPFS driver  https://review.openstack.org/25756514:45
openstackgerritMerged openstack/puppet-openstack-integration: all-in-one: install wget  https://review.openstack.org/25763414:46
openstackgerritMerged openstack/puppet-nova: Use auth_uri to get nova credentials  https://review.openstack.org/25232614:46
*** mgrosz has joined #puppet-openstack14:49
*** ducttape_ has quit IRC14:53
*** angdraug has joined #puppet-openstack14:53
*** ntpttr has joined #puppet-openstack14:58
iurygregoryspredzy, i can roll back the changes =)14:59
gchamoulsbadia: thx! ;-)15:03
sbadiagchamoul: np!15:06
mgrosziurygregory, spredzy, EmilienM: any chance patch set https://review.openstack.org/#/c/255431/ will be merged today?15:06
EmilienMmgrosz: we are in meeting15:07
EmilienMhttps://wiki.openstack.org/wiki/Meetings/PuppetOpenStack15:07
openstackgerritMerged openstack/puppet-ceph: Fix unit tests failing against Puppet 4.3.x  https://review.openstack.org/25409515:10
openstackgerritMerged openstack/puppet-trove: Put all the logging related parameters to the logging class  https://review.openstack.org/24239515:11
openstackgerritMerged openstack/puppet-swift: Document and example for storage filters use  https://review.openstack.org/25612815:12
*** xingchao has joined #puppet-openstack15:13
*** ducttape_ has joined #puppet-openstack15:19
*** paramite has quit IRC15:19
*** rleander is now known as leanderthal15:22
*** dhollinger has joined #puppet-openstack15:22
*** baker has joined #puppet-openstack15:24
*** bnemec has quit IRC15:25
*** mwhahaha_ has joined #puppet-openstack15:28
*** skolekonov has quit IRC15:29
*** mwhahaha has quit IRC15:30
*** mwhahaha_ is now known as mwhahaha15:30
*** ntpttr has quit IRC15:31
*** skolekonov has joined #puppet-openstack15:31
*** ntpttr has joined #puppet-openstack15:31
*** bnemec has joined #puppet-openstack15:32
*** skolekonov has quit IRC15:33
*** ntpttr has left #puppet-openstack15:33
*** skolekonov has joined #puppet-openstack15:37
openstackgerritMerged openstack/puppet-openstack-cookiecutter: remove real reference from variables  https://review.openstack.org/25657015:43
openstackgerritMerged openstack/puppet-sahara: Remove require on main init class  https://review.openstack.org/25539315:48
openstackgerritMerged openstack/puppet-neutron: Move Neutron to $::os_service_default - part 2  https://review.openstack.org/25341615:48
*** Guest99717 is now known as saneax_AFK15:48
*** xingchao has quit IRC15:52
*** admin0 has quit IRC15:53
pmaCould someone please review one-line change? https://review.openstack.org/#/c/251490/15:57
*** xingchao has joined #puppet-openstack15:57
*** rcernin has quit IRC15:58
*** jaosorior has joined #puppet-openstack16:03
openstackgerritColleen Murphy proposed openstack/puppet-keystone: Undeprecate the $keystone::endpoint::version param  https://review.openstack.org/25603116:04
*** mgrosz has quit IRC16:04
*** mdorman has joined #puppet-openstack16:05
*** skolekonov has quit IRC16:10
*** xingchao has quit IRC16:25
*** astupnikov has quit IRC16:26
*** davideagle has quit IRC16:36
openstackgerritEmilien Macchi proposed openstack/puppet-cinder: implement class and defined type for cinder GPFS driver  https://review.openstack.org/25756516:37
*** xarses has quit IRC16:40
*** aimon has quit IRC16:42
*** wenxin has quit IRC16:43
*** degorenko is now known as _degorenko|afk16:49
*** cwolferh has joined #puppet-openstack16:56
*** davideagle has joined #puppet-openstack16:57
*** davideagle has quit IRC17:03
*** davideagle has joined #puppet-openstack17:03
*** davideagle has quit IRC17:15
*** trown is now known as trown|lunch17:19
*** xarses has joined #puppet-openstack17:19
*** derekh has quit IRC17:21
*** dgurtner has joined #puppet-openstack17:22
*** dgurtner has quit IRC17:22
*** dgurtner has joined #puppet-openstack17:22
*** xarses has quit IRC17:25
*** xarses has joined #puppet-openstack17:25
*** skolekonov has joined #puppet-openstack17:31
*** rochaporto has quit IRC17:32
*** skolekonov has quit IRC17:33
*** toMeloos has joined #puppet-openstack17:33
*** jpena has quit IRC17:36
*** baker has quit IRC17:38
*** dgurtner has quit IRC17:40
*** aimon has joined #puppet-openstack17:40
*** markvoelker_ has joined #puppet-openstack17:40
*** markvoelker has quit IRC17:41
*** karimb_ has quit IRC17:42
*** RichardRaseley has joined #puppet-openstack17:42
*** baker has joined #puppet-openstack17:43
*** markvoelker_ has quit IRC17:47
*** markvoelker has joined #puppet-openstack17:49
*** baker has quit IRC17:59
*** baker has joined #puppet-openstack17:59
*** kindjal has joined #puppet-openstack18:12
*** foxhunt72 is now known as foxhunt_away18:13
*** dhollinger has quit IRC18:14
*** jistr has quit IRC18:22
*** gfidente is now known as gfidente|afk18:26
*** iurygregory has quit IRC18:36
*** ericksonsantos has quit IRC18:36
*** Guest73233 is now known as mgagne18:42
*** mgagne is now known as Guest7643418:42
*** trown|lunch is now known as trown18:46
*** skolekonov has joined #puppet-openstack18:48
*** skolekonov has quit IRC18:52
*** jtomasek has quit IRC18:55
*** jesusaur is now known as jesusaurus19:01
*** dhollinger has joined #puppet-openstack19:04
*** fvollero_ has joined #puppet-openstack19:05
*** rleander_ has joined #puppet-openstack19:05
*** fvollero has quit IRC19:08
*** morazi has quit IRC19:08
*** leanderthal has quit IRC19:08
*** rleander_ has quit IRC19:12
*** fvollero_ has quit IRC19:13
*** fvollero has joined #puppet-openstack19:13
*** kindjal has quit IRC19:23
*** jaosorior has quit IRC19:23
openstackgerritMerged openstack/puppet-keystone: Undeprecate the $keystone::endpoint::version param  https://review.openstack.org/25603119:23
*** rleander_ has joined #puppet-openstack19:25
EmilienMcrinkle: your patch has been merged, if you plan to backport it, I'll +219:25
openstackgerritColleen Murphy proposed openstack/puppet-keystone: Undeprecate the $keystone::endpoint::version param  https://review.openstack.org/25812019:29
crinkleEmilienM: ^ done19:29
EmilienMcool19:29
*** gfidente|afk has quit IRC19:30
*** openstackgerrit has quit IRC19:32
*** openstackgerrit has joined #puppet-openstack19:33
*** jaosorior has joined #puppet-openstack19:42
*** dprince has quit IRC19:47
dmsimardEmilienM: I can work on https://bugs.launchpad.net/bugs/1525385 but need to know what kind of implementation we want19:59
openstackLaunchpad bug 1525385 in puppet-openstack-integration "Make repositories configurable" [Wishlist,Triaged]19:59
EmilienMdmsimard: we could start by adding a boolean (true by default) to disable or enable the repo management20:00
dmsimardEmilienM: not a bad idea. And then configure that toggle based on an env variable (like the way the scenarios are set?)20:00
EmilienMdmsimard: in case of packaging job, I would suggest not using puppet to manage repos20:02
EmilienMand rather prepare them before running puppet scripts20:02
dmsimardEmilienM: yeah that's fine, I can set the toggle to false and configure the repositories myself before running run_tests20:02
EmilienMthat would be KISS20:02
dmsimardare you okay with the toggle being an environment variable like the scenario is ?20:02
EmilienMdmsimard: sure but puppet will not configure the repo ok?20:04
* dmsimard nods20:04
dmsimardreview coming right up20:04
EmilienMnice20:05
EmilienMdmsimard: do you have openstack jobs for packaging?20:05
*** foxhunt_away is now known as foxhunt7220:05
EmilienMwhat is the motivation?20:05
dmsimardEmilienM: what do you mean by "openstack jobs for packaging" ?20:05
EmilienMdmsimard: no I'm just asking why you want that now20:06
dmsimardEmilienM: This is part of the plan with the weirdo project so I can run your jobs upstream with the trunk repository before promoting to the tested repository (that I want you to use)20:08
dmsimardThis way RDO should in theory never be broken for you since I will be running your tests before you do :)20:08
EmilienMnice20:09
EmilienM++20:09
*** jaosorior has quit IRC20:12
dmsimardEmilienM: it looks like scenario003 wasn't updated to use the common manifests (i.e, repos, keystone, mysql, etc.)20:20
dmsimardIs it okay if I at least drop this part for the time being: https://github.com/openstack/puppet-openstack-integration/blob/master/fixtures/scenario003.pp#L19-L5720:20
dmsimard(I'll make sure it works..)20:21
EmilienMdmsimard: please use the common bits20:21
EmilienMif that works for you20:21
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration: Update scenario003 to use the common manifests  https://review.openstack.org/25813420:24
EmilienMdmsimard: +220:25
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration: Provide a toggle to disable repository management by puppet  https://review.openstack.org/25813620:29
dmsimardEmilienM: I actually ended up with the above because I did not see a clean way to pass a variable from run_tests to the fixtures so that these manifests could then call the repos class with the right value for manage_repos20:30
*** karimb has joined #puppet-openstack20:30
dmsimardomg a bash error20:30
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration: Provide a toggle to disable repository management by puppet  https://review.openstack.org/25813620:31
EmilienMdmsimard: maybe you can have a custom fact20:31
EmilienMthat says if this fact is here, let's disable the repo20:31
EmilienMotherwise enable it (by default)20:31
dmsimardthat's starting to be a bit over engineered :p20:31
*** jtomasek has joined #puppet-openstack20:32
EmilienMor even better20:32
EmilienMdmsimard: what you did is excellent20:32
EmilienMEXCELLENT !20:32
dmsimard\o/20:32
* dmsimard goes on twitter to brag he's awesome and like the tweet20:32
EmilienMdmsimard: but20:32
EmilienMthere something you missed20:33
dmsimardoh ?20:33
EmilienMin ::repo class, there are some bits that you'll need I think20:33
EmilienMselinux, and also package type20:33
* dmsimard looks20:33
*** saneax_AFK is now known as saneax20:34
dmsimardwell, yes and no20:34
*** saneax is now known as Guest2563720:34
dmsimardit just means that if I don't manage the repos, I also need to do stuff like install yum-plugin-priorities etc20:35
dmsimardunless I'm missing something20:35
dmsimardEmilienM: wait, you're telling me you need to explicitely install openstack-selinux ? It's not required by any packages ?20:36
EmilienMdmsimard: puppetlabs-rabbitmq requires the https://github.com/openstack/puppet-openstack-integration/blob/master/manifests/repos.pp#L1020:36
EmilienMand you'll have to move https://github.com/openstack/puppet-openstack-integration/blob/master/manifests/repos.pp#L16 in the manifests then (fixtures or anywhere you like)20:36
dmsimardEmilienM: it looks like putting it in init would make sense20:38
dmsimardwith a case on osfamily20:38
dmsimardI'll check20:39
EmilienMdmsimard: cool +120:39
dmsimardEmilienM: package_provider is already set explicitely in rabbitmq20:40
dmsimardhttps://github.com/openstack/puppet-openstack-integration/blob/master/manifests/rabbitmq.pp20:40
EmilienMdmsimard: but wrong for centos systems20:40
EmilienMdmsimard: it uses rpm by default20:41
EmilienMand we want yum20:41
dmsimardEmilienM: it's set right here: https://github.com/openstack/puppet-openstack-integration/blob/master/manifests/rabbitmq.pp#L7-L9 we should probably remove it from repos.pp if it's not expected elsewhere or something20:41
EmilienMdmsimard: right, we can drop it then from repos20:42
dmsimardok20:42
EmilienMdmsimard: so only thing is selinux20:42
EmilienMand we're good20:42
EmilienMyou can put it in init maybe20:42
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: scenario002: enable application catalog service  https://review.openstack.org/25043620:43
EmilienM_degorenko|afk: rebased ^20:43
*** toMeloos has quit IRC20:43
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration: Provide a toggle to disable repository management by puppet  https://review.openstack.org/25813620:45
dmsimardEmilienM: cool, murano :)20:45
EmilienMdmsimard: wait20:46
EmilienMthe puppet apply will run 2 times20:46
EmilienMmaybe we can run it outside the function20:46
EmilienMso:20:47
EmilienM1/ we install repo with puppet (or not)20:47
EmilienM2/ we run first puppet run20:47
EmilienM3/ we run second puppet run to test idempotency20:47
EmilienMinstead of running 4 times puppet20:47
dmsimardEmilienM: ah yes, I see what you mean20:47
EmilienMcool20:47
EmilienMotherwise LGTM20:47
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration: Provide a toggle to disable repository management by puppet  https://review.openstack.org/25813620:48
EmilienMdmsimard: +220:49
EmilienMI think we're good now, you can run your stuff20:49
EmilienMjust export the env install the delorean url yourself and it's ok20:49
dmsimardyup, should be good now20:53
dmsimardI need vacation now, thank god the holidays are coming :p20:53
mfischEmilienM: delattec on my my team discovered an issue with your fix for the RBD duplicate resource20:54
mfischI will propose a fix later today for it20:54
mfischhe's filing a bug20:54
EmilienMmfisch: nice20:54
EmilienMdmsimard: me too but I don't take holidays20:54
EmilienMI go in France Thursday for xmas20:55
bkeroEmilienM: safe travels20:55
EmilienMbkero: thanks!20:55
mfischyou dont take holidays no wonder they kicked you out of france lol20:56
EmilienMlol20:56
EmilienMthat's a troll !20:56
*** jaosorior has joined #puppet-openstack21:11
*** admin0 has joined #puppet-openstack21:22
*** kindjal has joined #puppet-openstack21:27
Guest76434I'm getting this error when trying to apply a manifest for keystone: Could not autoload puppet/type/keystone_tenant: interning empty string on node XXX21:33
*** Guest76434 is now known as mgagne21:33
mgagneand what was me21:33
mgagnethat*21:33
mgagneI see someone asking around on IRC about it but no answer :-/21:34
*** mgagne is now known as Guest16021:34
EmilienMGuest160: where?21:34
*** Guest160 has quit IRC21:34
*** Guest160 has joined #puppet-openstack21:34
Guest160on my server :D21:35
*** Guest160 is now known as mgagne21:35
mgagneok, trying to restore my nick21:35
openstackgerritMichael Chapman proposed openstack/puppet-neutron: Adds configuration support for OpenDaylight SDN Controller  https://review.openstack.org/24939021:37
*** cwolferh has quit IRC21:38
mgagnelooks like a ruby 1.8 specific behavior21:38
mgagneI think something is trying to run the equivalent of "".to_sym21:39
*** jaosorior has quit IRC21:41
*** imcsk8 has quit IRC21:50
*** imcsk8 has joined #puppet-openstack21:50
*** todin has quit IRC21:51
*** jayg is now known as jayg|g0n321:55
*** todin has joined #puppet-openstack21:58
*** bklei_ has joined #puppet-openstack21:59
*** jtomasek has quit IRC21:59
*** bklei has quit IRC22:02
*** bklei_ has quit IRC22:03
*** trown is now known as trown|outttypeww22:04
*** foxhunt72 is now known as foxhunt_away22:13
*** rcernin has joined #puppet-openstack22:16
openstackgerritGilles Dubreuil proposed openstack/puppet-ceph: Added default timeout for exec resources  https://review.openstack.org/25331722:27
*** admin0 has quit IRC22:28
*** gildub has joined #puppet-openstack22:32
xarseshas any one else worked through this? '$ bundle exec rake spec' -> 'Could not find minitest-5.8.3 in any of the sources'22:34
xarseshmm, `bundle install` from another repo fixed the deps22:38
xarsespuppet-ceph was my problem, puppet-cinder appears to have fixed it22:38
*** ericksonsantos has joined #puppet-openstack22:42
*** iurygregory has joined #puppet-openstack22:42
xarsesEmilienM: am I crazy or is this job still running the bad version of bundler? http://logs.openstack.org/11/257511/1/check/gate-puppet-ceph-puppet-unit-3.7-dsvm-centos7/a86daef/console.html#_2015-12-15_21_38_11_99122:43
openstackgerritAndrew Woodward proposed openstack/puppet-ceph: Move intergration tests out of acceptance  https://review.openstack.org/25751122:43
EmilienM1.11.1 is fixed22:43
EmilienMxarses: ^22:43
EmilienMI un-pined today22:44
EmilienMthey did a release22:44
*** ducttape_ has quit IRC22:46
xarsesok, fantastic22:46
xarsesso, then any pointers for why http://logs.openstack.org/11/257511/1/check/gate-puppet-ceph-puppet-unit-3.7-dsvm-centos7/a86daef/console.html#_2015-12-15_21_40_45_449 is occurring?22:46
*** ducttape_ has joined #puppet-openstack22:49
*** iurygregory has quit IRC22:53
*** iurygregory has joined #puppet-openstack22:53
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration: Provide a toggle to disable repository management by puppet  https://review.openstack.org/25813623:07
*** zhangjn has joined #puppet-openstack23:07
*** zhangjn has quit IRC23:09
_odyxarses: The require of spec_helper_acceptance kicks in beaker which defaults to using a vagrant based hypervisor which isn't available on nodepool instances.23:11
_odyA unit job shouldn't be kicking off integration jobs, I suspect.23:11
*** jtomasek has joined #puppet-openstack23:11
xarses_ody: yep, I see. I picked a bad word to name that folder, since rspec gobbles it up23:12
*** dhollinger has quit IRC23:13
*** jtomasek has quit IRC23:16
*** rcernin has quit IRC23:20
*** kindjal has quit IRC23:22
*** baker has quit IRC23:26
*** bklei has joined #puppet-openstack23:40
*** mwhahaha has quit IRC23:47
*** dgurtner has joined #puppet-openstack23:48
*** dgurtner has quit IRC23:48
*** dgurtner has joined #puppet-openstack23:48
*** gildub has quit IRC23:49
*** mwhahaha has joined #puppet-openstack23:50
*** aimon has quit IRC23:53
*** karimb has quit IRC23:56

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