Skip to content

[BlackboxBenchmarking] Remove dry run arg from aggregate_fuzzer_stats cron#5271

Merged
ViniciustCosta merged 1 commit into
masterfrom
dylanj/cron_args
May 12, 2026
Merged

[BlackboxBenchmarking] Remove dry run arg from aggregate_fuzzer_stats cron#5271
ViniciustCosta merged 1 commit into
masterfrom
dylanj/cron_args

Conversation

@dylanjew

@dylanjew dylanjew commented May 12, 2026

Copy link
Copy Markdown
Collaborator

removes dry run args and makes args optional in aggregate_fuzzer_stats cron job

The bot version of run_cron.py does not accept any args, while the local run_cron.py does accept args.

I previously used required args and this caused the cron job to fail in dev after I pushed the helm config.

Unhandled exception in cron: main() missing 1 required positional argument: 'argv'
Traceback (most recent call last):
  File "/mnt/scratch0/clusterfuzz/src/python/bot/startup/run_cron.py", line 68, in main
    return 0 if task_module.main() else 1
                ^^^^^^^^^^^^^^^^^^
TypeError: main() missing 1 required positional argument: 'argv'

@dylanjew dylanjew requested a review from a team as a code owner May 12, 2026 15:29
@dylanjew dylanjew requested a review from aakallam May 12, 2026 15:30
@dylanjew dylanjew force-pushed the dylanj/cron_args branch from 4ca856a to 614fe9c Compare May 12, 2026 15:42
@dylanjew dylanjew changed the title Remove dry run arg from aggregate_fuzzer_stats cron [BlackboxBenchmarking] Remove dry run arg from aggregate_fuzzer_stats cron May 12, 2026
@dylanjew dylanjew requested a review from ViniciustCosta May 12, 2026 16:12

@ViniciustCosta ViniciustCosta left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Yeah, the local run_cron.py is different from the prod one. I think this was a recent change unfortunatelly.

@ViniciustCosta ViniciustCosta merged commit 1cde4b3 into master May 12, 2026
10 checks passed
@ViniciustCosta ViniciustCosta deleted the dylanj/cron_args branch May 12, 2026 17:49
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