These are chat archives for ManageIQ/manageiq/performance

20th
Mar 2017
Keenan Brock
@kbrock
Mar 20 2017 20:33
@jrafanie were you able to look at ManageIQ/manageiq#14386 -- was trying to get into e
Joe Rafaniello
@jrafanie
Mar 20 2017 20:35
LGTM, is that BZ fixed by this PR?
Keenan Brock
@kbrock
Mar 20 2017 20:36
well, the BZ is "it is slow"
and this made it wicked fast ;)
well, 2x faster
Joe Rafaniello
@jrafanie
Mar 20 2017 20:36
So, fixed ;-)
Keenan Brock
@kbrock
Mar 20 2017 20:37
;)
Joe Rafaniello
@jrafanie
Mar 20 2017 20:37
yeah, I mean, if it's getting backported, it needs a POST BZ and it sounds like that one is not jsut related, it's fixed ;-)
Keenan Brock
@kbrock
Mar 20 2017 20:37
I have 2 PRs (merged) for the BZ
this is the last PR for the merge
would prefer to merge all 3 vs trying to split the BZ
ok - I'll follow up with gt
Joe Rafaniello
@jrafanie
Mar 20 2017 20:38
I can merge, just trying to make sure there's a BZ because we'll need that if it's for Euwe
Keenan Brock
@kbrock
Mar 20 2017 20:39
yes, BZ in related items section
I actually had it in there... next time will put at top
Joe Rafaniello
@jrafanie
Mar 20 2017 20:40
No worries, it wasn't clear if that's the BZ to be shipped or not
Keenan Brock
@kbrock
Mar 20 2017 20:40
ooh - thanks. will fix in future PRs
Joe Rafaniello
@jrafanie
Mar 20 2017 20:48
:+1: