Skip to content

bug(health): gate switch hosts and bmcs in spawn to avoid redfish cal…#1745

Open
mkoci wants to merge 1 commit into
NVIDIA:mainfrom
mkoci:bug_1744_no_redfish_for_switch_hosts
Open

bug(health): gate switch hosts and bmcs in spawn to avoid redfish cal…#1745
mkoci wants to merge 1 commit into
NVIDIA:mainfrom
mkoci:bug_1744_no_redfish_for_switch_hosts

Conversation

@mkoci
Copy link
Copy Markdown
Contributor

@mkoci mkoci commented May 17, 2026

…ls on switch hosts

Description

Fixes #1744

Type of Change

  • Add - New feature or capability
  • Change - Changes in existing functionality
  • Fix - Bug fixes
  • Remove - Removed features or deprecated functionality
  • Internal - Internal changes (refactoring, tests, docs, etc.)

Testing

  • Unit tests added/updated
  • Integration tests added/updated
  • Manual testing performed
  • No testing required (docs, internal refactor, etc.)

@mkoci mkoci requested a review from a team as a code owner May 17, 2026 02:28
@mkoci mkoci force-pushed the bug_1744_no_redfish_for_switch_hosts branch from acbce32 to 7f12d3a Compare May 17, 2026 02:31
@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot Bot commented May 17, 2026

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@mkoci mkoci force-pushed the bug_1744_no_redfish_for_switch_hosts branch from 7f12d3a to f645567 Compare May 17, 2026 02:33
…ls on switch hosts

Signed-off-by: mkoci <26286151+mkoci@users.noreply.github.com>
@mkoci mkoci force-pushed the bug_1744_no_redfish_for_switch_hosts branch from f645567 to 8ead8e7 Compare May 17, 2026 02:36
@yoks
Copy link
Copy Markdown
Contributor

yoks commented May 17, 2026

Why no Redfish for switch hosts? It is required to get leak data.

@yoks
Copy link
Copy Markdown
Contributor

yoks commented May 17, 2026

If Switch has different address for Redfish AND not Redfish interfaces, than it should be encoded somewhere else, or even better be separate Metadata, disabling redfish for all switches will disable cruical functionality

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug(health): Redfish SSE applies to Switch Hosts

2 participants