Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2013-12-26

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

All times shown according to UTC.

Time Nick Message
23:09 xarses joined #puppet-openstack
23:43 xarses joined #puppet-openstack
00:06 xarses joined #puppet-openstack
00:27 xarses joined #puppet-openstack
01:15 mgagne joined #puppet-openstack
01:49 tnoor joined #puppet-openstack
02:16 xingchao joined #puppet-openstack
02:18 rongze joined #puppet-openstack
02:22 xingchao joined #puppet-openstack
02:25 openstackgerrit Bob Callaway proposed a change to stackforge/puppet-cinder: Update manifest and spec to use new NetApp unified driver config options.  https://review.openstack.org/62733
02:26 blkperl_ joined #puppet-openstack
02:27 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-openstack: Remove duplicated parameters for keystone  https://review.openstack.org/64078
02:29 michchap_ joined #puppet-openstack
02:31 mgagne1 joined #puppet-openstack
02:40 ianw joined #puppet-openstack
03:16 openstackgerrit Xingchao Yu proposed a change to stackforge/puppet-nova: Fix configuring libvirt before installation  https://review.openstack.org/58123
04:01 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-neutron: Ensure ML2 plugin package is installed  https://review.openstack.org/64085
04:03 openstackgerrit Xingchao Yu proposed a change to stackforge/puppet-neutron: Fix configuring ml2 problem in RedHat  https://review.openstack.org/64086
04:04 mgagne1 xingchao: about the same time :O
04:05 xingchao mgagne: aha
04:05 mgagne don't know why I'm triaging bugs on christmas ^^'
04:06 xingchao mgagne: oh, Merry Christmas to you :)
04:06 mgagne xingchao: thanks, you too ;)
04:07 xingchao mgagne: don't you have a holiday today?
04:07 mgagne xingchao: yep, I'm alone at home, just woke up at 6pm :-/
04:08 mgagne xingchao: my mind just can't rest during holidays, vacation or days off. ^^'
04:10 xingchao mgagne: I guess your boss likes you very much :)
04:10 mgagne xingchao: I suppose ;)
04:11 mgagne xingchao: starting to get ideas to improve puppet modules
04:11 xingchao mgagne:   I see you use $ml2_server_package variable to decide whether install ml2 package or not, why not put it in ml2.pp  selector ? It should be installed in Redhat
04:11 mgagne xingchao: well, we already have a pattern used in neutron::agent to check if platform needs a package or not.
04:12 mgagne xingchao: I reused it
04:15 mgagne xingchao: one of the idea I had is: As openstack projects are starting to split common libs/code to oslo, we should try to do the same in our modules. Not sure yet how to accomplish it but that's the idea I had.
04:15 mgagne xingchao: and try to find a way to split database/messaging configuration from init.
04:18 xingchao mgagne: this topic has been discussed before, we could start from spliting database codes from modules.
04:18 mgagne xingchao: there is one main challenge: My understanding is that people expect to be able to include the XXX::init class and be done without the need to include and configuration an other class such as XXX::database or XXX::messaging. They expect XXX::init to be some sort of composition layer for those kind of configurations. This is challenging due to the way puppet works (being a declarative language)
04:19 mgagne xingchao: one simple challenge we faced recently is cinder::logging.
04:20 mgagne xingchao: you don't need cinder::logging to configure cinder. but people expected cinder::logging to be included and configured through cinder::init, repeating what I qualify the mess that is puppet-openstack.
04:24 xingchao mgagne: I think it's a normal way to use include another class to enable a new function, whtat I want is to reuse common codes, such as XXX:database and xxx:mesaging as you said.
04:26 mgagne xingchao: sure. My fear is the potential mess this desire to split common code could generate. The mess I'm referring to is puppet-openstack. puppet-openstack is some kind of composition layer and this is not what I would like for other puppet modules to be.
04:29 xingchao mgagne: I don't see cinder::logging in stackforge and review site, any links ?
04:29 mgagne xingchao: it got abandoned due to inactivity
04:29 mgagne https://review.openstack.org/#/c/59338/
04:32 xingchao mgagne: I would think about this issue, it's lunch time here :)
04:33 mgagne xingchao: alright, bon appétit
04:33 xingchao mgagne: what does "bon appétit" mean ? :)
04:34 mgagne xingchao: could be translated to "enjoy your meal"
04:34 xingchao mgagne: xiexie :D
04:34 mgagne xingchao: not sure if google translate did it right: 津津有味
04:35 mgagne ;)
04:35 xingchao mgagne: 津津有味 means it's very delicious
04:35 mgagne xingchao: come on google lol
04:42 mgagne xingchao: I think the idea is: classes shouldn't forward their data to other classes.
04:42 mgagne xingchao: idea found here: https://github.com/bodepd/openstack-cosi-blueprints/blob/37281175a9f96a85058740190ca6b513c096767a/blueprints/better_puppet_module.md
04:43 mgagne xingchao: "classes do not have to forward their data to other classes."
06:21 rongze joined #puppet-openstack
06:23 xarses joined #puppet-openstack
08:00 xingchao mgagne:  I have read the cinder::logging and understand your meaning. The idea about "classes do not have to forward their data to other classes " is pretty good, but b/c puppet current limitation, I don't have a idea realize it.  When introducing a new class into one class, Hiera can set automatically some params by different factor, but there are still some params must be set by end user.
08:14 mattymo|srt joined #puppet-openstack
08:45 piliman974 joined #puppet-openstack
09:49 piliman974 joined #puppet-openstack
10:18 openstackgerrit Xingchao Yu proposed a change to stackforge/puppet-neutron: Add ovs_use_veth option  https://review.openstack.org/64119
10:26 piliman974 joined #puppet-openstack
10:53 piliman974 joined #puppet-openstack
11:21 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-nova: Deprecate old libvirt config  https://review.openstack.org/63978
11:21 piliman974 joined #puppet-openstack
11:23 EmilienM mgagne: did some triage also
11:56 rongze joined #puppet-openstack
12:03 rongze joined #puppet-openstack
12:06 mmosesohn joined #puppet-openstack
12:47 piliman974 joined #puppet-openstack
13:43 xingchao joined #puppet-openstack
14:08 openstackgerrit Daniel Gollub proposed a change to stackforge/puppet-neutron: Support dnsmasq_config_file option for dhcp-agent  https://review.openstack.org/63520
14:24 piliman974 joined #puppet-openstack
14:31 piliman974 joined #puppet-openstack
14:56 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-heat: engine_life_check_timeout support  https://review.openstack.org/64146
15:15 tnoor joined #puppet-openstack
15:31 openstackgerrit Sebastien Badia proposed a change to stackforge/puppet-ceilometer: Fix package ceilometer-alarm type error on Debian  https://review.openstack.org/63935
15:58 tnoor1 joined #puppet-openstack
16:05 rongze joined #puppet-openstack
16:32 thumpba joined #puppet-openstack
16:41 angdraug joined #puppet-openstack
16:57 otherwiseguy joined #puppet-openstack
17:23 brdude joined #puppet-openstack
17:34 hogepodge joined #puppet-openstack
18:37 openstackgerrit A change was merged to stackforge/puppet-neutron: Ensure ML2 plugin package is installed  https://review.openstack.org/64085
18:43 openstackgerrit A change was merged to stackforge/puppet-neutron: Add new haproxy section  https://review.openstack.org/63944
18:56 openstackgerrit A change was merged to stackforge/puppet-keystone: Refactor keystone_user_role to be lazier  https://review.openstack.org/47891
19:40 thumpba joined #puppet-openstack
19:44 thumpba joined #puppet-openstack
20:06 EmilienM mgagne: Hi Mathieu, if you have one minute, I would like your thoughts about https://review.openstack.org/#/c/63520/2/manifests/agents/dhcp.pp
20:17 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-heat: engine_life_check_timeout support  https://review.openstack.org/64146
20:29 tnoor joined #puppet-openstack
20:37 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-cinder: Update GlusterFS driver  https://review.openstack.org/63979
20:37 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-cinder: Update GlusterFS driver  https://review.openstack.org/63979
21:20 tnoor joined #puppet-openstack
21:31 thumpba joined #puppet-openstack
21:35 thumpba joined #puppet-openstack
21:37 lolah joined #puppet-openstack
22:16 otherwiseguy joined #puppet-openstack
22:26 openstackgerrit A change was merged to stackforge/puppet-nova: Creating qpid_sasl_mechanisms parameter  https://review.openstack.org/58045
22:26 openstackgerrit A change was merged to stackforge/puppet-cinder: Creating qpid_sasl_mechanisms parameter  https://review.openstack.org/58003
22:32 openstackgerrit A change was merged to stackforge/puppet-keystone: Updated endpoints to support paths and ssl  https://review.openstack.org/55303

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