| 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 |
