Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2015-11-12

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

All times shown according to UTC.

Time Nick Message
00:14 _ducttape_ joined #puppet-openstack
00:36 gildub joined #puppet-openstack
00:52 nlahouti joined #puppet-openstack
01:03 zhangjn joined #puppet-openstack
01:05 openstackgerrit Adam Vinsh proposed openstack/puppet-swift: Manage swift with swiftinit service provider  https://review.openstack.org/203220
01:09 guessi joined #puppet-openstack
01:14 openstackgerrit Adam Vinsh proposed openstack/puppet-swift: Manage swift with swiftinit service provider  https://review.openstack.org/203220
01:22 openstackgerrit Adam Vinsh proposed openstack/puppet-swift: Manage swift with swiftinit service provider  https://review.openstack.org/203220
01:24 guessi left #puppet-openstack
01:32 xingchao joined #puppet-openstack
02:24 xingchao joined #puppet-openstack
02:37 lnxnut joined #puppet-openstack
03:54 fedexo joined #puppet-openstack
03:55 openstackgerrit Merged openstack/puppet-openstack-specs: Add AUTHORS and Changelog to .gitignore  https://review.openstack.org/240494
03:56 openstackgerrit Merged openstack/puppet-openstack-integration: tempest: use $::id fact for permissions  https://review.openstack.org/242179
04:03 Weeve joined #puppet-openstack
04:06 cwolferh joined #puppet-openstack
04:06 openstackgerrit Merged openstack/puppet-keystone: Add a new param for fernet tokens config  https://review.openstack.org/203675
04:19 karmix joined #puppet-openstack
04:29 khushbu_ joined #puppet-openstack
05:24 rahulait joined #puppet-openstack
05:59 rahulait joined #puppet-openstack
06:09 xingchao joined #puppet-openstack
06:22 gfidente joined #puppet-openstack
06:22 gfidente joined #puppet-openstack
06:33 dgurtner joined #puppet-openstack
06:44 nlahouti joined #puppet-openstack
06:46 nlahouti1 joined #puppet-openstack
06:53 openstackgerrit Andrew Spiers proposed openstack/puppet-designate: Configure rabbit_virtual_host  https://review.openstack.org/244469
07:01 dgurtner joined #puppet-openstack
07:01 dgurtner joined #puppet-openstack
07:29 zhangjn joined #puppet-openstack
07:33 zhangjn joined #puppet-openstack
07:42 gildub joined #puppet-openstack
07:55 dgurtner joined #puppet-openstack
07:58 khushbu_ joined #puppet-openstack
08:07 dgurtner joined #puppet-openstack
08:26 sleinen-AS559 joined #puppet-openstack
08:33 openstackgerrit Gael Chamoulaud proposed openstack/puppet-trove: Put all the logging related parameters to the logging class  https://review.openstack.org/242395
08:40 Pablo|off| joined #puppet-openstack
08:48 karimb joined #puppet-openstack
08:50 rcernin joined #puppet-openstack
08:53 sergmelikyan joined #puppet-openstack
08:56 dgurtner joined #puppet-openstack
09:03 jistr joined #puppet-openstack
09:15 sanjayu joined #puppet-openstack
09:18 openstackgerrit Jaume Devesa proposed openstack/puppet-tripleo: MidoNet services manifests  https://review.openstack.org/244184
09:28 paramite joined #puppet-openstack
09:47 openstackgerrit joined #puppet-openstack
09:48 karimb joined #puppet-openstack
09:56 IvanBerezovskiy joined #puppet-openstack
09:59 dgurtner joined #puppet-openstack
10:09 derekh joined #puppet-openstack
10:17 openstackgerrit joined #puppet-openstack
10:25 sergmelikyan joined #puppet-openstack
10:32 openstackgerrit Merged openstack/puppet-neutron: Support for Nuage Neutron plugin in puppet-neutron  https://review.openstack.org/214798
10:41 openstackgerrit Jaume Devesa proposed openstack/puppet-tripleo: MidoNet services manifests  https://review.openstack.org/244184
10:48 jistr_ joined #puppet-openstack
10:48 rcernin_ joined #puppet-openstack
10:48 khushbu_ joined #puppet-openstack
10:50 rcernin joined #puppet-openstack
11:01 khushbu_ joined #puppet-openstack
11:05 jistr_ joined #puppet-openstack
11:14 karimb_ joined #puppet-openstack
11:15 jerrygb joined #puppet-openstack
11:16 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-keystone: New provider for keystone domain configuration.  https://review.openstack.org/219289
11:16 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-keystone: Support for multiple ldap backend.  https://review.openstack.org/238164
11:18 iurygregory good morning people o/
11:19 mkarpin Hey!
11:25 chem joined #puppet-openstack
11:26 openstackgerrit Gael Chamoulaud proposed openstack/puppet-aodh: Remove class_parameter_defaults puppet-line check  https://review.openstack.org/244576
11:27 openstackgerrit Gael Chamoulaud proposed openstack/puppet-barbican: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244577
11:28 openstackgerrit Gael Chamoulaud proposed openstack/puppet-ceilometer: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244578
11:28 openstackgerrit Gael Chamoulaud proposed openstack/puppet-aodh: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244576
11:29 sergmelikyan joined #puppet-openstack
11:30 openstackgerrit Gael Chamoulaud proposed openstack/puppet-ceph: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244580
11:32 openstackgerrit Gael Chamoulaud proposed openstack/puppet-cinder: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244581
11:33 openstackgerrit Gael Chamoulaud proposed openstack/puppet-designate: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244583
11:34 openstackgerrit Gael Chamoulaud proposed openstack/puppet-glance: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244585
11:34 openstackgerrit Gael Chamoulaud proposed openstack/puppet-gnocchi: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244586
11:36 openstackgerrit Gael Chamoulaud proposed openstack/puppet-heat: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244587
11:36 openstackgerrit Gael Chamoulaud proposed openstack/puppet-horizon: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244589
11:37 openstackgerrit Gael Chamoulaud proposed openstack/puppet-ironic: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244590
11:38 openstackgerrit Gael Chamoulaud proposed openstack/puppet-keystone: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244591
11:40 openstackgerrit Gael Chamoulaud proposed openstack/puppet-manila: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244592
11:41 openstackgerrit Gael Chamoulaud proposed openstack/puppet-mistral: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244593
11:42 openstackgerrit Gael Chamoulaud proposed openstack/puppet-monasca: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244594
11:42 openstackgerrit Gael Chamoulaud proposed openstack/puppet-murano: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244595
11:43 openstackgerrit Gael Chamoulaud proposed openstack/puppet-neutron: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244596
11:45 openstackgerrit Gael Chamoulaud proposed openstack/puppet-nova: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244599
11:46 openstackgerrit Gael Chamoulaud proposed openstack/puppet-openstack_extras: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244600
11:47 openstackgerrit Gael Chamoulaud proposed openstack/puppet-openstack-integration: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244601
11:48 openstackgerrit Gael Chamoulaud proposed openstack/puppet-openstacklib: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244602
11:49 openstackgerrit Gael Chamoulaud proposed openstack/puppet-sahara: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244603
11:49 openstackgerrit Gael Chamoulaud proposed openstack/puppet-swift: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244604
11:50 openstackgerrit Gael Chamoulaud proposed openstack/puppet-tempest: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244605
11:51 openstackgerrit Gael Chamoulaud proposed openstack/puppet-tripleo: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244607
11:52 openstackgerrit Gael Chamoulaud proposed openstack/puppet-trove: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244608
11:52 openstackgerrit Gael Chamoulaud proposed openstack/puppet-tuskar: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244609
11:53 openstackgerrit Gael Chamoulaud proposed openstack/puppet-vswitch: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244610
11:54 openstackgerrit Gael Chamoulaud proposed openstack/puppet-zaqar: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244612
12:03 iurygregory WOW O.o
12:03 gchamoul sorry guys! :-p
12:07 iurygregory no problem kkkkkkkkkkk
12:08 iurygregory just to know, what is the reason to remove ?
12:08 dgurtner joined #puppet-openstack
12:08 dgurtner joined #puppet-openstack
12:10 xingchao joined #puppet-openstack
12:11 iurygregory anyone can help me to find out why the template is not applied properly?  https://review.openstack.org/#/c/216821/ ^^
12:13 degorenko gchamoul, yeah, why you want to remove it? :)
12:13 gchamoul iurygregory: just because we disable a check which has been removed in puppet-lint for a while
12:13 iurygregory oh ^^
12:14 degorenko ack
12:14 gchamoul https://review.openstack.org/#/c/242105/
12:15 degorenko gchamoul, please, set one topic for all reviews
12:15 degorenko i dont want -1 on all patches :D
12:16 iurygregory it's a good idea
12:16 gchamoul degorenko: ack
12:17 * degorenko away on lunch
12:24 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-keystone: Add keystone domain specific configuration.  https://review.openstack.org/219289
12:32 myatsenko Guys, plz look at https://review.openstack.org/#/c/233011/
12:33 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-keystone: Support for multiple ldap backend.  https://review.openstack.org/238164
12:34 mkarpin Hey! can anyone from core reviewers please check https://review.openstack.org/#/c/243030/? thank you!
12:34 mkarpin crinkle, mfisch, sbadia, spredzy : ^^
12:50 v1k0d3n joined #puppet-openstack
13:05 openstackgerrit Merged openstack/puppet-neutron: Split SR-IOV configuration file into driver and agent pieces  https://review.openstack.org/221991
13:05 sergmelikyan joined #puppet-openstack
13:08 ducttape_ joined #puppet-openstack
13:10 openstackgerrit Merged openstack/puppet-keystone: Add ability to configure paste ini options using config class  https://review.openstack.org/243030
13:14 dgurtner joined #puppet-openstack
13:14 dgurtner joined #puppet-openstack
13:14 aderyugin Hi, folks! Please take a look: https://review.openstack.org/#/c/220238/
13:15 aderyugin clayton, crinkle, mfisch, sbadia, spredzy : ^^^
13:15 dprince joined #puppet-openstack
13:15 jerrygb joined #puppet-openstack
13:18 jerrygb_ joined #puppet-openstack
13:24 clayton _ody is there an open issue for this Facter[factname] issue?
13:26 morazi joined #puppet-openstack
13:29 openstackgerrit Gael Chamoulaud proposed openstack/puppet-zaqar: metadata: Switch to OpenStack namespace and bump to Kilo  https://review.openstack.org/244650
13:31 sergmelikyan joined #puppet-openstack
13:33 openstackgerrit Gael Chamoulaud proposed openstack/puppet-barbican: metadata: Switch to OpenStack namespace and bump to Kilo  https://review.openstack.org/244654
13:41 bklei joined #puppet-openstack
13:43 openstackgerrit Gael Chamoulaud proposed openstack/puppet-barbican: metadata: Switch to OpenStack namespace and bump to Kilo  https://review.openstack.org/244654
13:47 iurygregory anyone can help me with templates?
13:50 openstackgerrit Gael Chamoulaud proposed openstack/puppet-mistral: metadata: Switch to OpenStack namespace and bump to Kilo  https://review.openstack.org/244667
13:54 richm joined #puppet-openstack
13:59 openstackgerrit Merged openstack/puppet-openstack-integration: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244601
14:00 openstackgerrit Merged openstack/puppet-openstacklib: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244602
14:02 openstackgerrit Merged openstack/puppet-swift: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244604
14:04 openstackgerrit Merged openstack/puppet-openstack_extras: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244600
14:06 openstackgerrit Merged openstack/puppet-monasca: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244594
14:07 openstackgerrit Merged openstack/puppet-mistral: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244593
14:08 openstackgerrit Merged openstack/puppet-nova: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244599
14:08 openstackgerrit Merged openstack/puppet-vswitch: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244610
14:10 openstackgerrit Merged openstack/puppet-trove: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244608
14:10 openstackgerrit Merged openstack/puppet-sahara: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244603
14:11 openstackgerrit John Trowbridge proposed openstack/puppet-ironic: [inspector] Specify SELinux type context for tftp dir  https://review.openstack.org/244675
14:11 openstackgerrit Merged openstack/puppet-murano: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244595
14:12 openstackgerrit Merged openstack/puppet-ironic: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244590
14:12 openstackgerrit Merged openstack/puppet-horizon: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244589
14:13 openstackgerrit Merged openstack/puppet-neutron: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244596
14:15 ducttape_ joined #puppet-openstack
14:15 openstackgerrit John Trowbridge proposed openstack/puppet-ironic: [inspector] Specify SELinux type context for tftp dir  https://review.openstack.org/244675
14:22 jerrygb joined #puppet-openstack
14:22 degorenko clayton, hi, what do you think my comment https://review.openstack.org/#/c/237970 ?
14:22 degorenko about*
14:22 clayton I'm not sure what you mean in the comment
14:23 openstackgerrit Merged openstack/puppet-keystone: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244591
14:23 clayton the *::db class changes that are going in will double the default number of connections for all services
14:23 clayton going in and have gone in
14:23 degorenko clayton, do you saw my reply in your ML about this issue?
14:24 clayton oh, let me look, sorry
14:24 clayton I redid my mail filters yesterday and apparently it's still not going to the right place
14:25 degorenko no problem
14:25 clayton so you're suggesting using $::os_service_default then?
14:25 degorenko not exactly
14:26 degorenko as you mentioned - those defaults are different from project
14:26 degorenko so, what i want
14:26 clayton that's not the case for the db classes though
14:26 clayton and the db class change is new for this cycle
14:26 degorenko we can raise warning about this
14:27 degorenko we just keep such parameters (which have default in puppet and this default is different from project) with puppet defaults
14:27 degorenko so, is there reason to change all modules now?
14:27 degorenko for those parameters
14:28 clayton I think people are going to upgrade from the kilo modules to the liberty modules and in some cases their db or load balancer is going to explode
14:28 clayton and I don't think this is an intentional change.
14:29 degorenko so, you suggest remove all those defaults and put undef? or what?
14:30 clayton I did the math the other day, if you're running 40 core servers and 3 control nodes and running neutron, glance, cinder, and keystone on your control nodes, that's 5 services * 40 workers * 3 servers * 5 connections vs 10 connections is 3000 vs 6000 db connections
14:30 itsuugo joined #puppet-openstack
14:31 clayton I think we need to either put $::os_service_default in place (for at least max_pool_size and and max_overflow or we need to reproduce the same functionality using the old style conditional where we ensure absent if it's not set
14:31 mwhahaha did the default values (10,20) get added during the liberty cycle?
14:32 clayton yes
14:32 degorenko i agree that os_service_default can solve this problem
14:32 mwhahaha so if we haven't released the values we should quickly fix them before we cut a release
14:32 degorenko but in the case of large envs you also should change another parameters
14:32 mwhahaha i think it would be ok not to do the warning thing
14:32 clayton mwhahaha they were added when spredzy started this refactoring in august I think.  I assume it wasn't intentional that max_pool_size and max_overflow were set to non-default values
14:33 clayton in fact, oslo_db doesn't even have defaults, it allows sqlalchemy to set the defaults
14:33 mwhahaha he probably copied from neutron where i think there were default values in the config file
14:33 mwhahaha or at least commented values
14:33 mwhahaha http://docs.openstack.org/liberty/config-reference/content/section_neutron.conf.html
14:33 clayton perhaps, neutron is the only service we've changed these values for, so that might be why we didn't see it before
14:34 spredzy clayton, I need to answer to the thread, but I used 10 and 20 because those were the values in nova or neutron module I can't remember. So I thought it was safe to use the same values everywhere. Anyway create the review to update the behavior
14:35 degorenko clayton, i think that https://review.openstack.org/#/c/237970/ should be merged as is - and after that we can update db class with os_service_defaults
14:45 openstackgerrit Alex Schultz proposed openstack/puppet-nova: Use os_package_type for libvirt service name  https://review.openstack.org/243912
14:48 clayton I don't think we should make the situation any worse until we know what the resolution is
14:48 clayton but I'm -1 on it, not -2, so someone else is welcome to merge it
14:48 dfisher joined #puppet-openstack
14:50 mwhahaha My personal opinion is that we should use service default for the db items
14:50 clayton ditto
14:50 mwhahaha my understanding is that we're good to start using service default in all classes based on the conversation from the trello board
14:51 clayton I felt like that was implied, but last I looked it didn't appear to be explicitly stated
14:51 mwhahaha If values were added between stable/kilo and are not in an official release, we should switch them prior to the release
14:51 mwhahaha if we cut a release with values then we will need to properly warn to switch them
14:53 degorenko so, then i see only one solution for my patch
14:53 degorenko wait for implementing os_service_default for glance
14:54 degorenko and then update my patch
14:54 mwhahaha at least a small patch to get it started
14:54 degorenko spredzy, what is your progress with glance and os_service_default?
14:54 mwhahaha maybe not a full conversion but at least the one that updates the tests and converts one class
14:55 mkarpin sorry that i am jumping in
14:55 mkarpin but will be ok this on https://review.openstack.org/#/c/237973/?
14:55 mkarpin https://review.openstack.org/#/c/237973/
14:56 spredzy degorenko, so far none since the summit. I have been on another project since I got back from PTO and haven't had time to look at it yet
14:56 spredzy will try to make some time tomm friday to push a review
14:56 mkarpin this patch is for glance os service default support ^^
14:57 degorenko spredzy, as mentioned mwhahaha can i upload small patch for spec_helper which will add possibility for os_service_default? and make my patch for db as dependency
14:57 ducttape_ joined #puppet-openstack
14:58 mwhahaha or mkarpin can remove the WIP and we can use those
14:58 ducttape_ joined #puppet-openstack
14:58 clayton degorenko: I don't see any reason why we have to wait for the modules to be converted over to os_service_default, in fact I think we should do the opposite
14:58 mkarpin yeah i added -1 because was not sure if it should be done
14:58 spredzy degorenko, not sure I know the context of what the patch to spec_helper would do. Could you remind me please ?
14:58 clayton I think we should decide on a cut-off and not accept any new patches past that date that introduce new parameters that don't use os_service_default
14:59 degorenko spredzy, https://review.openstack.org/#/c/237973/6/spec/spec_helper.rb
14:59 degorenko only this for small patch :)
14:59 clayton converting all the modules over to use os_service_default is a ton of work, but we can decide not to make it any worse
14:59 degorenko just for tests
14:59 clayton degorenko: I think that's great
15:00 mwhahaha yea we need the test update first
15:00 mwhahaha we can switch params later
15:00 degorenko so, we have patch from mkarpin , i think his patch should be update with only this change
15:00 degorenko mwhahaha, ^ thoughts?
15:01 mwhahaha i think his patch is ok as is
15:01 clayton has anyone investigated if it's possible to have the modules source a spec_helper in openstacklib?
15:01 clayton it'd be nice to have that sort of thing centralized instead of in every module
15:01 mwhahaha the problem is openstacklib may not be available to the tests
15:01 degorenko ^^ yes
15:01 mwhahaha i'm not sure the load ordering when you run the tests
15:01 clayton it's listed as a fixture for all the modules, isn't it?
15:02 degorenko yes it is
15:02 mwhahaha yea but then you end up including an rb in the fixtures
15:02 clayton doesn't rake do all the fixture work?
15:02 degorenko but can we use specs from fixtures module?
15:02 degorenko i guess no
15:02 mwhahaha it seems weird to me
15:02 bklei joined #puppet-openstack
15:02 clayton we do a lot of weird things ;)
15:03 mwhahaha indeed
15:03 degorenko let's do more? :)_
15:03 spredzy degorenko, the patch seems ok to me in the spec_helper, clayton's idea about having it centralized make totale sense also if possible
15:03 clayton well, we could probably accomplish the same goal with modulesync
15:03 openstackgerrit Sergey Kolekonov proposed openstack/puppet-neutron: Add parameters for Neutron QoS support  https://review.openstack.org/216654
15:03 degorenko actually
15:04 degorenko we can use $LOAD_PATH.push(File.join(File.dirname(__FILE__), 'fixtures', 'modules', 'openstacklib', 'lib')) - for fact
15:04 degorenko i think
15:04 clayton and if we used modulesync then we have less coupling between openstacklib and he other modules
15:04 clayton but that's more reviews
15:04 spredzy clayton, yes but we have an exponential of n reviews to review for every change
15:04 spredzy on the down side
15:04 clayton nod.  Although that's not too bad if module sync tags the reviews as being modulesync.  I don't know if it does
15:05 clayton ie, if we have a ton of reviews that all start with modulesync: subject, then I can just look at one of them and +2 the rest
15:05 spredzy since our modules are already pretty tied to openstacklib, I don't see it to bad that it is even tieder
15:05 spredzy clayton, yes you can do that, you decide the commit name so you can do "[msync] update os_service_default fact"
15:05 clayton my only complaint is that it makes following master harder, but I never get much sympathy for that complaint :)
15:05 spredzy hehe
15:06 ducttape_ joined #puppet-openstack
15:07 mwhahaha so i think the openstacklib thing goes along with something like https://review.openstack.org/#/c/242330/2/spec/spec_helper.rb
15:08 mwhahaha where we would do a require  'specs/fixtures/modules/openstacklib/spec/defaults.rb' or something of the sort
15:08 mwhahaha and *hopefully* that works :D
15:08 degorenko yeah, i'm also think about such solution
15:09 degorenko we can prepare fix for openstacklib and use depends on
15:09 degorenko and hope that it will work :)
15:09 mwhahaha which reminds me i need to fix that review
15:10 degorenko mwhahaha, one more comment for your patch
15:11 openstackgerrit Merged openstack/puppet-ceph: Replace all StackForge names to OpenStack  https://review.openstack.org/238521
15:12 openstackgerrit Alex Schultz proposed openstack/puppet-cinder: Use rspec-puppet-facts for fact management  https://review.openstack.org/242330
15:13 dhollinger joined #puppet-openstack
15:16 lnxnut joined #puppet-openstack
15:19 dmsimard gchamoul: https://review.openstack.org/#/c/244580/
15:42 ntpttr joined #puppet-openstack
15:46 openstackgerrit Nate Potter proposed openstack/puppet-magnum: puppet-magnum: Initial commit  https://review.openstack.org/242164
15:52 openstackgerrit Nate Potter proposed openstack/puppet-magnum: puppet-magnum: Initial commit  https://review.openstack.org/242164
15:55 angdraug joined #puppet-openstack
15:57 mdorman joined #puppet-openstack
15:58 sergmelikyan joined #puppet-openstack
16:01 zeus joined #puppet-openstack
16:11 openstackgerrit Merged openstack/puppet-manila: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244592
16:16 openstackgerrit Nate Potter proposed openstack/puppet-cinder: Add test coverage for cinder::db::mysql  https://review.openstack.org/244734
16:16 nlahouti joined #puppet-openstack
16:17 ntpttr ^^ hah IRC put a smiley face in my commit message
16:21 zeus joined #puppet-openstack
16:21 kbyrne joined #puppet-openstack
16:21 openstackgerrit Jaume Devesa proposed openstack/puppet-tripleo: MidoNet services manifests  https://review.openstack.org/244184
16:22 nlahouti joined #puppet-openstack
16:37 nlahouti joined #puppet-openstack
16:37 sergmelikyan joined #puppet-openstack
16:42 mdorman joined #puppet-openstack
16:43 sergmelikyan joined #puppet-openstack
16:54 IvanBerezovskiy left #puppet-openstack
16:56 openstackgerrit Jaume Devesa proposed openstack/puppet-tripleo: MidoNet services manifests  https://review.openstack.org/244184
16:59 DarthVigil joined #puppet-openstack
17:03 cwolferh joined #puppet-openstack
17:10 openstackgerrit Merged openstack/puppet-ceph: Remove class_parameter_defaults puppet-lint check  https://review.openstack.org/244580
17:12 DarthVigil joined #puppet-openstack
17:13 sanjayu joined #puppet-openstack
17:16 rmart04 joined #puppet-openstack
17:17 DarthVigil joined #puppet-openstack
17:21 darkxploit joined #puppet-openstack
17:21 rmart04 joined #puppet-openstack
17:22 xingchao joined #puppet-openstack
17:24 rcernin joined #puppet-openstack
17:26 rmart04_ joined #puppet-openstack
17:28 itsuugo joined #puppet-openstack
17:28 aojea_ joined #puppet-openstack
17:32 sergmelikyan joined #puppet-openstack
17:34 gfidente joined #puppet-openstack
17:40 rahulait joined #puppet-openstack
18:00 sergmelikyan joined #puppet-openstack
18:04 openstackgerrit Jaume Devesa proposed openstack/puppet-tripleo: MidoNet services manifests  https://review.openstack.org/244184
18:10 myatsenko crinle:  spredzy:   hello!!!  plz plz look at https://review.openstack.org/#/c/233011/
18:13 mdorman mfisch / clayton   around?  curious how you handle package versions in your puppet deployment.   (i watched your puppetconf talk the other day and now my mind is working)
18:13 clayton we always ensure latest for things we're ever going to upgrade
18:13 clayton and we have versioned repos, and change where the repo points when we want to upgrade
18:14 mdorman kk
18:14 mdorman do you do any rollback functionality?  or always fail forward?
18:15 clayton latter
18:15 clayton our rollback plan is PXE
18:15 mdorman yup
18:15 clayton very few of our nodes have any local state that isn't automated
18:16 mdorman right.
18:16 mdorman thanks for info.
18:16 mdorman we’re making plans to try to get this all going for us, too.
18:16 mdorman trying to narrow it down to some smaller first bites
18:16 clayton np, glad to answer questions
18:29 aimon joined #puppet-openstack
18:30 dgurtner joined #puppet-openstack
18:30 dgurtner joined #puppet-openstack
18:37 itsuugo joined #puppet-openstack
18:41 openstackgerrit John Trowbridge proposed openstack/puppet-ironic: [inspector] Specify SELinux type context for tftp dir  https://review.openstack.org/244675
18:44 darkxploit joined #puppet-openstack
18:46 EmilienM joined #puppet-openstack
18:53 ilbot3 joined #puppet-openstack
18:53 Topic for #puppet-openstack is now Puppet OpenStack modules - https://wiki.openstack.org/wiki/Puppet
18:59 mdorman joined #puppet-openstack
19:02 ntpttr joined #puppet-openstack
19:23 xingchao joined #puppet-openstack
19:24 nlahouti joined #puppet-openstack
19:27 _ody clayton: re: https://review.openstack.org/#/c/243849.  I had decided to do that because facter 3 being a reimplementation of facter in C++11 so free game for API changes but since you asked I check in with puppet-dev.
19:28 clayton it wasn't clear to me if the change was a workaround for a bug, or if the old way of doing this doesn't work with newer facter, or if just the recommended api changed
19:28 _ody They indicate that our old syntax should work but nothing in puppet core uses our syntax and we have modules bugs indicating a need to switch, https://tickets.puppetlabs.com/browse/MODULES-2246.
19:29 clayton wfm then
19:29 _ody I'll make the commit message better and include links to known issues.
19:29 clayton sounds good, thanks
19:30 * _ody would like to start testing against PC1
19:33 bnemec joined #puppet-openstack
19:40 angdraug joined #puppet-openstack
19:47 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-keystone: Add keystone domain specific configuration.  https://review.openstack.org/219289
19:47 iurygregory chem, can you help me with the template problem?
19:47 openstackgerrit John Trowbridge proposed openstack/puppet-ironic: [inspector] Specify SELinux type context for tftp dir  https://review.openstack.org/244675
19:48 chem iurygregory: which problem ?
19:49 ntpttr joined #puppet-openstack
19:49 iurygregory the CI is passing, but when i look at http://logs.openstack.org/21/216821/54/check/gate-puppet-keystone-puppet-beaker-rspec-dsvm-trusty/edaa060/logs/apache_config/10-keystone_wsgi_main.conf.txt.gz is only the initial apache configuration for keystone =/
19:50 chem iurygregory: strang as in my test vm I could see the file modified ...
19:50 iurygregory yeah =/
19:51 rmart04 joined #puppet-openstack
19:51 sergmelikyan joined #puppet-openstack
19:51 chem the thing is that the resulting apache conf is the one done by the last acceptance test, which is not necessarily yours.
19:51 chem this is very unhelpfull and confusing
19:51 iurygregory oh ...
19:51 chem the apache conf, to be usable in the console log should be backup after each acceptance test
19:52 chem you're just seeing the last one
19:52 chem which is not yours
19:52 chem :)
19:52 iurygregory make sense XD
19:52 iurygregory so i'll try to create a vagrant instance and see if the manifest is ok
19:52 chem iurygregory: the stuff work, but check by yourself :)
19:53 iurygregory yeah, can you do review and point anything o find? ^^
19:53 iurygregory when you have time
19:54 chem I'll do my best for tonight, but my guess would be tomorrow morning (in ~12h)
19:55 iurygregory no problem ;)
19:55 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-keystone: Support for multiple ldap backend.  https://review.openstack.org/238164
20:12 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-keystone: Example about composite namevar possibilities.  https://review.openstack.org/244846
20:34 ilbot3 joined #puppet-openstack
20:34 Topic for #puppet-openstack is now Puppet OpenStack modules - https://wiki.openstack.org/wiki/Puppet
20:55 samuelBartel joined #puppet-openstack
20:58 openstackgerrit Adam Vinsh proposed openstack/puppet-swift: Manage swift with swiftinit service provider  https://review.openstack.org/203220
21:01 mwhahaha anyone run across neutron_network resource failure because it can't find the specified keystone_tenant?
21:01 ducttape_ joined #puppet-openstack
21:05 mwhahaha argh i think it's related to the use of the openstack client to do an 'openstack domain list' which may not work because it defaults to v2
21:14 cwolferh joined #puppet-openstack
21:16 openstackgerrit Adam Vinsh proposed openstack/puppet-swift: Manage swift with swiftinit service provider  https://review.openstack.org/203220
21:17 itsuugo joined #puppet-openstack
21:28 xingchao joined #puppet-openstack
21:49 nlahouti joined #puppet-openstack
21:50 kylo joined #puppet-openstack
22:05 hogepodge has anyone else seen the neutron subnet provider breaking?
22:05 nlahouti joined #puppet-openstack
22:05 hogepodge it looks like the client has changed options?
22:06 hogepodge http://paste.openstack.org/show/478743/
22:06 ddmitriev1 joined #puppet-openstack
22:14 rahulait joined #puppet-openstack
22:15 ntpttr joined #puppet-openstack
22:16 sergmelikyan joined #puppet-openstack
22:29 mwhahaha why yes i just created a bug for that
22:29 mwhahaha https://bugs.launchpad.net/puppet-neutron/+bug/1515787
22:29 openstack Launchpad bug 1515787 in puppet-neutron "neutron_subnet provider fails when a identity v2 tenant is used" [Undecided,New]
22:30 mwhahaha er no that's a different one
22:30 mwhahaha hogepodge: what version of the openstack client do you have?
22:30 hogepodge It fails for the kilo python-neutronclient, I'm checking liberty right now
22:30 mwhahaha it works on liberty
22:31 mwhahaha it's failing because of the lack of --format=json
22:31 hogepodge or, I guess kilo version of the python-openstackclient
22:31 hogepodge so puppet modules only support one release back now?
22:31 mwhahaha no that was a regression
22:31 mwhahaha so we'll probably need to figure out an alternative
22:31 ducttape_ joined #puppet-openstack
22:31 mwhahaha please create a bug
22:32 hogepodge done
22:34 hogepodge mwhahaha: https://bugs.launchpad.net/puppet-neutron/+bug/1515786
22:34 openstack Launchpad bug 1515786 in puppet-neutron "Update https://review.openstack.org/#/c/238156/ breaks neutron provider" [Undecided,New]
22:34 gildub joined #puppet-openstack
22:40 openstackgerrit Alex Schultz proposed openstack/puppet-neutron: Add workaround for v2 tenant names  https://review.openstack.org/244900
22:42 mwhahaha degorenko richm please see https://review.openstack.org/#/c/244900/ I'm pretty sure that it's a dirty hack but I'm not sure the best way to fix it
22:42 gildub_ joined #puppet-openstack
22:46 ntpttr left #puppet-openstack
22:56 ducttape_ joined #puppet-openstack
22:58 mdorman joined #puppet-openstack
22:59 openstackgerrit Cody Herriges proposed openstack/puppet-glance: Make sure Facter is only executed on agent  https://review.openstack.org/243849
23:01 _ody clayton: After digging https://review.openstack.org/243849 was a completely different issue.
23:01 _ody Just the thing I ran into exposed it.
23:05 sanjayu joined #puppet-openstack
23:06 richm mwhahaha: does that make sense?
23:10 jpena|zzz joined #puppet-openstack
23:10 openstackgerrit Cody Herriges proposed openstack/puppet-swift: Make sure Facter is only executed on agent  https://review.openstack.org/243854
23:21 openstackgerrit Cody Herriges proposed openstack/puppet-neutron: Swamp Facter[].value with Facter.value()  https://review.openstack.org/243853
23:29 xingchao joined #puppet-openstack
23:29 aimon joined #puppet-openstack
23:39 nlahouti joined #puppet-openstack
23:40 nlahouti1 joined #puppet-openstack
23:47 ducttape_ joined #puppet-openstack
23:47 gildub joined #puppet-openstack
23:48 gildub_ joined #puppet-openstack

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