Skip to content

Comments

Change: Refactor GMP errors for import task and extend filter_clause#2666

Merged
greenbonebot merged 8 commits intomainfrom
fix-gmp-errors-for-import-task-filter-clause-agent-oci
Dec 4, 2025
Merged

Change: Refactor GMP errors for import task and extend filter_clause#2666
greenbonebot merged 8 commits intomainfrom
fix-gmp-errors-for-import-task-filter-clause-agent-oci

Conversation

@ozgen
Copy link
Member

@ozgen ozgen commented Dec 3, 2025

What

  • Updated GMP error messages to refer to import task instead of container task.
  • Extended filter_clause to support agent_group and oci_image_target.

Why

  • The previous wording in GMP errors was outdated and caused confusion, so aligning the terminology improves clarity.
  • agent_group and oci_image_target were not fully supported in filter_clause, which led to incorrect or incomplete filtering.

References

GEA-1406

@ozgen ozgen requested review from a team as code owners December 3, 2025 13:47
@greenbonebot greenbonebot enabled auto-merge (rebase) December 3, 2025 13:47
@ozgen ozgen force-pushed the fix-gmp-errors-for-import-task-filter-clause-agent-oci branch from 9e72f91 to 20ca230 Compare December 4, 2025 10:56
@ozgen ozgen requested a review from mattmundell December 4, 2025 11:11
Copy link
Contributor

@mattmundell mattmundell left a comment

Choose a reason for hiding this comment

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

I see another one in the return doc at manage_sql.c:9943.

@ozgen ozgen force-pushed the fix-gmp-errors-for-import-task-filter-clause-agent-oci branch from 52321f4 to 1495314 Compare December 4, 2025 11:59
Co-authored-by: Matt Mundell <32057441+mattmundell@users.noreply.github.com>
@ozgen ozgen force-pushed the fix-gmp-errors-for-import-task-filter-clause-agent-oci branch 2 times, most recently from d1b7a5e to 79a82c4 Compare December 4, 2025 12:05
Co-authored-by: Matt Mundell <32057441+mattmundell@users.noreply.github.com>
@ozgen ozgen force-pushed the fix-gmp-errors-for-import-task-filter-clause-agent-oci branch from 79a82c4 to d68abcf Compare December 4, 2025 12:06
@ozgen ozgen requested a review from mattmundell December 4, 2025 12:07
Copy link
Contributor

@mattmundell mattmundell left a comment

Choose a reason for hiding this comment

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

Last couple I can see.

ozgen and others added 3 commits December 4, 2025 13:41
Co-authored-by: Matt Mundell <32057441+mattmundell@users.noreply.github.com>
Co-authored-by: Matt Mundell <32057441+mattmundell@users.noreply.github.com>
…ilter-clause-agent-oci' into fix-gmp-errors-for-import-task-filter-clause-agent-oci
@ozgen ozgen requested a review from mattmundell December 4, 2025 12:43
@greenbonebot greenbonebot merged commit fd5794c into main Dec 4, 2025
29 of 30 checks passed
@greenbonebot greenbonebot deleted the fix-gmp-errors-for-import-task-filter-clause-agent-oci branch December 4, 2025 12:48
@mattmundell
Copy link
Contributor

@ozgen if you have a gap at any point, I'd like a review of /pull/2662.

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