redfish sessioncache error when redfish_password is not set

Bug #2097019 reported by Dr. Jens Harbott
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ironic
Fix Released
Undecided
Unassigned

Bug Description

The redfish_password option is marked as optional, however when it is not being set, these errors occur:

2025-01-30 09:28:40.340 7 WARNING ironic.drivers.modules.redfish.utils [None req-b9241d02-253f-4a66-82ea-15c0fbc24d69 - - - - - -] For node 8f18eb51-07f3-434a-ba84-f121acab7821, we received at AttributeError when attempting to utilize the client. A new client session shall be used upon the next attempt.Attribute Error: 'NoneType' object has no attribute 'encode'
2025-01-30 09:28:40.341 7 WARNING ironic.conductor.manager [None req-b9241d02-253f-4a66-82ea-15c0fbc24d69 - - - - - -] Failed to get sensor data for node 8f18eb51-07f3-434a-ba84-f121acab7821. Error: Redfish exception occurred. Error: 'NoneType' object has no attribute 'encode': ironic.common.exception.RedfishError: Redfish exception occurred. Error: 'NoneType' object has no attribute 'encode'

Changed in ironic:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ironic (master)

Reviewed: https://review.opendev.org/c/openstack/ironic/+/940431
Committed: https://opendev.org/openstack/ironic/commit/91b656d31cac77817db8c865d68471d546903314
Submitter: "Zuul (22348)"
Branch: master

commit 91b656d31cac77817db8c865d68471d546903314
Author: Dr. Jens Harbott <email address hidden>
Date: Thu Jan 30 10:50:24 2025 +0100

    Fix redfish session cache on missing password

    The redfish_password option is optional, make sure that the SessionCache
    does not throw an error when it is not set.

    Closes-Bug: 2097019

    Change-Id: Idf792c982a883a4c07ae1dad72e3c54bc73b96a1

Changed in ironic:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ironic (stable/2024.2)

Fix proposed to branch: stable/2024.2
Review: https://review.opendev.org/c/openstack/ironic/+/940634

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ironic (stable/2024.1)

Fix proposed to branch: stable/2024.1
Review: https://review.opendev.org/c/openstack/ironic/+/940635

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ironic (stable/2023.2)

Fix proposed to branch: stable/2023.2
Review: https://review.opendev.org/c/openstack/ironic/+/940636

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ironic (stable/2024.2)

Reviewed: https://review.opendev.org/c/openstack/ironic/+/940634
Committed: https://opendev.org/openstack/ironic/commit/60248a0b85b8e0205d47d47f6a784ca014ac60d5
Submitter: "Zuul (22348)"
Branch: stable/2024.2

commit 60248a0b85b8e0205d47d47f6a784ca014ac60d5
Author: Dr. Jens Harbott <email address hidden>
Date: Thu Jan 30 10:50:24 2025 +0100

    Fix redfish session cache on missing password

    The redfish_password option is optional, make sure that the SessionCache
    does not throw an error when it is not set.

    Closes-Bug: 2097019

    Change-Id: Idf792c982a883a4c07ae1dad72e3c54bc73b96a1
    (cherry picked from commit 91b656d31cac77817db8c865d68471d546903314)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ironic (stable/2023.2)

Reviewed: https://review.opendev.org/c/openstack/ironic/+/940636
Committed: https://opendev.org/openstack/ironic/commit/a3b0149658f438ef4417bcdf006cdbba23688553
Submitter: "Zuul (22348)"
Branch: stable/2023.2

commit a3b0149658f438ef4417bcdf006cdbba23688553
Author: Dr. Jens Harbott <email address hidden>
Date: Thu Jan 30 10:50:24 2025 +0100

    Fix redfish session cache on missing password

    The redfish_password option is optional, make sure that the SessionCache
    does not throw an error when it is not set.

    Closes-Bug: 2097019

    Change-Id: Idf792c982a883a4c07ae1dad72e3c54bc73b96a1
    (cherry picked from commit 91b656d31cac77817db8c865d68471d546903314)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/ironic 28.0.0

This issue was fixed in the openstack/ironic 28.0.0 Epoxy release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ironic (stable/2024.1)

Reviewed: https://review.opendev.org/c/openstack/ironic/+/940635
Committed: https://opendev.org/openstack/ironic/commit/9b97ec3bb2cc06676264e73bed8592fd3e0eac54
Submitter: "Zuul (22348)"
Branch: stable/2024.1

commit 9b97ec3bb2cc06676264e73bed8592fd3e0eac54
Author: Dr. Jens Harbott <email address hidden>
Date: Thu Jan 30 10:50:24 2025 +0100

    Fix redfish session cache on missing password

    The redfish_password option is optional, make sure that the SessionCache
    does not throw an error when it is not set.

    Closes-Bug: 2097019

    Change-Id: Idf792c982a883a4c07ae1dad72e3c54bc73b96a1
    (cherry picked from commit 91b656d31cac77817db8c865d68471d546903314)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/ironic 23.0.4

This issue was fixed in the openstack/ironic 23.0.4 Bobcat release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/ironic 24.1.4

This issue was fixed in the openstack/ironic 24.1.4 Caracal release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/ironic 26.1.2

This issue was fixed in the openstack/ironic 26.1.2 Dalmatian release.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.