Skip to content

show headers and query parameters in storage CI pipeline#4379

Merged
Jinming-Hu merged 4 commits intoAzure:mainfrom
Jinming-Hu:show_storage_headers
Feb 23, 2023
Merged

show headers and query parameters in storage CI pipeline#4379
Jinming-Hu merged 4 commits intoAzure:mainfrom
Jinming-Hu:show_storage_headers

Conversation

@Jinming-Hu
Copy link
Member

closes #4369

Pull Request Checklist

Please leverage this checklist as a reminder to address commonly occurring feedback when submitting a pull request to make sure your PR can be reviewed quickly:

See the detailed list in the contributing guide.

  • C++ Guidelines
  • Doxygen docs
  • Unit tests
  • No unwanted commits/changes
  • Descriptive title/description
    • PR is single purpose
    • Related issue listed
  • Comments in source
  • No typos
  • Update changelog
  • Not work-in-progress
  • External references or docs updated
  • Self review of PR done
  • Any breaking changes?

@Jinming-Hu
Copy link
Member Author

/azp run cpp - storage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@Jinming-Hu
Copy link
Member Author

/azp run cpp - storage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@Jinming-Hu
Copy link
Member Author

@LarryOsterman is it safe to ignore Create API Review for xxx CI failures? I don't think I did anything in this PR that could lead to this error.

@Jinming-Hu Jinming-Hu merged commit 8cd4ecb into Azure:main Feb 23, 2023
@Jinming-Hu Jinming-Hu deleted the show_storage_headers branch February 23, 2023 06:57
microzchang added a commit that referenced this pull request Mar 6, 2023
* update comment (#4364)

* update comment

* jghjg

* update broken link

* Update sdk/keyvault/tools/cleanup/src/cleanup.cpp

Co-authored-by: Anton Kolesnyk <[email protected]>

---------

Co-authored-by: Anton Kolesnyk <[email protected]>

* Docker comment (#4375)

* update comment

* add comment about vcpkg

* dsfs

* Trigger`keyvault` on proxy changes (#4343)

* autotrigger the keyvault and storage CI when a testproxy file is changed

* Generate API review for C++ using new parser (#4302)

* Add vmImage back to common perf.yml - Fixes #5466 - Partially reverts #5456 (#4376)

Co-authored-by: Mike Harder <[email protected]>

* Fix pipelines path (#4358)

* test path

* qwq

* dsda

* asas

* dsada

* sdsds

* Update sdk/keyvault/azure-security-keyvault-certificates/perf-tests.yml

Co-authored-by: Mike Harder <[email protected]>

* Update sdk/keyvault/azure-security-keyvault-certificates/perf-tests.yml

Co-authored-by: Mike Harder <[email protected]>

* remove warmup

---------

Co-authored-by: Mike Harder <[email protected]>

* Test proxy & storage tests improvements (#4241)

* Temporarily pin Node 18 to 18.13.0 - Fixes #5536 (#4378)

Co-authored-by: Mike Harder <[email protected]>

* Storage tests improvement (#4382)

* Update CODEOWNERS (#4380)

* show headers and query parameters in storage CI pipeline (#4379)

* Sync eng/common directory with azure-sdk-tools for PR 5431 #2501

Co-authored-by: Konrad Jamrozik <[email protected]>

* Sync eng/common directory with azure-sdk-tools for PR 5562 (#4384)

* Add todos to update packages to pick up the newest CODEOWNERS interpreter

* update

---------

Co-authored-by: Konrad Jamrozik <[email protected]>

* Fix Share Client failure #4377 (#4381)

* Fix Share Client failure #4377

* decrease request count to avoid throttling errors for storage tests (#4385)

* Sync eng/common directory with azure-sdk-tools for PR 5568 (#4387)

* we encourage folks to place their assets.jsons at the package level

* update generate-assets-json.ps1 to only include src/**/session-records so as to avoid picking up the duplicated 'target' sessionrecords

---------

Co-authored-by: scbedd <[email protected]>

* Add support to ignore invalid cert common name (#4361)

* update to version 7.4 for admin. update tests (#4388)

* update proxy version to include Info/Active against individual sessions + allow delayed response (#4391)

Co-authored-by: scbedd <[email protected]>

* Sync eng/common directory with azure-sdk-tools for PR 5540 (#4396)

* add parameter to set cadl emitter options

* remove emitter name in the additional parameter

---------

Co-authored-by: chunyu3 <[email protected]>

* Follow-up to update changelog to reflect community contribution (#4393)

* Follow-up to update changelog to reflect community contribution

* Upgrade cspell version from 0.1 to 0.2

* logging api post request body (#4404)

Co-authored-by: Albert Cheng <[email protected]>

* Sync eng/common directory with azure-sdk-tools for PR 5595 (#4400)

* Use "npm ci" to install cspell and respect package-lock.json

* Review feedback

* Pipe npm ci output to Write-Host

---------

Co-authored-by: Daniel Jurek <[email protected]>

---------

Co-authored-by: George Arama <[email protected]>
Co-authored-by: Anton Kolesnyk <[email protected]>
Co-authored-by: Scott Beddall <[email protected]>
Co-authored-by: Praven Kuttappan <[email protected]>
Co-authored-by: Azure SDK Bot <[email protected]>
Co-authored-by: Mike Harder <[email protected]>
Co-authored-by: JinmingHu <[email protected]>
Co-authored-by: Konrad Jamrozik <[email protected]>
Co-authored-by: chunyu3 <[email protected]>
Co-authored-by: Ahson Khan <[email protected]>
Co-authored-by: Albert Cheng <[email protected]>
Co-authored-by: Daniel Jurek <[email protected]>
microzchang added a commit to microzchang/azure-sdk-for-cpp that referenced this pull request Apr 4, 2023
* update comment (Azure#4364)

* update comment

* jghjg

* update broken link

* Update sdk/keyvault/tools/cleanup/src/cleanup.cpp

Co-authored-by: Anton Kolesnyk <[email protected]>

---------

Co-authored-by: Anton Kolesnyk <[email protected]>

* Docker comment (Azure#4375)

* update comment

* add comment about vcpkg

* dsfs

* Trigger`keyvault` on proxy changes (Azure#4343)

* autotrigger the keyvault and storage CI when a testproxy file is changed

* Generate API review for C++ using new parser (Azure#4302)

* Add vmImage back to common perf.yml - Fixes Azure#5466 - Partially reverts Azure#5456 (Azure#4376)

Co-authored-by: Mike Harder <[email protected]>

* Fix pipelines path (Azure#4358)

* test path

* qwq

* dsda

* asas

* dsada

* sdsds

* Update sdk/keyvault/azure-security-keyvault-certificates/perf-tests.yml

Co-authored-by: Mike Harder <[email protected]>

* Update sdk/keyvault/azure-security-keyvault-certificates/perf-tests.yml

Co-authored-by: Mike Harder <[email protected]>

* remove warmup

---------

Co-authored-by: Mike Harder <[email protected]>

* Test proxy & storage tests improvements (Azure#4241)

* Temporarily pin Node 18 to 18.13.0 - Fixes Azure#5536 (Azure#4378)

Co-authored-by: Mike Harder <[email protected]>

* Storage tests improvement (Azure#4382)

* Update CODEOWNERS (Azure#4380)

* show headers and query parameters in storage CI pipeline (Azure#4379)

* Sync eng/common directory with azure-sdk-tools for PR 5431 Azure#2501

Co-authored-by: Konrad Jamrozik <[email protected]>

* Sync eng/common directory with azure-sdk-tools for PR 5562 (Azure#4384)

* Add todos to update packages to pick up the newest CODEOWNERS interpreter

* update

---------

Co-authored-by: Konrad Jamrozik <[email protected]>

* Fix Share Client failure Azure#4377 (Azure#4381)

* Fix Share Client failure Azure#4377

* decrease request count to avoid throttling errors for storage tests (Azure#4385)

* Sync eng/common directory with azure-sdk-tools for PR 5568 (Azure#4387)

* we encourage folks to place their assets.jsons at the package level

* update generate-assets-json.ps1 to only include src/**/session-records so as to avoid picking up the duplicated 'target' sessionrecords

---------

Co-authored-by: scbedd <[email protected]>

* Add support to ignore invalid cert common name (Azure#4361)

* update to version 7.4 for admin. update tests (Azure#4388)

* update proxy version to include Info/Active against individual sessions + allow delayed response (Azure#4391)

Co-authored-by: scbedd <[email protected]>

* Sync eng/common directory with azure-sdk-tools for PR 5540 (Azure#4396)

* add parameter to set cadl emitter options

* remove emitter name in the additional parameter

---------

Co-authored-by: chunyu3 <[email protected]>

* Follow-up to update changelog to reflect community contribution (Azure#4393)

* Follow-up to update changelog to reflect community contribution

* Upgrade cspell version from 0.1 to 0.2

* logging api post request body (Azure#4404)

Co-authored-by: Albert Cheng <[email protected]>

* Sync eng/common directory with azure-sdk-tools for PR 5595 (Azure#4400)

* Use "npm ci" to install cspell and respect package-lock.json

* Review feedback

* Pipe npm ci output to Write-Host

---------

Co-authored-by: Daniel Jurek <[email protected]>

---------

Co-authored-by: George Arama <[email protected]>
Co-authored-by: Anton Kolesnyk <[email protected]>
Co-authored-by: Scott Beddall <[email protected]>
Co-authored-by: Praven Kuttappan <[email protected]>
Co-authored-by: Azure SDK Bot <[email protected]>
Co-authored-by: Mike Harder <[email protected]>
Co-authored-by: JinmingHu <[email protected]>
Co-authored-by: Konrad Jamrozik <[email protected]>
Co-authored-by: chunyu3 <[email protected]>
Co-authored-by: Ahson Khan <[email protected]>
Co-authored-by: Albert Cheng <[email protected]>
Co-authored-by: Daniel Jurek <[email protected]>
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) test-enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Display header values and query parameter values in pipeline testing log

3 participants