Project

General

Profile

Actions

Bug #37486

closed

Maximum sessions limit reached on iDRAC using Redfish as BMC provider

Added by Romuald Conty 5 months ago. Updated about 2 months ago.

Status:
Closed
Priority:
Normal
Category:
BMC
Target version:
-
Difficulty:
Triaged:
Yes
Fixed in Releases:
Found in Releases:

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.

Actions #1

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
Actions #2

Updated by Romuald Conty 5 months ago

  • Description updated (diff)
Actions #3

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
Actions #4

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
Actions #5

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)
Actions #6

Updated by The Foreman Bot 3 months ago

  • Pull request https://github.com/theforeman/smart-proxy/pull/895 added
Actions #7

Updated by The Foreman Bot 3 months ago

  • Fixed in Releases 3.12.0 added
Actions #8

Updated by Romuald Conty 3 months ago

  • Status changed from Ready For Testing to Closed
Actions #9

Updated by Ewoud Kohl van Wijngaarden about 2 months ago

  • Category set to BMC
  • Triaged changed from No to Yes
Actions

Also available in: Atom PDF