Add AMD Ryzen AI Developer Platform as a supported Linux platform - #52635
Add AMD Ryzen AI Developer Platform as a supported Linux platform#52635allenhouchins wants to merge 2 commits into
Conversation
Resolves #52630 AMD Ryzen AI Developer Platform ships its own /etc/os-release with ID=amd-ryzen-ai-developer-platform (ID_LIKE=debian), so osquery reports that string as the host platform. Since HostLinuxOSs and HOST_LINUX_PLATFORMS gate nearly every Linux check, these hosts enrolled but had no disk space, IP/MAC, users, or software inventory, were missed by linux-scoped policies and labels, and lost Run script in the UI. Add the platform to HostLinuxOSs, HostDebPackageOSs (deb-based), and HOST_LINUX_PLATFORMS. Regenerate understanding-host-vitals.md. LUKS support is intentionally not added; the reference hardware runs plain btrfs with no encryption layer, matching Debian's current treatment.
GitHub Actions
Error
Fleet now recognizes these hosts as Linux, allowing for population of host vitals, application of policies and labels, and execution of scripts.
GitHub Actions
Inactive
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #52635 +/- ##
=======================================
Coverage 75.91% 75.91%
=======================================
Files 4102 4102
Lines 247984 247984
Branches 14100 14100
=======================================
+ Hits 188266 188269 +3
+ Misses 59542 59539 -3
Partials 176 176
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
🟢 Approval recommended
The change consistently updates backend/frontend platform allowlists and includes targeted regression tests covering the newly supported platform behaviors.
Warning
- Copilot's review of this pull request may be incomplete because some of the changed files are excluded by your Copilot content exclusion settings. See Excluding content from Copilot for details.
Pull request overview
Adds support for the amd-ryzen-ai-developer-platform Linux os_version.platform value so Fleet treats these hosts as Linux for detail queries and script actions, resolving missing vitals/users/software inventory for that distro.
Changes:
- Add
amd-ryzen-ai-developer-platformto backend Linux platform allowlists and DEB package platforms. - Add the same platform to the frontend Linux platform allowlist so the UI enables Linux-only actions (e.g. Run script).
- Add/extend Go and frontend tests to cover os_version ingest, platform mapping, script filtering, and the Run script dropdown.
File summaries
| File | Description |
|---|---|
| server/service/osquery_utils/queries_test.go | Adds regression tests for ingesting os_version + OS inventory rows for the AMD Ryzen AI Developer Platform. |
| server/fleet/hosts.go | Extends HostLinuxOSs and HostDebPackageOSs to recognize the new platform string. |
| server/fleet/hosts_test.go | Verifies platform-to-linux mapping and confirms LUKS remains unsupported for this platform. |
| server/datastore/mysql/scripts_test.go | Ensures Linux script filtering applies to the new platform (preventing Windows scripts from leaking into lists). |
| frontend/interfaces/platform.ts | Adds the new platform to HOST_LINUX_PLATFORMS so frontend Linux checks include it. |
| frontend/pages/hosts/details/HostDetailsPage/HostActionsDropdown/HostActionsDropdown.tests.tsx | Confirms “Run script” renders for this Debian-like Linux platform ID. |
| changes/52630-amd-ryzen-ai-developer-platform-linux-support | Not reviewable here (content excluded by policy); appears to be the required changes entry for user-visible behavior. |
| docs/Contributing/product-groups/orchestration/understanding-host-vitals.md | Not reviewable here (content excluded by policy); referenced as regenerated via go generate. |
Review details
Files excluded by content exclusion policy (2)
- changes/52630-amd-ryzen-ai-developer-platform-linux-support
- docs/Contributing/product-groups/orchestration/understanding-host-vitals.md
- Files reviewed: 6/8 changed files
- Comments generated: 0
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Team Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (7)
Included review availability: Your plan provides up to 8 included reviews per hour; 6 remain after this review. WalkthroughAdded AMD Ryzen AI Developer Platform to frontend and server Linux platform registries. Added DEB package support and Linux classification coverage. Added osquery ingestion tests for normalized platform data and separate inventory identity. Added tests for shell-script filtering and the Run script host action. Merge Risk: ⚪ Minimal · up to AMD Ryzen AI Developer Platform hosts will now receive Linux host details, software inventory, and supported script actions without an identified current-head merge risk. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Docstring CoverageExplanation Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 5 functions across 6 files. (1 skipped: 1 unsupported.)
✨ Finishing Touches 💡 1📝 Generate docstrings 💡
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Confirmed fixed: 
|
Related issue: Resolves #52630
Summary
AMD Ryzen AI Developer Platform ships its own
/etc/os-releasewithID=amd-ryzen-ai-developer-platform(ID_LIKE=debian), so osquery reports that string asos_version.platform. Fleet stores the raw value on the host and filters detail queries by exact match againstHostLinuxOSs, which did not include it. Every Linux-only detail query (network_interface_unix,disk_space_unix,users,os_unix_like,software_linux) was therefore never sent to these hosts, which explains all six gaps listed in the issue. Nothing is wrong on the device: running the underlying osquery tables directly on an affected host (osquery 5.23.1) returns data formounts,interface_addresses/interface_details,users, anddeb_packages.This follows the same pattern as the Omarchy fix (#50069, commit 60ad78f):
amd-ryzen-ai-developer-platformtoHostLinuxOSsandHostDebPackageOSs(the distro is deb-based).HOST_LINUX_PLATFORMSin the frontend so the UI treats the host as Linux and shows Run script.understanding-host-vitals.mdviago generate.os_versioningest using the exact row a real host reports, OS inventory row, script filtering by platform, and the Run script dropdown.Intentionally not added to
IsLUKSSupported/DISK_ENCRYPTION_SUPPORTED_LINUX_PLATFORMS: the reference hardware runs plain btrfs with no LUKS layer, and Debian itself is not in those lists today. No OS-inventory aggregation is needed either; the name and 1.0.0 version parse cleanly to their own row.Checklist for submitter
changes/,orbit/changes/oree/fleetd-chrome/changes.See Changes files for more information.
Testing
Ran locally (in containers, since the dev machine is the affected host and has no Go/yarn/MySQL):
go vet+gofmton touched packagesgo test ./server/fleet/ ./server/service/osquery_utils/(targeted new/updated tests)MYSQL_TEST=1 go test ./server/datastore/mysql/ -run TestScripts/GetHostScriptDetailstsc --noEmit,eslinton changed frontend files, and the fullHostActionsDropdownJest suite (108 tests)Manual QA still needed after deploy to dogfood: confirm disk space, private/public IP, MAC, users, and software populate on the enrolled AMD Ryzen AI Halo host.
Frontend
No visual changes; only the platform allowlist gained an entry.
Summary by CodeRabbit
New Features
Bug Fixes