Bug 848564
| Summary: | System group limit of -1 is accepted | ||
|---|---|---|---|
| Product: | Red Hat Satellite | Reporter: | Jeff Weiss <jweiss> |
| Component: | WebUI | Assignee: | Walden Raines <walden> |
| Status: | CLOSED CURRENTRELEASE | QA Contact: | Sachin Ghai <sghai> |
| Severity: | medium | Docs Contact: | |
| Priority: | low | ||
| Version: | 6.0.0 | CC: | ckannan, dajohnso, ehelms, jmontleo, omaciel, sghai, walden |
| Target Milestone: | Unspecified | Keywords: | TestBlocker, Triaged |
| Target Release: | Unused | Flags: | walden:
needinfo+
|
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| URL: | http://projects.theforeman.org/issues/5163 | ||
| Whiteboard: | |||
| Fixed In Version: | Doc Type: | Bug Fix | |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2014-07-02 14:05:32 UTC | Type: | --- |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Embargoed: | |||
|
Description
Jeff Weiss
2012-08-15 21:25:52 UTC
I will add javascript check for browsers without HTML5 "spinner" support. Triage note: Re-targeting to CFSE 2.0, low impact. I am not sure how much the UI has changed, this might be still relevant. Note that you now have to enter -2 instead of -1 (as -1 represents unlimited) but this is still valid. Connecting redmine issue http://projects.theforeman.org/issues/5163 from this bug Connecting redmine issue http://projects.theforeman.org/issues/5163 from this bug commit ca4b35e3ab1f69a8508664ec7e931d6cada22344
Merge: 5dec326 f10db47
Author: Walden Raines <walden>
Date: Wed Jun 4 09:16:29 2014 -0400
Merge pull request #4203 from waldenraines/5163
Fixes #5163/BZ848564 - ensure limited host groups have >= 1 hosts.
commit f10db471b33b86cf15c03af0889a3fca85dec4be
Author: Walden Raines <walden>
Date: Tue Jun 3 11:06:34 2014 -0400
Fixes #5163/BZ848564 - ensure limited host groups have >= 1 hosts.
Verified with sat6 beta snap8 (Satellite-6.0.3-RHEL-6-20140604.0) So here is what I've verified. - I created a host collection ( system-group renamed to host collection) with "unlimited" content-host limit - And I tried to update the limit with some -ve value. If I set limit to any -ve value except -1 then, UI raises validation error and It is expected. But If I update the limit with "-1", as soon as I type "-1", the text box closes and points to "unlimited" checkbox. So I'm assuming "-1" still an accepted value to tick the "unlimited" checkbox ? Please confirm. (In reply to Sachin Ghai from comment #12) > But If I update the limit with "-1", as soon as I type "-1", the text box > closes and points to "unlimited" checkbox. So I'm assuming "-1" still an > accepted value to tick the "unlimited" checkbox ? Please confirm. Can you please file a new bug for the update issue and I'll fix that one as well? sure..thanks. Moving this bz to verified since if I set the limit to -1 on host-collection creation, the value is validated and higligted in red. So it appears UI doesn't allow -ve value on creation. However this issue appears on update so will log seperate issue for that. This was delivered with 6.0.3, which is the Satellite 6 Beta. |