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

DM-42698: Switch to the moto v5 API #81

Merged
merged 5 commits into from Jan 31, 2024
Merged

DM-42698: Switch to the moto v5 API #81

merged 5 commits into from Jan 31, 2024

Conversation

timj
Copy link
Member

@timj timj commented Jan 29, 2024

Checklist

  • ran Jenkins
  • added a release note for user-visible changes to doc/changes

@ktlim
Copy link
Contributor

ktlim commented Jan 29, 2024

Except I guess I don't approve the test failure :)

@timj
Copy link
Member Author

timj commented Jan 29, 2024

Yes. You jumped the gun a bit 😄. The failure is a bit odd because it is acting like one batch of writes is being completely ignored by moto. Works fine in v4.

Daf_butler needs fixing as well.

@timj
Copy link
Member Author

timj commented Jan 29, 2024

I am wondering if this is caused by getmoto/moto#7152

@timj
Copy link
Member Author

timj commented Jan 29, 2024

The tests are failing because pagination in moto doesn't seem to be paginating. It's only returning the first 1000 results and no second page.

Copy link

codecov bot commented Jan 29, 2024

Codecov Report

Attention: 19 lines in your changes are missing coverage. Please review.

Comparison is base (bee90fe) 86.84% compared to head (a757480) 86.79%.

Files Patch % Lines
.../resources/_resourceHandles/_baseResourceHandle.py 0.00% 0 Missing and 17 partials ⚠️
python/lsst/resources/utils.py 0.00% 0 Missing and 1 partial ⚠️
tests/test_s3.py 87.50% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #81      +/-   ##
==========================================
- Coverage   86.84%   86.79%   -0.05%     
==========================================
  Files          27       27              
  Lines        4248     4232      -16     
  Branches      858      860       +2     
==========================================
- Hits         3689     3673      -16     
+ Misses        438      416      -22     
- Partials      121      143      +22     

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

@timj
Copy link
Member Author

timj commented Jan 30, 2024

The moto problem is being fixed in getmoto/moto#7283

@timj
Copy link
Member Author

timj commented Jan 30, 2024

We could possibly change the constraint to !=5.0.0 since the fix will appear in 5.0.1.

The pagination test is currently failing with moto v5.0.0 but
should be fixed in v5.0.1.
@timj timj merged commit fe12363 into main Jan 31, 2024
15 of 17 checks passed
@timj timj deleted the tickets/DM-42698 branch January 31, 2024 16:50
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.

None yet

2 participants