Skip to content

platform: add fault description leaf and ACTION_FW_REPROGRAM #1455

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
brianneville wants to merge 8 commits into
base: master
Choose a base branch
from
+33 −1
Open

platform: add fault description leaf and ACTION_FW_REPROGRAM #1455

Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
877a4e4
platform: add ACTION_FW_REPROGRAM
brianneville Mar 23, 2026
bf9bbb5
update description to be more general, but note chassis as an example
brianneville Mar 23, 2026
104aa6d
add description leaf
brianneville Mar 23, 2026
b35c401
update revision
brianneville Mar 23, 2026
cdf2911
update description on ACTION_FW_REPROGRAM
brianneville Mar 23, 2026
90dc063
update description on ACTION_FW_REPROGRAM
brianneville Mar 23, 2026
8589174
add semicolon in description
brianneville Mar 30, 2026
a3337db
Merge branch 'master' into fault-action
dplore May 8, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
34 changes: 33 additions & 1 deletion release/models/platform/openconfig-platform-healthz-fault.yang
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,13 @@ module openconfig-platform-healthz-fault {
description
"This model defines device reported fault";

oc-ext:openconfig-version "0.1.0";
oc-ext:openconfig-version "0.1.1";
Copy link
Copy Markdown
Member

@dplore dplore May 12, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since we are adding yang, this should be a minor version increment per semver rules

Suggested change
oc-ext:openconfig-version "0.1.1";
oc-ext:openconfig-version "0.2.0";


revision "2026-03-23" {
description
"Add remediation description and ACTION_FW_REPROGRAM";
reference "0.1.1";
}

revision "2025-08-05" {
description
Expand Down Expand Up @@ -126,6 +132,26 @@ module openconfig-platform-healthz-fault {
"Replace component.";
}

identity ACTION_FW_REPROGRAM {
Comment thread
brianneville marked this conversation as resolved.
base ACTION_BASE;
description
"Force-reprogram the firmware for the targeted component.
The targeted component may need to be powercycled
for the reprogramming to take effect.
Comment on lines +139 to +140
Copy link
Copy Markdown
Member

@dplore dplore May 12, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Recommend updating to say something like:

if a power cycle is required, then ACTION_POWER_CYCLE will also be set as a remediations/remediation/action.


In comparison to ACTION_FACTORY_RESET action,
ACTION_FW_REPROGRAM does not necessarily involve resetting
the firmware back to its factory default. It also does not
involve resetting any certificates on disk or removing
any config files or OS images.
Comment on lines +142 to +146
Copy link
Copy Markdown
Member

@dplore dplore May 12, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you elaborate on where the firmware is stored? Is this reloading firmware stored in the device filesystem? In particular are there any other actions that may need to be performed for ACTION_FW_REPROGRAM ?


E.g. for the Chassis component this could mean
force-reprogramming all FPGAs and reloading
all drivers on the Chassis.
This reprogramming would be to the currently-installed
version, not the factory default version.";
}

grouping platform-health-faults {
description
"Grouping containing a component's active faults";
Expand Down Expand Up @@ -295,6 +321,12 @@ module openconfig-platform-healthz-fault {
disruptive.";
}

leaf description {
type string;
description
"Description for the remediating action";
}

leaf action {
description
"The specific action to be performed";
Expand Down
Loading