Cache SAS tokens used for container file management #224

anshuman-goel · 2020-10-28T20:20:12Z

Summary of the Pull Request

During scaling this operation is adding lot of calls to storage container.

Applies to work item: #xxx
CLA signed. If not, go over here and sign the CLI.
Tests added/passed
Requires documentation to be updated
I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

What does this include?

manually

src/cli/onefuzz/api.py

bmc-msft · 2020-10-29T14:21:23Z

Running integration tests

Adding cache for sas url

827f3af

anshuman-goel commented Oct 28, 2020

View reviewed changes

src/cli/onefuzz/api.py Outdated Show resolved Hide resolved

anshuman-goel added 3 commits October 28, 2020 13:35

adding ttl

3c27043

Merge branch 'main' into patch-5

d75270b

any complain

eafce14

anshuman-goel commented Oct 28, 2020

View reviewed changes

src/cli/onefuzz/api.py Outdated Show resolved Hide resolved

anshuman-goel requested a review from bmc-msft October 28, 2020 21:05

anshuman-goel and others added 3 commits October 28, 2020 14:18

adding cast

37f3bc0

Merge branch 'main' into patch-5

037bc68

Merge branch 'main' into patch-5

b4f1bcf

bmc-msft approved these changes Oct 29, 2020

View reviewed changes

bmc-msft changed the title ~~Adding cache for sas return~~ Cache SAS tokens used for container file management Oct 29, 2020

bmc-msft merged commit 99b69d3 into microsoft:main Oct 29, 2020

ghost locked as resolved and limited conversation to collaborators Apr 17, 2021