Skip to content

docs: add RPC health checker documentation - #880

Open
DevipriyaS17 wants to merge 3 commits into
mainfrom
rpc_health
Open

DevipriyaS17 wants to merge 3 commits into
mainfrom
rpc_health

Conversation

@DevipriyaS17

Copy link
Copy Markdown
Contributor

What are you changing?

This PR adds a new RPC guide that explains how to use rpc status/health to assess Intel® AMT device readiness before activation and manageability after activation. The document includes Linux and Windows command examples, ACM and CCM readiness options, optional management endpoint reachability checks, AMT password behavior for WSMAN-dependent checks, health check result categories, JSON output structure, and troubleshooting guidance.

=== "Windows"
```cmd
rpc.exe status
rpc.exe health

@amarnath-ac amarnath-ac Sep 22, 2026 •

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Suggestion: add an example output or list out all the checks, with this user will know what all details will be displayed.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

added the output screenshot

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can you place health checker under v3(Beta) as this is available only for v3

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

moved the file under v3

Comment thread mkdocs.yml Outdated
- Remote Provisioning Client:
- Overview: Reference/RPC/overview.md
- Build RPC-Go Manually: Reference/RPC/buildRPC_Manual.md
- RPC Health Checker: Reference/RPC/healthChecker.md

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please this under V3(Beta)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

updated

Comment thread docs/Reference/RPC/v2/commandsRPC.md Outdated
For more information,
see [Wireless Activation](../../../Tutorials/createWiFiConfig.md).

### status

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please move this under v3(Beta) its not part of v2

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

updated

Comment thread docs/Reference/RPC/healthChecker.md Outdated
@@ -0,0 +1,216 @@
# RPC Health and Readiness Checker

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
# RPC Health and Readiness Checker
# RPC Health Checker

@sudhir-intc sudhir-intc Sep 24, 2026 •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We call every where as RPC Health Checker, we could keep it the same

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

updated accordingly

Comment thread docs/Reference/RPC/healthChecker.md Outdated
@@ -0,0 +1,216 @@
# RPC Health and Readiness Checker

Use the RPC Health and Readiness Checker to assess an Intel® AMT device before or after activation. The report identifies configured prerequisites, outstanding requirements, and conditions that may prevent AMT management operations from working as expected.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
Use the RPC Health and Readiness Checker to assess an Intel® AMT device before or after activation. The report identifies configured prerequisites, outstanding requirements, and conditions that may prevent AMT management operations from working as expected.
Use the RPC Health Checker to assess the readiness of an Intel® AMT device before or after activation. The report identifies configured prerequisites, outstanding requirements, and conditions that may prevent AMT management operations from working as expected.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

updated

Comment thread docs/Reference/RPC/healthChecker.md Outdated

## Post-activation checks and the AMT password

The checker does not prompt for an AMT password. For an activated device, provide `--password` when you want RPC to run the additional WSMAN checks that require the password:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

could you provide the list of WSMAN checks that would be done.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

updated with the list

Comment thread docs/Reference/RPC/healthChecker.md Outdated

Each item in `checks` contains a check name, status, and optional message. Status values are `pass`, `warn`, `fail`, `skip`, or `unavailable`.

Example:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

can you provide the same earlier three examples in text format in json over here.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

updated with examples in json format

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

instead of images it might be better to place the command output as it also helps to modify information easily

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

updated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

instead of images it might be better to place the command output as it also helps to modify information easily

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

updated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

instead of images it might be better to place the command output as it also helps to modify information easily

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

updated

@sudhir-intc

Copy link
Copy Markdown
Contributor

@DevipriyaS17 : As there were too many modifications with the text, I created an proposal in this branch rpc_health_impr , diff can be seen here : https://github.com/device-management-toolkit/docs/compare/rpc_health..rpc_health_impr
Please see if that looks good, you can take in the content.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

Resolve credential-handling guidance and align the documented JSON examples and schema descriptions.

Get a fresh assessment by requesting another Copilot review.

Review effort: Lite
Findings: 1 High severity · 3 Medium severity

Open (4)
What changed in this PR

Adds v3 RPC health-checker documentation covering readiness checks, credentials, endpoint reachability, output formats, and troubleshooting.

Changes:

  • Adds the health-checker guide with Linux and Windows examples.
  • Documents ACM/CCM checks, password handling, result categories, and JSON output.
  • Adds navigation and overview links.
File Summary
mkdocs.yml Adds the health-checker page to v3 navigation.
docs/​Reference/​RPC/​v3/​healthChecker.md Adds comprehensive health-checker documentation and examples.
docs/​Reference/​RPC/​overview.md Links to the new guide.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread docs/Reference/RPC/v3/healthChecker.md Outdated
Comment thread docs/Reference/RPC/v3/healthChecker.md Outdated
Comment thread docs/Reference/RPC/v3/healthChecker.md Outdated
Comment thread docs/Reference/RPC/v3/healthChecker.md Outdated

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

The documented commands are unavailable in the shipped v3 CLI, and credential-handling and result-status issues remain unresolved.

Get a fresh assessment by requesting another Copilot review.

Review effort: Lite
Findings: 1 High severity · 3 Medium severity · 1 Low severity

Open (5)
Resolved since last review (4)

Comment thread docs/Reference/RPC/v3/healthChecker.md
Comment thread docs/Reference/RPC/v3/healthChecker.md Outdated
Comment thread docs/Reference/RPC/v3/healthChecker.md
Comment thread docs/Reference/RPC/v3/healthChecker.md Outdated
Comment thread docs/Reference/RPC/v3/healthChecker.md Outdated

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

Unresolved command-reference inconsistencies and credential-exposure issues remain in the documentation.

Get a fresh assessment by requesting another Copilot review.

Review effort: Lite
Findings: 1 Medium severity

Open (1)
Resolved since last review (5)

Comment on lines +131 to +133
set "AMT_PASSWORD=<AMT-password>"
rpc.exe status
set "AMT_PASSWORD="
@DevipriyaS17
DevipriyaS17 marked this pull request as ready for review September 25, 2026 17:58

## Run the health checker

=== "Linux"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Do we need both linux/windows commands everywhere, generic rpc status is sufficient. If there is difference we can add as a note. ex: On Linux, use sudo. On Windows, open an Administrator Command Prompt. as note is sufficient.

This branch has not been deployed

No deployments
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.

5 participants