Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2014-03-06

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

All times shown according to UTC.

Time Nick Message
00:16 xarses joined #puppet-openstack
00:19 rharrison joined #puppet-openstack
00:26 mgagne joined #puppet-openstack
00:31 kitp joined #puppet-openstack
00:31 britthouser joined #puppet-openstack
00:40 openstackgerrit joined #puppet-openstack
00:43 openstackgerrit Sebastien Badia proposed a change to stackforge/puppet-swift: Fix test files (address is now removed)  https://review.openstack.org/78481
00:58 thumpba joined #puppet-openstack
00:58 britthouser joined #puppet-openstack
00:59 otherwiseguy joined #puppet-openstack
01:01 prad joined #puppet-openstack
01:10 fandi joined #puppet-openstack
01:13 openstackgerrit Sebastien Badia proposed a change to stackforge/puppet-swift: Add client_package_ensure parameter and swift::client class  https://review.openstack.org/78468
01:24 ryanycoleman joined #puppet-openstack
01:44 dprince joined #puppet-openstack
01:59 xarses joined #puppet-openstack
02:01 rmoe joined #puppet-openstack
02:02 fandi joined #puppet-openstack
02:08 xingchao joined #puppet-openstack
02:19 markvoelker1 joined #puppet-openstack
02:46 starmer joined #puppet-openstack
02:54 rharrison joined #puppet-openstack
03:00 kitp joined #puppet-openstack
03:07 kitp joined #puppet-openstack
03:10 openstackgerrit A change was merged to stackforge/puppet-nova: compute: RBD backend support  https://review.openstack.org/76462
03:24 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-keystone: Load user un-lazily if needed  https://review.openstack.org/78333
03:28 otherwiseguy joined #puppet-openstack
03:29 thumpba_ joined #puppet-openstack
03:33 ryanycoleman joined #puppet-openstack
03:34 markvoelker1 mgagne: ^^ good timing, I was just about to kick off a deployment.  I'll take the new revision for a spin.
03:34 mgagne markvoelker1: ^^'
03:37 markvoelker1 mgagne: by the way, there's a stable backport of that keystone patch that was bugging you the other day if you're interested: https://review.openstack.org/#/c/78419/
03:38 mgagne markvoelker1: cool, thanks! reviewed
03:38 markvoelker1 np, thanks for the review
04:15 xingchao joined #puppet-openstack
04:18 xarses joined #puppet-openstack
05:43 openstackgerrit A change was merged to stackforge/puppet_openstack_builder: Expose libvirt disk_cachemodes setting  https://review.openstack.org/78227
05:58 starmer joined #puppet-openstack
06:04 openstackgerrit A change was merged to stackforge/puppet-keystone: Don't set keystone endpoint by default  https://review.openstack.org/78419
06:09 kitp joined #puppet-openstack
06:11 openstackgerrit A change was merged to stackforge/puppet_openstack_builder: Expose libvirt_cpu_mode parameter  https://review.openstack.org/77877
06:20 starmer joined #puppet-openstack
06:38 starmer joined #puppet-openstack
06:49 openstackgerrit A change was merged to stackforge/puppet-horizon: Parameterize OPENSTACK_HYPERVISOR_FEATURES settings  https://review.openstack.org/68433
06:51 bauzas joined #puppet-openstack
07:10 kitp joined #puppet-openstack
07:22 saju_m joined #puppet-openstack
07:46 bauzas joined #puppet-openstack
07:58 bauzas joined #puppet-openstack
08:06 thumpba__ joined #puppet-openstack
08:10 kitp joined #puppet-openstack
08:18 saju_m joined #puppet-openstack
08:18 mmagr joined #puppet-openstack
08:32 dgollub joined #puppet-openstack
08:34 openstackgerrit joined #puppet-openstack
08:36 saju_m joined #puppet-openstack
08:52 openstackgerrit Mark T. Voelker proposed a change to stackforge/puppet-neutron: Modernize Cisco plugin settings  https://review.openstack.org/78550
08:55 bauzas1 joined #puppet-openstack
08:59 openstackgerrit Mark T. Voelker proposed a change to stackforge/puppet-neutron: Modernize Cisco plugin settings  https://review.openstack.org/78550
09:00 openstackgerrit A change was merged to stackforge/puppet-neutron: Fix FWaaS race condition  https://review.openstack.org/78423
09:05 comptona joined #puppet-openstack
09:11 kitp joined #puppet-openstack
09:28 openstackgerrit Pierre Rambaud proposed a change to stackforge/puppet-nova: Add options to use remove_unused_* parameters  https://review.openstack.org/77881
09:58 saju_m joined #puppet-openstack
10:03 comptona joined #puppet-openstack
10:12 kitp joined #puppet-openstack
10:34 dvanders joined #puppet-openstack
10:37 dvanders left #puppet-openstack
10:56 openstackgerrit Michael Chapman proposed a change to stackforge/puppet_openstack_builder: Remove controller and compute class groups.  https://review.openstack.org/78579
11:03 saju_m joined #puppet-openstack
11:13 kitp joined #puppet-openstack
11:19 openstackgerrit Michael Chapman proposed a change to stackforge/puppet_openstack_builder: Remove advanced data mappings  https://review.openstack.org/78585
12:13 openstackgerrit A change was merged to stackforge/puppet_openstack_builder: Add swift mappings for full_ha scenario  https://review.openstack.org/78223
12:13 kitp joined #puppet-openstack
12:40 morazi joined #puppet-openstack
12:42 sputnik13 joined #puppet-openstack
12:46 xingchao joined #puppet-openstack
12:55 markvoelker joined #puppet-openstack
13:14 kitp joined #puppet-openstack
13:29 EmilienM I introduced a bug in puppet-neutron: https://bugs.launchpad.net/​puppet-neutron/+bug/1288741
13:29 EmilienM i'm working on the fix right now.
13:48 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-neutron: ml2: fix wrong package dependency for Debian/Ubuntu  https://review.openstack.org/78603
13:51 EmilienM michchap: still around?
14:07 markvoelker EmilienM: I'll have a look in a few minutes
14:07 EmilienM markvoelker: oh thanks
14:15 kitp joined #puppet-openstack
14:17 otherwiseguy joined #puppet-openstack
14:19 britthouser joined #puppet-openstack
14:19 markvoelker EmilienM: haven't tested this yet, but just from eyeballing it something looks odd.  I don't think Ubuntu Cloud Archive has a "neutron" package does it?
14:20 markvoelker EmilienM: 'neutron-server' instead perhaps?
14:20 EmilienM markvoelker: the resource is neutron-common afik
14:20 EmilienM let me check
14:21 EmilienM markvoelker: yeah, look in manifests/init.pp L196
14:21 EmilienM 196   package { 'neutron':
14:21 EmilienM 197     ensure => $package_ensure,
14:21 EmilienM 198     name   => $::neutron::params::package_name,
14:21 EmilienM 199   }
14:21 EmilienM it's neutron-common in reality
14:22 markvoelker ah, I see
14:22 markvoelker thanks
14:22 EmilienM markvoelker: i introduced this bug :(
14:22 EmilienM it's up to me to fix it :)
14:23 markvoelker EmilienM: I'll run a quick test momentarily
14:23 EmilienM ok
14:23 EmilienM and i'll backport it
14:48 britthouser joined #puppet-openstack
14:58 kitp joined #puppet-openstack
15:00 EmilienM markvoelker: thanks for the quick review, i'll backport it once approved
15:00 markvoelker EmilienM: sure, feel free to add me as a reviewer for that as well
15:00 EmilienM markvoelker: thx!
15:08 mgagne joined #puppet-openstack
15:12 meylor joined #puppet-openstack
15:14 openstackgerrit Benedikt Trefzer proposed a change to stackforge/puppet-glance: add parameter for endpoint protocols  https://review.openstack.org/78625
15:28 openstackgerrit Benedikt Trefzer proposed a change to stackforge/puppet-neutron: add parameter for endpoint protocols  https://review.openstack.org/78634
15:29 openstackgerrit Benedikt Trefzer proposed a change to stackforge/puppet-glance: add parameter for endpoint protocols  https://review.openstack.org/78625
15:32 Aimon joined #puppet-openstack
15:36 britthouser joined #puppet-openstack
15:41 openstackgerrit Mark T. Voelker proposed a change to stackforge/puppet-neutron: Modernize Cisco plugin settings  https://review.openstack.org/78550
15:42 britthou_ joined #puppet-openstack
15:57 dgollub1 joined #puppet-openstack
16:05 dtalton joined #puppet-openstack
16:07 openstackgerrit Mark T. Voelker proposed a change to stackforge/puppet-neutron: Modernize Cisco plugin settings  https://review.openstack.org/78550
16:10 ryanycoleman joined #puppet-openstack
16:13 openstackgerrit Benedikt Trefzer proposed a change to stackforge/puppet-nova: add parameter for endpoint protocols  https://review.openstack.org/78647
16:15 prad_ joined #puppet-openstack
16:23 thumpba_ joined #puppet-openstack
16:31 openstackgerrit Donald Talton proposed a change to stackforge/puppet_openstack_builder: Set rbd as storage default for the full_ha scenario  https://review.openstack.org/78656
16:31 mgagne markvoelker: ping
16:33 markvoelker mgagne: pong
16:33 mgagne markvoelker: had a chance to test keystone patch?
16:34 markvoelker mgagne: Did you see the comment I left on patchset 2?  Still seeing an error, but I can't see why from eyeballing it.
16:34 mgagne markvoelker: I patched my setup today with latest patch set and it worked
16:35 mgagne markvoelker: are you using the latest version?
16:35 markvoelker Patchset 2, yes
16:35 markvoelker I also confirmed the error goes away without the patch
16:35 mgagne :-/
16:36 markvoelker I will dig into it some more this afternoon if possible...I don't understand where that error would come from.
16:36 mgagne With patch set 1, I made the mistake to dev/test elsewhere and copy/paste the patch in puppet-keystone (with mistakes)
16:36 EmilienM mgagne: if you need some help on testing, i have a ci here with havana working
16:37 mgagne markvoelker: unless someone still uses (from elsewhere) the previous get_user() which accepted 2 args
16:37 markvoelker Yeah, it made sense with patchset 1, but patchset 2 should have cleared that up.
16:37 openstackgerrit Donald Talton proposed a change to stackforge/puppet_openstack_builder: Removed cinder and glance scenario dirs  https://review.openstack.org/78660
16:37 mgagne EmilienM: https://review.openstack.org/#/c/78333/
16:37 markvoelker I grepped around a bit and couldn't find anywhere that used 2 args
16:37 mgagne markvoelker: did you restart puppetmaster?
16:37 EmilienM mgagne: ack
16:38 markvoelker mgagne: it was a fresh build from scratch
16:38 mgagne markvoelker: updating customer types might require a puppetmaster restart
16:38 mgagne markvoelker: oh you =)
16:38 * markvoelker is a troublemaker today
16:41 markvoelker EmilienM: on a fresh build this morning, I'm seeing some weird database issues w/Neutron & Ubuntu. Wondering if it might be related to https://review.openstack.org/#/c/76358 so retesting with that backed out. Will let you know what I find.
16:41 EmilienM markvoelker: it could be related to my bug i solved today
16:41 EmilienM markvoelker: using ML2 ?
16:42 markvoelker EmilienM: nope, ovs
16:42 EmilienM markvoelker: as a plugin?
16:42 markvoelker EmilienM: yes
16:42 EmilienM ah
16:42 EmilienM markvoelker: (you should use ml2 between us) :)
16:43 markvoelker EmilienM: Yeah, already have a bunch of deployments using ovs though
16:43 EmilienM ok :)
16:43 EmilienM mgagne: if you want to approve: https://review.openstack.org/78603
16:43 EmilienM mgagne: related to https://bugs.launchpad.net/​puppet-neutron/+bug/1288741
16:48 openstackgerrit A change was merged to stackforge/puppet-neutron: ml2: fix wrong package dependency for Debian/Ubuntu  https://review.openstack.org/78603
16:50 JonnyNomad joined #puppet-openstack
16:51 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-neutron: ml2: fix wrong package dependency for Debian/Ubuntu  https://review.openstack.org/78666
16:51 EmilienM and the backport ^
17:08 rwsu joined #puppet-openstack
17:13 derekh joined #puppet-openstack
17:15 openstackgerrit A change was merged to stackforge/puppet-neutron: ml2: fix wrong package dependency for Debian/Ubuntu  https://review.openstack.org/78666
17:18 EmilienM markvoelker: mgagne thx btw
17:20 markvoelker EmilienM: sure
17:22 markvoelker EmilienM: so I just confirmed that if I set db_sync=false on Ubuntu w/the ovs plugin, the database error I was seeing goes away.  I think why the new default behavior breaks it.  Will file a bug in a bit.
17:23 xarses joined #puppet-openstack
17:23 mgagne markvoelker: FYI, we had a lot of issues with db_sync when upgrading to havana... so I don't like much db_sync being added to puppet-neutron without thorough testing, there is a lot of corner cases and pitfalls
17:25 EmilienM mgagne: i did the functionnal testing on havana
17:25 mgagne EmilienM: "there is a lot of corner cases and pitfalls"
17:28 markvoelker I'm ok with the patch being there, it's just that the default behavior change to run the db sync in Havana seems to have caused problems for my deployments.
17:28 markvoelker I think I see why, but I'll have to investigate a bit later this afternoon...have to run off to a meeting shortly. =/
17:28 rmoe joined #puppet-openstack
17:42 openstackgerrit A change was merged to stackforge/puppet_openstack_builder: Fixes Heat AES Key Error  https://review.openstack.org/77978
17:42 openstackgerrit A change was merged to stackforge/puppet_openstack_builder: Add missing data mapping for swift_hash  https://review.openstack.org/78014
17:45 bauzas joined #puppet-openstack
17:50 britthouser joined #puppet-openstack
18:04 ryanycoleman joined #puppet-openstack
18:05 angdraug joined #puppet-openstack
18:06 openstackgerrit Chris Ricker proposed a change to stackforge/puppet_openstack_builder: Typo in example parameter in user.common.yaml  https://review.openstack.org/78691
18:06 ryanycoleman joined #puppet-openstack
18:07 ryanycol_ joined #puppet-openstack
18:12 ryanycoleman joined #puppet-openstack
18:15 ryanycoleman joined #puppet-openstack
18:18 hogepodge joined #puppet-openstack
18:19 dmsimard joined #puppet-openstack
18:32 openstackgerrit A change was merged to stackforge/puppet-ceilometer: Move service polling credentials to service_credentials  https://review.openstack.org/78428
18:35 ryanycoleman joined #puppet-openstack
18:38 openstackgerrit A change was merged to stackforge/puppet-swift: Fix test files (address is now removed)  https://review.openstack.org/78481
18:39 dprince joined #puppet-openstack
18:43 Aimon joined #puppet-openstack
18:46 openstackgerrit A change was merged to stackforge/puppet-openstack: Ensure swift.conf created in all scenarios  https://review.openstack.org/78427
18:48 dgollub joined #puppet-openstack
19:03 openstackgerrit A change was merged to stackforge/puppet_openstack_builder: Removed cinder and glance scenario dirs  https://review.openstack.org/78660
19:05 hogepodge joined #puppet-openstack
19:06 openstackgerrit A change was merged to stackforge/puppet-heat: Adds Support for Database Idle Timeout  https://review.openstack.org/78007
19:11 openstackgerrit A change was merged to stackforge/puppet-nova: Add support for latest puppetlabs-rabbitmq  https://review.openstack.org/76896
19:17 openstackgerrit A change was merged to stackforge/puppet-ceilometer: fix alarm_package_name for Debian/Ubuntu (only havana)  https://review.openstack.org/77183
19:21 openstackgerrit Donald Talton proposed a change to stackforge/puppet_openstack_builder: Options for configuring live migration on NFS  https://review.openstack.org/78721
19:28 openstackgerrit Daneyon Hansen proposed a change to stackforge/puppet-heat: Adds Support for Database Idle Timeout  https://review.openstack.org/78724
19:29 openstackgerrit Chris Ricker proposed a change to stackforge/puppet_openstack_builder: Remove config parameters from global_hiera_params  https://review.openstack.org/78725
19:39 ryanycoleman joined #puppet-openstack
19:41 hogepodge joined #puppet-openstack
19:47 bauzas joined #puppet-openstack
19:47 ryanycoleman joined #puppet-openstack
19:56 openstackgerrit Daneyon Hansen proposed a change to stackforge/puppet-heat: Adds Support for Database Idle Timeout  https://review.openstack.org/78724
20:00 openstackgerrit_ joined #puppet-openstack
20:03 JonnyNomad joined #puppet-openstack
20:27 markvoelker EmilienM: https://bugs.launchpad.net/​puppet-neutron/+bug/1288975 =(
20:27 markvoelker mgagne: ^^ This is the sync_db thing we were discussing earlier
20:28 markvoelker For the sake of getting somethign in motion here, I'm going to submit a patch to change the sync_db default to false.  Only question is whether to do that in stable/havana only, or master too?
20:29 mgagne markvoelker: havana != icehouse in that regard ?
20:30 markvoelker mgagne: I haven't tested Icehouse (and likely won't have time to soon...deadline coming up tomorrow).
20:30 markvoelker mgagne: my thought was that in stable/havana it's a behavior change
20:30 markvoelker That might be ok in master
20:30 markvoelker But I'm thinking even in master this looks like there may be a lot of scenarios where it will cause problems, so might be safest to set to false there as well
20:30 mgagne markvoelker: also, I was more/less thinking about a way for puppet to tell the end user to stamp his database version manually if not already done (and abort puppet run)
20:31 mgagne markvoelker: that's my opinion
20:31 markvoelker Ok, I'll send up a patch to master and if others have a different opinion we'll discuss it in gerrit.
20:44 openstackgerrit Mark T. Voelker proposed a change to stackforge/puppet-neutron: Don't sync database by default  https://review.openstack.org/78752
20:47 markvoelker mgagne: ^^
20:47 markvoelker EmilienM: ^^
20:55 thumpba_ joined #puppet-openstack
21:09 sputnik13 joined #puppet-openstack
21:10 ryanycoleman joined #puppet-openstack
21:13 sputnik13 joined #puppet-openstack
21:30 openstackgerrit Mark T. Voelker proposed a change to stackforge/puppet_openstack_builder: Explicitly disable Neutron DB sync by default  https://review.openstack.org/78776
21:32 ryanycoleman joined #puppet-openstack
21:33 mwoodson i am running grizzly with stackforge  modules.  We have about 1300 user, 100 tenants.
21:33 mwoodson doing a puppet run takes forEVER.  We looked at the debug and trace and saw thousands of keyston queires
21:34 mwoodson is thos normal?
21:34 thumpba_ joined #puppet-openstack
21:38 otherwiseguy joined #puppet-openstack
21:51 openstackgerrit Thomas Bechtold proposed a change to stackforge/puppet-nova: Add Nova Aggregate support  https://review.openstack.org/70344
22:02 michchap mwoodson: there  was a patch in havana to reduce the # of queries by loading lazily, which wouldn't be on the grizzly branch
22:03 mwoodson michchap: k, thanks
22:03 michchap mwoodson: take a look through the commit log and you could probably backport it, but we are currently having issues with it, try to ping mgagne
22:03 michchap mwoodson: https://review.openstack.org/#/c/78333/
22:03 mgagne michchap: they should all be on grizzly branch, I needed them and backported them
22:04 michchap mgagne: oh I didn't realise that
22:04 mgagne michchap: except I no longer use grizzly branch so won't backport stuff anymore
22:05 dtalton joined #puppet-openstack
22:15 dtalton2 joined #puppet-openstack
22:28 sputnik13 joined #puppet-openstack
22:37 openstackgerrit Bob Callaway proposed a change to stackforge/puppet-cinder: Add support for copy offload, E-Series driver  https://review.openstack.org/78796
22:39 sputnik13 joined #puppet-openstack
22:59 sputnik13 joined #puppet-openstack
23:03 mgagne markvoelker: ok, db_sync is sh*tting all over the place in our setup now
23:30 kkirkpatrick joined #puppet-openstack

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