RHEL Engineering is moving the tracking of its product development work on RHEL 6 through RHEL 9 to Red Hat Jira (issues.redhat.com). If you're a Red Hat customer, please continue to file support cases via the Red Hat customer portal. If you're not, please head to the "RHEL project" in Red Hat Jira and file new tickets here. Individual Bugzilla bugs in the statuses "NEW", "ASSIGNED", and "POST" are being migrated throughout September 2023. Bugs of Red Hat partners with an assigned Engineering Partner Manager (EPM) are migrated in late September as per pre-agreed dates. Bugs against components "kernel", "kernel-rt", and "kpatch" are only migrated if still in "NEW" or "ASSIGNED". If you cannot log in to RH Jira, please consult article #7032570. That failing, please send an e-mail to the RH Jira admins at rh-issues@redhat.com to troubleshoot your issue as a user management inquiry. The email creates a ServiceNow ticket with Red Hat. Individual Bugzilla bugs that are migrated will be moved to status "CLOSED", resolution "MIGRATED", and set with "MigratedToJIRA" in "Keywords". The link to the successor Jira issue will be found under "Links", have a little "two-footprint" icon next to it, and direct you to the "RHEL project" in Red Hat Jira (issue links are of type "https://issues.redhat.com/browse/RHEL-XXXX", where "X" is a digit). This same link will be available in a blue banner at the top of the page informing you that that bug has been migrated.
Bug 818985 - [ipa webui] Searchsize limit is not used
Summary: [ipa webui] Searchsize limit is not used
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Enterprise Linux 6
Classification: Red Hat
Component: doc-Identity_Management_Guide
Version: 6.2
Hardware: Unspecified
OS: Unspecified
urgent
urgent
Target Milestone: rc
: ---
Assignee: Deon Ballard
QA Contact: ecs-bugs
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-05-04 14:04 UTC by Namita Soman
Modified: 2012-07-02 19:31 UTC (History)
8 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-07-02 19:31:47 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description Namita Soman 2012-05-04 14:04:30 UTC
Description of problem:
In configuration, set the search size limit to be 5
With 10 users added, go to user page, and expect only 5 to be listed...but all 10 are listed. 
Same with groups.

But in config page, dropdown to list "Default Users group" lists only 5. 

Tried in cli, and ipa user-find brings back only 5 users

Version-Release number of selected component (if applicable):
ipa-server-2.2.0-12.el6.x86_64

How reproducible:
always

Steps to Reproduce:
1. Add 10 users
2. Set search size limit to be 5
3. Go to user page
  
Actual results:
display all 10 users and admin

Expected results:
display only 5 users total

Additional info:

Comment 2 Namita Soman 2012-05-04 14:48:34 UTC
Discussed this with Petr.

======
he said <snippets from irc> :
web ui uses kinda its own search limit for search pages to support paging.
Right now it is hardcoded to 20. It behaves this way: UI sends xxx-find command
with --pkey-only option. The --pkey-only bypasses the limit and gets all pkeys
then UI selects 20 of them (depend on current page) and calls batch command to
fetch data.

--pkey-only limit is 5000

In other controls (mentioned dropdown) the --pkey-only option is not used so
the result is limited.

IMHO this behavior is OK. What we can do is to make the search page size
configurable. For example user-specific config value stored in browser (hmtl 5
local storage).

in dropdowns like selecting a group, limit is enforced. In search pages may be
fetched first 5000 pkeys+DNs and then selected 20 of them to fetch remaining
values

not a bug - it was designed this way

=====

Beahviour is as expected...so closing

Comment 3 Jenny Severance 2012-05-04 15:16:12 UTC
I would like more discussion around this since setting the search limits now can be very confusing ... maybe that the minimum should be 20 if the UI is setting it as a hard limit??

Comment 5 Petr Vobornik 2012-05-07 11:43:31 UTC
There is no point to create some minimum because UI uses limit which can be larger than config 'search limit'. What would we achieve by that?

If we want to do some conclusion we should first mention/define purposes of various limits.

So we have 3 limits:
 1) search limit defined in configuration (default 100 records)
 2) pkey-only limit: In above comment I didn't say it correctly. The limit is 0, that means that it is only limited by ldap server limit which might more than 5000.
 3) page size - hardcoded to 20

I'm not sure what is the original purpose of 'search limit'. IMO it is to limit the amount of data transferred and not to overwhelm user with tons of text. This limit is applied to all commands except those with --pkey-only option. The only thing which uses --pkey-only options are search pages in Web UI. We can conclude that all parts of CLI and Web UI except search behaves consistently and now we should only discuss search pages.

So the question is how many records should the search page show? I don't see a point in limiting the amount of records to 'search limit' if there is a space for displaying the records. Is there a reason the display the same amount of records as the search limit? Also no. The search limit can be much higher - ie 200. It would make the paging somehow weird. User should not scroll vertically a lot. The reasonable amount can be about 1-3 screens which is approximately 20-80 records depending on window size.

So enforcing the 'search limit' in search page is not the right thing.
Is the hard-coded limit to 20 records good? No. But it is the better than 'search limit'. So that's why I mentioned that we might want to make this limit configurable. I think it should be user specific. Storing the value in user record is bad because it is application specific data. So that's the reasoning for storing it in browser. Another option is to make the option dynamic, it can be computed from the size of window (with some minimum size of course).

Conclusion:
IMO we should do one of following options:
 a) leave it as is
 b) make page_size configurable in browser
 c) make page_size dependant on window size

Comment 6 Dmitri Pal 2012-05-09 12:26:09 UTC
IMO we should document all of the above in a special section talking about search limits. Reassigning.

Comment 7 Deon Ballard 2012-06-27 03:09:34 UTC
I added a new subsection in the part on search settings:
http://docs.redhat.com/docs/en-US/Red_Hat_Enterprise_Linux/6/html/Identity_Management_Guide/searches.html#types-of-search-limits

Comment 10 Deon Ballard 2012-07-02 19:31:47 UTC
The internal url has been changed to example.com.


Note You need to log in before you can comment on or make changes to this bug.