Storage - Content Validation Decoder Stress Tests#49255
Open
gunjansingh-msft wants to merge 27 commits into
Open
Storage - Content Validation Decoder Stress Tests#49255gunjansingh-msft wants to merge 27 commits into
gunjansingh-msft wants to merge 27 commits into
Conversation
* removed enableDeterministic * change .delete() to .deleteIfExists() * remove Sinks.EmitFailureHandler.FAIL_FAST from CrcInputStream - read functions had FAIL_FAST which would throw an error when the stream had reached then end and we wanted to read from the stream again. So we removed from both reads. - refactor code so that the exit criteria is a tthe beginning - refactor the emitContentInfo for dry * prevent crashes on reattempted close on stream - changed emitValue to tryEmitValue - remove Sinks.EmitFailureHandler.FAIL_FAST so that multiple closes does not cause an error to be thrown * fix telemetry so that it doesnt swallow errors * roll back two deps because they were causing failures in the containers - opentelemetry-runtime-telemetry-java8 from 2.24.0-alpha -> 2.15.0-alpha - opentelemetry-logback-appender-1.0 from 2.24.0-alpha -> 2.15.0-alpha * rollback azure-client-sdk-parent linting extensions from 1.0.0-beta.2 t0 beta.1 * revert linting extensions to beta2 * remove comments with old code * add logging for errors * remove catches for double close issue and okay status * recursively delete files then delete the directory * change to sync deletes, refactor for easier reading * restructing share clean up so super calls only once * incorporate copilot suggestions * incorporate copilot suggestions * incorporate copilot suggestions * incorporate copilot suggestions * fix all deletes to make sync and wrap in try-catch * fix tests so that super.globalCleanupAsync() is only called once * change telemetry to loggin only returns final state instead of failed retries when ultimately successful * undo versio downgrade for linting-extensions * Fixing spacing in error messages Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * refactor datalake delete all so that it is easier to read * refactor runAsync in ShareScenarioBase so retry failures dont show as failures upon success --------- Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Export the fault injector certificate as PEM and wait for it before importing it into the Java truststore so storage stress tests fail fast instead of hitting PKIX errors. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
gunjansingh-msft
requested review from
a team,
alzimmermsft,
browndav-msft,
ibrandes,
kyleknap and
seanmcc-msft
as code owners
gunjansingh-msft
requested review from
a team,
ArthurMa1978,
XiaofeiCao,
axisc,
haolingdong-msft,
hmlam,
j7nw4r,
samvaity,
sjkwak,
srnagar and
weidongxu-microsoft
as code owners
gunjansingh-msft
requested review from
jpalvarezl,
kaylieee and
trrwilson
as code owners
gunjansingh-msft
requested review from
JonathanCrd,
KarishmaGhiya,
anannya03,
maorleger,
minhanh-phan and
vukelich
gunjansingh-msft
force-pushed
the
feature/storage/content-validation-stress-fix
branch
from
2fbc006 to
635064f
Compare
ibrandes
changed the title
…om/Azure/azure-sdk-for-java into feature/storage/content-validation-stress-fix
ibrandes
approved these changes
Jun 1, 2026
browndav-msft
approved these changes
Jun 1, 2026
Member
browndav-msft
left a comment
browndav-msft
left a comment
There was a problem hiding this comment.
I think it looks good. I don't see anything glaring, but I don't exactly know the cv download stuff.
Member
There was a problem hiding this comment.
@ibrandes do we want to get rid of this? I created this as an easy way to delete resource groups. It was the problem that Wes pointed out when we hit to 250 storage account limit or some other kind of limit that came from running stress tests multiple times.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.