Skip to content

fix: fix agent group command by adding scheduler_cron_time#1345

Merged
greenbonebot merged 3 commits into
mainfrom
fix/agent-group-scheduler-cron-time
Jun 3, 2026
Merged

fix: fix agent group command by adding scheduler_cron_time#1345
greenbonebot merged 3 commits into
mainfrom
fix/agent-group-scheduler-cron-time

Conversation

@ozgen
Copy link
Copy Markdown
Member

@ozgen ozgen commented Jun 3, 2026

What

Added the mandatory scheduler_cron_time parameter to create and modify agent group commands and updated the related tests.

Why

This is needed because scheduler_cron_time is now required for agent group comments.

Checklist

  • Tests

Update the create and modify agent group command to include
the mandatory scheduler_cron_time parameter.
@ozgen ozgen requested a review from a team as a code owner June 3, 2026 09:11
@greenbonebot greenbonebot enabled auto-merge (rebase) June 3, 2026 09:11
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 3, 2026

Dependency Review

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

Snapshot Warnings

⚠️: No snapshots were found for the head SHA 317cafc.
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

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 3, 2026

Conventional Commits Report

Type Number
Bug Fixes 1

🚀 Conventional commits found.

Shut down the Unix socket test server before closing it and wait for the
server thread to finish.

This avoids a race where serve_forever can access a closed file descriptor
and the client can fail with BrokenPipeError in CI.
Comment thread tests/connections/test_unix_socket_connection.py Fixed
Comment thread tests/connections/test_unix_socket_connection.py Fixed
@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 3, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.28%. Comparing base (acf6ccf) to head (317cafc).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1345   +/-   ##
=======================================
  Coverage   98.28%   98.28%           
=======================================
  Files          99       99           
  Lines        6415     6421    +6     
  Branches     1146     1148    +2     
=======================================
+ Hits         6305     6311    +6     
  Misses         77       77           
  Partials       33       33           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
@greenbonebot greenbonebot merged commit 8dfcbef into main Jun 3, 2026
26 checks passed
@greenbonebot greenbonebot deleted the fix/agent-group-scheduler-cron-time branch June 3, 2026 09:56
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.

4 participants