Bug 1649011
| Summary: | hammer os list fails with default organization set | ||
|---|---|---|---|
| Product: | Red Hat Satellite | Reporter: | Ryan Kimbrell <ryan.e.kimbrell> |
| Component: | API | Assignee: | satellite6-bugs <satellite6-bugs> |
| Status: | CLOSED ERRATA | QA Contact: | Shweta Singh <shwsingh> |
| Severity: | medium | Docs Contact: | |
| Priority: | unspecified | ||
| Version: | 6.4.0 | CC: | ahumbe, apatel, dgross, dhlavacd, gscott, jsenkyri, juwatts, kgaikwad, mbacovsk, mhulan, mkalyat, nkathole, rabajaj, rcavalca, sadas, saydas, sfroemer, vchepkov, vijsingh |
| Target Milestone: | 6.10.0 | Keywords: | Triaged |
| Target Release: | Unused | ||
| Hardware: | x86_64 | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Fixed In Version: | Doc Type: | If docs needed, set a value | |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2021-11-16 14:08:27 UTC | Type: | Bug |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Embargoed: | |||
|
Description
Ryan Kimbrell
2018-11-12 17:15:23 UTC
The organization_id and location_id parameters were addad to API documentation of all API commands to allow setting the org/loc context for the operation. It is similar to Org/Loc selector in the UI. Through the API it is propagated to the hammer options automatically. The correct behavior should be the API accepts the options and sets the context properly. It is ignored for resources that are not taxable (same as in UI). Changing the component to API. As a workaround you can suppress passing the defaults with --organization NIL --location NIL. With Hammer >= 0.15.0 (probably Satellite >= 6.5) there comes ability to turn off defaults from configuration or per command basis with --no-use-defaults. *** Bug 1640617 has been marked as a duplicate of this bug. *** @Martin - Thank you, the NIL option is a successful workaround for me. I also tried specifying --organization-id and --location-id, but that still yields 'Association not found for location'. I also tried creating a new OS via `hammer os create` and specified --organization-id and --location-id in the create command. The newly created OS also does not return when executing `hammer os list --organization-id 1 --location-id 2`. It seems os is not taxable, but I believe that is what you were trying to tell me in Comment 1. *** Bug 1667428 has been marked as a duplicate of this bug. *** I have seen other similar bugs marked as duplicates of this one, so unless someone tells me that this is indeed a wholly different bug, I'll focus here. On Sat6.7.4, if a user will get the same error when attempting to list the settings. # hammer settings list Association not found for organization This is precisely because they followed the Hammer CLI guide and set a default organization for the hammer client on an earlier version: https://access.redhat.com/documentation/en-us/red_hat_satellite/6.5/html/hammer_cli_guide/chap-cli_guide-introduction_to_hammer#sect-CLI_Guide-Setting_a_Default_Organization And it can be worked around by using --no-use-defaults: # hammer --no-use-defaults settings list However, without the use of --no-use-defaults, it does trigger an Internal Server Error, logged in ~/.hammer/log/hammer.log ~~~ [ERROR 2020-11-12T17:09:23 API] 500 Internal Server Error [ERROR 2020-11-12T17:09:23 Exception] Association not found for organization [ERROR 2020-11-12T17:09:23 Exception] RestClient::InternalServerError (500 Internal Server Error): /opt/theforeman/tfm/root/usr/share/gems/gems/rest-client-2.0.2/lib/restclient/abstract_response.rb:223:in `exception_with_response' /opt/theforeman/tfm/root/usr/share/gems/gems/rest-client-2.0.2/lib/restclient/abstract_response.rb:103:in `return!' /opt/theforeman/tfm/root/usr/share/gems/gems/apipie-bindings-0.3.0/lib/apipie_bindings/api.rb:353:in `block in rest_client_call_block' /opt/theforeman/tfm/root/usr/share/gems/gems/rest-client-2.0.2/lib/restclient/request.rb:807:in `process_result' /opt/theforeman/tfm/root/usr/share/gems/gems/rest-client-2.0.2/lib/restclient/request.rb:725:in `block in transmit' /opt/rh/rh-ruby25/root/usr/share/ruby/net/http.rb:910:in `start' /opt/theforeman/tfm/root/usr/share/gems/gems/rest-client-2.0.2/lib/restclient/request.rb:715:in `transmit' /opt/theforeman/tfm/root/usr/share/gems/gems/rest-client-2.0.2/lib/restclient/request.rb:145:in `execute' /opt/theforeman/tfm/root/usr/share/gems/gems/rest-client-2.0.2/lib/restclient/request.rb:52:in `execute' /opt/theforeman/tfm/root/usr/share/gems/gems/rest-client-2.0.2/lib/restclient/resource.rb:51:in `get' /opt/theforeman/tfm/root/usr/share/gems/gems/apipie-bindings-0.3.0/lib/apipie_bindings/api.rb:327:in `call_client' /opt/theforeman/tfm/root/usr/share/gems/gems/apipie-bindings-0.3.0/lib/apipie_bindings/api.rb:240:in `http_call' /opt/theforeman/tfm/root/usr/share/gems/gems/apipie-bindings-0.3.0/lib/apipie_bindings/api.rb:190:in `call_action' /opt/theforeman/tfm/root/usr/share/gems/gems/apipie-bindings-0.3.0/lib/apipie_bindings/api.rb:185:in `call' /opt/theforeman/tfm/root/usr/share/gems/gems/apipie-bindings-0.3.0/lib/apipie_bindings/resource.rb:21:in `call' /opt/theforeman/tfm/root/usr/share/gems/gems/hammer_cli-0.19.2.1/lib/hammer_cli/apipie/command.rb:53:in `send_request' /opt/theforeman/tfm/root/usr/share/gems/gems/hammer_cli_foreman-0.19.6.5/lib/hammer_cli_foreman/commands.rb:188:in `send_request' /opt/theforeman/tfm/root/usr/share/gems/gems/hammer_cli_foreman-0.19.6.5/lib/hammer_cli_foreman/commands.rb:247:in `send_request' /opt/theforeman/tfm/root/usr/share/gems/gems/hammer_cli_foreman-0.19.6.5/lib/hammer_cli_foreman/commands.rb:298:in `retrieve_all' /opt/theforeman/tfm/root/usr/share/gems/gems/hammer_cli_foreman-0.19.6.5/lib/hammer_cli_foreman/commands.rb:266:in `execute' /opt/theforeman/tfm/root/usr/share/gems/gems/clamp-1.1.2/lib/clamp/command.rb:63:in `run' /opt/theforeman/tfm/root/usr/share/gems/gems/hammer_cli-0.19.2.1/lib/hammer_cli/abstract.rb:76:in `run' /opt/theforeman/tfm/root/usr/share/gems/gems/clamp-1.1.2/lib/clamp/subcommand/execution.rb:11:in `execute' /opt/theforeman/tfm/root/usr/share/gems/gems/clamp-1.1.2/lib/clamp/command.rb:63:in `run' /opt/theforeman/tfm/root/usr/share/gems/gems/hammer_cli-0.19.2.1/lib/hammer_cli/abstract.rb:76:in `run' /opt/theforeman/tfm/root/usr/share/gems/gems/clamp-1.1.2/lib/clamp/subcommand/execution.rb:11:in `execute' /opt/theforeman/tfm/root/usr/share/gems/gems/clamp-1.1.2/lib/clamp/command.rb:63:in `run' /opt/theforeman/tfm/root/usr/share/gems/gems/hammer_cli-0.19.2.1/lib/hammer_cli/abstract.rb:76:in `run' /opt/theforeman/tfm/root/usr/share/gems/gems/clamp-1.1.2/lib/clamp/command.rb:132:in `run' /opt/theforeman/tfm/root/usr/share/gems/gems/hammer_cli-0.19.2.1/bin/hammer:147:in `<top (required)>' /bin/hammer:23:in `load' /bin/hammer:23:in `<main>' ~~~ If fixing the "settings list" subcommand call to the API is not within the scope of this BZ, please let me know and I'll open a different Bug. Created redmine issue https://projects.theforeman.org/issues/31384 from this bug Upstream bug assigned to ymakias Upstream bug assigned to ymakias Moving this bug to POST for triage into Satellite since the upstream issue https://projects.theforeman.org/issues/31384 has been resolved. Verified "hammer os list" displays list of os system in expected format. [root@dhcp-2-196 ~]# hammer defaults add --param-name organization_id --param-value 1 Added organization_id default-option with value 1. [root@dhcp-2-196 ~]# hammer os list ---|------------|--------------|------- ID | TITLE | RELEASE NAME | FAMILY ---|------------|--------------|------- 1 | RedHat 7.9 | | Redhat ---|------------|--------------|------- 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.10 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-2021:4702 |