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

job-archive: improve logging on parse job error #4422

Merged

Conversation

chu11
Copy link
Member

@chu11 chu11 commented Jul 23, 2022

Problem: The error

job-archive.err[0]: job_info_lookup_continuation: parse job

was recently observed, which did not provide insight into the deeper
problem.

Solution: Improve the error message by using json_unpack_ex() to
get parsing error information. Also add the job ID to the error
output.

Fixes #4403

Problem: The error

job-archive.err[0]: job_info_lookup_continuation: parse job

was recently observed, which did not provide insight into the deeper
problem.

Solution: Improve the error message by using json_unpack_ex() to
get parsing error information.  Also add the job ID to the error
output.

Fixes flux-framework#4403
@codecov
Copy link

codecov bot commented Jul 23, 2022

Codecov Report

Merging #4422 (22bfcf2) into master (4c2ba65) will decrease coverage by 0.00%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##           master    #4422      +/-   ##
==========================================
- Coverage   83.32%   83.32%   -0.01%     
==========================================
  Files         395      395              
  Lines       66644    66648       +4     
==========================================
+ Hits        55529    55532       +3     
- Misses      11115    11116       +1     
Impacted Files Coverage Δ
src/modules/job-archive/job-archive.c 62.62% <50.00%> (-0.18%) ⬇️
src/broker/overlay.c 86.38% <0.00%> (-0.42%) ⬇️
src/common/libsubprocess/subprocess.c 87.89% <0.00%> (-0.30%) ⬇️
src/common/libterminus/terminus.c 85.82% <0.00%> (-0.25%) ⬇️
src/cmd/flux-job.c 87.29% <0.00%> (-0.14%) ⬇️
src/broker/state_machine.c 81.70% <0.00%> (+0.79%) ⬆️
src/common/libpmi/pmi2.c 89.87% <0.00%> (+1.26%) ⬆️
src/common/libpmi/simple_client.c 87.21% <0.00%> (+1.82%) ⬆️

Copy link
Member

@garlick garlick left a comment

Choose a reason for hiding this comment

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

Good improvement, thanks!

@mergify mergify bot merged commit e5da9e9 into flux-framework:master Jul 25, 2022
@chu11 chu11 deleted the issue4403_job_archive_parse_job branch July 25, 2022 21:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

job-archive.err[0]: job_info_lookup_continuation: parse job
2 participants