Bug #37486
closedMaximum sessions limit reached on iDRAC using Redfish as BMC provider
Description
Foreman smart-proxy uses `redfish_client` gem as backend to implement the Redfish implementation for BMC.
`redfish_client` is a wrapper around the Redfish REST API available on BMC.
This client automatically authenticates the connection using basic auth, then retrieve a Redfish's session which is a token used in Redfish implementation.
Redfish specifications allow single authenticated calls but prefer the session opening before running multiple commands.
Basically, multiple commands should be ran using this scheme:
1. login: ie. retrieve a token (session) using basic auth
2. send many commands using the token in headers
3. logout: ie. request the sessions endpoint to delete the current one
But in Foreman's implementation, we login but never logout.
As side effect, when I go to BMC tab on (old) host UI, many connections to BMC are opened, each one produce a new session and when sessions limit is reached, BMC info can not be retrieved.
To test, I used iDRAC 8 using latest firmware (2.86.86.86) which allow me to debug the session creation by Foreman's Smart Proxy, internally made by `redfish_client`.
PS: I already have some working code and will provide a PR.
Updated by Romuald Conty 5 months ago
- Subject changed from Maximum sessions limit reach on iDRAC using Redfish as BMC provider to Maximum sessions limit reached on iDRAC using Redfish as BMC provider
Updated by The Foreman Bot 5 months ago
- Status changed from New to Ready For Testing
- Pull request https://github.com/theforeman/smart-proxy/pull/895 added
Updated by The Foreman Bot 5 months ago
- Assignee set to Ewoud Kohl van Wijngaarden
- Pull request https://github.com/theforeman/smart-proxy/pull/896 added
Updated by The Foreman Bot 3 months ago
- Pull request deleted (
https://github.com/theforeman/smart-proxy/pull/895, https://github.com/theforeman/smart-proxy/pull/896)
Updated by The Foreman Bot 3 months ago
- Pull request https://github.com/theforeman/smart-proxy/pull/895 added
Updated by Romuald Conty 3 months ago
- Status changed from Ready For Testing to Closed
Applied in changeset d287e3cc01698852c9f6ff85eac246263d01a5b3.
Updated by Ewoud Kohl van Wijngaarden about 1 month ago
- Category set to BMC
- Triaged changed from No to Yes