Time |
Nick |
Message |
00:18 |
|
britthouser joined #puppet-openstack |
01:33 |
|
sputnik1_ joined #puppet-openstack |
01:36 |
|
sputnik__ joined #puppet-openstack |
01:56 |
|
sputnik1_ joined #puppet-openstack |
02:17 |
|
mgagne joined #puppet-openstack |
03:58 |
|
rmoe joined #puppet-openstack |
04:43 |
|
mgagne1 joined #puppet-openstack |
04:50 |
|
starmer joined #puppet-openstack |
06:32 |
|
starmer joined #puppet-openstack |
10:02 |
|
ryanycoleman joined #puppet-openstack |
11:38 |
|
andi- joined #puppet-openstack |
12:19 |
openstackgerrit |
A change was merged to stackforge/puppet-neutron: Add missing quotas parameters https://review.openstack.org/83823 |
12:28 |
openstackgerrit |
Michael Chapman proposed a change to stackforge/puppet_openstack_builder: Remove controller and compute class groups. https://review.openstack.org/78579 |
12:34 |
openstackgerrit |
Michael Chapman proposed a change to stackforge/puppet_openstack_builder: Rewrite data model to be pure hiera https://review.openstack.org/81241 |
12:37 |
EmilienM |
michchap: sunday evening is a good time to make puppet, huh? :) |
12:38 |
michchap |
EmilienM: yep :) |
12:39 |
michchap |
EmilienM: all times are good times |
12:42 |
michchap |
EmilienM: I have a lot of trouble remembering which things I have reviewed... =/ |
12:45 |
michchap |
EmilienM: what do you think about https://review.openstack.org/#/c/83341/ and https://review.openstack.org/#/c/83458/ ? do you think we should have a nova::notify class or put everything into nova? |
12:56 |
EmilienM |
michchap: looking |
12:58 |
EmilienM |
michchap: interesting question |
12:58 |
EmilienM |
michchap: afik, glance has it |
12:58 |
michchap |
EmilienM: I think the patch is good and we dont' need notify, because in http://docs.openstack.org/trunk/config-reference/content/list-of-compute-config-options.html is it listed under common |
12:58 |
EmilienM |
yeah |
12:59 |
michchap |
EmilienM: but maybe in general we could try to stick to the categories that upstream has for its conf, sicne nova has so many and the init class has ~50 parameters now |
12:59 |
EmilienM |
michchap: it makes sense, I agree |
12:59 |
michchap |
EmilienM: like we do for the quota class, for example. |
12:59 |
EmilienM |
michchap: but it's usual to see that |
12:59 |
EmilienM |
michchap: look in rabbitmq module |
12:59 |
EmilienM |
or apache |
13:01 |
michchap |
EmilienM: I guess I like to include classes, and by looking at the classes I'm including I can tell roughly what the config is. I don't like those modules and the way they have lots of parameters that tell the init class to include other classes |
13:01 |
EmilienM |
michchap: i agree and I have another example more critical in Nova: database |
13:01 |
michchap |
EmilienM: I feel that having the user explicitly include more than just the init class is quite nice. |
13:02 |
EmilienM |
since conductor, compute do not need database access anymore, and they still have it in nova class which is very bad |
13:02 |
michchap |
EmilienM: Yeah...and it is bad for security. |
13:02 |
EmilienM |
yep |
13:03 |
michchap |
EmilienM: I noticed it too. I think it would have to be a breaking change though. |
13:03 |
EmilienM |
michchap: we should define a way to fix it in J |
13:03 |
EmilienM |
I already created a blueprint by the way |
13:34 |
|
saju_m joined #puppet-openstack |
14:03 |
|
ryanycoleman joined #puppet-openstack |
14:39 |
openstackgerrit |
Benedikt Trefzer proposed a change to stackforge/puppet-neutron: add https support options to metadata agent https://review.openstack.org/83769 |
14:50 |
|
thuc joined #puppet-openstack |
15:01 |
|
hogepodge joined #puppet-openstack |
15:43 |
|
thuc joined #puppet-openstack |
15:48 |
|
morazi joined #puppet-openstack |
16:04 |
|
ryanycoleman joined #puppet-openstack |
16:48 |
|
Aimon joined #puppet-openstack |
16:53 |
|
thuc joined #puppet-openstack |
16:56 |
|
ryanycoleman joined #puppet-openstack |
17:04 |
|
ryanycoleman joined #puppet-openstack |
17:14 |
|
thuc joined #puppet-openstack |
17:16 |
|
thuc joined #puppet-openstack |
17:35 |
|
thuc joined #puppet-openstack |
17:51 |
|
thuc joined #puppet-openstack |
19:01 |
|
thuc joined #puppet-openstack |
19:07 |
|
thuc joined #puppet-openstack |
19:18 |
|
thuc joined #puppet-openstack |
19:48 |
|
thuc joined #puppet-openstack |
19:49 |
|
thuc_ joined #puppet-openstack |
20:02 |
|
thuc joined #puppet-openstack |
20:12 |
|
thuc joined #puppet-openstack |
20:42 |
|
kkirkpatrick joined #puppet-openstack |
20:42 |
|
thuc joined #puppet-openstack |
21:15 |
|
thuc joined #puppet-openstack |
21:20 |
|
thuc joined #puppet-openstack |
21:49 |
|
thuc joined #puppet-openstack |
22:40 |
|
ryanycoleman joined #puppet-openstack |
22:52 |
|
ianw joined #puppet-openstack |
23:58 |
|
ryanycoleman joined #puppet-openstack |