Skip to content

Fix task exception never retrieved due to aiobotocore sessions#1714

Merged
deep1401 merged 5 commits intomainfrom
fix/task-exception-all
Apr 1, 2026
Merged

Fix task exception never retrieved due to aiobotocore sessions#1714
deep1401 merged 5 commits intomainfrom
fix/task-exception-all

Conversation

@deep1401
Copy link
Copy Markdown
Member

@deep1401 deep1401 commented Apr 1, 2026

No description provided.

@paragon-review
Copy link
Copy Markdown

paragon-review bot commented Apr 1, 2026

Paragon Summary

This pull request review analyzed 4 files and found no issues. The review examined code changes, potential bugs, security vulnerabilities, performance issues, and code quality concerns using automated analysis tools.

Paragon did not detect any problems in the current diff. Proceed with merge after your normal checks.

This PR fixes unhandled task exceptions caused by aiobotocore session handling in storage, preventing noisy runtime errors and improving async stability. It also aligns the relevant API and SDK dependency versions to support the fix.

Key changes:

  • Fix aiobotocore session handling to prevent “task exception was never retrieved” errors
  • Update lab-sdk/src/lab/storage.py to use safer storage/session lifecycle management
  • Align dependency/config versions across api, localprovider, and lab-sdk pyproject files

Confidence score: 5/5

  • This PR has low risk with no critical or high-priority issues identified
  • Score reflects clean code review with only minor suggestions or no issues found
  • Code quality checks passed - safe to proceed with merge

4 files reviewed, 0 comments


Tip: @paragon-run <instructions> to chat with our agent or push fixes!

Dashboard

@sentry
Copy link
Copy Markdown

sentry bot commented Apr 1, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Copy link
Copy Markdown
Member

@dadmobile dadmobile left a comment

Choose a reason for hiding this comment

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

Question: You only did this to some functions and I couldn't quickly figure out why?

Copy link
Copy Markdown
Member Author

@deep1401 deep1401 left a comment

Choose a reason for hiding this comment

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

Question: You only did this to some functions and I couldn't quickly figure out why?

Missed 4 functions because they were not used in launch_template. Fixed now

@deep1401 deep1401 merged commit 96f8244 into main Apr 1, 2026
14 checks passed
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.

2 participants