Description of problem: Version-Release number of selected component (if applicable): Katello Version: 1.1.4-1.git.74.4bc9528.el6_3 How reproducible: Steps to Reproduce: 1. Create a system group 2. Select it 3. change the Limit from unlimited to -1 Actual results: System group updated Expected results: Error notification like "System group limit must be positive integer". Additional info: already automated.
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.
PR: https://github.com/Katello/katello/pull/4203
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.