Bug 2131788

Summary: Documentation bug for the compute_resource module
Product: Red Hat Satellite Reporter: Odilon Sousa <osousa>
Component: Ansible CollectionAssignee: Evgeni Golov <egolov>
Status: CLOSED ERRATA QA Contact: Griffin Sullivan <gsulliva>
Severity: low Docs Contact:
Priority: unspecified    
Version: 6.9.0CC: egolov, osousa, zhunting
Target Milestone: 6.11.4Keywords: Triaged
Target Release: Unused   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: ansible-collection-redhat-satellite-3.6.0 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: 1990119 Environment:
Last Closed: 2022-10-27 13:08:49 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 Odilon Sousa 2022-10-03 16:03:24 UTC
+++ This bug was initially created as a clone of Bug #1990119 +++

Description of problem:

When following the documentation at https://console.redhat.com/ansible/automation-hub/repo/published/redhat/satellite/content/module/compute_resource the URL of the provider is wrong, at least for the libvirt provider.


Version-Release number of selected component (if applicable):
2.1.2 (Sat 6.9 -> 2.0.1)

How reproducible:
Always

Steps to Reproduce:
1. Using the following Ansible playbook snippet:
provider_params:
      url: libvirt.example.com
      display_type: vnc
2. Run the playbook


Actual results:

fatal: [localhost]: FAILED! => {"changed": false, "error": {"errors": {"url": ["is invalid"]}, "full_messages": ["URL is invalid"], "id": null}, "msg": "Error while performing create on compute_resources: 422 Client Error: Unprocessable Entity"}                                             


Expected results:

TASK [compute-resources : Create livirt compute resource] *********************************************
changed: [localhost]



Additional info:

Correct would be:

- name: Create livirt compute resource
  redhat.satellite.compute_resource:
    name: libvirt-hypervisor
    locations: "BER"
    organizations: "{{ organization }}"
    provider: libvirt
    provider_params:
      url: qemu+ssh://root.com/system
      display_type: spice
      set_console_password: no
    server_url: "{{ sat_url }}"
    username: "{{ sat_user }}"
    password: "{{ sat_passwd }}"
    state: present

BTW: display_type is also undocumented and vnc is not really state of the art compared to spice.

Thanks,

Luc

--- Additional comment from  on 2022-09-09T15:58:25Z 

Verified in 6.12 snap 10

compute_resource docs are updated with a proper libvirt url and new display_type.

Steps to Reproduce:

1) #  ansible-doc -M /usr/share/ansible/collections/ansible_collections/redhat/satellite/plugins/modules compute_resource

Results:

- name: Create livirt compute resource
  redhat.satellite.compute_resource:
    name: example_compute_resource
    locations:
      - Munich
    organizations:
      - ACME
    provider: libvirt
    provider_params:
      url: qemu+ssh://root.com/system
      display_type: spice
    server_url: "https://satellite.example.com"
    username: "admin"
    password: "changeme"
    state: present


Notes:

Original link is also updated properly.

Comment 2 Griffin Sullivan 2022-10-11 15:19:09 UTC
Verified in 6.11.4 snap 1

compute_resource docs are updated with a proper libvirt url and new display_type.

Steps to Reproduce:

1) #  ansible-doc -M /usr/share/ansible/collections/ansible_collections/redhat/satellite/plugins/modules compute_resource

Results:

- name: Create livirt compute resource
  redhat.satellite.compute_resource:
    name: example_compute_resource
    locations:
      - Munich
    organizations:
      - ACME
    provider: libvirt
    provider_params:
      url: qemu+ssh://root.com/system
      display_type: spice
    server_url: "https://satellite.example.com"
    username: "admin"
    password: "changeme"
    state: present

Comment 7 errata-xmlrpc 2022-10-27 13:08:49 UTC
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 (Important: Satellite 6.11.4 Async Security Update), 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:7242