Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2013-09-20

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

All times shown according to UTC.

Time Nick Message
00:09 michchap joined #puppet-openstack
00:34 dmsimard joined #puppet-openstack
01:09 otherwiseguy joined #puppet-openstack
01:27 ari joined #puppet-openstack
01:46 ari joined #puppet-openstack
02:55 michchap_ joined #puppet-openstack
03:10 openstackgerrit A change was merged to stackforge/puppet-openstack: update test sec group rules  https://review.openstack.org/47256
03:11 morazi joined #puppet-openstack
03:33 michchap joined #puppet-openstack
03:35 michchap joined #puppet-openstack
03:37 michchap joined #puppet-openstack
03:39 michchap_ joined #puppet-openstack
04:15 rcrit joined #puppet-openstack
04:55 openstackgerrit Mark T. Voelker proposed a change to stackforge/puppet-openstack: Enable RBD-backed Glance in openstack::all  https://review.openstack.org/47520
06:03 michchap joined #puppet-openstack
07:41 openstackgerrit A change was merged to stackforge/puppet-cinder: Creates an empty init script override file if necessary  https://review.openstack.org/47252
07:44 openstackgerrit A change was merged to stackforge/puppet-cinder: Add ratelimiting parameters to cinder::api  https://review.openstack.org/46672
07:44 openstackgerrit A change was merged to stackforge/puppet-nova: Add ratelimiting options to nova::api  https://review.openstack.org/45577
07:47 openstackgerrit A change was merged to stackforge/puppet-swift: Use swift_dispersion_config  https://review.openstack.org/43642
07:49 openstackgerrit A change was merged to stackforge/puppet-openstack: Enable RBD-backed Glance in openstack::all  https://review.openstack.org/47520
07:50 openstackgerrit A change was merged to stackforge/puppet-openstack: Expose glance rbd parameters in openstack::controller  https://review.openstack.org/47227
07:55 openstackgerrit A change was merged to stackforge/puppet-glance: Update glance::backend::rbd options  https://review.openstack.org/47297
07:56 openstackgerrit A change was merged to stackforge/puppet-glance: Update glance::notify::rabbitmq options  https://review.openstack.org/47296
08:00 openstackgerrit A change was merged to stackforge/puppet-openstack: Add allow_overlapping_ips parameter to controller and neutron classes  https://review.openstack.org/45865
08:02 openstackgerrit A change was merged to stackforge/puppet-openstack: Add support for rbd-backed volumes to ::all class  https://review.openstack.org/47310
08:07 openstackgerrit Francois Deppierraz proposed a change to stackforge/puppet-openstack: Add syslog support to the openstack module  https://review.openstack.org/47530
08:19 mmagr joined #puppet-openstack
09:32 leizhang joined #puppet-openstack
09:56 leizhang joined #puppet-openstack
12:55 mjeanson joined #puppet-openstack
13:11 xingchao joined #puppet-openstack
13:32 prad joined #puppet-openstack
13:33 openstackgerrit François Charlier proposed a change to stackforge/puppet-ceilometer: Allow to set full url for endpoints  https://review.openstack.org/47406
13:40 dmsimard joined #puppet-openstack
13:46 rcrit joined #puppet-openstack
13:52 prad joined #puppet-openstack
13:55 dmsimard joined #puppet-openstack
14:40 mjblack joined #puppet-openstack
14:56 technolo-g joined #puppet-openstack
15:02 openstackgerrit Matthew J Black proposed a change to stackforge/puppet-openstack: Expose parameter to pass keystone host  https://review.openstack.org/47585
15:17 ari_ joined #puppet-openstack
15:22 prad joined #puppet-openstack
15:29 mmagr joined #puppet-openstack
15:30 badiane_ka joined #puppet-openstack
15:33 openstackgerrit joined #puppet-openstack
15:41 otherwiseguy joined #puppet-openstack
15:43 otherwiseguy joined #puppet-openstack
15:43 otherwiseguy joined #puppet-openstack
15:55 badiane_ka joined #puppet-openstack
16:23 michchap joined #puppet-openstack
16:43 mgagne We should find a way to address this bug: https://bugs.launchpad.net/p​uppet-keystone/+bug/1224179 =)
16:43 hogepodge joined #puppet-openstack
16:45 EmilienM mgagne: do you manage keystone users by puppet, right ?
16:48 hogepodge mgagne: ouch on that
16:50 openstackgerrit Mark T. Voelker proposed a change to stackforge/puppet-openstack: Add support for rbd-backed volumes to ::all class  https://review.openstack.org/47605
17:06 openstackgerrit Mark T. Voelker proposed a change to stackforge/puppet-openstack: Enable RBD-backed Glance in openstack::all  https://review.openstack.org/47607
17:38 badiane_ka joined #puppet-openstack
18:12 technolo-g joined #puppet-openstack
18:30 hogepodge joined #puppet-openstack
18:54 bodepd mgagne: I have a feeling I could reduce the computational complexity of the whole thing if I just sat down for an hour or so and fixed it
18:54 bodepd mgagne: I know it's making too many calls
18:55 bodepd mgagne: I think the real issue is that the subcalls are happpening way too often
18:57 hogepodge joined #puppet-openstack
19:11 hogepodge joined #puppet-openstack
19:14 mgagne bodepd: puppet-keystone is prefetching everything at each run which is wrong
19:18 digicloseup joined #puppet-openstack
19:26 bodepd mgagne: it should not be that expensive to prefetch everything
19:26 bodepd mgagne: even that shoudl only be 4/5 calls
19:26 mgagne bodepd: try with 10k users =)
19:26 bodepd mgagne: ah, to make the call without pagination?
19:27 mgagne bodepd: puppet doesn't have to prefetch everything just to manage 10 users
19:27 bodepd what if there were 10k users? how expensive is the get_all_users call?
19:27 bodepd mgagne: the issue is way nastier than that
19:27 mgagne bodepd: worst idea ever
19:27 bodepd it fetches thinngs to retrieve the values of other things
19:27 bodepd (like to get tenant info for a user)
19:27 bodepd that code is broken and what causes issues
19:27 mgagne bodepd: users -> tenants -> roles -> user-roles
19:28 bodepd mgagne: the point is that it calls all of that like for every user
19:28 mgagne bodepd: I mean, it's exponential
19:28 bodepd mgagne: yep, that is hte bug
19:29 bodepd mgagne: it should only have to call each of those things once
19:29 bodepd mgagne: then lookup the results
19:29 bodepd mgagne: from that single call
19:29 bodepd mgagne: I always mean to go back and fix that code, just never quite looped back around to it
19:29 mgagne bodepd: it shouldn't fetch all users, only the ones puppet has an interest in
19:30 mgagne bodepd: I might revisit this code myself
19:30 bodepd mgagne: I think the answer there is 'it depends'
19:30 bodepd mgagne: I would fix the inefficient exponential lookups first
19:30 bodepd mgagne: and consider that question as a separate question for later
19:32 mgagne bodepd: my opinion stays the same. it's not a good idea to load the whole database only to check a few entries.
19:32 bodepd mgagne: I think you are worried about performance issues, but looking at the wrong part of the code to fix them
19:33 bodepd mgagne: querying all for most cases should be cheaper
19:33 bodepd mgagne: it is also required for purging of users
19:33 mgagne bodepd: I am concerned because it's affecting my production and my coworker asked who wrote this code.
19:33 bodepd mgagne: the issue with the code is that it is re-looking up the same info too many times
19:34 bodepd mgagne: and I wrote the code, and can speak to the fact that it has known performance issues
19:36 otherwiseguy 651735
19:37 otherwiseguy oops, wrong window. :)
19:59 otherwiseguy joined #puppet-openstack
20:21 hogepodge joined #puppet-openstack
21:06 otherwiseguy joined #puppet-openstack
21:23 EmilienM joined #puppet-openstack
21:28 ari joined #puppet-openstack
22:36 ari joined #puppet-openstack
22:43 openstackgerrit Paul Belanger proposed a change to stackforge/puppet-keystone: Update documentation for sql_connection credentials  https://review.openstack.org/47649
22:50 pabelanger joined #puppet-openstack
22:50 pabelanger guess I should idle here
23:24 bodepd_ joined #puppet-openstack
23:24 ari joined #puppet-openstack
23:29 bodepd_ yet another round of build breakages b/c of puppetlabs upstream compat breakages
23:30 blkperl bodepd_: which module this time?
23:30 bodepd_ postgresql
23:30 blkperl :(
23:30 bodepd_ probably need to do the same branch creation method
23:30 bodepd_ I'm getting a little sick of them as upstream
23:31 blkperl maybe send an email to puppet-dev
23:31 bodepd_ actually, I should not say that until I check if they created a compatibility branch with a release
23:32 blkperl looks like there going to make current master as version 3
23:33 bodepd_ so, did they cut a 2.x branch :)
23:34 blkperl ashp: please make 2.x branch ^ :)
23:34 michchap joined #puppet-openstack
23:36 bodepd_ intersting that module is not even in the Modulefile
23:36 bodepd_ maybe we should just delete all of the postgresql code if noone is using it
23:40 bodepd_ # end rant
23:48 openstackgerrit Paul Belanger proposed a change to stackforge/puppet-glance: Fix puppet-lint issue with documentation  https://review.openstack.org/47660

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