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

Structured message cherrypick stg96 #45626

Conversation

jaschrep-msft
Copy link
Member

Contributing to the Azure SDK

Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.

For specific information about pull request etiquette and best practices, see this section.

jaschrep-msft and others added 25 commits August 12, 2024 11:17
* Initial implementation and basic test

* seek/write tests

* fix test param

* fix exceptions
* enum rename and footer read/write methods

* align encode/decode tests | update encoding stream

* decode stream footer

* rename
* decode tests & bugfixes

* roundtrip tests

* more tests

* better errors | remove duplicate test
* regenerate and stage block uses structured message

* page and append

* testproxy

* cleanup

* fix datalake/share tests

* testproxy

* testproxy

* re-add null-safe access
* download range structured message

* testproxy

* throw when service fails to give back structured message

* test

* testproxy
* putblob structured message

* testproxy

* fixes
* datalake append

* null fix

* fixes
* retriable decode

* rewind mock test

* bugfix

* bugfix

* tests
* blobs retriable structured message download

* test proxy

* testproxy

* remove commented code
* blockblob working

* revert testing change

* page/append

* datalake file

* testfix

* bug fixes | test fixes

* disable new API for presenting CRC from structured message

* fix nunit

* whitespace

* fix/test-proxy

* csproj

* more csproj removeals

This is building fine locally idk what's up

* Trigger Fresh Build

* fileshare testproxy

* fix mock

* Update macos image from 11 to latest (Azure#44607)

* Update macos image from 11 to latest

* Update eng/pipelines/templates/jobs/ci.mgmt.yml

Co-authored-by: Ben Broderick Phillips <ben@benbp.net>

---------

Co-authored-by: Ben Broderick Phillips <ben@benbp.net>

* Revert "Update macos image from 11 to latest (Azure#44607)"

this is causing too many problems. skipping macos tests for now. They'll
run when this feature branch merges into main.

This reverts commit 29e87b4.

---------

Co-authored-by: Wes Haggard <weshaggard@users.noreply.github.com>
Co-authored-by: Ben Broderick Phillips <ben@benbp.net>
* impl

* testproxy

* shares fix

* testproxy
* block blob fix

* testproxy
* rename and validation

* fix
* expose crc from structured message

* testproxy

* undo typo

* exportapi
rename 'Size' in ContentRange. Name was misleading.
Change partitioned downloader to rely on calculating downloaded length
based on Content-Range, as Content-Length is no longer reliable for blob
data.
@github-actions github-actions bot added the Storage Storage Service (Queues, Blobs, Files) label Aug 21, 2024
@azure-sdk
Copy link
Collaborator

azure-sdk commented Aug 21, 2024

API change check

APIView has identified API level changes in this PR and created following API reviews.

Azure.Storage.Blobs
Azure.Storage.Common
Azure.Storage.Files.DataLake
Azure.Storage.Files.Shares
Azure.Storage.Queues

@jaschrep-msft jaschrep-msft force-pushed the structured-message-cherrypick-stg96 branch from f2c85fa to 1f9983b Compare September 24, 2024 20:19
@seanmcc-msft seanmcc-msft merged commit 99c02de into Azure:feature/storage/stg96base2 Sep 25, 2024
27 of 30 checks passed
seanmcc-msft added a commit that referenced this pull request Sep 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Storage Storage Service (Queues, Blobs, Files)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants