Skip to content

Update dagster-obstore lockfile#312

Merged
cmpadden merged 4 commits into
mainfrom
update-dagster-obstore-lockfile
May 22, 2026
Merged

Update dagster-obstore lockfile#312
cmpadden merged 4 commits into
mainfrom
update-dagster-obstore-lockfile

Conversation

@cmpadden

Copy link
Copy Markdown
Contributor

Summary

  • refresh dagster-obstore uv.lock with latest compatible dependency versions

Testing

  • uv lock --upgrade in libraries/dagster-obstore

@cmpadden cmpadden requested a review from ion-elgreco as a code owner May 22, 2026 18:12
@cmpadden

Copy link
Copy Markdown
Contributor Author

Pulled the failing check logs with gh. The failure was from the upgraded Ruff formatter reporting files that needed formatting. Ran uv run ruff format in libraries/dagster-obstore, committed the formatting-only fix, and pushed it to this branch.

@cmpadden

Copy link
Copy Markdown
Contributor Author

Pulled the new failing check logs with gh after the formatting fix. The Azure tests failed because azure-storage-blob 12.29.0 defaults to service API version 2026-04-06, which the Azurite version used in CI does not support. Constrained the dev dependency to azure-storage-blob>=12.24.1,<12.29 and regenerated the lockfile. Ran uv run ruff check and uv run ruff format --check locally for libraries/dagster-obstore, then pushed the fix.

@cmpadden

Copy link
Copy Markdown
Contributor Author

Pulled the latest failing logs with gh. Azure Blob 12.28.0 still defaulted to a service API version unsupported by the Azurite image in CI (2026-02-06). Tightened the dev dependency to azure-storage-blob>=12.24.1,<12.25, which preserves the original Azurite-compatible SDK line, regenerated the lockfile, and pushed the fix.

@cmpadden cmpadden merged commit f7b5b44 into main May 22, 2026
7 checks passed
@cmpadden cmpadden deleted the update-dagster-obstore-lockfile branch May 22, 2026 18: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.

1 participant