Skip to content

Change/agent update list payload#1072

Merged
greenbonebot merged 4 commits into
mainfrom
change/agent-update-list-payload
May 19, 2026
Merged

Change/agent update list payload#1072
greenbonebot merged 4 commits into
mainfrom
change/agent-update-list-payload

Conversation

@ozgen
Copy link
Copy Markdown
Member

@ozgen ozgen commented May 19, 2026

What

  • Changed agent update payload handling to use a list of agent-specific updates.
  • Updated PATCH payload creation to build updates from agent_controller_agent_update_list_t.
  • Added agent_id and base config handling to each update entry.
  • Updated config merging so missing update values are filled from the base config.
  • Updated and extended unit tests for the new update-list payload flow.
  • Verified with the agent controller unit tests.

Why

  • The previous update flow used one shared update template for multiple agents.
  • The new structure allows each agent to have its own update values.
  • This makes the payload clearer and closer to the Agent Controller API format.
  • It also makes future changes easier, especially when different agents need different update data.

References

GEA-1791

Checklist

  • Tests

ozgen added 3 commits May 19, 2026 12:02
Change the agent update flow to build PATCH payloads from a list of
agent-specific updates.

Each update now carries its own agent ID, optional config override, base
config, authorization flag, and update-to-latest flag. This makes the payload
construction independent from the agent list and allows different updates to be
sent for different agents in the same request.

Also update config merging so missing override values are resolved from the
provided base config.
@ozgen ozgen requested review from a team as code owners May 19, 2026 10:14
@greenbonebot greenbonebot enabled auto-merge (rebase) May 19, 2026 10:14
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 19, 2026

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA f3221a2.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

Scanned Files

None

Comment thread agent_controller/agent_controller.c Outdated
Comment thread agent_controller/agent_controller.c Outdated
Comment thread agent_controller/agent_controller.c Outdated
Comment thread agent_controller/agent_controller.h Outdated
Comment thread agent_controller/agent_controller.c Outdated
@ozgen ozgen requested a review from timopollmeier May 19, 2026 12:34
@ozgen ozgen force-pushed the change/agent-update-list-payload branch from 07cd376 to f3221a2 Compare May 19, 2026 12:59
@greenbonebot greenbonebot merged commit 2764578 into main May 19, 2026
24 checks passed
@greenbonebot greenbonebot deleted the change/agent-update-list-payload branch May 19, 2026 13:04
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.

3 participants