Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2013-04-23

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

All times shown according to UTC.

Time Nick Message
01:53 dprince joined #puppet-openstack
02:15 e1mer joined #puppet-openstack
06:05 aimon I have a bunch of changes for quantum
06:06 aimon I ahve it wokring.. sans network seeding
06:06 aimon module is missing needed packages and config
06:06 aimon and has improper default ovs driver
06:07 aimon I'll be writing providers for network management.. it will use API
06:07 aimon will try to engage module maintainer
06:31 d3u joined #puppet-openstack
06:51 needcoffee joined #puppet-openstack
07:00 dachary joined #puppet-openstack
07:24 dachary1 joined #puppet-openstack
07:28 dachary joined #puppet-openstack
07:32 bodepd_ aimon: what did you use as a starting place?
07:32 bodepd_ aimon: I already put a bunch of effort into the bodepd version
07:33 bodepd_ aimon: even added redhat support. I was planning on putting it on stackforge this week
07:34 dachary1 joined #puppet-openstack
07:35 aimon bodepd_: only hting I have changed in code so far in the ovs driver
07:35 aimon *is
07:35 aimon rest was config additions from outside
07:35 aimon the Hybrid driver is more compatible then what is in tehre atm
07:35 aimon I should look at your
07:35 aimon *yours
07:36 bodepd_ let me know. I also more or less rewrote the vswitch module
07:36 aimon hehe
07:36 aimon [remote "upstream"]
07:36 aimon url = git://github.com/bodepd/puppet-quantum.git
07:36 aimon fetch = +refs/heads/*:refs/remotes/upstream/*
07:36 aimon I guess I am using yours ;)
07:36 bodepd_ yep. perfect.
07:37 bodepd_ getting that on stackforge is one of my top three goals for this week
07:37 aimon I'll rebase and see where we are at :)
07:37 bodepd_ I'm eager to see those pull requests
07:37 aimon have you done any work on providers for network items?
07:37 bodepd_ none at all
07:37 bodepd_ I'm happy to help you do it :)
07:37 aimon cool :)
07:37 taziden left #puppet-openstack
07:37 bodepd_ I would really love to get that stuff in providers though
07:38 aimon let me get PRs in (tomorrow) for the config stuff and missing ubuntu packages
07:38 aimon then we can talk
07:38 bodepd_ cool.
07:38 aimon I am excited atm I ahve it all happy in my envs.. my vns get ips automaticcally fo any project, and they all have internet.. woot!
07:38 aimon the Hybrid driver makes things much easier
07:39 aimon it behaves a bit more like nova-netowrk in the way it auto assigns
07:39 bodepd_ I dont know much about the hybrid driver
07:39 aimon 'DEFAULT/interface_driver':         value =>  'quantum.agent.linux.inter​face.OVSInterfaceDriver';
07:39 aimon I'll send in PR tomorrow with details
07:40 aimon lol wrong line
07:41 aimon ahh its in openstack module
07:41 aimon modules/openstack/manifests/compute.pp:      libvirt_vif_driver  =>  'nova.virt.libvirt.vif.Libv​irtHybridOVSBridgeDriver',
07:42 aimon I am going to have a not so fun time next 2 days soring out all my changes :)
07:42 aimon *sorting
07:43 aimon we also need to add ability for gre tunnel pools
07:44 aimon bodepd_ u have a preferred document collaboration format? So we can write up a spec...
07:44 aimon we usually use gdocs
07:45 bodepd_ maybe a blueprint?
07:45 aimon ahh that would be proper now wouldnt it..
07:46 aimon I ahve an internal doc I am writing which consolidates the netowrk creation stuff.. I'll see if I can get perm to allow you to see it
07:46 aimon (I don't see why not)
07:55 dachary joined #puppet-openstack
08:00 openstackgerrit Dan Bode proposed a change to stackforge/puppet-openstack: fix nova::network::quantum parameters  https://review.openstack.org/27323
08:06 EmilienM bodepd_: do you think "keystone_host" variable in puppet-openstack is confusing ? if we read the README, we should use "controller_host", no ? I was surprise to see "${keystone_host}:9696" for Quantum
08:21 aimon bodepd_ sent u link
08:21 aimon I started doc today.. will add mroe to it over next day or so
08:45 derekh joined #puppet-openstack
08:58 d3u joined #puppet-openstack
09:01 d3u joined #puppet-openstack
10:37 _ilbot joined #puppet-openstack
10:37 Topic for #puppet-openstack is now Place to collaborate on Puppet/OpenStack tools: logs at http://irclog.perlgeek.de/puppet-openstack/today
11:45 openstackgerrit Dan Prince proposed a change to stackforge/puppet-cinder: Add support for configuring qpid.  https://review.openstack.org/27231
11:45 openstackgerrit Dan Prince proposed a change to stackforge/puppet-cinder: Add cinder::rabbitmq class.  https://review.openstack.org/27230
12:54 openstackgerrit François Charlier proposed a change to stackforge/puppet-nova: Fix the nova-consoleauth package name on Debian OS  https://review.openstack.org/27340
13:02 dprince joined #puppet-openstack
13:22 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-openstack: Rename keystone_host by controller_host  https://review.openstack.org/27342
14:04 d3u joined #puppet-openstack
14:18 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-openstack: Rename keystone_host by controller_host  https://review.openstack.org/27342
14:37 dachary joined #puppet-openstack
14:45 EmilienM fc__: sileht : dummy question, but the mailing list for OpenStack puppetlabs modules is openstack-dev ?
14:45 EmilienM or should I use puppet labs ML ?
14:45 sileht EmilienM, nop
14:45 EmilienM sileht: no what :P
14:46 sileht EmilienM, https://groups.google.com/a/puppetlabs.com​/forum/?fromgroups#!forum/puppet-openstack
14:47 EmilienM sileht: kind of you
15:24 gmi joined #puppet-openstack
15:48 openstackgerrit Dan Prince proposed a change to stackforge/puppet-nova: Add nova::db::postgresql class.  https://review.openstack.org/27353
16:04 openstackgerrit Dan Prince proposed a change to stackforge/puppet-cinder: Add cinder::db::postgresql class.  https://review.openstack.org/27355
16:10 openstackgerrit Dan Prince proposed a change to stackforge/puppet-glance: Add glance::db::postgresql class.  https://review.openstack.org/27357
16:12 openstackgerrit Dan Prince proposed a change to stackforge/puppet-cinder: Drop params require from mysql.pp.  https://review.openstack.org/27358
16:17 gmi Can sombeody help a bit with the openstack puppet modules? I installed the puppetlabs-openstack modules, then "cat /etc/puppet/modules/openstack/examples/site.pp >> /etc/puppet/manifests/site.pp" and edited that file, but now when I run "puppet agent --test --verbose --noop" on the node that should be a controller, I get "Error: Failed to apply catalog: Parameter name failed on Database_user[@]: Invalid database user @"
16:18 gmi I can provide the full site.pp if needed
16:39 gmi nobody can help?
16:41 openstackgerrit Dan Prince proposed a change to stackforge/puppet-keystone: Add tests on keystone:db::postgresql.  https://review.openstack.org/27362
16:58 openstackgerrit Dan Prince proposed a change to stackforge/puppet-cinder: Add cinder::db::postgresql class.  https://review.openstack.org/27355
16:58 openstackgerrit Dan Prince proposed a change to stackforge/puppet-nova: Add nova::db::postgresql class.  https://review.openstack.org/27353
16:58 openstackgerrit Dan Prince proposed a change to stackforge/puppet-glance: Add glance::db::postgresql class.  https://review.openstack.org/27357
17:33 dprince joined #puppet-openstack
18:29 dachary joined #puppet-openstack
18:29 odyi joined #puppet-openstack
18:50 openstackgerrit Dan Prince proposed a change to stackforge/puppet-nova: Drop dependency on apt module.  https://review.openstack.org/27371
18:52 bodepd_ EmilienM: I saw the patch. I have a bit of backlog atm, I'll get to it when I can
19:20 mgagne Is there a known issue with gerrit? Approved reviews seem to pile up
19:39 openstackgerrit A change was merged to stackforge/puppet-cinder: Add cinder::rabbitmq class.  https://review.openstack.org/27230
19:39 openstackgerrit A change was merged to stackforge/puppet-cinder: Add support for configuring qpid.  https://review.openstack.org/27231
19:45 d3u joined #puppet-openstack
19:47 openstackgerrit A change was merged to stackforge/puppet-cinder: Drop params require from mysql.pp.  https://review.openstack.org/27358
20:13 openstackgerrit Aimon Bustardo proposed a change to stackforge/puppet-cinder: Add Nexenta Support  https://review.openstack.org/26817
20:13 openstackgerrit Aimon Bustardo proposed a change to stackforge/puppet-openstack: add rabbit_virtual_host passthrough to cinder  https://review.openstack.org/26934
20:14 openstackgerrit Aimon Bustardo proposed a change to stackforge/puppet-openstack: Add vncproxy_host option  https://review.openstack.org/26947
20:21 openstackgerrit Dan Bode proposed a change to stackforge/puppet-openstack_dev_env: fix gui_mode  https://review.openstack.org/27376
20:23 openstackgerrit A change was merged to stackforge/puppet-openstack_dev_env: fix gui_mode  https://review.openstack.org/27376
20:23 openstackgerrit A change was merged to stackforge/puppet-nova: Fix the nova-consoleauth package name on Debian OS  https://review.openstack.org/27340
20:24 openstackgerrit A change was merged to stackforge/puppet-openstack: fix nova::network::quantum parameters  https://review.openstack.org/27323
20:26 openstackgerrit A change was merged to stackforge/puppet-glance: Add glance::db::postgresql class.  https://review.openstack.org/27357
20:26 openstackgerrit A change was merged to stackforge/puppet-nova: Add nova::db::postgresql class.  https://review.openstack.org/27353
20:26 openstackgerrit A change was merged to stackforge/puppet-keystone: Add tests on keystone:db::postgresql.  https://review.openstack.org/27362
20:32 openstackgerrit Aimon Bustardo proposed a change to stackforge/puppet-openstack: add rabbit_virtual_host passthrough to cinder  https://review.openstack.org/26934
20:32 openstackgerrit Aimon Bustardo proposed a change to stackforge/puppet-openstack: Add vncproxy_host option  https://review.openstack.org/26947
20:33 aimon bodepd_ can I get your to approve this one today: https://review.openstack.org/#/c/26817/
20:42 openstackgerrit A change was merged to stackforge/puppet-openstack: Add vncproxy_host option  https://review.openstack.org/26947
20:45 mgagne I would need input regarding https://review.openstack.org/#/c/27117/
20:45 mgagne I proposed using environment variable to pass around the password. Is it the right way?
20:56 aimon mgagne why not use -- flags ?
20:56 aimon ah env does save tyoing....
20:56 aimon *typing
20:56 mgagne that's not the point
20:57 mgagne gonna show up in processlist. there was an issue opened in github regarding security concerns.
20:57 aimon ahhh
20:57 mgagne env variables aren't the definitive answer I would say as you can still read env variables of other processes.
20:58 aimon yeah, I was just thinking similar.. there really is no way ti hide it completely from a smart individual
20:59 aimon but taking out of ps is in the right direction
20:59 mgagne can't we cat the password from a temporary file instead?
20:59 aimon the best way IMHO is to use the rest API and never show it period
20:59 mgagne file which would only be readable by root
20:59 mgagne hmmm
21:00 mgagne it's true
21:00 aimon hmm.. how to feed it to command
21:00 aimon ^ file I mean
21:00 aimon if was postgres.. easy.. nova.. not so sure ;)
21:00 mgagne $(cat /root/the_secret)
21:00 aimon lol I was afraid you would say that
21:00 mgagne :P
21:00 aimon totally feasible
21:00 mgagne better use the API then... =)
21:00 aimon and it *is* secure
21:01 mgagne long term objective
21:01 aimon API will require some work
21:01 aimon yeah :)
21:01 bodepd_ man. you guys are busy!
21:01 aimon nuice thing is as soon as one of us does some API work the rest can share the basics
21:01 bodepd_ I'm going to try to run some manual integration tests
21:01 aimon and then it will be a copy and paste job roughly
21:01 mgagne hehe, always the first step that's difficult
21:01 aimon yep
21:01 aimon I'll be taking that step today or tomorrow
21:02 aimon will keep ya'll posted
21:02 mgagne sure
21:03 aimon hola bodepd_
21:03 bodepd_ aimon: I thought that patch was havana specific?
21:03 aimon which patch?
21:03 aimon ahh the nexenta one?
21:03 aimon not at all
21:03 aimon we use it now in Grizz
21:04 aimon and it should work for H also
21:04 aimon We got the cinder driver change merged also to cinder project
21:05 aimon its released now
21:05 aimon G wont be really stable for another few weeks...
21:06 aimon I have 4 manually applied patches to keep it runnig atm
21:06 bodepd_ aimon: I had one commet about ruby style
21:06 aimon waiting on Ubuntu to package them :/
21:06 aimon ok checking
21:06 bodepd_ what versions are your patches  accepted in?
21:06 aimon one patch.. grizz
21:06 aimon rest or 3rd part patches
21:07 bodepd_ oh, the patches were backported? can you send me a link?
21:07 aimon *party.. all to fix critical bugs
21:07 aimon links.. hold
21:07 bodepd_ but, the basic support for a nexenta cinder driver exists in cinder?
21:07 aimon its broken.. completely for G
21:07 aimon but does exist
21:08 aimon patch was easy enough...
21:09 aimon I'll dig them all out
21:09 aimon # https://bugs.launchpad.net/keystone/+bug/1170186
21:09 aimon # https://bugs.launchpad.net/oslo/+bug/1158958
21:09 bodepd_ no worries.
21:10 aimon https://bugs.launchpad.net/cinder/+bug/1166607
21:11 aimon ^ that was my patch..
21:11 bodepd_ I checked the cinder code and verified that in the grizzly release has the nexenta volume driver
21:11 aimon yep
21:11 bodepd_ that supports all of the parameters you listed in the class
21:11 bodepd_ that is really what I cared about
21:11 bodepd_ sorry if that was not clea
21:11 bodepd_ r
21:11 aimon understood
21:12 bodepd_ fix that formatting issue (you can actually omit the braces)
21:12 bodepd_ and we are good to merge
21:12 bodepd_ aimon: one more thing
21:13 bodepd_ aimon: do you mind putting your details in the file so that people can know you are the 'nexenta' expert
21:13 aimon https://bugs.launchpad.net/pyt​hon-quantumclient/+bug/1165962
21:13 aimon Sure np
21:14 aimon I htik that was the last one we apply atm :)
21:14 aimon that one is silly… have to install python-cliff from source
21:14 aimon or python-quantumclient craps out
21:15 aimon ahh yes, our ruby devs keep telling me to use do instead of braces
21:16 bodepd_ you can't for hashes
21:16 bodepd_ but you are using: do {
21:16 bodepd_ which is bad ruby style
21:16 bodepd_ should be:
21:16 bodepd_ do
21:16 bodepd_ {
21:16 aimon oh..
21:18 aimon hehe thats what I get for using the netapp one as a template ;)
21:18 mgagne oh, I forgot I had to fix those netapp tests :P
21:19 aimon bodepd_ so like:
21:19 aimon let foo do
21:19 aimon {
21:19 aimon blah
21:19 aimon }
21:19 aimon done
21:19 aimon or shoudl the braces be at same level as let ?
21:19 aimon or u have example? lol
21:19 aimon I am looking in code and seeing a few different ways
21:20 mgagne let :params do
21:20 mgagne { :test  => 'test',
21:20 mgagne :test2 => 'test
21:20 mgagne }
21:20 mgagne end
21:20 mgagne or
21:20 mgagne let :params do
21:20 mgagne {
21:20 mgagne :test  => 'test',
21:20 mgagne :test2 => 'test
21:20 mgagne }
21:20 mgagne end
21:20 aimon :test is indented twice?
21:20 mgagne that's my style =)
21:21 aimon looks clean in vim.. I like
21:22 mgagne I saw some people putting the { and } on the same line as do and end
21:22 openstackgerrit Aimon Bustardo proposed a change to stackforge/puppet-cinder: Add Nexenta Support  https://review.openstack.org/26817
21:22 openstackgerrit Aimon Bustardo proposed a change to stackforge/puppet-cinder: Add Nexenta Support  https://review.openstack.org/26817
21:23 mgagne oh, netapp test :P
21:23 aimon yeah :)
21:23 aimon bodepd_ ^ cleaned
21:24 aimon forgot auth info
21:24 aimon adding
21:27 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-cinder: Fix cinder::volume::netapp rspec tests  https://review.openstack.org/27381
21:28 openstackgerrit Aimon Bustardo proposed a change to stackforge/puppet-cinder: Add Nexenta Support  https://review.openstack.org/26817
21:29 bodepd_ sorry I keep rejecting your code.
21:29 bodepd_ the test was not correct
21:29 aimon haha I know you just hate me :p
21:30 aimon no worries! I am learning and won't learn more unless I am corrected
21:30 mgagne is there an inline comment bodepd_?
21:30 aimon escpecially around spec/rake .. I am very rough
21:32 aimon bodepd_ think you missede the comment…
21:38 dachary joined #puppet-openstack
21:47 mgagne aimon: You can take a look at my fix for netapp tests to see what I tried to accomplish with default_params. I don't think my example in the best form thought.
21:52 mgagne joined #puppet-openstack
21:55 aimon k will do
22:17 openstackgerrit Dan Prince proposed a change to stackforge/puppet-cinder: Add cinder::db::postgresql class.  https://review.openstack.org/27355
22:44 openstackgerrit A change was merged to stackforge/puppet-cinder: Add cinder::db::postgresql class.  https://review.openstack.org/27355
22:55 dachary joined #puppet-openstack
23:02 e1mer joined #puppet-openstack
23:02 e1mer joined #puppet-openstack
23:04 mgagne I'm working on linuxbridge support in quantum: https://github.com/mgagne/puppet-quantum/com​mit/a9e86ef37bb2b8f0879b0714454b8ba3109ca39c
23:05 mgagne Should I force people to provide the network_vlan_ranges and physical_interface_mappings? Or should I let OpenStack provides its default values?
23:20 aimon I have been setting defaults in puppet to match defaults in OpenStack except where specific need/reason not to
23:21 mgagne aimon: Can we expect the defaults to not change over time?
23:21 aimon any non default default should probably contain a comment explenation of the change
23:21 aimon so far they have not much..
23:21 aimon docs are near identicle in most cases between essx and grizzly
23:22 aimon changes there were around inclusion of cinder and quantum
23:22 aimon for H i doubt much change at all
23:23 aimon fortunately all doc changes require tickets .. so we shoyuld be able to track defaults fairly easily
23:24 aimon just my 2c :)
23:27 aimon here is linked ticket to the python client one: https://bugs.launchpad.net/ubuntu/s-series​/+source/python-quantumclient/+bug/1170849
23:28 aimon looks like as long as you stay on stable repo this wont effect you
23:29 mgagne my concerns were about default values and what we should provide
23:30 mgagne I see what puppet-quantum did for ovs but I'm not convinced it's a good example
23:37 mgagne aimon: https://bugs.launchpad.net/quantum/+bug/1121855
23:37 mgagne aimon: Is NoopDriver the default firewall_driver due to the plugin architecture? Should puppet therefore provide the "right" driver from the start?
23:37 dachary joined #puppet-openstack
23:51 aimon I would like to see a   securitygroups boolean which enables the addition of the additional config
23:51 aimon .. if its alwaysw a single arg for driver.. then the absence could be the False…
23:52 aimon in either case we will have to chose a reasonabgle default for the users
23:52 aimon I have been leaning towards Folsom behavior compatability
23:53 aimon which is why  use the Hybrid ovs driver
23:55 mgagne Unfortunately, I don't have the opportunity to test with Folsom. So I don't know if providing SECURITYGROUP/firewall_driver with linuxbridge in Folsom will work or not.
23:56 mgagne IMO, quantum without security group support is kind of useless so I would prefer to ship a working configuration from the start.
23:57 aimon +1 basic feature parity with Folsom

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