These are chat archives for ManageIQ/manageiq/performance

28th
Sep 2016
Keenan Brock
@kbrock
Sep 28 2016 12:33
@dmetzger57 if a PR works, but has no business signoff - is that a WIP? ref: #11046
Dennis Metzger
@dmetzger57
Sep 28 2016 12:39
Since it does change (granted in a slight way) the UX, buy in is needed before going into the product - so WIP for now. Would be good to have rough numbers in the PR to show the beneficial UX impact.
Keenan Brock
@kbrock
Sep 28 2016 13:31
+1 thanks
Keenan Brock
@kbrock
Sep 28 2016 15:17
nice - every hit to /report/explorer spends the time to lookup the trees. but it then does an ajax fetch to lookup those values again and refresh the left hand side
fun bug
I'll bin/update and see if I'm still having this issue
since the refresh of the tree can be >20 seconds, and then the dom displaying it - it can tack on an additional minute or two to the page
Dan Clarizio
@dclarizio
Sep 28 2016 15:37
@kbrock fyi, I'd like to remove the reports from the main tree and only show them on the right, like we did with VMs
except in this case, no option to turn them on, they really don't serve much purpose there
can probably do it on both the saved reports and the reports accordion
remove the leafs, basically
Dennis Metzger
@dmetzger57
Sep 28 2016 15:42
👍
Oleg Barenboim
@chessbyte
Sep 28 2016 15:48
sounds like the no-leaf solution is a good pattern
Keenan Brock
@kbrock
Sep 28 2016 15:51
glad to hear it is getting traction. that sounds great
Dan Clarizio
@dclarizio
Sep 28 2016 15:57
and in this area, there is no "matching VMware's UI", so I'll run it by a few folks and get a story going
another "feature" that would help here is a limit on how many of each report we keep . . . would need to be per report tho
settable by the users
then a scheduled delete of reports over the limits
Keenan Brock
@kbrock
Sep 28 2016 16:37
@Fryguy could you please look at ManageIQ/manageiq#11374 -- it is 1 line change and is blocking me in a number of areas
Jason Frey
@Fryguy
Sep 28 2016 18:59

another "feature" that would help here is a limit on how many of each report we keep . . . would need to be per report tho

that already exists...but it's done at the purger level

so keep the last "N" reports, or keep reports for the last "X" days