Skip to content

Conversation

@yarikoptic
Copy link
Member

@yarikoptic yarikoptic commented Jul 24, 2025

Add error handling to gracefully skip CircleCI jobs that return 404 when attempting to fetch v1.1 job details. This prevents crashes when some jobs are no longer accessible via the API.

Fixes #210

🤖 Generated with Claude Code

TODOs


(venv) (tinuous-dev) datalad@smaug:/mnt/datasets/datalad/ci/afni$ grep -i warn /tmp/afni-tinuous-2025.07.25T11.27.01-*
/tmp/afni-tinuous-2025.07.25T11.27.01-stderr:2025-07-25T11:27:17-0400 [WARNING ] tinuous: Job 26170 returned 404; skipping logs
/tmp/afni-tinuous-2025.07.25T11.27.01-stderr:2025-07-25T11:27:17-0400 [WARNING ] tinuous: Job 26167 returned 404; skipping logs
/tmp/afni-tinuous-2025.07.25T11.27.01-stderr:2025-07-25T11:27:17-0400 [WARNING ] tinuous: Job 26169 returned 404; skipping logs
/tmp/afni-tinuous-2025.07.25T11.27.01-stderr:2025-07-25T11:27:17-0400 [WARNING ] tinuous: Job 26168 returned 404; skipping logs
/tmp/afni-tinuous-2025.07.25T11.27.01-stderr:2025-07-25T11:27:49-0400 [WARNING ] tinuous: Job 26082 returned 404; skipping logs
/tmp/afni-tinuous-2025.07.25T11.27.01-stderr:2025-07-25T11:27:49-0400 [WARNING ] tinuous: Job 26083 returned 404; skipping logs
/tmp/afni-tinuous-2025.07.25T11.27.01-stderr:2025-07-25T11:27:49-0400 [WARNING ] tinuous: Job 26084 returned 404; skipping logs
/tmp/afni-tinuous-2025.07.25T11.27.01-stderr:2025-07-25T11:27:49-0400 [WARNING ] tinuous: Job 26087 returned 404; skipping logs
/tmp/afni-tinuous-2025.07.25T11.27.01-stderr:2025-07-25T11:27:49-0400 [WARNING ] tinuous: Job 26085 returned 404; skipping logs
/tmp/afni-tinuous-2025.07.25T11.27.01-stderr:2025-07-25T11:27:49-0400 [WARNING ] tinuous: Job 26086 returned 404; skipping logs
/tmp/afni-tinuous-2025.07.25T11.27.01-stderr:2025-07-25T11:27:49-0400 [WARNING ] tinuous: Job 26088 returned 404; skipping logs
/tmp/afni-tinuous-2025.07.25T11.27.01-stderr:2025-07-25T11:27:51-0400 [WARNING ] tinuous: Job 26076 returned 404; skipping logs
/tmp/afni-tinuous-2025.07.25T11.27.01-stderr:2025-07-25T11:27:51-0400 [WARNING ] tinuous: Job 26077 returned 404; skipping logs
/tmp/afni-tinuous-2025.07.25T11.27.01-stderr:2025-07-25T11:27:51-0400 [WARNING ] tinuous: Job 26078 returned 404; skipping logs
/tmp/afni-tinuous-2025.07.25T11.27.01-stderr:2025-07-25T11:27:51-0400 [WARNING ] tinuous: Job 26079 returned 404; skipping logs
/tmp/afni-tinuous-2025.07.25T11.27.01-stderr:2025-07-25T11:27:51-0400 [WARNING ] tinuous: Job 26080 returned 404; skipping logs

@codecov
Copy link

codecov bot commented Jul 24, 2025

Codecov Report

❌ Patch coverage is 37.50000% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 65.39%. Comparing base (3323d63) to head (7a0283d).
⚠️ Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
src/tinuous/circleci.py 37.50% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #211      +/-   ##
==========================================
+ Coverage   64.85%   65.39%   +0.53%     
==========================================
  Files          10       10              
  Lines        1383     1390       +7     
  Branches      211      212       +1     
==========================================
+ Hits          897      909      +12     
+ Misses        405      400       -5     
  Partials       81       81              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Add error handling to gracefully skip CircleCI jobs that return 404
when attempting to fetch v1.1 job details. This prevents crashes
when some jobs are no longer accessible via the API.

Fixes #210

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

Trying it out on AFNI lead to many jobs skipping:

	(venv) (tinuous-dev) datalad@smaug:/mnt/datasets/datalad/ci/afni$ grep -i warn /tmp/afni-tinuous-2025.07.25T11.27.01-*
	/tmp/afni-tinuous-2025.07.25T11.27.01-stderr:2025-07-25T11:27:17-0400 [WARNING ] tinuous: Job 26170 returned 404; skipping logs
	/tmp/afni-tinuous-2025.07.25T11.27.01-stderr:2025-07-25T11:27:17-0400 [WARNING ] tinuous: Job 26167 returned 404; skipping logs
	/tmp/afni-tinuous-2025.07.25T11.27.01-stderr:2025-07-25T11:27:17-0400 [WARNING ] tinuous: Job 26169 returned 404; skipping logs
	/tmp/afni-tinuous-2025.07.25T11.27.01-stderr:2025-07-25T11:27:17-0400 [WARNING ] tinuous: Job 26168 returned 404; skipping logs
	/tmp/afni-tinuous-2025.07.25T11.27.01-stderr:2025-07-25T11:27:49-0400 [WARNING ] tinuous: Job 26082 returned 404; skipping logs
	/tmp/afni-tinuous-2025.07.25T11.27.01-stderr:2025-07-25T11:27:49-0400 [WARNING ] tinuous: Job 26083 returned 404; skipping logs
	/tmp/afni-tinuous-2025.07.25T11.27.01-stderr:2025-07-25T11:27:49-0400 [WARNING ] tinuous: Job 26084 returned 404; skipping logs
	/tmp/afni-tinuous-2025.07.25T11.27.01-stderr:2025-07-25T11:27:49-0400 [WARNING ] tinuous: Job 26087 returned 404; skipping logs
	/tmp/afni-tinuous-2025.07.25T11.27.01-stderr:2025-07-25T11:27:49-0400 [WARNING ] tinuous: Job 26085 returned 404; skipping logs
	/tmp/afni-tinuous-2025.07.25T11.27.01-stderr:2025-07-25T11:27:49-0400 [WARNING ] tinuous: Job 26086 returned 404; skipping logs
	/tmp/afni-tinuous-2025.07.25T11.27.01-stderr:2025-07-25T11:27:49-0400 [WARNING ] tinuous: Job 26088 returned 404; skipping logs
	/tmp/afni-tinuous-2025.07.25T11.27.01-stderr:2025-07-25T11:27:51-0400 [WARNING ] tinuous: Job 26076 returned 404; skipping logs
	/tmp/afni-tinuous-2025.07.25T11.27.01-stderr:2025-07-25T11:27:51-0400 [WARNING ] tinuous: Job 26077 returned 404; skipping logs
	/tmp/afni-tinuous-2025.07.25T11.27.01-stderr:2025-07-25T11:27:51-0400 [WARNING ] tinuous: Job 26078 returned 404; skipping logs
	/tmp/afni-tinuous-2025.07.25T11.27.01-stderr:2025-07-25T11:27:51-0400 [WARNING ] tinuous: Job 26079 returned 404; skipping logs
	/tmp/afni-tinuous-2025.07.25T11.27.01-stderr:2025-07-25T11:27:51-0400 [WARNING ] tinuous: Job 26080 returned 404; skipping logs
@yarikoptic yarikoptic marked this pull request as draft July 25, 2025 19:17
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.

Still having issues with AFNI circle-ci logs

2 participants