Add SetSessionService to redfish_config

[tejabailey]

adding SetSessionService command to redfish_config
to set BMC default session timeout policy.

Fixes #5008

ISSUE TYPE

• Feature Pull Request

COMPONENT NAME

redfish_config

[tejabailey]

(venv) tj@C02GF4NZMD6R Tests % ANSIBLE_COLLECTIONS_PATH=/Users/tj/ansible ansible-playbook -i s2r8node65.s2r8, SetSessionServiceTests.yml
[WARNING]: Collection community.general does not support Ansible version 2.10.7

PLAY [all] **********************************************************************************************************************************************************************************************************************

TASK [Gather initial BMC lan information] ***************************************************************************************************************************************************************************************
ok: [s2r8node65.s2r8]

TASK [Set BMC IP address as fact] ***********************************************************************************************************************************************************************************************
ok: [s2r8node65.s2r8]

TASK [Set something for SessionService] *****************************************************************************************************************************************************************************************
changed: [s2r8node65.s2r8]

TASK [debug] ********************************************************************************************************************************************************************************************************************
ok: [s2r8node65.s2r8] => {
    "redfish_set_sessionservice": {
        "changed": true,
        "failed": false,
        "msg": "Modified SessionService"
    }
}

PLAY RECAP **********************************************************************************************************************************************************************************************************************
s2r8node65.s2r8            : ok=4    changed=1    unreachable=0    failed=0    skipped=0    rescued=0    ignored=0   

(venv) tj@C02GF4NZMD6R Tests % cat SetSessionServiceTests.yml 
---
- hosts: all
  gather_facts: false
  vars:
    bmc_username: root
    bmc_password: hunter2
    default_uri_timeout: 50
    default_uri_retries: 5
  tasks:
  - name: Gather initial BMC lan information
    command: ipmitool lan print
    register: ipmitool_output
    changed_when: ipmitool_output.rc != 0
    become: true

  - name: Set BMC IP address as fact
    set_fact:
      bmc_address: "{{ ipmitool_output.stdout | regex_search('IP Address\\s+:\\s+(\\d{1,3}\\.\\d{1,3}\\.\\d{1,3}\\.\\d{1,3})', '\\1')
                                              | first }}"

  - name: Set something for SessionService
    community.general.redfish_config:
      category: SessionService
      command: SetSessionService
      sessionservice_config:
        SessionTimeout: 800
      baseuri: "{{ bmc_address }}"
      username: "{{ bmc_username }}"
      password: "{{ bmc_password }}"
      timeout: "{{ default_uri_timeout }}"
    retries: "{{ default_uri_retries }}"
    register: redfish_set_sessionservice

  - debug:
      var: redfish_set_sessionservice

[ansibullbot]

cc @Bhavya06 @mraineri @rajeevkallur @renxulei @tomasg2012 @xmadsen
click here for bot help

[felixfontein]

Thanks for your contribution! Can you take a look at @mraineri's suggestion as well (https://github.com/ansible-collections/community.general/pull/5009/files#r930114637)? Thanks.

[russoz]

Hi @tejabailey thanks for your contribution!

I left a couple of comments.

[russoz]

@felixfontein I missed the discussion about this Docs Build feature in the PRs. Why do they get so long? It is quite annoying to work in the PR having to scroll multiple pages in the browser up and down.

[tejabailey]

@felixfontein working on it, thank you.

[tejabailey]

Hi! I The last pylint error has been corrected. Is this PR ok to merge now?

[felixfontein]

One last change request, then we can merge (assuming that @mraineri does not have further comments).

[mraineri]

One last change request, then we can merge (assuming that @mraineri does not have further comments).

I do not have anything else; everything looks good to me so far.

[felixfontein]

(The REUSE fail is because the test was changed to not rebase. That way I don't have to manually approve every run :) At least for this PR it can be safely ignored.)

[patchback]

Backport to stable-5: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-5/d9d830a1680f7dec4601b41200323a9e1534a4bb/pr-5009

Backported as #5311

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

[felixfontein]

@tejabailey thanks for this contribution!
@mraineri @russoz thanks for reviewing!