flux-framework / flux-accounting

bank/accounting interface for the Flux resource manager
https://flux-framework.readthedocs.io/projects/flux-accounting/en/latest/index.html
GNU Lesser General Public License v3.0
3 stars 10 forks source link

fetch-job-records: set `max_entries=0` #516

Closed cmoussa1 closed 1 month ago

cmoussa1 commented 1 month ago

Problem

flux account-fetch-job-records uses flux.job.job_list_inactive() to fetch all jobs that have finished since a certain time. However, it does not set a value for max_entries, which defaults to 1000. This would mean that it could miss fetching certain jobs if more than 1000 completed since a certain time.


This PR sets max_entries to 0 to get all jobs that have finished since a certain timestamp.

Fixes #515

cmoussa1 commented 1 month ago

Thanks! Setting MWP here