feat(core): per-agent add-on status read model (#3425) #3455

Merged

mfreeman451 merged 2 commits from feat/native-addon-edge-ops into staging 2026-05-29 17:18:15 +00:00

Conversation 1 Commits 2 Files changed 14 +823 -3

mfreeman451 commented 2026-05-29 15:58:40 +00:00

Owner

Copy link

Implements task 7.2 of add-native-addon-edge-ops: the control plane now records the per-add-on status agents report. Previously the agent capability status (service_name 'agent') was received and dropped; StatusHandler now routes it to AddonStatusIngestor, which decodes the payload, extracts the addon: sidecar entries (ignoring real sidecars like netprobe), and upserts a ServiceRadar.Plugins.AddonStatus row per add-on keyed by {agent_uid, addon_id} with state, active, degradation_reason, pid, restart_count, and last_health_at.

AddonStatus lives in the platform schema (migration 20260529010000) and is registered in the Plugins domain, so Edge Ops can reconcile desired assignments against observed state. version/arch columns exist but are nullable: the agent does not yet report them per add-on (task 7.1, agent-side enrichment); they can be populated without a schema change once it does. Rust-SRQL entity registration for addon_statuses is a follow-up.

Tests (addon_status_ingestor_test.exs, verified against the srql-fixtures CNPG cluster, 2 tests / 0 failures): addon sidecars parsed into the read model, non-addon sidecars ignored, re-ingest upserts the existing row. mix compile --warnings-as-errors green; openspec validate --strict green.

Co-Authored-By: Claude Opus 4.8 noreply@anthropic.com

IMPORTANT: Please sign the Developer Certificate of Origin

Thank you for your contribution to ServiceRadar. Please note, when contributing, the developer must include
a DCO sign-off statement indicating the DCO acceptance in one commit message. Here
is an example DCO Signed-off-by line in a commit message:

Signed-off-by: J. Doe <j.doe@domain.com>

Describe your changes

Issue ticket number and link

Code checklist before requesting a review

• I have signed the DCO?
• The build completes without errors?
• All tests are passing when running make test?

Implements task 7.2 of add-native-addon-edge-ops: the control plane now records the per-add-on status agents report. Previously the agent capability status (service_name 'agent') was received and dropped; StatusHandler now routes it to AddonStatusIngestor, which decodes the payload, extracts the addon:<id> sidecar entries (ignoring real sidecars like netprobe), and upserts a ServiceRadar.Plugins.AddonStatus row per add-on keyed by {agent_uid, addon_id} with state, active, degradation_reason, pid, restart_count, and last_health_at. AddonStatus lives in the platform schema (migration 20260529010000) and is registered in the Plugins domain, so Edge Ops can reconcile desired assignments against observed state. version/arch columns exist but are nullable: the agent does not yet report them per add-on (task 7.1, agent-side enrichment); they can be populated without a schema change once it does. Rust-SRQL entity registration for addon_statuses is a follow-up. Tests (addon_status_ingestor_test.exs, verified against the srql-fixtures CNPG cluster, 2 tests / 0 failures): addon sidecars parsed into the read model, non-addon sidecars ignored, re-ingest upserts the existing row. mix compile --warnings-as-errors green; openspec validate --strict green. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> ## IMPORTANT: Please sign the Developer Certificate of Origin Thank you for your contribution to ServiceRadar. Please note, when contributing, the developer must include a [DCO sign-off statement]( https://developercertificate.org/) indicating the DCO acceptance in one commit message. Here is an example DCO Signed-off-by line in a commit message: ``` Signed-off-by: J. Doe <j.doe@domain.com> ``` ## Describe your changes ## Issue ticket number and link ## Code checklist before requesting a review - [ ] I have signed the DCO? - [ ] The build completes without errors? - [ ] All tests are passing when running make test?

mfreeman451 added 1 commit 2026-05-29 15:58:40 +00:00

feat(core): per-agent add-on status read model (#3425) ... 32f73c3988

Implements task 7.2 of add-native-addon-edge-ops: the control plane now records the per-add-on status agents report. Previously the agent capability status (service_name 'agent') was received and dropped; StatusHandler now routes it to AddonStatusIngestor, which decodes the payload, extracts the addon:<id> sidecar entries (ignoring real sidecars like netprobe), and upserts a ServiceRadar.Plugins.AddonStatus row per add-on keyed by {agent_uid, addon_id} with state, active, degradation_reason, pid, restart_count, and last_health_at.

AddonStatus lives in the platform schema (migration 20260529010000) and is registered in the Plugins domain, so Edge Ops can reconcile desired assignments against observed state. version/arch columns exist but are nullable: the agent does not yet report them per add-on (task 7.1, agent-side enrichment); they can be populated without a schema change once it does. Rust-SRQL entity registration for addon_statuses is a follow-up.

Tests (addon_status_ingestor_test.exs, verified against the srql-fixtures CNPG cluster, 2 tests / 0 failures): addon sidecars parsed into the read model, non-addon sidecars ignored, re-ingest upserts the existing row. mix compile --warnings-as-errors green; openspec validate --strict green.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

mfreeman451 added 1 commit 2026-05-29 16:24:06 +00:00

feat(srql): add addon_statuses SRQL entity (#3425) ... dc9899d7ad

Completes add-native-addon-edge-ops task 1.2: per-agent add-on status is now queryable via SRQL (e.g. in:addon_statuses agent_uid:agent-1 state:unhealthy). SRQL runs in-process in web-ng as a Rustler NIF over the rust/srql crate (the NIF translates SRQL -> SQL; Elixir executes it), so the entity is registered on the live translate_request path.

Adds Entity::AddonStatuses + parse mapping (parser.rs), the addon_statuses Diesel table (schema.rs, unqualified like ocsf_agents so the platform schema resolves via search_path), AddonStatusRow (models.rs), and a query/addon_statuses.rs executor mirroring agents.rs (text filters agent_uid/addon_id/state/version/arch, reported_at time-range + default desc ordering). Wired into both exhaustive dispatch matches (translate_request used by the NIF, execute_query used by the standalone server) and the viz metadata match.

Verified with cargo test (translation is pure, no DB): the example test asserts in:addon_statuses translates to SQL against addon_statuses with the expected filters/ordering; full srql lib suite 179/0. cargo fmt + clippy clean. The prebuilt NIF .so is not git-tracked (built in CI from this source).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

mfreeman451 force-pushed feat/native-addon-edge-ops from dc9899d7ad to b628716eaa 2026-05-29 17:16:28 +00:00 Compare

mfreeman451 reviewed 2026-05-29 17:17:57 +00:00

mfreeman451 left a comment

Copy link

lgtm

lgtm

mfreeman451 merged commit 35d7d0658f into staging 2026-05-29 17:18:15 +00:00

mfreeman451 referenced this pull request from a commit 2026-05-29 17:18:17 +00:00

Merge pull request 'feat(core): per-agent add-on status read model (#3425)' (#3455) from feat/native-addon-edge-ops into staging

mfreeman451 referenced this pull request 2026-05-29 17:19:49 +00:00

Tracker: host-network-visibility + native add-on framework OpenSpec proposals #3456

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

1week

  

2weeks

  

Failed compliance check

  

IP cameras

  

NATS

NATS JetStream

  

Possible security concern

  

Review effort 1/5

  

Review effort 2/5

  

Review effort 3/5

  

Review effort 4/5

  

Review effort 5/5

  

UI

  

aardvark

  

accessibility

  

amd64

  

api

  

arm64

  

auth

  

back-end

  

bgp

  

blog

  

bug

Something isn't working

  

build

  

checkers

  

ci-cd

continuous integration-continuous deployments

  

cleanup

  

cnpg

cloud-native postgres

  

codex

  

core

core service

  

dependencies

Pull requests that update a dependency file

  

device-management

  

documentation

Improvements or additions to documentation

  

duplicate

This issue or pull request already exists

  

dusk

  

ebpf

  

enhancement

New feature or request

  

eta 1d

  

eta 1hr

  

eta 3d

  

eta 3hr

  

feature

  

fieldsurvey

  

github_actions

Pull requests that update GitHub Actions code

  

go

Pull requests that update Go code

  

good first issue

Good for newcomers

  

help wanted

Extra attention is needed

  

invalid

This doesn't seem right

  

javascript

Pull requests that update Javascript code

  

k8s

  

log-collector

  

mapper

  

mtr

multi traceroute

  

needs-triage

  

netflow

  

network-sweep

  

observability

  

oracle

Oracle Linux related issues

  

otel

opentelemetry logs, traces, metrics

  

plug-in

  

proton

timeplus proton streaming database

  

python

  

question

Further information is requested

  

reddit

  

redhat

  

research

  

rperf

  

rperf-checker

  

rust

Pull requests that update rust code

  

sdk

  

security

  

serviceradar-agent

  

serviceradar-agent-gateway

  

serviceradar-web

  

serviceradar-web-ng

  

siem

  

snmp

  

sysmon

  

topology

  

ubiquiti

  

wasm

  

wontfix

This will not be worked on

  

zen-engine

No labels

1week

2weeks

Failed compliance check

IP cameras

NATS

Possible security concern

Review effort 1/5

Review effort 2/5

Review effort 3/5

Review effort 4/5

Review effort 5/5

UI

aardvark

accessibility

amd64

api

arm64

auth

back-end

bgp

blog

bug

build

checkers

ci-cd

cleanup

cnpg

codex

core

dependencies

device-management

documentation

duplicate

dusk

ebpf

enhancement

eta 1d

eta 1hr

eta 3d

eta 3hr

feature

fieldsurvey

github_actions

go

good first issue

help wanted

invalid

javascript

k8s

log-collector

mapper

mtr

needs-triage

netflow

network-sweep

observability

oracle

otel

plug-in

proton

python

question

reddit

redhat

research

rperf

rperf-checker

rust

sdk

security

serviceradar-agent

serviceradar-agent-gateway

serviceradar-web

serviceradar-web-ng

siem

snmp

sysmon

topology

ubiquiti

wasm

wontfix

zen-engine

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

carverauto/serviceradar!3455

No description provided.