Skip to content

[GEA-1178] Add target_type to tasks table - #2994

Open
rdttmr wants to merge 66 commits into
mainfrom
GEA-1178_tasks_target_type
Open

[GEA-1178] Add target_type to tasks table#2994
rdttmr wants to merge 66 commits into
mainfrom
GEA-1178_tasks_target_type

Conversation

@rdttmr

@rdttmr rdttmr commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

What

Add target_type column to tasks table.
Remove agent_group, oci_image_target and web_application_target (+ *_location) columns from tasks table

This PR only refactors the "backend", it intentionally does not modify any APIs, refactors behavior or simplifies other low hanging fruit that could be achieved after the refactoring.
It only intents to swap out database columns, while not changing any behavior. (The old columns are not yet dropped, which is a future endeavor once the changes have been working in production)

One side effect of this is, that the task trend feature is now available for all target types.

Why

Modernize code- and database, make it easier to add different types of targets in the future.

References

GEA-1178

Comment thread src/manage_migrators.c
@rdttmr
rdttmr force-pushed the GEA-1178_tasks_target_type branch from 074efab to a65b25d Compare August 5, 2026 09:14
@rdttmr
rdttmr force-pushed the GEA-1178_tasks_target_type branch 2 times, most recently from e1c2968 to 7e0f1b7 Compare August 17, 2026 10:47
@rdttmr
rdttmr marked this pull request as ready for review August 19, 2026 07:57
@rdttmr
rdttmr requested review from a team as code owners August 19, 2026 07:57
@greenbonebot
greenbonebot enabled auto-merge (rebase) August 19, 2026 07:57
rdttmr added 23 commits August 19, 2026 10:03
…target, agent_group, oci_image_target, web_application_target)
@rdttmr
rdttmr force-pushed the GEA-1178_tasks_target_type branch from 7e0f1b7 to 9b70677 Compare August 19, 2026 08:04
Comment thread src/manage_pg.c Outdated
Comment thread src/manage_pg.c
Comment thread src/manage_tasks.h Outdated
@rdttmr
rdttmr requested a review from timopollmeier August 20, 2026 11:21
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