Skip to content

[action] [PR:818] [PMON] Bmcctld improvements & thermalctld: push thermals from Switch-Host --> BMC#70

Merged
mssonicbld merged 1 commit into
Azure:202608from
mssonicbld:cherry/msft-202608/818
Jul 1, 2026
Merged

[action] [PR:818] [PMON] Bmcctld improvements & thermalctld: push thermals from Switch-Host --> BMC#70
mssonicbld merged 1 commit into
Azure:202608from
mssonicbld:cherry/msft-202608/818

Conversation

@mssonicbld

Copy link
Copy Markdown
Collaborator

Changes in this PR


Fixes issue: sonic-net/sonic-platform-daemons#824

sonic-bmcctld/scripts/bmcctld

  • Conditional SWITCH_HOST_POWER_ON_DELAY - applied only on REBOOT_CAUSE_POWER_LOSS; skipped on warm/fast/soft reboot.
  • GracefulShutdownHandler.execute() cleaned up
  • Event-log de-duplication across handler functions
  • Rack manager alert : skip when incoming alert matches to existing alert level, prevents unnecessary pub/sub events
  • chassis module admin_status : skip when admin_status don't change, prevents unnecessary pub/sub events
  • Uses self.chassis.is_liquid_cooled() platform API.
  • Update the init_host_state API to derive the device_power_state from get_oper_status platform API which will either ONLINE/OFFLINE.
  • Set the default graceful_shutdown_timeout = 0 till we support GNOI graceful shutdown

sonic-thermalctld/scripts/thermalctld

  • EventLogger class added - stores critical events to syslog and /host/bmc/event.log.
  • Switch-Host ↔ BMC, thermals are pushed to BMC, via TEMPERATURE_INFO_STATE_DB table
    - critical temperature breaches logged into BMC event log.
  • LiquidCoolingUpdater improvements:
    - writes SYSTEM_LEAK_STATUS only on transitions
    - Per-sensor state updates also only on transitions
  • Added the Leak severity escalation logic (MINOR → CRITICAL after max_minor_duration_sec).
  • Critical leaks logged via event_logger.
  • Add logic to use self.chassis.is_liquid_cooled() also to identify liquid cooled platforms"

Signed-off-by: Sonic Build Admin sonicbld@microsoft.com

…Host --> BMC

***
**Changes in this PR**
***

Fixes issue: sonic-net/sonic-platform-daemons#824

**sonic-bmcctld/scripts/bmcctld**

- Conditional SWITCH_HOST_POWER_ON_DELAY - applied only on REBOOT_CAUSE_POWER_LOSS; skipped on warm/fast/soft reboot.
- GracefulShutdownHandler.execute() cleaned up
- Event-log de-duplication across handler functions
- Rack manager alert : skip when incoming alert matches to existing alert level, prevents unnecessary pub/sub events
- chassis module admin_status : skip when admin_status don't change, prevents unnecessary pub/sub events
- Uses self.chassis.is_liquid_cooled() platform API.
- Update the init_host_state API to derive the device_power_state from get_oper_status platform API which will either ONLINE/OFFLINE.
- Set the default graceful_shutdown_timeout = 0 till we support GNOI graceful shutdown

**sonic-thermalctld/scripts/thermalctld**

- EventLogger class added - stores critical events to syslog and /host/bmc/event.log.
- Switch-Host ↔ BMC, thermals are pushed to BMC, via TEMPERATURE_INFO_STATE_DB table
      - critical temperature breaches logged into BMC event log.
- LiquidCoolingUpdater improvements:
                 - writes SYSTEM_LEAK_STATUS only on transitions
                 - Per-sensor state updates also only on transitions
- Added the Leak severity escalation logic (MINOR → CRITICAL after max_minor_duration_sec).
- Critical leaks logged via event_logger.
- Add logic to use self.chassis.is_liquid_cooled() also to identify liquid cooled platforms"

Signed-off-by: Sonic Build Admin <sonicbld@microsoft.com>
@mssonicbld

Copy link
Copy Markdown
Collaborator Author

Original PR: sonic-net/sonic-platform-daemons#818

@mssonicbld

Copy link
Copy Markdown
Collaborator Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
No pipelines are associated with this pull request.

@mssonicbld mssonicbld merged commit 3d513c8 into Azure:202608 Jul 1, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant