These are chat archives for ManageIQ/manageiq/performance
process_filterseems to be calling rbac for every value even blank ones
findstatement that we dropped? ==> ManageIQ/manageiq#8718
ActsAsActiveRecorddo not implement
|pglogical_subscription.rb||find||Y||can be converted to sql|
find, then rbac won't do anything for tenants or limiting at all?
- if klass.respond_to?(:find) - targets, total_count, auth_count = find_targets_with_rbac(klass, scope, user_filters, find_options, user, miq_group) - else - total_count = targets.length - auth_count = targets.length - end + targets, total_count, auth_count = find_targets_with_rbac(klass, scope, user_filters, find_options, user, miq_group)
allor SOMETHING that represents "look this up in the 'table'"
Yay @yrudman found the line ==> here which I find very confusing.
We run rbac and generate non rbac classes. Then we run rbac on THAT. Seems pagination (and maybe filter) is the only thing the rbac routine could possibly do.
@Fryguy @jrafanie thoughts on converting that to something other than rbac? (we already know it will skip the find and the core of rbac)
Eand roll back this change