Bug 2082241

Hi

In production.log I only see the successful test (Test1, without --unlimited-hosts="True"), not the failures:

2022-05-05T12:19:23 [I|app|e257f9c4] Started POST "/katello/api/organizations/24/host_collections" for 10.1.2.241 at 2022-05-05 12:19:23 -0400
2022-05-05T12:19:23 [I|app|e257f9c4] Processing by Katello::Api::V2::HostCollectionsController#create as JSON
2022-05-05T12:19:23 [I|app|e257f9c4]   Parameters: {"name"=>"Test1", "api_version"=>"v2", "organization_id"=>"24", "host_collection"=>{"name"=>"Test1"}}
2022-05-05T12:19:23 [I|app|e257f9c4] Authorized user admin(Admin User)
2022-05-05T12:19:23 [I|aud|e257f9c4] Katello::HostCollection (1) create event on name Test1
2022-05-05T12:19:23 [I|aud|e257f9c4] Katello::HostCollection (1) create event on description 
2022-05-05T12:19:23 [I|aud|e257f9c4] Katello::HostCollection (1) create event on max_hosts 
2022-05-05T12:19:23 [I|aud|e257f9c4] Katello::HostCollection (1) create event on organization_id 24
2022-05-05T12:19:23 [I|aud|e257f9c4] Katello::HostCollection (1) create event on unlimited_hosts true
2022-05-05T12:19:23 [I|app|e257f9c4]   Rendering /usr/share/gems/gems/katello-4.3.0.33/app/views/katello/api/v2/host_collections/create.json.rabl within katello/api/v2/layouts/resource
2022-05-05T12:19:23 [I|app|e257f9c4]   Rendered /usr/share/gems/gems/katello-4.3.0.33/app/views/katello/api/v2/host_collections/create.json.rabl within katello/api/v2/layouts/resource (Duration: 29.3ms | Allocations: 4952)
2022-05-05T12:19:23 [I|app|e257f9c4] Completed 201 Created in 305ms (Views: 42.7ms | ActiveRecord: 43.1ms | Allocations: 26902)

Hey Stephen,

Have you tried 
`hammer host-collection create --name="Test2" --organization-id=1 --unlimited-hosts`

That should work. '--unlimited-hosts' is a flag not an option.

Hi Partha

That raises a question, why did test automation pass before?


E             'unlimited-hosts': 'True'}

Testing on 6.11.0-19.0-rhel7.9
~]# hammer --version
hammer (3.1.0.1)
 * hammer_cli_foreman (3.1.0)
 * hammer_cli_foreman_admin (1.1.0)

* hammer_cli_katello (1.3.1.2

[root@xxx ~]# rpm -q satellite
satellite-6.11.0-0.11.beta.el7sat.noarch
[root@xxx ~]# hammer host-collection create --name="Test1" --organization-id=1
Host collection created.

[root@xxx ~]# hammer host-collection info --name="Test1" --organization-id=1 
Id:          1
Name:        Test1
Limit:       None   <----- Note (so that is the default)
Description: 
Total Hosts: 0

[root@xxx ~]# hammer host-collection create --name="Test2" --organization-id=1 --unlimited-hosts
Host collection created.

[root@xxx ~]# hammer host-collection info --name="Test2" --organization-id=1 
Id:          2
Name:        Test2
Limit:       None
Description: 
Total Hosts: 0


~]# hammer host-collection create --name="Test3" --organization-id=1 --max-hosts 10
Host collection created.
[root@xxx ~]# hammer host-collection info --name="Test3" --organization-id=1 
Id:          3
Name:        Test3
Limit:       10
Description: 
Total Hosts: 0


Another question, why does it require a value in the case of updating?

[root@xxx ~]# hammer host-collection update --name="Test3" --organization-id=1 --unlimited-hosts
Could not update the the host collection:
  Error: Option '--unlimited-hosts': Value must be one of true/false, yes/no, 1/0..
  
  See: 'hammer host-collection update --help'.

[root@xxx ~]# hammer host-collection update --name="Test3" --organization-id=1 --unlimited-hosts=1   <--- NOTE
Host collection updated.
[root@xxx ~]# hammer host-collection info --name="Test3" --organization-id=1 
Id:          3
Name:        Test3
Limit:       None
Description: 
Total Hosts: 0


Thank you

Re ""That raises a question, why did test automation pass before? ""

This got added as part of ->  https://bugzilla.redhat.com/show_bug.cgi?id=1813624

"""
Another question, why does it require a value in the case of updating?
"""

Sounds like a bug . I 'll put in a upstream issue for that.

Connecting redmine issue https://projects.theforeman.org/issues/34889 from this bug

Upstream bug assigned to paji

Upstream bug assigned to paji

(In reply to Partha Aji from comment #4)
> Re ""That raises a question, why did test automation pass before? ""
> 
> This got added as part of -> 
> https://bugzilla.redhat.com/show_bug.cgi?id=1813624
> 

Has the suggestion in the "Additional info:" section[1] has not been accepted?


[1] https://bugzilla.redhat.com/show_bug.cgi?id=1813624#c0

Moving this bug to POST for triage into Satellite since the upstream issue https://projects.theforeman.org/issues/34889 has been resolved.

Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory (Moderate: Satellite 6.11 Release), and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2022:5498