Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2013-09-10

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

All times shown according to UTC.

Time Nick Message
00:09 dmsimard joined #puppet-openstack
00:42 ari joined #puppet-openstack
00:44 badiane_ka joined #puppet-openstack
01:18 ari joined #puppet-openstack
01:18 xingchao joined #puppet-openstack
01:26 openstackgerrit Xingchao Yu proposed a change to stackforge/puppet-ceilometer: Use connection to replace deprecated database_connection  https://review.openstack.org/45658
01:53 ari joined #puppet-openstack
02:26 ari_ joined #puppet-openstack
03:08 ari_ joined #puppet-openstack
05:04 dguitarbite joined #puppet-openstack
05:38 openstackgerrit Ian Wienand proposed a change to stackforge/puppet-heat: Add heat::keystone::auth class  https://review.openstack.org/45380
05:45 xingchao joined #puppet-openstack
06:38 xingchao joined #puppet-openstack
07:30 mmagr joined #puppet-openstack
07:50 bauzas hi folks
07:53 bauzas I have a quick question
07:54 bauzas does anyone know how to override a default value in Neutron conf (allow_overlapping_ips=False, to be set to true) without providing a change to openstack::controller if Neutron deployed by it ?
07:54 bauzas I mean openstack::neutron class sorry
07:55 bauzas openstack::neutron is declaring ::neutron with some default values and leaving the others as it is
08:19 openstackgerrit A change was merged to stackforge/puppet-ceilometer: Use connection to replace deprecated database_connection  https://review.openstack.org/45658
08:33 derekh joined #puppet-openstack
08:39 e1mer joined #puppet-openstack
08:39 e1mer joined #puppet-openstack
08:52 francois1 joined #puppet-openstack
08:54 francois1 hi there!
08:54 openstackgerrit Seif Lotfy proposed a change to stackforge/puppet-nova: Add ratelimiting options to nova::api  https://review.openstack.org/45577
09:21 badiane_ka joined #puppet-openstack
09:21 ikke-t joined #puppet-openstack
09:21 mgagne1 joined #puppet-openstack
09:21 francois1 joined #puppet-openstack
09:21 xingchao joined #puppet-openstack
09:21 dguitarbite joined #puppet-openstack
09:21 openstackgerrit joined #puppet-openstack
09:21 dachary joined #puppet-openstack
09:21 digicloseup joined #puppet-openstack
09:21 rcrit joined #puppet-openstack
09:21 JoeJulian joined #puppet-openstack
09:21 bodepd joined #puppet-openstack
09:21 andi- joined #puppet-openstack
09:21 fvollero|gone joined #puppet-openstack
09:21 bauzas joined #puppet-openstack
09:21 odyi joined #puppet-openstack
09:21 comptona joined #puppet-openstack
09:21 EmilienM joined #puppet-openstack
09:21 Hunner joined #puppet-openstack
09:21 CoilDomain joined #puppet-openstack
09:21 sileht joined #puppet-openstack
09:21 openf|y joined #puppet-openstack
09:21 fc__ joined #puppet-openstack
09:21 tavasti joined #puppet-openstack
09:21 iwi joined #puppet-openstack
09:21 MxG joined #puppet-openstack
09:21 sbadia joined #puppet-openstack
09:21 ashp joined #puppet-openstack
10:02 openstackgerrit Xingchao Yu proposed a change to stackforge/puppet-keystone: Add provider option for setting token  https://review.openstack.org/45837
11:15 e1mer joined #puppet-openstack
11:54 morazi joined #puppet-openstack
12:13 markvoelker joined #puppet-openstack
12:23 mjblack joined #puppet-openstack
12:30 bauzas hi team,
12:31 bauzas I want to provide allow_overlapping_ips in openstack::controller and openstack::neutron classes, do you know if I have to refer to any BP ?
12:32 bauzas (this time, running puppet-rspec before the patchset :D)
12:35 dmsimard joined #puppet-openstack
12:38 bauzas ok, no worries, will patch it without mention
12:39 dmsimard1 joined #puppet-openstack
12:55 dprince joined #puppet-openstack
12:56 openstackgerrit Sylvain Bauza proposed a change to stackforge/puppet-openstack: Add allow_overlapping_ips parameter to controller and neutron classes  https://review.openstack.org/45865
13:17 mjblack bauzas, just one thing that might get you dinged is you'll more than likely need to update the rspec
13:18 bauzas mmm ok
13:18 bauzas I still need to ramp up on writing rspec unittests
13:19 bauzas lemme look which one is impacted by the change
13:20 mjblack you would not need to start completely from scratch but you will need to add your parameter to the existing test cases
13:21 bauzas ok, aiui I would have to edit openstack_controller_spec.rb by adding overlapping_ips param to the default params ?
13:22 bauzas and also lines 632 and below ?
13:23 bauzas shall I specifically add an unit test for testing the param when set to true ?
13:29 bauzas ok I think I get the point
13:30 bauzas I'll provide a patchset #2 with the updated test cases
13:30 mjblack it sounds like I'm seeing a one sided conversation
13:30 mjblack but yeah you only need to add it to the existing test cases
13:33 openstackgerrit Sylvain Bauza proposed a change to stackforge/puppet-openstack: Add allow_overlapping_ips parameter to controller and neutron classes  https://review.openstack.org/45865
13:35 bauzas mjblack: thanks
13:35 mjblack bauzas: np
13:49 prad joined #puppet-openstack
14:03 otherwiseguy joined #puppet-openstack
14:39 marun joined #puppet-openstack
14:49 blentz joined #puppet-openstack
14:53 mjeanson joined #puppet-openstack
14:58 technolo-g joined #puppet-openstack
15:00 Donckers joined #puppet-openstack
15:00 ari joined #puppet-openstack
15:01 Donckers joined #puppet-openstack
15:03 terryw joined #puppet-openstack
15:22 ari joined #puppet-openstack
15:25 marun joined #puppet-openstack
15:41 otherwiseguy It makes me happy when I install a test environment with packstack on Red Hat RDO and I see it installing puppet 3.2.4.
15:41 otherwiseguy So much better that seeing it install puppet 2.6. :)
16:00 mjblack_ joined #puppet-openstack
16:01 mjblack_ joined #puppet-openstack
16:35 marun is there any support, planned or otherwise, for upgrading openstack releases via puppet?
16:40 EmilienM ouch, nice topic :)
16:42 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-nova: Add Nova Cells support  https://review.openstack.org/41908
17:09 dprince joined #puppet-openstack
17:27 ari__ joined #puppet-openstack
17:35 mjblack joined #puppet-openstack
17:38 badiane_ka joined #puppet-openstack
17:44 mjblack joined #puppet-openstack
18:36 bodepd otherwiseguy: that's huge!
18:36 bodepd otherwiseguy: congrats!
18:36 bodepd marun: I would like for there to be plans
18:52 ari__ joined #puppet-openstack
18:54 dachary joined #puppet-openstack
19:00 bodepd mgagne: what do you think about deprecating the sql_connection params?
19:01 bodepd (did I already ask you that)
19:01 mgagne bodepd: context?
19:01 bodepd and instead replacing them with their components
19:01 bodepd if you think about the way that data flows into the class parameters
19:01 bodepd the sql_connection requires duplicating bits of data that are needed by other parts of the system
19:02 bodepd you should only have to specify a single key called db_host
19:02 bodepd and not resupply it as part of a sql_connection
19:02 bodepd (this is all part of an effort to better model things to avoid needing higher level composition classes)
19:03 bodepd mgagne: context ^^^
19:03 bodepd there are various other class parameters that also need to be refactored (ie: broken up into their bits)
19:04 bodepd the other issue with sql_conneciton as a class param is that it conflates values that should have defaults:
19:04 bodepd db_name, db_user, db_type
19:04 bodepd with things that do not (db_host, db_passwd) which means that you can't really supply defaults for them anywhere
19:27 openstackgerrit François Charlier proposed a change to stackforge/puppet-keystone: Update creds_keystone exception handling  https://review.openstack.org/45931
19:27 otherwiseguy joined #puppet-openstack
19:28 ari__ joined #puppet-openstack
19:37 openstackgerrit Dan Prince proposed a change to stackforge/puppet-keystone: Switch from signing/format --> token/provider.  https://review.openstack.org/45933
19:39 openstackgerrit François Charlier proposed a change to stackforge/puppet-keystone: Remove the get_service_id method and inline python  https://review.openstack.org/45934
20:10 openstackgerrit François Charlier proposed a change to stackforge/puppet-keystone: Enable serving keystone from apache mod_wsgi  https://review.openstack.org/29059
20:24 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-neutron: Fix module name in Modulefile  https://review.openstack.org/45945
20:28 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-neutron: Fix module name in Modulefile  https://review.openstack.org/45947
20:35 mgagne bodepd: I know what you are referring to. I just don't have time to forge an opinion/solution for it.
21:01 prad joined #puppet-openstack
21:03 bodepd joined #puppet-openstack
21:04 bodepd mgagne: the solution is to add more parameters and then either have them compose the sql_connection (and let sql_connection override) or deprecarte it
21:04 bodepd mgagne: no problem. Given that I'm moving to the new composition model, I will likely just submit patches for the change that I need
21:04 mgagne bodepd: my fear is that at one point, db_* parameters will get out of control.
21:06 openstackgerrit A change was merged to stackforge/puppet-neutron: Fix module name in Modulefile  https://review.openstack.org/45945
21:06 openstackgerrit A change was merged to stackforge/puppet-neutron: Fix module name in Modulefile  https://review.openstack.org/45947
21:06 ari joined #puppet-openstack
21:06 bodepd mgagne: it would initially be limited to the parameters that comprise the sql_connection url
21:08 mgagne bodepd: just look at what happened to db_* in puppet-openstack. Someone needed mysql ssl support and had to add parameters to all classes to support it. Unless we come up with a clever way to handle such situation/flexibility, I don't see it as a good idea.
21:15 bodepd mgagne: I think for that, you would need another parameter called connection_options
21:16 bodepd mgagne: I don't find that to be too messy, but as I mentioned, I will just open up the patches that I need
21:18 openstackgerrit A change was merged to stackforge/puppet-keystone: Update creds_keystone exception handling  https://review.openstack.org/45931
21:27 openstackgerrit A change was merged to stackforge/puppet-openstack: Added missing param to neutron and cinder classes  https://review.openstack.org/45696
21:27 openstackgerrit A change was merged to stackforge/puppet-openstack: Added missing param to quantum and cinder classes  https://review.openstack.org/45703
21:54 prad joined #puppet-openstack
22:17 prad joined #puppet-openstack
23:16 mjblack_ joined #puppet-openstack
23:19 openstackgerrit Dan Bode proposed a change to stackforge/puppet-neutron: Update invalid sql_connection erorr message  https://review.openstack.org/45972
23:21 openstackgerrit Dan Bode proposed a change to stackforge/puppet-neutron: Update invalid sql_connection erorr message  https://review.openstack.org/45973
23:39 openstackgerrit Dan Bode proposed a change to stackforge/puppet-neutron: Update invalid sql_connection error message  https://review.openstack.org/45972
23:40 mgagne bodepd: why not outputting the actual parameter name?
23:45 openstackgerrit Dan Bode proposed a change to stackforge/puppet-neutron: Update invalid sql_connection error message  https://review.openstack.org/45973
23:49 bodepd mgagne: not any reason in particular
23:50 bodepd if you want to reject it for that reason, just document it inline, and I'll update it

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