[00:21] Someone here have experienced to use juju with puppet? === CyberJacob is now known as CyberJacob|Away === CyberJacob|Away is now known as CyberJacob === uru_ is now known as urulama === CyberJacob is now known as CyberJacob|Away [08:25] gnuoy, just syncing up with neutron-openvswitch - https://code.launchpad.net/~james-page/charms/trusty/neutron-api/hyperv-support/+merge/237370 [08:27] jamespage, approved [08:34] jamespage, fix for glance on juno https://code.launchpad.net/~gnuoy/charms/trusty/glance/list-known-stores/+merge/237372 [08:56] gnuoy, https://code.launchpad.net/~james-page/charm-helpers/fixup-haproxy-maas/+merge/237376 [08:57] fairly urgent - my landings yesterday broke on MAAS [08:57] sure, looking [08:59] jamespage, approved [09:01] gnuoy, not going to raise MP's for the re-sync [09:01] sure [09:01] as I consider those trivials [09:06] gnuoy, feedback on glance charm fixes [09:06] gnuoy, needs to be more conditional on relations being present [09:08] I did think about doing that but I didn't see the point tbh. Still, it's trivial either way [09:21] jamespage, I've updated https://code.launchpad.net/~gnuoy/charms/trusty/glance/list-known-stores/+merge/237372 [09:22] gnuoy, almost! [09:22] aarrghhh [09:22] gnuoy, glance can support multiple backend stores with a default option now [09:23] gnuoy, so having the list is valid, but we need to populate it based on what's related to glance [09:23] ack, will do === ericsnow is now known as ericsnow__ === ericsnow is now known as ericsnow__ === rbasak_ is now known as rbasak === lovea_ is now known as lovea [12:06] gnuoy, quicky for you - https://code.launchpad.net/~james-page/charm-helpers/worker-context/+merge/237409 [12:06] just sweeping up on the worker scaling fixes we did in nova-cc -> neutron-api, glance, cinder and keystone [12:25] jamespage, approved [12:42] gnuoy, and landing for that in: [12:42] https://code.launchpad.net/~james-page/charms/trusty/nova-cloud-controller/worker-config/+merge/237417 [12:42] https://code.launchpad.net/~james-page/charms/trusty/cinder/worker-config/+merge/237413 [12:42] https://code.launchpad.net/~james-page/charms/trusty/neutron-api/worker-config/+merge/237416 [12:42] https://code.launchpad.net/~james-page/charms/trusty/keystone/worker-config/+merge/237415 [12:42] https://code.launchpad.net/~james-page/charms/trusty/glance/worker-config/+merge/237414 [12:42] sorry ;-) [12:42] I've just finished testing that lot === revagomes_ is now known as revagomes [14:05] jamespage, I testing a utopic-juno deploy now but assuming that works I think I've addressed your comments in http://pad.ubuntu.com/nova-cell-feedback [14:05] gnuoy, utopic-juno is bust [14:05] https://bugs.launchpad.net/ubuntu/+source/mysql-5.5/+bug/1378359 [14:05] Bug #1378359: mysql-server crash when syncing neutron database schema [14:05] dealing with that atm [14:06] jamespage, ok, cell charms are working with trusty-icehouse so if you get a moment to have a look that'd be great [14:07] gnuoy, ack will do === jcw4|travelling is now known as jcw4 [14:48] jamespage, 73rd time lucky https://code.launchpad.net/~gnuoy/charms/trusty/glance/list-known-stores/+merge/237372 [14:55] jamespage, we're defaulting the number of glance workers to be twice the machine cpu count but upstream seems to suggest it should match the cpu count. What's the thinking there ? [14:59] gnuoy, ok - look in a sec [14:59] gnuoy, we discussed worker count a while back, and based on the testing I did last cycle at scale, most servers can deal with x4 cpu's as workers [15:00] however that seemed a little extreme so we defaulted to 2x [15:00] kk [15:06] jamespage, looking at http://docs.openstack.org/icehouse/config-reference/content/setting-flags-in-cinder-conf-file.html and http://docs.openstack.org/trunk/config-reference/content/section_cinder.conf.html [15:06] osapi_volume_workers is not listed for icehouse [15:07] gnuoy, hmm [15:07] This looks like a documentation bug tbh. Did you test on icehouse ? [15:07] gnuoy, it is a documentation bug - I tested on icehouse OK [15:08] gnuoy, its in the sample conf file in the source tree for icehouse [15:08] kk [15:10] gnuoy, so close... [15:10] * gnuoy weeps [15:13] gnuoy, sorry - feedback on MP [15:13] gnuoy, you context was a bit non-standard as well [15:19] jamespage, got a sec for another look? [15:20] gnuoy, beautiful - unit test? [15:20] then +1 [15:21] tip top, ta [15:51] jamespage, https://code.launchpad.net/~gnuoy/charms/trusty/glance/list-known-stores/+merge/237372 <- now added unit tests [16:35] it [16:56] gnuoy, +1 on that - please merge away! === roadmr is now known as roadmr_afk === jcw4 is now known as jcw4|away === CyberJacob|Away is now known as CyberJacob === scuttle|afk is now known as scuttlemonkey === jcw4|away is now known as jcw4 === jcw4 is now known as jcw4|away === roadmr_afk is now known as roadmr [22:20] Juys, I'm trying something like this: http://www.slideshare.net/lynxmanuk/juju-puppet-puppetconf-2011 [22:20] So.. [22:20] When deploy an app using puppet master, it take care about relations, right? [22:21] I should use somethink like: juju deploy puppetmaster, juju deploy mysql, juju deploy wordpress, and then use puppet master for manage the relations. [22:22] Or, should I make something different? [22:33] And also, do you guys would install charms using puppet dsl? (: === jcw4|away is now known as jcw4 === rogpeppe is now known as acmeboy