Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2013-08-06

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

All times shown according to UTC.

Time Nick Message
00:32 openstackgerrit A change was merged to stackforge/puppet-heat: Replace deprecated :rubygems source  https://review.openstack.org/39948
00:43 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-openstack: Add ability to configure endpoint_type in openrc  https://review.openstack.org/40318
01:09 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-openstack: Add ability to configure endpoint_type in openrc  https://review.openstack.org/40318
01:38 xingchao joined #puppet-openstack
02:06 prad joined #puppet-openstack
06:21 openstackgerrit Daneyon Hansen proposed a change to stackforge/puppet-swift: Fix Several Issues Related to the Proxy Signing Directory  https://review.openstack.org/40356
06:22 openstackgerrit A change was merged to stackforge/puppet-openstack: Disable dhcp on the provisioned public subnet.  https://review.openstack.org/40313
07:49 fvollero EmilienM: Bonjour! You've some suggestion for https://review.openstack.org/#/c/40188/ ?
07:50 EmilienM fvollero: hey
07:50 * EmilienM reading
07:50 * fvollero wait drinking coffee
07:52 EmilienM fvollero: actually, rabbit is needed only for heat-engine
07:52 EmilienM other services are just API, connected to RabbitMQ
07:52 EmilienM afik
07:53 EmilienM but indeed, the module needs to be clean-up...
07:57 fvollero EmilienM: well, the api are also needed to be configured :)
07:58 fvollero EmilienM: the point it that mgagne is surely sleeping now, so I was wondering if you've any suggestions on how to approach the problem.
08:03 EmilienM fvollero: I'm quite sure the API does not need AMQP configuration
08:03 EmilienM fvollero: let me think about the approach
08:04 fvollero EmilienM: in the mean time i'll ask the heat guys and get you back on this :)
08:04 EmilienM fvollero: yeah
08:04 EmilienM they are quite responsive
08:05 EmilienM and very cool
08:05 EmilienM fvollero: but it seems the puppet-heat module needs a big cleanup :)
08:06 fvollero EmilienM: Yeah, I totally agree on that. From ruby to puppet, you can't see at the beginning how things have to be... Now I started to have a clear picture :)
08:07 fvollero EmilienM: after the cleanup, the lint fixing!
08:13 EmilienM fvollero: oops it seems I was wrong about RPC :/
08:14 fvollero Lol... it's common... luckily there wasn't shardy or he would yelled at me :)
08:15 EmilienM fvollero: "pas de problème" by the way
08:15 EmilienM ;P
08:15 fvollero EmilienM: Merci :) I'm really rusty on french :)
08:16 fvollero EmilienM: you should have to force me in pvt to write in french :)
08:23 derekh joined #puppet-openstack
09:29 red_trela joined #puppet-openstack
10:09 openstackgerrit A change was merged to stackforge/puppet-openstack: Allow ovs_enable_tunneling to be Passed Through  https://review.openstack.org/40146
12:20 markvoelker joined #puppet-openstack
12:47 fvollero \o/ my router exploded after 4 years of duty!!
12:48 dprince joined #puppet-openstack
12:53 xingchao_ joined #puppet-openstack
13:08 dmsimard joined #puppet-openstack
13:22 dmsimard joined #puppet-openstack
13:57 morazi joined #puppet-openstack
14:50 otherwiseguy joined #puppet-openstack
15:40 EmilienM mgagne: I'm working on https://review.openstack.org/#/c/39980/
15:40 EmilienM mgagne: I let rabbit stuff only on init.pp and write the tests
15:41 mgagne EmilienM: sounds much better
15:45 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-heat: Update the module for new config file method  https://review.openstack.org/39980
15:47 EmilienM mgagne: there is a lot of tests to write. I could do them, but maybe I'll need some help for reviews. is it ok for you ?
15:47 mgagne EmilienM: np for me
15:48 EmilienM mgagne: if you don't mind, could you review https://review.openstack.org/#/c/39836/ ?
15:49 EmilienM which is also in grizzly branch
15:49 mgagne EmlienM: quantum got branched right?
15:49 EmilienM https://review.openstack.org/#/c/40263/
15:50 mgagne ok, hold on, finishing some stuff
15:50 EmilienM mgagne: right
15:52 EmilienM bodepd: since we have now master for neutron, could you tell me if https://review.openstack.org/#/c/40062/ is good now ?
15:57 EmilienM mgagne: for https://review.openstack.org/#/c/40122/ why do you want to move it into nova::network, since it's not mandatory to install nova-network on compute nodes. IMO that should stay in compute nodes.
15:57 EmilienM or maybe I'm wrong ?
15:57 mgagne EmilienM: commit message says that it's only required with nova-network.
15:59 openstackgerrit Daneyon Hansen proposed a change to stackforge/puppet-swift: Fix Several Issues Related to the Proxy Signing Directory  https://review.openstack.org/40356
15:59 mgagne EmlienM: but is it true? I use linuxbridge with quantum and I need bridge-utils installed too but the quantum-plugin-linuxbridge-agent package installs it for me.
16:02 mgagne EmilienM: I don't feel adding this kind of parameter is a good idea. nova::compute shouldn't be aware of the use of nova-network or quantum
16:03 EmilienM mgagne: how do you detest on compute nodes that you run quantum or nova-network then ?
16:03 EmilienM detect**
16:05 mgagne EmilienM: you don't. someone running nova-network will include nova::network or one of the subclass
16:05 EmilienM mgagne: ok I see. So maybe should I move the package dependency on nova::network and add a new one for the linuxbridge plugin ? (i've never used it)
16:06 mgagne You don't need anything for linuxbridge, package dependencies are well defined
16:07 gmi joined #puppet-openstack
16:07 EmilienM ok
16:08 EmilienM so I move it to nova::network
16:08 mgagne EmilienM: well, somewhere around there =)
16:11 EmilienM mgagne: \o/ manifests/network.pp +79
16:11 mgagne haha nice
16:18 EmilienM mgagne: in fact I disagree, sorry. Let me explain again.
16:18 EmilienM mgagne: you have 2 cases in nova-network: multi host and non multi host
16:18 EmilienM mgagne: for both, computes need bridge utils
16:19 EmilienM mgagne: so If I move the dependency into nova::network, compute nodes won't have the package installed
16:19 EmilienM mgagne: since you don't need nova::network when you setup a compute in non-multi node mode
16:19 EmilienM multi-host*
16:20 EmilienM mgagne: make sense ?
16:31 mgagne EmilienM: I never used nova-network =)
16:32 EmilienM mgagne: I did :)
16:32 EmilienM mgagne: I can point you the doc, a sec
16:33 EmilienM mgagne: http://docs.openstack.org/trunk/openstack-co​mpute/admin/content/networking-options.html
16:33 EmilienM "All network managers operate in either single-host or multi-host mode."
16:33 EmilienM "In single-host mode, there is just 1 instance of nova-network which is used as a default gateway for VMs and hosts a single DHCP server (dnsmasq), whereas in multi-host mode every compute node has its own nova-network"
16:34 EmilienM so in both cases you need bridge-utils
16:34 mgagne and in both cases, nova-network is required?
16:35 EmilienM no
16:35 mgagne in single-node: one network node is running nova-network?
16:35 EmilienM yep
16:35 EmilienM and then you have a SPOF
16:36 EmilienM that's why you should run nova-network in multi host :)
16:36 EmilienM as we do on our public cloud
16:36 mgagne EmilienM: and how is networking done on the compute node? do you need to configure something?
16:37 EmilienM mgagne: no, just install bridge-utils and nova-compute will assume the work -> you have a single bridge where all the traffic goes in and go to nova-network node each time
16:38 EmilienM imagine a L2 bridge on all compute nodes
16:38 EmilienM and a big l2 + virtual router on nova-network node
16:38 EmilienM that's the single host model
16:38 EmilienM the multi host is : L2 + L3 on each compute
16:38 mgagne EmilienM: I was just wondering where the magic was, how is the bridge discovered/configured
16:38 EmilienM mgagne: by nova-compute
16:39 EmilienM mgagne: https://github.com/openstack/nova/bl​ob/master/nova/network/linux_net.py
16:40 EmilienM wrong file
16:41 EmilienM in nova/compute/manager.py
16:43 EmilienM mgagne: https://github.com/openstack/nova/blob/​master/nova/compute/manager.py#L619-648
16:54 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-heat: Update the module for new config file method  https://review.openstack.org/39980
16:56 mgagne EmilienM: I can't find enable_metadata_proxy in l3_agent.ini on stable/grizzly
16:56 EmilienM oops
16:56 EmilienM let me check if I did wrong
16:56 mgagne https://github.com/openstack/neutron/commi​t/a299091d00d0fcb4d43cf09aba1aefd9c43eb22c
16:57 mgagne 2013.2.b2 2013.2.b1
16:58 EmilienM you're right
16:58 EmilienM let me check if it's going to be backported in 2013.1.3
16:59 mgagne just split the change so we can cherry-pick between master and stable/grizzly
17:00 EmilienM it's not going to be backported, so I split the change
17:00 EmilienM mgagne: my bad.
17:00 mgagne np
17:00 mgagne fun, I don't have merge rights on stable/grizzly
17:01 mgagne try cherry-pick -x next time, it will add the line (cherry picked from commit ...)
17:02 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-quantum: Improve the L3 Agent support  https://review.openstack.org/40453
17:04 EmilienM mgagne: \o/
17:04 EmilienM mgagne: ok thx for the tips
17:07 EmilienM mgagne: do you have any idea why https://review.openstack.org/#/c/39980/ fails ?
17:07 EmilienM I'm reading http://logs.openstack.org/80/39980/3/check/gat​e-puppet-heat-puppet-lint/13d6977/console.html
17:07 mgagne are you concerned about lint or unit test?
17:07 mgagne lint is non-voting
17:08 mgagne EmilienM: http://logs.openstack.org/80/39980/3/check/gate-​puppet-heat-puppet-unit-2.7/cb3248f/console.html
17:08 EmilienM ok
17:08 mgagne it_configures is a helper I created on some modules, it's not a native method in rspec
17:08 mgagne try to rebase over https://review.openstack.org/#/c/40201/
17:09 EmilienM mgagne: yep
17:10 mgagne EmilienM: it was more or less an experience I did back then that stayed
17:11 EmilienM mgagne: I did a git review instead git review stable/grizzly on https://review.openstack.org/#/c/40263/
17:11 EmilienM do you know how could I revert ?
17:11 mgagne git review over it?
17:11 mgagne it will replace it
17:12 openstackgerrit A change was merged to stackforge/puppet-heat: Initial setup for the test environment in puppet-heat  https://review.openstack.org/40201
17:14 starmer joined #puppet-openstack
17:15 EmilienM all done
17:15 EmilienM I retry the heat stuff and I leave :)
17:25 EmilienM mgagne: I'm not familiar enough with rebase. I'm following https://wiki.openstack.org/wiki/Gerrit_Workflow. when I do a "git rebase -i master" I don't really know what to do in the editor
17:26 mgagne EmilienM: the change got merged anyway =)
17:26 mgagne just rebase on master
17:26 EmilienM oh cool
17:27 EmilienM let's try again
17:27 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-heat: Update the module for new config file method  https://review.openstack.org/39980
17:27 mgagne otherwise, I checkout the patch set and rebase over it
17:27 xingchao joined #puppet-openstack
17:27 mgagne there's a link in the patch set section for checkout
17:28 mgagne EmilienM: will have to wait on that one for stable/grizzly: https://review.openstack.org/#/c/40456/
17:29 EmilienM mgagne: indeed :P
17:31 mgagne nice
17:32 EmilienM mgagne: still -1, damn
17:33 EmilienM I try again with checkout method
17:33 mgagne no
17:33 mgagne inline comment =)
17:34 EmilienM mgagne: ahh that's what I add at te first patchset, but I though it was bad to have heat alone :P
17:34 mgagne haha
17:35 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-heat: Update the module for new config file method  https://review.openstack.org/39980
17:35 EmilienM mgagne: don't kidding me, I should have left for 30 minutes \o/
17:36 mgagne EmilienM: hehe, much "better". now we have real tests =)
17:36 EmilienM mgagne: still not. i'm wondering what i'm doing wrong
17:36 mgagne Invalid parameter rabbit_virtual_host at line 3
17:36 EmilienM indeed
17:36 mgagne to me, it's explicit
17:37 EmilienM ahah yeah it is
17:37 EmilienM since we have tests :
17:37 mgagne it's rabbit_virtualhost
17:37 EmilienM mgagne: tomorrow morning I'll make the tests for other manifests
17:37 mgagne cool
17:37 mgagne I don't mind contributing but I was confused with the multiplication of rabbit configs =)
17:37 EmilienM mgagne: I need to manage my next sprint to have more time on puppet stuff. I'll know tomorrow afternoon
17:38 mgagne rabbit_virtual_host is used by nova instead of rabbit_virtualhost
17:38 EmilienM yeah I know you don't mind :) that's cool
17:38 mgagne EmilienM: sprint? I know that =)
17:38 EmilienM mgagne: :P
17:38 EmilienM mgagne: TBH, I'm trying to convince my boss to let me the whole next sprint on puppet stuffs
17:39 EmilienM neutron stuff was a great argument
17:40 mgagne hehe
17:45 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-heat: Update the module for new config file method  https://review.openstack.org/39980
17:48 EmilienM still failing, it seems far enough for today. will see that later
18:03 starmer joined #puppet-openstack
18:16 bodepd EmilienM: I'm going to give chris a little time to get the grizzly branches together
18:16 mgagne bodepd: he won't be able to do the release until https://review.openstack.org/#/c/40456/ is merged
18:16 bodepd EmilienM: then I can review your patch
18:17 bodepd mgagne: I love how you're always at least one step ahead of me
18:21 mgagne =)
19:19 prad joined #puppet-openstack
19:32 prad joined #puppet-openstack
19:55 prad hey guys, whats the best practice to target a patch for two release branches  .. I have a patch for puppet-quantum that i need to send for havana as well as stable/grizzly
19:56 mgagne propose to master and cherry-pick to stable
19:57 mgagne I'm still trying to figure out the flow but I think we should use bugs a bit more. So we can have better traceability and catch what needs to be backported to stable/grizzly.
20:07 prad thx mgagne
20:09 EmilienM bodepd: ok !
20:09 EmilienM bodepd: sorry for lag, I was out for dinner
20:13 EmilienM mgagne: regarding to the flow, in OpenStack we use "grizzly-backport" tag in launchpad
20:13 EmilienM mgagne: maybe could we use the same system
20:13 mgagne EmilienM: yes, I was looking for a similar flow
20:13 EmilienM we file a bug with a feature, and if needed, we backport it
20:13 mgagne EmilienM: I think we want to use most of the openstack flow
20:14 mgagne reuse*
20:14 EmilienM but do follow that, we need to follow openstack versions, and have milestones
20:14 mgagne EmilienM: true
20:14 EmilienM s/do/to
20:14 mgagne EmilienM: bug management is hard because we have only 1 LP project for all puppet modules
20:14 EmilienM I was wondering why the project has no PTL
20:15 mgagne hard to define a milestone when some modules are getting behind or ahead
20:15 EmilienM IMO this is a quite big enough project
20:16 mgagne I think we were mostly playing catch up during the last months
20:19 EmilienM mgagne: I count 146173 lines of code for all puppet modules in OpenStack
20:19 mgagne impossibru!
20:19 openstackgerrit Pradeep Kilambi proposed a change to stackforge/puppet-quantum: Update QUANTUM_PLUGIN_CONFIG for cisco plugin.  https://review.openstack.org/40491
20:19 EmilienM Glance has 159947
20:21 mgagne I count about 70974 =)
20:21 EmilienM mgagne: we need to scale :P
20:24 prad mgagne: one clarification, when you said propose to master and cherry pick, did you mean once the patch is merged to master just cherry pick to stable/grizzly? would that be another review request, I assume?
20:25 mgagne yes, it will require 2 reviews
20:25 prad ok so i create another brand of stable/grizzly, cherry pick commit from master and then git review
20:25 mgagne you can cherry-pick an "in progress" review but I think it will be more trouble if the one in master gets updated meanwhile
20:26 mgagne prad: yes, that's what OpenStack does
20:26 mgagne git cherry-pick -x XYZ
20:26 prad ok
20:26 mgagne https://wiki.openstack.org/​wiki/StableBranch#Workflow
20:51 fvollero joined #puppet-openstack
20:52 EmilienM people with knowledges in nova-network could review this patch > https://review.openstack.org/#/c/40122/
21:06 EmilienM If people here is familiar with tests, I have an issue with tests in Heat module, and I can't figure out the first problem > http://paste.ubuntu.com/5956481/
21:07 EmilienM the test file : http://paste.ubuntu.com/5956488/
21:08 mgagne hi
21:08 EmilienM mgagne: hi !
21:08 mgagne is it with the manifest that's currently in master?
21:09 EmilienM mgagne: the manifest : http://paste.ubuntu.com/5956494/
21:09 EmilienM but should be the same yeah
21:09 mgagne EmilienM: rspec checks the catalog
21:09 mgagne EmilienM: I don't see the File['/etc/heat/heat.cfg'] or any rabbit configs
21:10 mgagne EmilienM: rspec is telling you it's not finding what you were expecting
21:11 EmilienM and lib/puppet/provider/heat_config ?
21:11 mgagne I don't see any in your manifest?
21:11 mgagne EmilienM: rabbit parameters won't configure themselves in heat.conf by some black magic ;)
21:12 EmilienM mgagne: I do it in init.pp
21:12 mgagne EmilienM: did you send me the latest version? I don't see anything related to heat_config in it
21:12 derekh joined #puppet-openstack
21:13 EmilienM mgagne: http://paste.openstack.org/​show/IDf2GnchynYaggVdweoW/
21:13 EmilienM oh
21:13 EmilienM wrong file
21:14 mgagne EmilienM: yha, I though so
21:14 EmilienM oh gotcha
21:14 mgagne EmilienM: "tests are failing and my manifest is mostly empty. why is that?" =)
21:14 EmilienM mgagne: Heat_config is missing in init.pp
21:15 mgagne EmilienM: missing is an understatement =)
21:15 EmilienM \o/
21:17 mgagne wth... "Enable passwordless SSH login between all your hosts if you want to use the resize or migration functionality."
21:19 EmilienM mgagne: for Nova ?
21:19 mgagne EmilienM: yes
21:19 EmilienM mgagne: it's not exactly like this.
21:19 EmilienM mgagne: indeed, you need a passwordless SSH login for nova, but only for a specific user, and a specific folder.
21:20 EmilienM actually used by libvirt for migration
21:20 mgagne http://paste.openstack.org/show/43354/
21:20 mgagne come on, nova has a rpc system and can't use it to create the folder... now I'm a sad panda
21:20 EmilienM mgagne: indeed
21:21 EmilienM mgagne: here, we create a dedicated key for nova and push it on compute nodes on /var/lib/nova/.ssh/authorized_keys
21:21 EmilienM without that, no resize, no migration...
21:21 mgagne EmilienM: yha, someone did it for an other infra and I was flabbergasted by this requirement
21:24 EmilienM mgagne: http://paste.openstack.org/​show/fJPvrVTZlJT0Yyekb2Bx/
21:24 EmilienM mgagne: AFIK it's not a problem from nova
21:24 mgagne yes
21:24 EmilienM mgagne: from libvirt
21:24 EmilienM libvirt uses scp to transfer VMs
21:25 mgagne for your test: check nova/manifests/init.pp, you will see File['/etc/nova/nova.conf'] is defined
21:25 EmilienM if you have a privilege escalation on compute nodes, that means the guy could access to all VM of all the cloud if he has 'nova' account
21:25 mgagne EmilienM: yes and it worries me
21:25 mgagne EmilienM: it shouldn't be like that
21:26 EmilienM mgagne: s/nova/heat :P
21:26 mgagne just add the file resource in init.pp or remove the test
21:27 EmilienM yeah it's working
21:28 EmilienM mgagne: I don't know if nova core plan to change it
21:28 mgagne EmilienM: well I guess that if nobody complains, it's gonna stay that way
21:28 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-heat: Update the module for new config file method  https://review.openstack.org/39980
21:28 EmilienM should work now ^
21:29 EmilienM mgagne: we have a nova dev here, I'll check with him tomorrow what could be done
21:30 EmilienM \o/ my first test on puppet heat module
21:30 mgagne :D
21:30 mgagne rspec-puppet is really just about testing the resulting catalog
21:31 EmilienM mgagne: gotcha. tomorrow morning I'll play with other manifests. thank's for helping :)
21:31 mgagne you're welcome ;)
21:32 EmilienM if somebody cares, I have a patch which could be reviewed > https://review.openstack.org/#/c/39836/ (backported in grizzly also)
21:34 mgagne EmilienM: still waiting for gerrit ACL to be merged
21:34 EmilienM mgagne: it's in master :P
21:34 EmilienM the other one, I know I have to wait
21:34 mgagne EmilienM: true, forgot to mention I was referring to the one for stable/grizzly
21:34 EmilienM mgagne: you won a nova-network course for https://review.openstack.org/#/c/40122/
21:34 EmilienM :P
21:35 EmilienM a free course by the way ;)
21:35 mgagne hehe, what's up with nova-network? is it going to die in Havana?
21:35 EmilienM no
21:35 mgagne =(
21:35 EmilienM not until a long time.
21:35 EmilienM it's not a bad thing
21:35 EmilienM they want backward compatibility
21:35 * mgagne is a sad panda
21:35 EmilienM there is still some use case where quantum is too much
21:36 EmilienM \o/
21:36 EmilienM personnally, I've never used nova-network
21:36 EmilienM but I understand people who does
21:36 mgagne but I feel it's a burden to support =)
21:36 EmilienM yep
21:36 EmilienM until we don't have multi host in quantum for L3, we should keep nova-network
21:37 EmilienM it does the job, that's it.
21:37 EmilienM mgagne: I'm about leaving now. In my todo, I continue on the tests for heat module tomorrow morning
21:37 EmilienM and the afternoon maybe I'll try puppet-neutron :P
21:38 mgagne EmilienM: hehe
21:38 EmilienM mgagne: bonne soirée !
21:38 mgagne EmilienM: a+
21:48 starmer joined #puppet-openstack
21:53 openstackgerrit Chris Hoge proposed a change to stackforge/puppet-keystone: release Preparing puppet-keystone 2.1.0 release  https://review.openstack.org/40508
22:29 openstackgerrit Pradeep Kilambi proposed a change to stackforge/puppet-quantum: Update QUANTUM_PLUGIN_CONFIG for cisco plugin.  https://review.openstack.org/40491
23:00 openstackgerrit Daneyon Hansen proposed a change to stackforge/puppet-swift: Fix Issues Related to Swift Proxy Signing Directory  https://review.openstack.org/40356
23:13 openstackgerrit Daneyon Hansen proposed a change to stackforge/puppet-openstack: Removes Unneeded ovs_local_ip Error Message  https://review.openstack.org/39966

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