Skip to content

fix: pin adlfs for azure public ingest regression#4333

Merged
badGarnet merged 3 commits intovk/enable-vertical-text-detection-for-rotated-pagesfrom
rn/azure-public-anon-pin-test1
Apr 10, 2026
Merged

fix: pin adlfs for azure public ingest regression#4333
badGarnet merged 3 commits intovk/enable-vertical-text-detection-for-rotated-pagesfrom
rn/azure-public-anon-pin-test1

Conversation

@ryannikolaidis
Copy link
Copy Markdown
Contributor

@ryannikolaidis ryannikolaidis commented Apr 10, 2026

adlfs changed in 2026.4.0 with a breaking auth change: the default anon behavior flipped to False, so code that previously hit public Azure blobs anonymously now tries DefaultAzureCredential unless anon=True is set explicitly. that matches the CI failure here.

this pins adlfs==2026.2.0 to temporarily unblock failing changes when bumping to latest


Note

Medium Risk
Pins adlfs to an older version to work around an Azure public-container ingest regression, which can affect Azure connector behavior at runtime. The rest is CI-only fixture update plumbing and is low risk.

Overview
Pins adlfs to ==2026.2.0 via pyproject.toml UV constraints (and updates uv.lock) to avoid a regression affecting anonymous access to public Azure blob containers.

Updates the ingest-test-fixtures-update-pr GitHub Actions workflow to also generate and include expected-structured-output-markdown fixtures in the auto-created PRs, and records this change in the CHANGELOG.md.

Reviewed by Cursor Bugbot for commit 88a7801. Bugbot is set up for automated code reviews on this repo. Configure here.

@ryannikolaidis ryannikolaidis force-pushed the rn/azure-public-anon-pin-test1 branch from 1900588 to 9da64ac Compare April 10, 2026 02:43
@ryannikolaidis ryannikolaidis changed the base branch from main to vk/enable-vertical-text-detection-for-rotated-pages April 10, 2026 02:43
@ryannikolaidis ryannikolaidis changed the base branch from vk/enable-vertical-text-detection-for-rotated-pages to main April 10, 2026 02:44
@ryannikolaidis ryannikolaidis changed the base branch from main to vk/enable-vertical-text-detection-for-rotated-pages April 10, 2026 04:30
@ryannikolaidis ryannikolaidis marked this pull request as ready for review April 10, 2026 04:33
@ryannikolaidis ryannikolaidis changed the title test: pin adlfs for azure public ingest regression fix: pin adlfs for azure public ingest regression Apr 10, 2026
@badGarnet badGarnet merged commit f1f33fd into vk/enable-vertical-text-detection-for-rotated-pages Apr 10, 2026
94 of 98 checks passed
@badGarnet badGarnet deleted the rn/azure-public-anon-pin-test1 branch April 10, 2026 20:54
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.

3 participants