Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

jobs_completed_total counter not covering all job completions #1297

Closed
hacktobeer opened this issue Jun 26, 2023 · 3 comments
Closed

jobs_completed_total counter not covering all job completions #1297

hacktobeer opened this issue Jun 26, 2023 · 3 comments
Assignees
Labels

Comments

@hacktobeer
Copy link
Collaborator

The jobs_completed_total counter is not implemented to cover all possible ways a job can be set to completed and as such is incorrectly representing the total jobs completed.

turbinia_jobs_completed_total = Gauge(

@hacktobeer hacktobeer self-assigned this Jun 26, 2023
@hacktobeer hacktobeer added the bug label Jun 26, 2023
@hacktobeer
Copy link
Collaborator Author

It looks like it does cover all jobs. Some jobs error out because eg a target file to analyse can not be found. This is WAI....but I think we can improve the code to not throw a job error but let the job finish with a relevant message instead of just a job error.

@aarontp
Copy link
Member

aarontp commented Jul 24, 2023

@hacktobeer Should we rename/update this issue to track the job error issue you mentioned then? Can you point to where this happens?

@hacktobeer
Copy link
Collaborator Author

Let's close this for now, as I changed the metrics type from Gauge to Counter, which should give a better representation of the different job metrics. I will test when the new release is deployed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants