Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2014-10-21

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

All times shown according to UTC.

Time Nick Message
00:08 hogepodge crinkle, fwiw, most of the US goes off of daylight savings at the end of this month. This means for the west coast 1400 UTC is going to be 6 am.
00:09 crinkle oh :/
00:09 hogepodge I just realized it myself after putting a bunch of stuff into my calendar an hour off :-P
00:12 nibalizer yea 14:00.... maybe if i stay up lol
00:24 rmoe joined #puppet-openstack
00:35 aimon joined #puppet-openstack
01:08 michchap crinkle: as long as the agenda gets up early I can easily just stay up if I see something that justifies it.
01:09 michchap crinkle: so I'm fine with putting at a time that works for EU/US East/US West and if I have something important I'll just drink caffeine.
01:13 crinkle I think it would be good to re-poll with more than 3 options
01:14 crinkle as hoge pointed out 1400 will actually be pretty tough for me
01:15 michchap yeah just push it a bit later imo, it's more important that you're able to attend than xc or myself.
01:16 crinkle :/
01:16 michchap I'm about to start working on a project with some us east folks anyway so I might be awake at strange hours
01:32 xarses joined #puppet-openstack
02:06 morazi joined #puppet-openstack
02:08 xingchao joined #puppet-openstack
03:03 mmaglana joined #puppet-openstack
03:42 mmaglana joined #puppet-openstack
04:03 sanjayu joined #puppet-openstack
04:04 jamespd joined #puppet-openstack
05:05 topshare joined #puppet-openstack
05:05 daneyon joined #puppet-openstack
05:34 snk joined #puppet-openstack
06:36 topshare joined #puppet-openstack
06:49 mmagr joined #puppet-openstack
07:08 mmagr joined #puppet-openstack
07:13 dgurtner joined #puppet-openstack
07:34 dklepikov joined #puppet-openstack
07:40 holser joined #puppet-openstack
08:04 daneyon joined #puppet-openstack
08:08 daneyon joined #puppet-openstack
08:10 berendt joined #puppet-openstack
08:12 sanjayu joined #puppet-openstack
08:14 qba73 joined #puppet-openstack
08:27 mmaglana joined #puppet-openstack
08:34 markvoelker joined #puppet-openstack
08:35 openstackgerrit joined #puppet-openstack
08:35 sileht joined #puppet-openstack
09:10 dkaigarodsev joined #puppet-openstack
10:35 dgurtner joined #puppet-openstack
11:10 alokj joined #puppet-openstack
11:22 rlaurika joined #puppet-openstack
11:37 larsks joined #puppet-openstack
11:37 larsks joined #puppet-openstack
11:37 rwsu joined #puppet-openstack
11:37 kbyrne joined #puppet-openstack
11:38 igajsin joined #puppet-openstack
11:41 openstackgerrit John Patterson proposed a change to stackforge/puppet-neutron: Adding neutron_service_provider type  https://review.openstack.org/129360
11:53 vb joined #puppet-openstack
12:04 EmilienM good morning!
12:12 dgurtner joined #puppet-openstack
12:19 Mosibi joined #puppet-openstack
12:43 rcallawa joined #puppet-openstack
13:06 richm joined #puppet-openstack
13:22 morazi joined #puppet-openstack
13:27 dmsimard_away joined #puppet-openstack
13:58 openstackgerrit Adrian Joian proposed a change to stackforge/puppet-nova: Adding support for file injects  https://review.openstack.org/129913
14:00 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-openstack-specs: Common OpenStack Identity Resources  https://review.openstack.org/129700
14:01 EmilienM mgagne: bogdando: I've addressed all your comments, please review again. I would like to go ahead next week and don't want to keep this blueprint a lot of time. It's just refactorization for now.
14:24 rmoe joined #puppet-openstack
14:34 dgurtner joined #puppet-openstack
14:34 topshare joined #puppet-openstack
14:35 topshare joined #puppet-openstack
14:39 derekh joined #puppet-openstack
14:42 mmaglana joined #puppet-openstack
14:43 ericpeterson joined #puppet-openstack
14:43 rmoe joined #puppet-openstack
14:43 mdorman joined #puppet-openstack
14:45 michchap EmilienM: I think I made something similar to your spec. https://github.com/aptira/puppet-stacktira/blob/master/manifests/openstack/service_auth.pp
14:46 EmilienM michchap: hey, you here so late. Looking
14:47 EmilienM michchap: nice! it's almost what I planned to do
14:47 michchap EmilienM: I think use that in each module, and maybe also make a wrapper that accepts a hash, so we can do multi-region?
14:48 michchap EmilienM: That's what I did with mine, anyway. I thought it worked OK, but the wrapper got very complicated as it needed a lot of defaults for every service: https://github.com/aptira/puppet-stacktira/blob/master/manifests/openstack/region_auth.pp
14:48 michchap EmilienM: maybe you can do better.
14:49 EmilienM michchap: if you read my specs, I'm also talking about Hiera for multi regions
14:49 EmilienM michchap: I would need your help on reviewing the code
14:50 EmilienM to be sure we are flexible enough with data modeling
14:50 michchap EmilienM: My test case was two regions with a shared keystone and glance. Once I could do that I felt I had enough flexibility, but having sane defaults is a bit difficult.
14:53 topshare joined #puppet-openstack
14:56 openstackgerrit Adrian Joian proposed a change to stackforge/puppet-nova: Adding support for file injects  https://review.openstack.org/129913
15:00 EmilienM michchap: bookmarking your code ;)
15:21 EmilienM sbadia: could we go ahead with metadata.json patches?
15:22 sbadia hi
15:22 sbadia EmilienM: yep, i'll take 10min to update all this patches
15:22 EmilienM sbadia: thx
15:23 EmilienM spredzy|afk:  I have the same remark regarding all policy.json patches. Can we have an update?
15:25 spredzy|afk EmilienM, yep need to update those patches will try to do it by Wed.
15:26 spredzy|afk I mean Thu. I though today was Monday
15:26 EmilienM spredzy|afk: nice!
15:27 EmilienM we have a bunch of puppet-vswitch paches, don't know if someone cares about them
15:28 EmilienM they don't have tests anyway... -1
15:32 EmilienM anycore around could have a look at https://review.openstack.org/#/c/119507/? mgagne crinkle fvollero fc__ markvoelker
15:37 mgagne EmilienM: done
15:39 EmilienM mgagne: thx
15:40 openstackgerrit A change was merged to stackforge/puppet-trove: Migrate mysql backend to use openstacklib::db::mysql  https://review.openstack.org/119507
15:41 mschmitt joined #puppet-openstack
15:44 openstackgerrit Richard Megginson proposed a change to stackforge/puppet-keystone: support the ldap user_enabled_invert parameter  https://review.openstack.org/129951
15:47 pradk joined #puppet-openstack
15:47 rcallawa joined #puppet-openstack
16:02 cjellick joined #puppet-openstack
16:09 rmoe joined #puppet-openstack
16:25 cjellick joined #puppet-openstack
16:39 aimon_ joined #puppet-openstack
16:53 sputnik13 joined #puppet-openstack
17:00 cjellick joined #puppet-openstack
17:01 openstackgerrit Richard Megginson proposed a change to stackforge/puppet-keystone: support the ldap user_enabled_invert parameter  https://review.openstack.org/129951
17:46 EmilienM mgagne: hey, what do you think about the comment ? https://review.openstack.org/#/c/124543/3/spec/classes/neutron_init_spec.rb
17:46 EmilienM I think we cannot blame spredzy to test augeas here. It's actually kind of fair. But I wanted your thoughts
17:50 openstackgerrit Yanis Guenane proposed a change to stackforge/puppet-neutron: policy.json: Allow one to manage them from the puppet module  https://review.openstack.org/124543
17:51 EmilienM sounds like spredzy updated! sounds good to me
17:52 berendt joined #puppet-openstack
17:52 EmilienM spredzy: does policies bits don't go only for API server?
17:52 spredzy EmilienM, mgagne yep, this way if something changes in the implementation of openstacklib::policy those test won't be affected as long as they're called
17:53 EmilienM spredzy: I think the policy things in Neutron should go in neutron::server
17:53 * EmilienM runs
17:53 spredzy looking right now, for heat and co also, but yep you might be right
17:56 xarses joined #puppet-openstack
18:11 openstackgerrit Chris Hoge proposed a change to stackforge/puppet-sahara: Added sahara.conf provider and unit tests.  https://review.openstack.org/128675
18:16 xarses joined #puppet-openstack
18:29 mgagne EmilienM: Can't a policy be enforced later down the stack on a non-api service or am I wrong?
19:06 derekh joined #puppet-openstack
19:08 EmilienM mgagne: to me, a policy is only for api services
19:11 mgagne EmilienM: what about https://github.com/openstack/nova/blob/master/nova/compute/api.py ? Can we assume that the only consumer is and will be nova-api?
19:12 EmilienM mgagne: I think so
19:37 rcallawa joined #puppet-openstack
19:43 openstackgerrit Yanis Guenane proposed a change to stackforge/puppet-neutron: policy.json: Allow one to manage them from the puppet module  https://review.openstack.org/124543
19:46 spredzy EmilienM, so for stuff like heat, I should inlcude it in all 3 apis file, right ?
19:46 EmilienM spredzy: I think so
19:47 EmilienM mgagne was not sure though
19:47 EmilienM to me, it makes sence policy think is only for API
19:47 mgagne EmilienM: won't they conflict each others?
19:47 EmilienM mgagne: with default you mean?
19:48 spredzy EmilienM, imagine, you have the case where you set the 3 apis on the same node (most common case)
19:48 spredzy you might end up with Duplicated Ressource
19:48 mgagne EmilienM: ^
19:50 spredzy mgagne, since default is empty, can't we rely on deployer knowledge to not deploy twice the same propery on the same node ?
19:51 spredzy I mean by default, if no policies is passed it will never conflict
19:51 mgagne spredzy: don't you think someone will come back later and ask for the resource to be wrapped in if defined ? :D
19:52 spredzy mgagne, :(
19:52 spredzy Ok will wrap it up
19:52 mgagne spredzy: we shouldn't implement something that could conflict under normal usage
19:53 spredzy on the top of your head do you guys know if defined(Class[$ANARRAY]) works ?
19:53 spredzy my bad haven't said anything
19:53 spredzy forget about that
19:54 mgagne spredzy: how about we create neutron::policy and ask neutron::server to include it? if people wish to manage policy, they should include that class first with all needed parameters and then include the neutron::server class
19:54 mgagne spredzy: or use hiera
19:54 * mgagne takes cover
19:56 spredzy for the if defined I would implement it there https://github.com/stackforge/puppet-openstacklib/blob/master/manifests/policy/base.pp#L25
19:56 spredzy agree ?
19:56 mmagr joined #puppet-openstack
19:56 mgagne spredzy: I would go down that path: create neutron::policy. Move the code here. Add include ::neutron::policy to neutron::server. Ask users to use Hiera to override policies or include neutron::policy before neutron::server and define their policies there
19:56 spredzy for the second suggestion the idea is that by default the default policy will be used, nothing touched
19:56 spredzy if and only if the user wants to change a policy it will have to specify a polocy_path + 1 or many policies
19:56 mgagne spredzy: we are touching neutron::server =)
19:58 spredzy mgagne, ok, for the path, the same logic will have to be enabled for each component
19:58 mgagne spredzy: we should test first =)
19:59 spredzy f:q!
19:59 spredzy oops
19:59 spredzy wrong window
19:59 mgagne spredzy: can you include (with include dha) a class already previously declared with class { 'foobar': } syntax
20:01 spredzy Nop
20:01 spredzy [root@a modules]# puppet apply -e 'include ::apache class { "apache" : }'
20:01 spredzy Error: Duplicate declaration: Class[Apache] is already declared; cannot redeclare at line 1 on node a
20:01 spredzy mgagne, ^
20:02 mgagne spredzy: how about the other way around
20:02 spredzy mgagne, funny the other way around it works
20:03 mgagne spredzy: that's what I suggest to do
20:03 spredzy since class apache already existe and the second call is an include it doesn't force it
20:04 spredzy so in neutron::server create a if $::neutron_manage_policy { include ::neutron::policy } and let the user specify the policies through hiera /
20:04 spredzy ?
20:04 mgagne spredzy: http://paste.openstack.org/show/122830/
20:05 mgagne http://paste.openstack.org/show/122831/
20:05 spredzy ok
20:06 mgagne spredzy: that's for people not using hiera. if you are using hiera, you don't need class { '::neutron::policy': }
20:06 spredzy should I create a flag $neutron_manage_policy ?
20:06 mgagne spredzy: why would you?
20:06 spredzy or since the default is to do nothing it isn't worth it
20:06 mgagne spredzy: that's my opinion unless someone has a valid concern
20:07 spredzy ok will go down this road, if someone has a say against it, it will be on gerrit
20:07 * mgagne goes on gerrit and -1 his own suggestion /troll
20:08 * spredzy smiled
20:09 mgagne I should be careful with caffeine if I want to make it to Paris
20:10 mgagne someone posted a chart about caffeine consumption and effects on health on the fridge, next to the coffee machine. not sure if it was targeted at me...
20:22 spredzy Caffeine is not bad per se, abuse his :)
20:30 openstackgerrit Yanis Guenane proposed a change to stackforge/puppet-neutron: policy.json: Allow one to manage them from the puppet module  https://review.openstack.org/124543
20:32 openstackgerrit Yanis Guenane proposed a change to stackforge/puppet-neutron: policy.json: Allow one to manage them from the puppet module  https://review.openstack.org/124543
20:32 spredzy mgagne, do we agree on this pattern ?
20:33 mgagne spredzy: I guess I do =)
20:36 mgagne always fun to see or be remembered that your coworker comes to work in a kayak
20:36 spredzy I give him 2 weeks to stop :)
20:36 mgagne spredzy: we will see, he said we was planning to still use it in winter
20:37 spredzy When river is frozen ?
20:37 rcallawa joined #puppet-openstack
20:38 openstackgerrit Yanis Guenane proposed a change to stackforge/puppet-openstacklib: policy : Add a if defined wrapper to avoid duplicate resource  https://review.openstack.org/130031
20:38 spredzy mgagne, the if defined() ^
20:38 mgagne spredzy: might not all be frozen but I'm still curious myself about how he will manage to do it
20:39 mgagne spredzy: I don't think we need it tbh
20:39 spredzy optimistically, temperature will be negative in two week
20:39 mgagne spredzy: if the above pattern is used, you shouldn't encounter duplicated resources
20:41 spredzy mgagne, imagine a deployer uses heat-api, heat-api-cfn and heat-api-cloudwatch in 3 different nodes, and set a bunch of policies (some being the same), that's fine, now if the deployer decide to use them in a single node, it will trigger this issues unless he changes his code first
20:41 spredzy but we get back to what you mentioned earlier
20:42 mgagne spredzy: yep, that's why I proposed it
20:45 spredzy ahm, so all three would include ::heat::policy and the hiera conf will be fine
20:45 * spredzy think its time to go home :(
20:47 mgagne yes, IMO it's a good plan
20:48 spredzy mgagne, will wait for the feedback on that one and will apply to all the components
20:48 mgagne spredzy: cool
20:53 igajsin joined #puppet-openstack
20:56 rcallawa joined #puppet-openstack
20:59 dgurtner joined #puppet-openstack
20:59 dgurtner joined #puppet-openstack
21:47 morazi joined #puppet-openstack
22:00 openstackgerrit Spencer Krum proposed a change to stackforge/puppet-nova: Initial rspec-beaker testing scaffolding  https://review.openstack.org/102020
22:05 nibalizer great news everyone, the rspec-beaker scaffoling merged
22:05 nibalizer so soon we should be running nonvoting tests for openstack-infra/puppet-storyboard
22:05 nibalizer then all other puppet openstack things can do similar
22:09 EmilienM nibalizer: awesome
22:59 nibalizer EmilienM: so actually i misspoke
22:59 nibalizer the tes is failing
22:59 nibalizer https://review.openstack.org/#/c/126086/
22:59 nibalizer if you look in the non-voting column
22:59 nibalizer crinkle: ^^
23:01 crinkle yay failing tests \o/
23:05 derekh joined #puppet-openstack
23:11 thumpba joined #puppet-openstack

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