Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2014-11-26

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

All times shown according to UTC.

Time Nick Message
00:19 aimon joined #puppet-openstack
00:26 sputnik13 joined #puppet-openstack
00:49 vinsh joined #puppet-openstack
00:53 ericpeterson joined #puppet-openstack
00:58 ericpeterson joined #puppet-openstack
01:17 rmoe joined #puppet-openstack
01:22 gildub joined #puppet-openstack
01:22 mmaglana joined #puppet-openstack
01:22 gildub joined #puppet-openstack
01:35 topshare joined #puppet-openstack
01:56 mmaglana joined #puppet-openstack
01:59 xingchao joined #puppet-openstack
02:02 vinsh joined #puppet-openstack
02:15 snk joined #puppet-openstack
02:47 ilbot3 joined #puppet-openstack
02:47 Topic for #puppet-openstack is now Place to collaborate on Puppet/OpenStack tools: logs at http://irclog.perlgeek.de/puppet-openstack/today
02:51 vinsh joined #puppet-openstack
02:52 sputnik13 joined #puppet-openstack
02:56 xarses joined #puppet-openstack
03:03 mmaglana joined #puppet-openstack
03:19 EmilienM mgagne: yes afik
03:24 EmilienM mgagne: https://github.com/openstack/nova/tree/master/nova/db/sqlalchemy/migrate_repo/versions
03:37 gildub joined #puppet-openstack
04:28 vinsh joined #puppet-openstack
05:15 fedexo joined #puppet-openstack
06:00 sanjayu joined #puppet-openstack
07:45 openstackgerrit Merged stackforge/puppet-cinder: Remove deprecated parameters  https://review.openstack.org/137170
07:47 sputnik13 joined #puppet-openstack
07:56 openstackgerrit Merged stackforge/puppet-nova: Remove deprecated parameters  https://review.openstack.org/137186
07:56 openstackgerrit Merged stackforge/puppet-swift: Remove deprecated class  https://review.openstack.org/137187
07:57 openstackgerrit Merged stackforge/puppet-horizon: Remove deprecated parameters  https://review.openstack.org/137174
07:59 openstackgerrit Merged stackforge/puppet-keystone: Remove deprecated parameters  https://review.openstack.org/137178
08:02 openstackgerrit Merged stackforge/puppet-glance: Remove deprecated parameters  https://review.openstack.org/137172
08:07 openstackgerrit Merged stackforge/puppet-neutron: Remove deprecated parameters  https://review.openstack.org/137180
08:11 michchap joined #puppet-openstack
08:17 henditan joined #puppet-openstack
08:24 dgurtner joined #puppet-openstack
08:24 stamak joined #puppet-openstack
08:26 mmagr joined #puppet-openstack
08:38 tfz joined #puppet-openstack
08:41 openstackgerrit Bogdan Dobrelya proposed stackforge/puppet-neutron: Add 'state_path' and 'lock_path' for Neutron::Server  https://review.openstack.org/136333
08:57 zz_avozza joined #puppet-openstack
08:57 berendt joined #puppet-openstack
08:58 topshare joined #puppet-openstack
08:59 tfz joined #puppet-openstack
09:04 arnaud_orange joined #puppet-openstack
09:12 derekh joined #puppet-openstack
09:37 openstackgerrit Michal Skalski proposed stackforge/puppet-swift: Enlarging the scope of ssh module  https://review.openstack.org/113550
10:50 ddmitriev joined #puppet-openstack
11:22 qba73 joined #puppet-openstack
11:29 mmaglana joined #puppet-openstack
11:41 avozza left #puppet-openstack
13:01 morazi joined #puppet-openstack
13:08 openstackgerrit Michal Skalski proposed stackforge/puppet-swift: Enlarging the scope of ssh module  https://review.openstack.org/113550
13:11 EmilienM good morning!
13:16 topshare joined #puppet-openstack
13:20 mmaglana joined #puppet-openstack
13:23 openstackgerrit Merged stackforge/puppet-ceilometer: Add support for configuring coordination/backend_url  https://review.openstack.org/134356
13:27 francois joined #puppet-openstack
13:28 openstackgerrit Emilien Macchi proposed stackforge/puppet-nova: Configure database parameters on the right nodes  https://review.openstack.org/135840
13:37 ericpeterson joined #puppet-openstack
13:39 michchap EmilienM: morning
13:39 michchap spredzy: ping
13:39 spredzy michchap, pong
13:40 michchap spredzy: where do you think firewall definitions should go? in a firewall profile or connected to the individual profiles? or lower level? I am leaning towards each profile providing a daemon having the firewall defaults in there but it's awkward.
13:41 michchap spredzy: basically should firewall defs be centralised or not.
13:41 spredzy michchap, connected to the indeividual profile. I just feel it makes it clearer
13:43 michchap spredzy: cheers. I'm thinking a hash + maybe 1 param for setting source, since that's likely to be different per site.
13:43 michchap spredzy: but that's going to end up in almost every class.
13:44 spredzy michchap, EmilienM did something similar in puppet-openstack-cloud if you want to take a look at it. Each component with a hash of option to create a firewall ressource
13:46 michchap spredzy: EmilienM I will take a look. That's pretty much what I had in mind, but I don't want the user to have to override the entire hash just to set source.
13:48 spredzy michchap, eventually make a special param for source and merge this param into an hash with default value. This how does that look like
13:48 michchap spredzy: will do exactly that.
13:51 patcable any thoughts on https://review.openstack.org/137227? I want to get feedback on this before i push the change out to the other modules as well
13:51 spredzy michchap, :)
13:51 patcable though i put my reason in a comment rather than the commit because i did a git commit -m per habit... hopefully thats ok
13:53 michchap patcable: I do that sometimes too :) I think you should do a --amend and put it in the commit message. Do you know what specific problem occurs if both are defined? Surely one is ignored. In any case I agree with your patch, just fix the commit message.
13:54 patcable michchap: it seems like rabbit_host has precedent if both are defined -- this way if someone goes from using rabbit_host to rabbit_hosts, the old value gets tossed
13:54 patcable michchap: since there's already an if statement there
13:55 patcable michchap: would I just do a git commit --amend and then git push gerrit?
13:55 michchap patcable: git commit --amend and then 'git review'
13:55 patcable ah ok
13:55 michchap patcable: that will automatically push to the right place
13:55 patcable wasnt sure if that was going to create a new review
13:55 patcable neat
13:56 openstackgerrit Patrick Cable proposed stackforge/puppet-cinder: fixed behavior for rabbit_hosts variable  https://review.openstack.org/137227
13:58 michchap spredzy: I like this. it's neat. : https://github.com/enovance/puppet-openstack-cloud/blob/master/manifests/firewall/rule.pp
13:58 EmilienM michchap: https://github.com/enovance/puppet-openstack-cloud/blob/master/manifests/image/api.pp#L231-L236
13:58 EmilienM michchap: cool, thx
13:58 michchap EmilienM: yeah I was *just* looking at it
13:58 EmilienM lol
13:59 michchap EmilienM: spredzy so...does a define like that belong in openstack_extras or in the composition layer?
13:59 EmilienM michchap: to me, it's a good candidate for extras
13:59 EmilienM (and now you may wonder why I did not push in extra)
14:00 michchap EmilienM: I agree. I actually think we should add firewall support to the modules themselves
14:00 michchap EmilienM: because you need to get work done, haha.
14:00 EmilienM michchap: I asked and mgagne did not like the idea
14:00 EmilienM if I remember correctly
14:01 michchap EmilienM: oh? Because it's not a part of openstack? I feel the whole point of extras is to cover that sort of thing, where we need something configured correctly but it's not an openstack project or doesn't belong in another module.
14:02 michchap EmilienM: I would even say, why not add it to lib and make nova::api::firewall?
14:03 EmilienM michchap: that would make sense
14:03 michchap EmilienM: out of the box, no traffic is allowed by default on redhat platforms, so it's a bit strange to leave firewall definitions out when most of the users of modules are going to hit this problem.
14:07 patcable michchap: all fixed
14:09 michchap EmilienM: which do you think? extras or in each module+lib ?
14:09 michchap spredzy: ^
14:10 EmilienM michchap: I would say, module+lib
14:10 EmilienM michchap: lib with the code I did in cloud::firewall::rule
14:10 EmilienM to allow Hiera control everything
14:10 michchap EmilienM: spredzy if we look at puppetlabs modules like apache and mysql, they don't have firewall definitions. So we'd be breaking from the standard.
14:10 EmilienM michchap: OpenStack is breaking standards :)
14:11 michchap EmilienM: actually there was another comment in a review from Spencer and mgagne...on whether we assume data bindings are in use.
14:12 michchap EmilienM: What do you think?
14:12 EmilienM michchap: I agree
14:15 spredzy michchap, I understand its not the PuppetLabs (and community way) of doing it. But having the firewall rule closer to the port it actually firewalled make it logically easier to follow IMO
14:17 tchaypo joined #puppet-openstack
14:25 michchap EmilienM: sent mail to list about data bindings
14:25 EmilienM ok cool
14:26 michchap EmilienM: if you don't make a spec about firewall rules I will make one tomorrow. Going to sleep now.
14:26 Guest81366 joined #puppet-openstack
14:26 topshare joined #puppet-openstack
14:28 snk joined #puppet-openstack
14:29 ericpeterson joined #puppet-openstack
14:29 EmilienM michchap: I won't make it, I have no time this week, sorry
14:29 EmilienM michchap: good night
14:38 ericpeterson joined #puppet-openstack
14:42 mikeit joined #puppet-openstack
14:42 ericpeterson joined #puppet-openstack
14:46 rharrison left #puppet-openstack
14:47 mikeit joined #puppet-openstack
14:56 britthouser joined #puppet-openstack
15:01 xingchao joined #puppet-openstack
15:02 ajayaa joined #puppet-openstack
15:04 ajayaa Hi. guys. I am trying to set up a nova controller node through stakforge/puppet-nova.
15:05 ajayaa The issue I am facing is circular dependency.
15:05 ajayaa Can someone please look into my manifest file?
15:06 EmilienM ajayaa: if you don't paste it that will be hard
15:09 ajayaa EmilienM https://gist.github.com/ajayaa/33b235c751df83ba86b4
15:09 mmaglana joined #puppet-openstack
15:12 ajayaa EmilienM, If I remove line number 31 it says nova::db::mysql depends on it. And after I add it I get a circular dependency problem.
15:12 EmilienM ajayaa: everything looks fine to me
15:13 ajayaa http://paste.openstack.org/show/138861/ This is the error. EmilienM
15:17 snk joined #puppet-openstack
15:17 snk joined #puppet-openstack
15:43 ericpeterson joined #puppet-openstack
15:52 JayJ joined #puppet-openstack
15:54 vinsh joined #puppet-openstack
15:55 aarefiev joined #puppet-openstack
16:00 nibalizer michchap: thanks for sending that email
16:07 mmagr joined #puppet-openstack
16:10 mmaglana joined #puppet-openstack
16:11 bogdando joined #puppet-openstack
16:13 JayJ joined #puppet-openstack
16:18 JayJ joined #puppet-openstack
16:22 mmaglana joined #puppet-openstack
16:23 JayJ joined #puppet-openstack
16:30 EmilienM nibalizer: any pointer of the patch?
16:40 ajayaa EmilienM, When I move line declaring mysql to the very top, it works surprisingly. I am novice in puppet. Can you think if a reason?
16:41 nibalizer EmilienM: which patch?
16:41 nibalizer link?
16:51 crinkle I am not a big fan of sticking firewall things in lib...it sort of breaks the modularity of the modules in my mind
16:51 crinkle modules should do one thing well, firewall rules are kind of beyond a module's responsibility
16:51 crinkle i would be in favor of sticking firewall rules in each profile
16:51 crinkle my 2c
17:02 mmaglana joined #puppet-openstack
17:05 openstackgerrit Colleen Murphy proposed stackforge/puppet-keystone: Use openstackclient for keystone_tenant  https://review.openstack.org/134844
17:05 openstackgerrit François Charlier proposed stackforge/puppet-keystone: Allow adding a delay before flushing tokens  https://review.openstack.org/137406
17:12 aimon joined #puppet-openstack
17:14 dgurtner joined #puppet-openstack
17:30 openstackgerrit François Charlier proposed stackforge/puppet-keystone: Allow adding a delay before flushing tokens  https://review.openstack.org/137406
17:37 EmilienM nibalizer: the one michchap is mentioning in the email
17:42 nibalizer this i think
17:42 nibalizer https://review.openstack.org/#/c/126458/
17:42 EmilienM nibalizer: thx
17:43 EmilienM nibalizer: mine?
17:52 nibalizer ya
18:00 rmoe joined #puppet-openstack
18:06 stamak joined #puppet-openstack
18:18 openstackgerrit Emilien Macchi proposed stackforge/puppet-nova: Configure database parameters on the right nodes  https://review.openstack.org/135840
18:31 stamak joined #puppet-openstack
18:35 pradk joined #puppet-openstack
18:42 ajayaa joined #puppet-openstack
18:47 ericpeterson joined #puppet-openstack
18:59 patcable this is interesting - Error: Could not retrieve catalog from remote server: Error 400 on SERVER: undefined method `split' for #<Array:0x007f47edc60390> at /etc/puppet/environments/blah/modules/openstack/manifests/resources/controller.pp:7 on node
18:59 patcable has_ip_address($api_address) is the function
19:00 patcable and its just pulling from $::openstack::config::controller_address_api which is definitely a string
19:05 stamak joined #puppet-openstack
19:11 xarses joined #puppet-openstack
19:13 EmilienM patcable: do you have stdlib?
19:13 patcable i do
19:15 patcable odd, must not have been included somehow. i'm trying out some different stuff with the composition layer, so one of the other modules i called must have brought it in
19:19 tfz joined #puppet-openstack
19:28 bitblt joined #puppet-openstack
19:29 bitblt left #puppet-openstack
19:39 patcable eh, ran into it again. I had removed a role and didnt realize it
19:40 patcable EmilienM: i've tried explicitly including stdlib on my ENC for fun and still running into weird split issue
19:54 andi-_ joined #puppet-openstack
19:56 ToMiles joined #puppet-openstack
20:14 rcallawa joined #puppet-openstack
20:26 openstackgerrit Emilien Macchi proposed stackforge/puppet-nova: Configure database parameters on the right nodes  https://review.openstack.org/135840
20:27 EmilienM mgagne: ^ thx
20:28 mgagne EmilienM: could you use pick() to reduce the number of line of code? https://gist.github.com/mgagne/6ed00808815b0af0fe0e
21:01 EmilienM mgagne: of course I can
21:36 JayJ joined #puppet-openstack
21:38 imcsk8 hello i have a problem, i'm adding the allowed_start and allowed_end parameters to the network create part of the nova puppet module but i'm getting an execution error when i test the manifest, the weird part is that if i execute the command directly it works, here's the output: http://pastebin.com/SkUxMpwB
21:41 mgagne imcsk8: are you copy/pasting the command?
21:41 mgagne nova help network-create shows that the network name should be placed at the end
21:42 imcsk8 mgagne: yes
21:42 mgagne my python-novaclient version is 2.17.0
21:45 openstackgerrit Emilien Macchi proposed stackforge/puppet-nova: Configure database parameters on the right nodes  https://review.openstack.org/135840
21:46 imcsk8 mgagne: i'm testing with the network name at the end
21:47 f10d4 imcsk8: are you trey?
21:47 imcsk8 f10d4: i'm ivan chavero
21:48 f10d4 then, you are not trey (gmicsko)
21:48 f10d4 you ve just a similar sounding nickname :)
21:48 imcsk8 f10d4: yeah hehehe
21:48 EmilienM imcsk8: using nova network?
21:48 f10d4 in that case, no rottendotkom urls will be posted to ya ;)
21:50 imcsk8 mgagne: i found the problem, in nova/lib/puppet/provider/nova_network/nova_manage.rb  had ' --fixed-range-v4 ' , changed it to '--fixed-range-v4' and it worked. pretty odd since the spaces don't give a problem in the command line
21:50 mgagne imcsk8: try running the same with single-quotes
21:50 imcsk8 EmilienM: yeah it's a RFE
21:50 EmilienM imcsk8: ok
21:51 imcsk8 EmilienM: nova network now can set ip ranges for the clients with allow_start and allow_end, so they asked for it in the module
21:52 EmilienM yeah it does
21:52 EmilienM never used it though
21:52 imcsk8 EmilienM: the module does that? i couldn't find those options
21:53 EmilienM mgagne: ready for upgrade ? or tomorrow ? lol
21:55 mgagne EmilienM: sorry? what upgrade? :)
21:55 EmilienM mgagne: I saw your tweet
21:55 mgagne EmilienM: oh lol
21:55 mgagne EmilienM: have been delaying the updates for months
21:56 mgagne EmilienM: nobody ain't got time for that
21:58 imcsk8 EmilienM, mgagne BTW i'm not sure if this is correct, the nova puppet module uses the nova-manage command for creating the newtwork and i had to change it to nova because nova-manage does not support the --allow_start and --allow_end switches: http://pastebin.com/unKA2Eb9
21:58 mgagne imcsk8: I think nova_network is one of the oldest feature in puppet-nova and nova-manage used to be THE tool to manage network
21:59 mgagne imcsk8: from now on, I think everything is moved to API and no more feature is added to nova-manage
21:59 ericpeterson joined #puppet-openstack
22:00 imcsk8 mgagne: so my change makes sense then...
22:01 mgagne imcsk8: about using nova? yes
22:02 imcsk8 mgagne: thanks i was a little worried about taking this path
22:02 mgagne imcsk8: the inconvenient of the nova command is that you need api credentials while nova-manage only needs nova.conf to be present and database credentials are read and used from it
22:03 imcsk8 mgagne: yes, i've noticed that. could this pose a problem for the module users?
22:04 mgagne imcsk8: using the nova command or anything interacting with the API from puppet is a huge thing we are still working on
22:05 mgagne imcsk8: not that it's controversial but there is huge work/refactor around that area atm
22:05 imcsk8 mgagne: i get it, i noticed that it uses nova_manage for everything...
22:06 mgagne imcsk8: yes, you are going down a rabbit hole, this part of our module hasn't been updated for months =)
22:07 imcsk8 mgagne: i'll submit my patch for you guys to check it out, i'm more than glad to help
22:07 mgagne imcsk8: thanks!
22:08 imcsk8 mgagne: and since i've had to read and compare both commands i could help refactoring the module :)
22:08 mgagne =)
22:15 * social got all puzzled from the neutron configuration
22:15 social I can have several neutron plugins? eg openvswitch and ml2?
22:21 EmilienM social: you have ml2 with OVS
22:21 mgagne social: grep core_plugin /etc/neutron/neutron.conf
22:23 social well ok lets say I have ovs plugin and I set ovs agent
22:23 social I have ovs plugin with vxlan
22:23 social I set ovs agent with tunnel_types vxlan and gre
22:23 social what would be the usecase there?
22:24 social I mean ovs allows only one tenant network type
22:47 EmilienM joined #puppet-openstack
23:24 gildub joined #puppet-openstack

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