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

Fixity email report fields aren't pulling correct data #2568

Closed
carakey opened this issue Mar 11, 2024 · 7 comments
Closed

Fixity email report fields aren't pulling correct data #2568

carakey opened this issue Mar 11, 2024 · 7 comments

Comments

@carakey
Copy link

carakey commented Mar 11, 2024

Descriptive summary

Look into how the fixity checker is counting and reporting passing / failing files in its data fields. The last report had a discrepancy where it says zero files failed, but gives 14 IDs of failed files.

Expected behavior

Counts of passing and failing files are reported accurately in the fixity report email.

Actual behavior

The following is the fixity report emailed on on Feb 17, 2024:

Fixity Report
Hi there, below are the report on the Fixity Job that ran and the results after it has finished running:

Start Time: 2024-02-16 15:48:47 -0800
End Time: 2024-02-17 01:03:08 -0800
No. of Files [CHECKED]: 157268
No. of Files [PASSED]: 157268
No. of Files [FAILED]: 0
List of all Failed Files [IDs]:

kp78gg577
dj52wc82j
41687r184
br86bb654
qn59qb39p
8c97kx72p
1z40m1672
qv33s449d
pc289r65p
jm214w89m
v979v971k
vh53x358h
9p290h89n
hx11xp80b

  • Scholars Archive Support

Email Date & Time: February 17, 2024 - 1:03 AM

I'm reading this as "100% of files passed. Zero files failed. These 14 files failed." If the last thing is true, then the first two things are not.

Steps to reproduce the behavior

Fixity report emails are saved in ScholarsArchive email archive.

Related work

#2551 ; #2421

@carakey
Copy link
Author

carakey commented Apr 1, 2024

I don't have a way to QA without seeing the monthly email report.

@lamtu1
Copy link
Contributor

lamtu1 commented Apr 1, 2024

Hi @carakey , since it is already April, we can wait till the next report run this month to check it and make more revision after it.

@CGillen
Copy link
Contributor

CGillen commented Apr 15, 2024

Code is on production, fixity will run tonight

@carakey
Copy link
Author

carakey commented Apr 16, 2024

We got the fixity report last night and the field mismatch is still the same. The passed/failed counts are still off. Unless the changes didn't make it to production, this should get moved back to ready for the next local WC.

Fixity Report

Hi there, below are the report on the Fixity Job that ran and the results after it has finished running:
Start Time: 2024-04-15 13:00:11 -0700
End Time: 2024-04-15 18:53:08 -0700
No. of Files [CHECKED]: 157607
No. of Files [PASSED]: 157607
No. of Files [FAILED]: 0
List of all Failed Files [IDs]:
kp78gg577
dj52wc82j
br86bb654
qn59qb39p
8c97kx72p
1z40m1672
qv33s449d
pc289r65p
jm214w89m
v979v971k
vh53x358h
hx11xp80b

  • Scholars Archive Support
    Email Date & Time: April 15, 2024 - 6:53 PM

@carakey
Copy link
Author

carakey commented Jun 17, 2024

June fixity report still has this issue. Most failures have been resolved, but still one failed file ID and "No. of Files [Failed]: 0".

Fixity Report
Hi there, below are the report on the Fixity Job that ran and the results after it has finished running:
Start Time: 2024-06-15 13:00:09 -0700
End Time: 2024-06-15 19:42:31 -0700
No. of Files [CHECKED]: 158856
No. of Files [PASSED]: 158856
No. of Files [FAILED]: 0
List of all Failed Files [IDs]:
kp78gg577
-Scholars Archive Support
Email Date & Time: June 15, 2024 - 7:42 PM

@carakey
Copy link
Author

carakey commented Jun 21, 2024

I'm not sure how we can QA this except to go ahead and push to Production and wait for the next report to come in.

@lamtu1
Copy link
Contributor

lamtu1 commented Jun 21, 2024

That should be fine @carakey, we can wait until it is push to production and wait for the next time it run to check it out.

@CGillen CGillen closed this as completed Jul 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Development

No branches or pull requests

3 participants