* Stress test (#3820)
* mem leak fix
* PR comments fix
* PR comments
* seems to work?
* clang
* curl again
* add first stress test
* also update gitignore
* missing line
* only linux
* clang typo format
* typo 2
* PR comments
* cspell
* remove terminator
* Fixed the metadata for better handling (#3824)
Co-authored-by: sima-zhu <sizhu@microsoft.com>
* Update attestation core vcpkg dependency to latest GA version that is required. (#3830)
* Update attestation core vcpkg dependency to latest GA version that is required.
* Update vcpkg config.cmake to match the dependency version needed.
* Remove winhttp (#3832)
* remove refs to winhttp
* this tsst only curl
* Move perf.yml to eng/common (#3833)
Co-authored-by: Mike Harder <mharder@microsoft.com>
* Sync eng/common directory with azure-sdk-tools for PR 3656 (#3834)
* someone wants to reference the test-proxy startup scripts externally. to make this easy on them we're parameterizing the root of the eng/common for easy use in that scenario
* two leading $ signs on the definition of the certificate path was causing some issues!
Co-authored-by: scbedd <45376673+scbedd@users.noreply.github.com>
* Update casing for 'verison' (#3821)
Co-authored-by: Daniel Jurek <djurek@microsoft.com>
* Print additional result summary formats to pipelines UI (#3842)
Co-authored-by: Mike Harder <mharder@microsoft.com>
* Sync eng/common directory with azure-sdk-tools for PR 3702 (#3843)
* Add language-service to list of valid sample slugs
* Update link to taxonomies
Co-authored-by: Heath Stewart <heaths@microsoft.com>
* Fix incorrect failure notification in vcpkg publishing (#3838)
* Set up for testing of template pipeline
* Override branches, set up for template GA release
* More TODOs to prevent merging an unintended change
* More removal of TestPipeline
* Increment template version number
* Use script to set package version
* Check out the correct branch
* branch parameter in the command
* Use $(PublishToVcpkg) to determine if we should check for changes
* dictionary syntax
* Set GA package version to validate GA publish scenario
* Update changelog
* create-pull-request.yml optionally pushes changes
* Output GH PR URI
* Macro syntax with a variable set earlier
* Move up to 1.2.0-beta.2
* beta.1
* Revert changelog
* Revert testing-related changes
* Revert vcpkg-clone.yml
* Revert eng/common changes, ensure global $(HasChanges) is set properly
* Changes to enable testing
* 1.1.0-beta.1
* vcpkg clone should clone at configured branch
* Clone "main" branch of vcpkg betas
* Undo test-specific changes
* More PR cleanup
* PR cleanup
* Helm charts (#3841)
* helm chart
* first pipe setup
* poi
* magiks
* ewewe
* stress
* cleanup
* pr
* QFE and readme
* clang
* stupid clang , waste of time
* ewqwewewrqrewewrqewrqwrqr
* stupid clang
* cspell
* PR
* try try again
* clang again
* Sync eng/common directory with azure-sdk-tools for PR 3661 (#3846)
* Add full clone fallback to sparse checkout
* Improve clone handling and overrides for sparse checkout
* Use SkipSparseCheckout variable name
Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com>
* Sync eng/common directory with azure-sdk-tools for PR 3735 (#3845)
* Detect API changes using new snadboxinx approach
* Added strictmode
* Remove unset variable
* Changes as per strict mode 3
* Rervert strict mode to allow language level fixes to merge first
Co-authored-by: praveenkuttappan <prmarott@microsoft.com>
* Increment version for keyvault releases (#3809)
* Increment package version after release of azure-security-keyvault-keys
* Increment package version after release of azure-security-keyvault-secrets
* Increment package version after release of azure-security-keyvault-certificates
* Removed hard dependency on opentelemetry version (#3844)
* Revert "Add full clone fallback to sparse checkout (#3661)" (#3851)
This reverts commit 7605ead00308dd20f20f2afe5acc4ec9900a2c47.
Co-authored-by: Ben Broderick Phillips <ben@benbp.net>
* Sync eng/common directory with azure-sdk-tools for PR 3753 (#3857)
* Bump MacOs version to macos-11
* Update eng/common/scripts/job-matrix/tests/job-matrix-functions.tests.ps1
Co-authored-by: Wes Haggard <weshaggard@users.noreply.github.com>
* Update eng/common/scripts/job-matrix/tests/job-matrix-functions.tests.ps1
Co-authored-by: Wes Haggard <weshaggard@users.noreply.github.com>
Co-authored-by: sima-zhu <sizhu@microsoft.com>
Co-authored-by: Sima Zhu <48036328+sima-zhu@users.noreply.github.com>
Co-authored-by: Wes Haggard <weshaggard@users.noreply.github.com>
* Bump macOs version to macos-11 (#3853)
* Update CHANGELOG.md
* Fix misleading step name in az module install (#3859)
Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com>
* Retry poll calls on EINTR (#3858)
When signals are delievered to the process, calls here to poll may be
interrupted and return with a spurious failure. The call instead should be
restarted.
* Fix bad path to sample matrix json (#3860)
Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com>
* Sync eng/common directory with azure-sdk-tools for PR 3826 (#3863)
* Add link checking for stress CI
* Remove region segment from stress testing links
Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com>
* Core August releases (#3862)
* Core August releases
* cspell
Co-authored-by: Anton Kolesnyk <antkmsft@users.noreply.github.com>
* Increment version for core releases (#3865)
* Increment package version after release of azure-core
* Increment package version after release of azure-core-tracing-opentelemetry
* Storage Aug GA release (#3864)
* CG only runs on internal (#3866)
* Condition added and indentation changes
* STG 79 80 81 Features (#3850)
* Sync eng/common directory with azure-sdk-tools for PR 3860 (#3871)
* updating pfx and cert
* update the targeted version of the proxy as well
Co-authored-by: scbedd <45376673+scbedd@users.noreply.github.com>
* Storage August Preview Release (#3868)
* Storage August Preview Release
* update versions
* Increment version for storage releases (#3875)
* Increment package version after release of azure-storage-common
* Increment package version after release of azure-storage-blobs
* Fix an issue that reason phrase is missing as expected from HTTP/2 server (#3879)
* Fix an issue that reason phrase is missing as expected from HTTP/2 server
* reverse the condition
* compact comments
* fix clang format
* Update CODEOWNERS (#3881)
* updated targeted proxy version to one that properly allows consumption of TLS certificates (#3888)
Co-authored-by: scbedd <45376673+scbedd@users.noreply.github.com>
* Fix `azure-storage-blobs-cpp` beta install (#3889)
Co-authored-by: Anton Kolesnyk <antkmsft@users.noreply.github.com>
* Fixed#3899 - Handle new OSX version correctly; reformatted platform-matrix JSON files (#3900)
* Simpler get-binarysizes check for OSX (#3901)
* Simpler get-binarysizes check for OSX
* Missed wildcard
* Sync eng/common directory with azure-sdk-tools for PR 3874 (#3907)
* Add sparse checkout to perf template
* Rename base template, add parameters
* Add comment
* Add TODO
* Fix copy/pase bug, add language to artifacts path
* Fix language ref
* Fix python language
* perf.yml extends perf-base.yml
* Remove unnecessary base yml
* Fetch pull request refs
* Fix refs path
* Fetch commitish if matches PR syntax
* Print command before running
* Add PR commitish to remote.origin.fetch
* Add pool and vmimage parameters
* Map ref under "refs/remotes/origin"
* Add ref to refs/remotes/origin
Co-authored-by: Mike Harder <mharder@microsoft.com>
* File Share Features until STG82 (#3908)
* Empty file or existing file won't be created/overwritten if the blob to be downloaded doesn't exist. (#3910)
* Remove unnecessary checkout: none that causes conflicts with sparse checkout declarations (#3915)
Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com>
* Core September Release (#3913)
Co-authored-by: Anton Kolesnyk <antkmsft@users.noreply.github.com>
* Increment package version after release of azure-core (#3919)
* Migrate SkipDefaultCheckout to SkipCheckoutNone (#3916)
Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com>
* Storage Sept Release (#3914)
* Increment version for storage releases (#3924)
* Increment package version after release of azure-storage-common
* Increment package version after release of azure-storage-blobs
* Increment package version after release of azure-storage-files-shares
* Add github actions based check enforcer (#3922)
* Add `$schema` to `cgmanifest.json` (#3918)
Co-authored-by: Jamie Magee <jamie.magee@microsoft.com>
* Remove SkipDefaultCheckout parameter (#3930)
Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com>
* Compatibility improvement for striped blob (#3932)
* add a parameter 'condition' to the test-proxy ci invocations. (#3931)
Co-authored-by: scbedd <45376673+scbedd@users.noreply.github.com>
* storage ad-hoc release (#3934)
* Increment package version after release of azure-storage-blobs (#3935)
* Update `AttestationClient::AttestTpm` API to match existing `AttestOpenEnclave` and `AttestSgxmEnclave` (#3928)
* Fix broken link and typo in contributing.md
* Use vector<uint8_t> for attest instead of strings
* remove options
* fix comments
* update release version
* remove versionig
* revert changelog
* add the change
* update comment
* Update sdk/attestation/azure-security-attestation/CHANGELOG.md
Co-authored-by: Larry Osterman <LarryOsterman@users.noreply.github.com>
* fix formatting
* address pr comment
* fix formating
* update a comment
* remove the attest tpm comment
Co-authored-by: Peng Li <pengfeli@microsoft.com>
Co-authored-by: Larry Osterman <LarryOsterman@users.noreply.github.com>
* Update check enforcer yaml permissions and name (#3944)
* Set VCPKG_ROOT (#3938)
* Set VCPKG_ROOT
* Update eng/pipelines/templates/steps/vcpkg-clone.yml
Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com>
Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com>
* Sync the devops helper scripts between azure-sdk repo and tools repo (#3949)
This should fix https://github.com/Azure/azure-sdk-tools/issues/3538
as it will now upgrade the devops extension to the latest version
which has the auth fix needed.
Co-authored-by: Wes Haggard <Wes.Haggard@microsoft.com>
* Add Win32FileTimeConverter (#3941)
* Sync eng/common directory with azure-sdk-tools for PR 4169 (#3950)
* pin newest version of proxy
Co-authored-by: scbedd <45376673+scbedd@users.noreply.github.com>
* Enable Query() test in blob SDK (#3960)
* Sync eng/common directory with azure-sdk-tools for PR 4212 (#3962)
* version including lock fixes
* target the correct proxy version
Co-authored-by: scbedd <45376673+scbedd@users.noreply.github.com>
* Pipeline core (#3953)
* test1
* glob
* quick cleanup
* ddasda
* ddsa
* envs
* dsds
* Revert "ddsa"
This reverts commit 6d9e385284ba37736fb5bc41e56065943abd7937.
* wq
* comment out issues
* valid test
* maybe
* qwewqewq
* lang steps
* Vcpkg pre-steps in the place Daniel should've recommend they go
* RepoOwner
* Correct repo owner name
* Remove vcpkg.yml (we don't need to write)
* PR trigger
* small change
* Update eng/pipelines/templates/jobs/perf.yml
Co-authored-by: Mike Harder <mharder@microsoft.com>
* Update eng/pipelines/templates/jobs/perf.yml
Co-authored-by: Mike Harder <mharder@microsoft.com>
* Update sdk/core/perf.yml
Co-authored-by: Mike Harder <mharder@microsoft.com>
Co-authored-by: Daniel Jurek <djurek@microsoft.com>
Co-authored-by: Mike Harder <mharder@microsoft.com>
* Win32FileTimeConverter (#3967)
* Sync eng/common directory with azure-sdk-tools for PR 4240 (#3965)
* use version of docker with CLI present
* target repaired executable
Co-authored-by: scbedd <45376673+scbedd@users.noreply.github.com>
* Revert "Add Win32FileTimeConverter (#3941)" (#3966)
This reverts commit 14e4f52b91.
* add test case for batch options on snapshots and versions (#3970)
* add test case for batch options on snapshots and versions
* recording
* Merge feature/datalakestg82 (#3973)
* Build subscription configuration from cloud config values (#3976)
* fix LROs in keyvault (#3952)
* fix LROs in keyvault
* qfe
* some cleanup
* fdd
* clangs
* correct fix
* oops
* Pull all Websockets changes to main except for WebSockets itself. (#3954)
* Pulled WebSocket fixes into main except for WebSockets functionality
Co-authored-by: Rick Winter <rick.winter@microsoft.com>
Co-authored-by: Daniel Jurek <djurek@microsoft.com>
Co-authored-by: George Arama <50641385+gearama@users.noreply.github.com>
Co-authored-by: Ahson Khan <ahson_ahmedk@yahoo.com>
Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com>
Co-authored-by: Anton Kolesnyk <antkmsft@users.noreply.github.com>
* Upadte Attestation changelog.md (#3992)
* Remove cspell directive from being visible (#3993)
* Update Attestation readme.md (#3991)
* Fixed memory leak in curl (#3995)
* Fixed memory leak in curl
* fixed documentation in unique_handle.hpp
* Fixup comment in unique_handle.hpp to reference curl_easy_cleanup rather than curl_free. (#3997)
* Azure Core October Release (#3996)
* Azure Core October Release
Co-authored-by: Anton Kolesnyk <antkmsft@users.noreply.github.com>
Co-authored-by: Larry Osterman <LarryOsterman@users.noreply.github.com>
Co-authored-by: Ahson Khan <ahkha@microsoft.com>
* Sync eng/common directory with azure-sdk-tools for PR 4315 (#3999)
* update target version to one that includes integration discovered bugfixes
Co-authored-by: scbedd <45376673+scbedd@users.noreply.github.com>
* Increment package version after release of azure-core (#4000)
* Fix 50x errors while loading page links (#4001)
We have hit a few 50x errors while pulling a page to get
links and that has caused the rest of the link checking to
terminate early. To fix that we switching to LogError which will
output an error in devops instead of Write-Error which terminates
immediately. We also add some retry count to the page retrieval
and cache file retrieval to help with these transitent 50x issues.
Co-authored-by: Wes Haggard <Wes.Haggard@microsoft.com>
* enable datalake softdelete tests (#3974)
* Small updates for core tests to work with the main automation. (#4004)
* core tests
* clang
* datalakestg82/change IsServerEncrypted not nullable (#4005)
* Add default .assets directory to .gitignore (#4008)
* Storage October release (#3975)
* Increment version for storage releases (#4013)
* Increment package version after release of azure-storage-blobs
* Increment package version after release of azure-storage-files-datalake
* Increment package version after release of azure-storage-files-shares
* Key Vault October releases (#4010)
* prepScript
* Updates for changelogs for 7.3
* target version with fixes (#4003)
Co-authored-by: scbedd <45376673+scbedd@users.noreply.github.com>
* Increment version for keyvault releases (#4014)
* Increment package version after release of azure-security-keyvault-keys
* Increment package version after release of azure-security-keyvault-certificates
* Increment package version after release of azure-security-keyvault-secrets
* Attestation October (#4017)
* Updated attestation change log to add periods for consistency (#4016)
* Updated attestation change log to add periods for consistency
* Update wording for a breaking change.
* Add back-ticks.
* Increment package version after release of azure-security-attestation (#4018)
* Bump the code owner retriver version. (#4020)
Co-authored-by: sima-zhu <sizhu@microsoft.com>
* Improved discoverability of tests; handle exceptions thrown during test execution (#4021)
* Improved discoverability of tests on WIndows; handle exceptions thrown during test execution better
* Vcpkg macro version for customers (#3909)
* macro version for customers
* remove other macros
* remove variable
* Core test (#4006)
* core tests
* clang
* vcpkg test
* dswsw
* first take on the vcpkg
* vcpkg
* get vcpkg version
* fix cmake
* separators
* vcpkg
* vcpkg
* test1
* disable verify agent od
* put back verify
* put back yml
* Update VCPKG Baseline due to zlib (#4027)
* override zlib version
* shwitch baseline
* macro update
* override zlib
* Fixed CURL build breaks discovered by newer openssl baselines (#4034)
* Vcpkg baseline to latest (#4036)
* vcpkg baseline
* f4b262b259145adb2ab0116a390b08642489d32b
* 79fa9d85cb4510b05c2657bd7ec8a6a2cb266dc5
* 6ca56aeb457f033d344a7106cb3f9f1abf8f4e98
* override
* 1.4.1`
* Removed version override for opentelemetry (#4037)
* Removed version override for opentelemetry
* Disable deprecation warnings for open telemety tests to guard against future deprecations
* Added explanation for 4996 error; removed InMemoryExporter from documentation
* Sync eng/common directory with azure-sdk-tools for PR 3790 (#4041)
* Added yaml support for job matrix creation
* autogen scenario matrix for stress test
* Temporary Working State
* update to default sparse
* pr comments and some error handling
* custom matrixfilename and ordering of generatedValues.yaml
* common module import
* JobMatrix write host
Co-authored-by: Albert Cheng <albertcheng@microsoft.com>
* Storageblob (#4043)
* first take
* grrrrr
* qwqwq
* ewqw
* eewew
* put back
* pipeline
* Set read-only VCPKG_BINARY_SOURCES_SECRET
* Pass InstallLanguageSteps through the perf template
* blop pipeline
* path
* ';l'l;
* add support for sync param
* --sync flag support
* typo
* sync
* sync to y
* for PR
* clang
Co-authored-by: Daniel Jurek <djurek@microsoft.com>
* Upgrade cspell packages to version `^6.12.0` (#4044)
to address dependabot issue https://github.com/Azure/azure-sdk-for-js/security/dependabot/13
Co-authored-by: Jeremy Meng <yumeng@microsoft.com>
* Sync eng/common directory with azure-sdk-tools for PR 4480 (#4046)
* Add extra parameters for .net readme path
* Use another way for default value
* Remove custom source
Co-authored-by: sima-zhu <sizhu@microsoft.com>
* Converted WinHTTP to Async. (#4015)
* Async WinHTTP
* Added cancellation test; added support for request cancellation.
Co-authored-by: Anton Kolesnyk <41349689+antkmsft@users.noreply.github.com>
Co-authored-by: Ahson Khan <ahkha@microsoft.com>
Co-authored-by: Rick Winter <rick.winter@microsoft.com>
* Updated OpenTelemetry tests to use their own `SpanExporter` rather than depend on the `InMemorySpanExporter`. (#4042)
* Removed version override for opentelemetry
* Disable deprecation warnings for open telemety tests to guard against future deprecations
* Added explanation for 4996 error; removed InMemoryExporter from documentation
* Checkpoint adding manual telemetry exporter
* Switch from using OpenTelemetry's InMemoryExporter (which is an internal-to-opentelemetry test hook) to our own SpanExporter
* removed unused variables because gcc doesn't like them
* Fixed doxygen comments
* Removed a bunch of unnecessary headers
* Update sdk/core/azure-core-tracing-opentelemetry/test/ut/test_exporter.hpp
Co-authored-by: Rick Winter <rick.winter@microsoft.com>
Co-authored-by: Rick Winter <rick.winter@microsoft.com>
* [Perf] Include GA versions of core package (#4053)
* Fix to for a SAL annotation (#4054)
* In-memory Identity token cache (#4024)
* identicalize the comments type (#4063)
* Improve scenario matrix edge case handling (#4065)
Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com>
* docV1 (#4051)
* docV1
* Update doc/PerformanceTesting.md
Co-authored-by: Larry Osterman <LarryOsterman@users.noreply.github.com>
* Update doc/PerformanceTesting.md
Co-authored-by: Larry Osterman <LarryOsterman@users.noreply.github.com>
* Update doc/PerformanceTesting.md
Co-authored-by: Larry Osterman <LarryOsterman@users.noreply.github.com>
* Update doc/PerformanceTesting.md
Co-authored-by: Larry Osterman <LarryOsterman@users.noreply.github.com>
* comment
* some more updates
* Update doc/PerformanceTesting.md
Co-authored-by: Larry Osterman <LarryOsterman@users.noreply.github.com>
* Update doc/PerformanceTesting.md
Co-authored-by: Larry Osterman <LarryOsterman@users.noreply.github.com>
* Update doc/PerformanceTesting.md
Co-authored-by: Larry Osterman <LarryOsterman@users.noreply.github.com>
* PR
* Update doc/PerformanceTesting.md
Co-authored-by: Larry Osterman <LarryOsterman@users.noreply.github.com>
Co-authored-by: Larry Osterman <LarryOsterman@users.noreply.github.com>
* bump version to newest (#4066)
Co-authored-by: scbedd <45376673+scbedd@users.noreply.github.com>
* Async WInHTTP changes to WebSockets; fixed a couple of websocket bugs reported by cognitive.
* cspell; clang-format
* Sync eng/common directory with azure-sdk-tools for PR 4543 (#4071)
* stress test addons version check
* cleanup
Co-authored-by: Albert Cheng <albertcheng@microsoft.com>
* Disable proxy test on mac
* Fixed crash caused by premature destruction of WinHttpRequest object.
* Disable proxy tests if proxy tests are disabled
* Fixed race condition shutting down connections due to TLS violations
* cspell
* moved cspell out of doxygen comments
* moved cspell out of doxygen comments
* Added the ability to select OpenSSL 1.1.1n if desired. (#4045)
* Added ability to switch to OpenSSL 1.1.1n
* Fixed OpenSSL compilation challenges - the CRL stuff works now
* Added instructions on using OpenSSL 1.1.1 to README
* Added vcpkg args to handle openssl variants
* Improved names of SetEvent and ResetEvent
* cspell
* clang-format
* Traces to figure out test failure
* More traces
* Another try closing the TLS cert failure window
* More atempts at improving CI
* clang-format
Co-authored-by: George Arama <50641385+gearama@users.noreply.github.com>
Co-authored-by: Azure SDK Bot <53356347+azure-sdk@users.noreply.github.com>
Co-authored-by: sima-zhu <sizhu@microsoft.com>
Co-authored-by: Ahson Khan <ahson_ahmedk@yahoo.com>
Co-authored-by: Mike Harder <mharder@microsoft.com>
Co-authored-by: scbedd <45376673+scbedd@users.noreply.github.com>
Co-authored-by: Daniel Jurek <djurek@microsoft.com>
Co-authored-by: Heath Stewart <heaths@microsoft.com>
Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com>
Co-authored-by: praveenkuttappan <prmarott@microsoft.com>
Co-authored-by: Ben Broderick Phillips <ben@benbp.net>
Co-authored-by: Sima Zhu <48036328+sima-zhu@users.noreply.github.com>
Co-authored-by: Wes Haggard <weshaggard@users.noreply.github.com>
Co-authored-by: Rick Winter <rick.winter@microsoft.com>
Co-authored-by: John Heffner <john@topofcenter.net>
Co-authored-by: Anton Kolesnyk <41349689+antkmsft@users.noreply.github.com>
Co-authored-by: Anton Kolesnyk <antkmsft@users.noreply.github.com>
Co-authored-by: JinmingHu <jinmhu@microsoft.com>
Co-authored-by: Jonathan Cárdenas <JonathanCrd@users.noreply.github.com>
Co-authored-by: keshen-msft <53491277+keshen-msft@users.noreply.github.com>
Co-authored-by: microzchang <110015819+microzchang@users.noreply.github.com>
Co-authored-by: Jamie Magee <jamie.magee@gmail.com>
Co-authored-by: Jamie Magee <jamie.magee@microsoft.com>
Co-authored-by: Peng Li <86324823+penglimsft@users.noreply.github.com>
Co-authored-by: Peng Li <pengfeli@microsoft.com>
Co-authored-by: Wes Haggard <Wes.Haggard@microsoft.com>
Co-authored-by: Ahson Khan <ahkha@microsoft.com>
Co-authored-by: James Suplizio <jasupliz@microsoft.com>
Co-authored-by: Victor Vazquez <victor.vazquez@microsoft.com>
Co-authored-by: Albert Cheng <albertcheng@microsoft.com>
Co-authored-by: Jeremy Meng <yumeng@microsoft.com>
* Stress test (#3820)
* mem leak fix
* PR comments fix
* PR comments
* seems to work?
* clang
* curl again
* add first stress test
* also update gitignore
* missing line
* only linux
* clang typo format
* typo 2
* PR comments
* cspell
* remove terminator
* Fixed the metadata for better handling (#3824)
Co-authored-by: sima-zhu <sizhu@microsoft.com>
* Update attestation core vcpkg dependency to latest GA version that is required. (#3830)
* Update attestation core vcpkg dependency to latest GA version that is required.
* Update vcpkg config.cmake to match the dependency version needed.
* Remove winhttp (#3832)
* remove refs to winhttp
* this tsst only curl
* Move perf.yml to eng/common (#3833)
Co-authored-by: Mike Harder <mharder@microsoft.com>
* Sync eng/common directory with azure-sdk-tools for PR 3656 (#3834)
* someone wants to reference the test-proxy startup scripts externally. to make this easy on them we're parameterizing the root of the eng/common for easy use in that scenario
* two leading $ signs on the definition of the certificate path was causing some issues!
Co-authored-by: scbedd <45376673+scbedd@users.noreply.github.com>
* Update casing for 'verison' (#3821)
Co-authored-by: Daniel Jurek <djurek@microsoft.com>
* Print additional result summary formats to pipelines UI (#3842)
Co-authored-by: Mike Harder <mharder@microsoft.com>
* Sync eng/common directory with azure-sdk-tools for PR 3702 (#3843)
* Add language-service to list of valid sample slugs
* Update link to taxonomies
Co-authored-by: Heath Stewart <heaths@microsoft.com>
* Fix incorrect failure notification in vcpkg publishing (#3838)
* Set up for testing of template pipeline
* Override branches, set up for template GA release
* More TODOs to prevent merging an unintended change
* More removal of TestPipeline
* Increment template version number
* Use script to set package version
* Check out the correct branch
* branch parameter in the command
* Use $(PublishToVcpkg) to determine if we should check for changes
* dictionary syntax
* Set GA package version to validate GA publish scenario
* Update changelog
* create-pull-request.yml optionally pushes changes
* Output GH PR URI
* Macro syntax with a variable set earlier
* Move up to 1.2.0-beta.2
* beta.1
* Revert changelog
* Revert testing-related changes
* Revert vcpkg-clone.yml
* Revert eng/common changes, ensure global $(HasChanges) is set properly
* Changes to enable testing
* 1.1.0-beta.1
* vcpkg clone should clone at configured branch
* Clone "main" branch of vcpkg betas
* Undo test-specific changes
* More PR cleanup
* PR cleanup
* Helm charts (#3841)
* helm chart
* first pipe setup
* poi
* magiks
* ewewe
* stress
* cleanup
* pr
* QFE and readme
* clang
* stupid clang , waste of time
* ewqwewewrqrewewrqewrqwrqr
* stupid clang
* cspell
* PR
* try try again
* clang again
* Sync eng/common directory with azure-sdk-tools for PR 3661 (#3846)
* Add full clone fallback to sparse checkout
* Improve clone handling and overrides for sparse checkout
* Use SkipSparseCheckout variable name
Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com>
* Sync eng/common directory with azure-sdk-tools for PR 3735 (#3845)
* Detect API changes using new snadboxinx approach
* Added strictmode
* Remove unset variable
* Changes as per strict mode 3
* Rervert strict mode to allow language level fixes to merge first
Co-authored-by: praveenkuttappan <prmarott@microsoft.com>
* Increment version for keyvault releases (#3809)
* Increment package version after release of azure-security-keyvault-keys
* Increment package version after release of azure-security-keyvault-secrets
* Increment package version after release of azure-security-keyvault-certificates
* Removed hard dependency on opentelemetry version (#3844)
* Revert "Add full clone fallback to sparse checkout (#3661)" (#3851)
This reverts commit 7605ead00308dd20f20f2afe5acc4ec9900a2c47.
Co-authored-by: Ben Broderick Phillips <ben@benbp.net>
* Sync eng/common directory with azure-sdk-tools for PR 3753 (#3857)
* Bump MacOs version to macos-11
* Update eng/common/scripts/job-matrix/tests/job-matrix-functions.tests.ps1
Co-authored-by: Wes Haggard <weshaggard@users.noreply.github.com>
* Update eng/common/scripts/job-matrix/tests/job-matrix-functions.tests.ps1
Co-authored-by: Wes Haggard <weshaggard@users.noreply.github.com>
Co-authored-by: sima-zhu <sizhu@microsoft.com>
Co-authored-by: Sima Zhu <48036328+sima-zhu@users.noreply.github.com>
Co-authored-by: Wes Haggard <weshaggard@users.noreply.github.com>
* Bump macOs version to macos-11 (#3853)
* Update CHANGELOG.md
* Fix misleading step name in az module install (#3859)
Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com>
* Retry poll calls on EINTR (#3858)
When signals are delievered to the process, calls here to poll may be
interrupted and return with a spurious failure. The call instead should be
restarted.
* Fix bad path to sample matrix json (#3860)
Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com>
* Sync eng/common directory with azure-sdk-tools for PR 3826 (#3863)
* Add link checking for stress CI
* Remove region segment from stress testing links
Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com>
* Core August releases (#3862)
* Core August releases
* cspell
Co-authored-by: Anton Kolesnyk <antkmsft@users.noreply.github.com>
* Increment version for core releases (#3865)
* Increment package version after release of azure-core
* Increment package version after release of azure-core-tracing-opentelemetry
* Storage Aug GA release (#3864)
* CG only runs on internal (#3866)
* Condition added and indentation changes
* STG 79 80 81 Features (#3850)
* Sync eng/common directory with azure-sdk-tools for PR 3860 (#3871)
* updating pfx and cert
* update the targeted version of the proxy as well
Co-authored-by: scbedd <45376673+scbedd@users.noreply.github.com>
* Storage August Preview Release (#3868)
* Storage August Preview Release
* update versions
* Increment version for storage releases (#3875)
* Increment package version after release of azure-storage-common
* Increment package version after release of azure-storage-blobs
* Fix an issue that reason phrase is missing as expected from HTTP/2 server (#3879)
* Fix an issue that reason phrase is missing as expected from HTTP/2 server
* reverse the condition
* compact comments
* fix clang format
* Update CODEOWNERS (#3881)
* updated targeted proxy version to one that properly allows consumption of TLS certificates (#3888)
Co-authored-by: scbedd <45376673+scbedd@users.noreply.github.com>
* Fix `azure-storage-blobs-cpp` beta install (#3889)
Co-authored-by: Anton Kolesnyk <antkmsft@users.noreply.github.com>
* Fixed#3899 - Handle new OSX version correctly; reformatted platform-matrix JSON files (#3900)
* Simpler get-binarysizes check for OSX (#3901)
* Simpler get-binarysizes check for OSX
* Missed wildcard
* Sync eng/common directory with azure-sdk-tools for PR 3874 (#3907)
* Add sparse checkout to perf template
* Rename base template, add parameters
* Add comment
* Add TODO
* Fix copy/pase bug, add language to artifacts path
* Fix language ref
* Fix python language
* perf.yml extends perf-base.yml
* Remove unnecessary base yml
* Fetch pull request refs
* Fix refs path
* Fetch commitish if matches PR syntax
* Print command before running
* Add PR commitish to remote.origin.fetch
* Add pool and vmimage parameters
* Map ref under "refs/remotes/origin"
* Add ref to refs/remotes/origin
Co-authored-by: Mike Harder <mharder@microsoft.com>
* File Share Features until STG82 (#3908)
* Empty file or existing file won't be created/overwritten if the blob to be downloaded doesn't exist. (#3910)
* Remove unnecessary checkout: none that causes conflicts with sparse checkout declarations (#3915)
Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com>
* Core September Release (#3913)
Co-authored-by: Anton Kolesnyk <antkmsft@users.noreply.github.com>
* Increment package version after release of azure-core (#3919)
* Migrate SkipDefaultCheckout to SkipCheckoutNone (#3916)
Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com>
* Storage Sept Release (#3914)
* Increment version for storage releases (#3924)
* Increment package version after release of azure-storage-common
* Increment package version after release of azure-storage-blobs
* Increment package version after release of azure-storage-files-shares
* Add github actions based check enforcer (#3922)
* Add `$schema` to `cgmanifest.json` (#3918)
Co-authored-by: Jamie Magee <jamie.magee@microsoft.com>
* Remove SkipDefaultCheckout parameter (#3930)
Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com>
* Compatibility improvement for striped blob (#3932)
* add a parameter 'condition' to the test-proxy ci invocations. (#3931)
Co-authored-by: scbedd <45376673+scbedd@users.noreply.github.com>
* storage ad-hoc release (#3934)
* Increment package version after release of azure-storage-blobs (#3935)
* Update `AttestationClient::AttestTpm` API to match existing `AttestOpenEnclave` and `AttestSgxmEnclave` (#3928)
* Fix broken link and typo in contributing.md
* Use vector<uint8_t> for attest instead of strings
* remove options
* fix comments
* update release version
* remove versionig
* revert changelog
* add the change
* update comment
* Update sdk/attestation/azure-security-attestation/CHANGELOG.md
Co-authored-by: Larry Osterman <LarryOsterman@users.noreply.github.com>
* fix formatting
* address pr comment
* fix formating
* update a comment
* remove the attest tpm comment
Co-authored-by: Peng Li <pengfeli@microsoft.com>
Co-authored-by: Larry Osterman <LarryOsterman@users.noreply.github.com>
* Update check enforcer yaml permissions and name (#3944)
* Set VCPKG_ROOT (#3938)
* Set VCPKG_ROOT
* Update eng/pipelines/templates/steps/vcpkg-clone.yml
Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com>
Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com>
* Sync the devops helper scripts between azure-sdk repo and tools repo (#3949)
This should fix https://github.com/Azure/azure-sdk-tools/issues/3538
as it will now upgrade the devops extension to the latest version
which has the auth fix needed.
Co-authored-by: Wes Haggard <Wes.Haggard@microsoft.com>
* Add Win32FileTimeConverter (#3941)
* Sync eng/common directory with azure-sdk-tools for PR 4169 (#3950)
* pin newest version of proxy
Co-authored-by: scbedd <45376673+scbedd@users.noreply.github.com>
* Enable Query() test in blob SDK (#3960)
* Sync eng/common directory with azure-sdk-tools for PR 4212 (#3962)
* version including lock fixes
* target the correct proxy version
Co-authored-by: scbedd <45376673+scbedd@users.noreply.github.com>
* Pipeline core (#3953)
* test1
* glob
* quick cleanup
* ddasda
* ddsa
* envs
* dsds
* Revert "ddsa"
This reverts commit 6d9e385284ba37736fb5bc41e56065943abd7937.
* wq
* comment out issues
* valid test
* maybe
* qwewqewq
* lang steps
* Vcpkg pre-steps in the place Daniel should've recommend they go
* RepoOwner
* Correct repo owner name
* Remove vcpkg.yml (we don't need to write)
* PR trigger
* small change
* Update eng/pipelines/templates/jobs/perf.yml
Co-authored-by: Mike Harder <mharder@microsoft.com>
* Update eng/pipelines/templates/jobs/perf.yml
Co-authored-by: Mike Harder <mharder@microsoft.com>
* Update sdk/core/perf.yml
Co-authored-by: Mike Harder <mharder@microsoft.com>
Co-authored-by: Daniel Jurek <djurek@microsoft.com>
Co-authored-by: Mike Harder <mharder@microsoft.com>
* Win32FileTimeConverter (#3967)
* Sync eng/common directory with azure-sdk-tools for PR 4240 (#3965)
* use version of docker with CLI present
* target repaired executable
Co-authored-by: scbedd <45376673+scbedd@users.noreply.github.com>
* Revert "Add Win32FileTimeConverter (#3941)" (#3966)
This reverts commit 14e4f52b91.
* add test case for batch options on snapshots and versions (#3970)
* add test case for batch options on snapshots and versions
* recording
* Merge feature/datalakestg82 (#3973)
* Build subscription configuration from cloud config values (#3976)
* fix LROs in keyvault (#3952)
* fix LROs in keyvault
* qfe
* some cleanup
* fdd
* clangs
* correct fix
* oops
* Pull all Websockets changes to main except for WebSockets itself. (#3954)
* Pulled WebSocket fixes into main except for WebSockets functionality
Co-authored-by: Rick Winter <rick.winter@microsoft.com>
Co-authored-by: Daniel Jurek <djurek@microsoft.com>
Co-authored-by: George Arama <50641385+gearama@users.noreply.github.com>
Co-authored-by: Ahson Khan <ahson_ahmedk@yahoo.com>
Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com>
Co-authored-by: Anton Kolesnyk <antkmsft@users.noreply.github.com>
* Upadte Attestation changelog.md (#3992)
* Remove cspell directive from being visible (#3993)
* Post-merge changes to websockets code
* Update Attestation readme.md (#3991)
* Added back WebSocket_server.py
* Added Start-WebSocketServer.ps1 to list of files to copy
* clang-format
* Fixed memory leak in curl (#3995)
* Fixed memory leak in curl
* fixed documentation in unique_handle.hpp
Co-authored-by: George Arama <50641385+gearama@users.noreply.github.com>
Co-authored-by: Azure SDK Bot <53356347+azure-sdk@users.noreply.github.com>
Co-authored-by: sima-zhu <sizhu@microsoft.com>
Co-authored-by: Ahson Khan <ahson_ahmedk@yahoo.com>
Co-authored-by: Mike Harder <mharder@microsoft.com>
Co-authored-by: scbedd <45376673+scbedd@users.noreply.github.com>
Co-authored-by: Daniel Jurek <djurek@microsoft.com>
Co-authored-by: Heath Stewart <heaths@microsoft.com>
Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com>
Co-authored-by: praveenkuttappan <prmarott@microsoft.com>
Co-authored-by: Ben Broderick Phillips <ben@benbp.net>
Co-authored-by: Sima Zhu <48036328+sima-zhu@users.noreply.github.com>
Co-authored-by: Wes Haggard <weshaggard@users.noreply.github.com>
Co-authored-by: Rick Winter <rick.winter@microsoft.com>
Co-authored-by: John Heffner <john@topofcenter.net>
Co-authored-by: Anton Kolesnyk <41349689+antkmsft@users.noreply.github.com>
Co-authored-by: Anton Kolesnyk <antkmsft@users.noreply.github.com>
Co-authored-by: JinmingHu <jinmhu@microsoft.com>
Co-authored-by: Jonathan Cárdenas <JonathanCrd@users.noreply.github.com>
Co-authored-by: keshen-msft <53491277+keshen-msft@users.noreply.github.com>
Co-authored-by: microzchang <110015819+microzchang@users.noreply.github.com>
Co-authored-by: Jamie Magee <jamie.magee@gmail.com>
Co-authored-by: Jamie Magee <jamie.magee@microsoft.com>
Co-authored-by: Peng Li <86324823+penglimsft@users.noreply.github.com>
Co-authored-by: Peng Li <pengfeli@microsoft.com>
Co-authored-by: Wes Haggard <Wes.Haggard@microsoft.com>
* Stress test (#3820)
* mem leak fix
* PR comments fix
* PR comments
* seems to work?
* clang
* curl again
* add first stress test
* also update gitignore
* missing line
* only linux
* clang typo format
* typo 2
* PR comments
* cspell
* remove terminator
* Fixed the metadata for better handling (#3824)
Co-authored-by: sima-zhu <sizhu@microsoft.com>
* Update attestation core vcpkg dependency to latest GA version that is required. (#3830)
* Update attestation core vcpkg dependency to latest GA version that is required.
* Update vcpkg config.cmake to match the dependency version needed.
* Remove winhttp (#3832)
* remove refs to winhttp
* this tsst only curl
* Move perf.yml to eng/common (#3833)
Co-authored-by: Mike Harder <mharder@microsoft.com>
* Sync eng/common directory with azure-sdk-tools for PR 3656 (#3834)
* someone wants to reference the test-proxy startup scripts externally. to make this easy on them we're parameterizing the root of the eng/common for easy use in that scenario
* two leading $ signs on the definition of the certificate path was causing some issues!
Co-authored-by: scbedd <45376673+scbedd@users.noreply.github.com>
* Update casing for 'verison' (#3821)
Co-authored-by: Daniel Jurek <djurek@microsoft.com>
* Print additional result summary formats to pipelines UI (#3842)
Co-authored-by: Mike Harder <mharder@microsoft.com>
* Sync eng/common directory with azure-sdk-tools for PR 3702 (#3843)
* Add language-service to list of valid sample slugs
* Update link to taxonomies
Co-authored-by: Heath Stewart <heaths@microsoft.com>
* Fix incorrect failure notification in vcpkg publishing (#3838)
* Set up for testing of template pipeline
* Override branches, set up for template GA release
* More TODOs to prevent merging an unintended change
* More removal of TestPipeline
* Increment template version number
* Use script to set package version
* Check out the correct branch
* branch parameter in the command
* Use $(PublishToVcpkg) to determine if we should check for changes
* dictionary syntax
* Set GA package version to validate GA publish scenario
* Update changelog
* create-pull-request.yml optionally pushes changes
* Output GH PR URI
* Macro syntax with a variable set earlier
* Move up to 1.2.0-beta.2
* beta.1
* Revert changelog
* Revert testing-related changes
* Revert vcpkg-clone.yml
* Revert eng/common changes, ensure global $(HasChanges) is set properly
* Changes to enable testing
* 1.1.0-beta.1
* vcpkg clone should clone at configured branch
* Clone "main" branch of vcpkg betas
* Undo test-specific changes
* More PR cleanup
* PR cleanup
* Helm charts (#3841)
* helm chart
* first pipe setup
* poi
* magiks
* ewewe
* stress
* cleanup
* pr
* QFE and readme
* clang
* stupid clang , waste of time
* ewqwewewrqrewewrqewrqwrqr
* stupid clang
* cspell
* PR
* try try again
* clang again
* Sync eng/common directory with azure-sdk-tools for PR 3661 (#3846)
* Add full clone fallback to sparse checkout
* Improve clone handling and overrides for sparse checkout
* Use SkipSparseCheckout variable name
Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com>
* Sync eng/common directory with azure-sdk-tools for PR 3735 (#3845)
* Detect API changes using new snadboxinx approach
* Added strictmode
* Remove unset variable
* Changes as per strict mode 3
* Rervert strict mode to allow language level fixes to merge first
Co-authored-by: praveenkuttappan <prmarott@microsoft.com>
* Increment version for keyvault releases (#3809)
* Increment package version after release of azure-security-keyvault-keys
* Increment package version after release of azure-security-keyvault-secrets
* Increment package version after release of azure-security-keyvault-certificates
* Removed hard dependency on opentelemetry version (#3844)
* Revert "Add full clone fallback to sparse checkout (#3661)" (#3851)
This reverts commit 7605ead00308dd20f20f2afe5acc4ec9900a2c47.
Co-authored-by: Ben Broderick Phillips <ben@benbp.net>
* Sync eng/common directory with azure-sdk-tools for PR 3753 (#3857)
* Bump MacOs version to macos-11
* Update eng/common/scripts/job-matrix/tests/job-matrix-functions.tests.ps1
Co-authored-by: Wes Haggard <weshaggard@users.noreply.github.com>
* Update eng/common/scripts/job-matrix/tests/job-matrix-functions.tests.ps1
Co-authored-by: Wes Haggard <weshaggard@users.noreply.github.com>
Co-authored-by: sima-zhu <sizhu@microsoft.com>
Co-authored-by: Sima Zhu <48036328+sima-zhu@users.noreply.github.com>
Co-authored-by: Wes Haggard <weshaggard@users.noreply.github.com>
* Bump macOs version to macos-11 (#3853)
* Update CHANGELOG.md
* Fix misleading step name in az module install (#3859)
Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com>
* Retry poll calls on EINTR (#3858)
When signals are delievered to the process, calls here to poll may be
interrupted and return with a spurious failure. The call instead should be
restarted.
* Fix bad path to sample matrix json (#3860)
Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com>
* Sync eng/common directory with azure-sdk-tools for PR 3826 (#3863)
* Add link checking for stress CI
* Remove region segment from stress testing links
Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com>
* Core August releases (#3862)
* Core August releases
* cspell
Co-authored-by: Anton Kolesnyk <antkmsft@users.noreply.github.com>
* Increment version for core releases (#3865)
* Increment package version after release of azure-core
* Increment package version after release of azure-core-tracing-opentelemetry
* Storage Aug GA release (#3864)
* CG only runs on internal (#3866)
* Condition added and indentation changes
* STG 79 80 81 Features (#3850)
* Sync eng/common directory with azure-sdk-tools for PR 3860 (#3871)
* updating pfx and cert
* update the targeted version of the proxy as well
Co-authored-by: scbedd <45376673+scbedd@users.noreply.github.com>
* Storage August Preview Release (#3868)
* Storage August Preview Release
* update versions
* Increment version for storage releases (#3875)
* Increment package version after release of azure-storage-common
* Increment package version after release of azure-storage-blobs
* Fix an issue that reason phrase is missing as expected from HTTP/2 server (#3879)
* Fix an issue that reason phrase is missing as expected from HTTP/2 server
* reverse the condition
* compact comments
* fix clang format
* Update CODEOWNERS (#3881)
* updated targeted proxy version to one that properly allows consumption of TLS certificates (#3888)
Co-authored-by: scbedd <45376673+scbedd@users.noreply.github.com>
* Fix `azure-storage-blobs-cpp` beta install (#3889)
Co-authored-by: Anton Kolesnyk <antkmsft@users.noreply.github.com>
* Fixed#3899 - Handle new OSX version correctly; reformatted platform-matrix JSON files (#3900)
* Simpler get-binarysizes check for OSX (#3901)
* Simpler get-binarysizes check for OSX
* Missed wildcard
* Sync eng/common directory with azure-sdk-tools for PR 3874 (#3907)
* Add sparse checkout to perf template
* Rename base template, add parameters
* Add comment
* Add TODO
* Fix copy/pase bug, add language to artifacts path
* Fix language ref
* Fix python language
* perf.yml extends perf-base.yml
* Remove unnecessary base yml
* Fetch pull request refs
* Fix refs path
* Fetch commitish if matches PR syntax
* Print command before running
* Add PR commitish to remote.origin.fetch
* Add pool and vmimage parameters
* Map ref under "refs/remotes/origin"
* Add ref to refs/remotes/origin
Co-authored-by: Mike Harder <mharder@microsoft.com>
* File Share Features until STG82 (#3908)
* Empty file or existing file won't be created/overwritten if the blob to be downloaded doesn't exist. (#3910)
* Remove unnecessary checkout: none that causes conflicts with sparse checkout declarations (#3915)
Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com>
* Core September Release (#3913)
Co-authored-by: Anton Kolesnyk <antkmsft@users.noreply.github.com>
* Increment package version after release of azure-core (#3919)
* Migrate SkipDefaultCheckout to SkipCheckoutNone (#3916)
Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com>
* Storage Sept Release (#3914)
* Increment version for storage releases (#3924)
* Increment package version after release of azure-storage-common
* Increment package version after release of azure-storage-blobs
* Increment package version after release of azure-storage-files-shares
* Add github actions based check enforcer (#3922)
* Add `$schema` to `cgmanifest.json` (#3918)
Co-authored-by: Jamie Magee <jamie.magee@microsoft.com>
* Remove SkipDefaultCheckout parameter (#3930)
Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com>
* Compatibility improvement for striped blob (#3932)
* add a parameter 'condition' to the test-proxy ci invocations. (#3931)
Co-authored-by: scbedd <45376673+scbedd@users.noreply.github.com>
* storage ad-hoc release (#3934)
* Increment package version after release of azure-storage-blobs (#3935)
Co-authored-by: George Arama <50641385+gearama@users.noreply.github.com>
Co-authored-by: Azure SDK Bot <53356347+azure-sdk@users.noreply.github.com>
Co-authored-by: sima-zhu <sizhu@microsoft.com>
Co-authored-by: Ahson Khan <ahson_ahmedk@yahoo.com>
Co-authored-by: Mike Harder <mharder@microsoft.com>
Co-authored-by: scbedd <45376673+scbedd@users.noreply.github.com>
Co-authored-by: Daniel Jurek <djurek@microsoft.com>
Co-authored-by: Heath Stewart <heaths@microsoft.com>
Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com>
Co-authored-by: praveenkuttappan <prmarott@microsoft.com>
Co-authored-by: Ben Broderick Phillips <ben@benbp.net>
Co-authored-by: Sima Zhu <48036328+sima-zhu@users.noreply.github.com>
Co-authored-by: Wes Haggard <weshaggard@users.noreply.github.com>
Co-authored-by: Rick Winter <rick.winter@microsoft.com>
Co-authored-by: John Heffner <john@topofcenter.net>
Co-authored-by: Anton Kolesnyk <41349689+antkmsft@users.noreply.github.com>
Co-authored-by: Anton Kolesnyk <antkmsft@users.noreply.github.com>
Co-authored-by: JinmingHu <jinmhu@microsoft.com>
Co-authored-by: Jonathan Cárdenas <JonathanCrd@users.noreply.github.com>
Co-authored-by: keshen-msft <53491277+keshen-msft@users.noreply.github.com>
Co-authored-by: microzchang <110015819+microzchang@users.noreply.github.com>
Co-authored-by: Jamie Magee <jamie.magee@gmail.com>
Co-authored-by: Jamie Magee <jamie.magee@microsoft.com>
Jeff has pointed out that the current practice in the SDK of having a ServiceVersion which contains the current API versions of the service (for instance:
```c++
class ServiceVersion final {
public:
explicit ServiceVersion(std::string version) : m_version(std::move(version)) {}
AZ_STORAGE_QUEUES_DLLEXPORT const static ServiceVersion V2018_03_28;
AZ_STORAGE_QUEUES_DLLEXPORT const static ServiceVersion V2020_10_01;
};
)
```
Has a problem because the `ServiceVersion` construct has an implication that each of the `ServiceVersion` values listed is fully supported by the SDK.
The reality is that the SDK client team only tests the most recent API version listed in the SDK (the value which is the default version listed in the `ServiceClient` constructor).
How do we resolve this issue?
There are a few possible solutions that we’ve explored:
1) Test all the API versions listed in the `ServiceVersion` enumeration.
2) Remove the unsupported values from the `ServiceVersion` enumeration.
3) Remove the `ServiceVersion` enumeration
4) Remove the ability to set the API version at all.
Each of these solutions has some fairly significant drawbacks.
1) Test all the API versions listed.
The core problem with this is that the SDK team is small and adding tests to support every possible API version is going to be prohibitively expensive.
2) Remove the unsupported values from the `ServiceVersion` enumeration.
This is a breaking change and it means that moving to a new API version requires a breaking change to the SDK, even if the changes between API versions is strictly additive.
3) Remove the ServiceVersion enumeration.
This is also a breaking change for shipping SDKs (specifically KeyVault and Storage Queues). However, it is a one-time breaking change and we don’t have evidence of customers actually using the feature.
4) Remove the ability to set the API version at all.
Having *some* mechanism to set the API version is an important “escape hatch” which will allow customers to specify a specific API version even if that API version is not fully supported.
After discussing this a LOT, [@Ahson Khan](mailto:ahkha@microsoft.com), [@Rick Winter](mailto:Rick.Winter@microsoft.com), [@Jeffrey Richter](mailto:jeffreyr@microsoft.com), [@George Arama](mailto:George.Arama@microsoft.com), and [@Larry Osterman](mailto:Larry.Osterman@microsoft.com) came to the conclusion that we should probably take option #3, but leave the ClientOptions.Version value as a std::string.
* Implement tracing for Attestation and Template services
* Pipeline no longer requires service name if opting into distributed tracing; enable tracing in attestation service
* Generate user-agent header from request activity policy
* Added test to catch the redacted header regression
* Updated documentation to reflect API surface changes
* Prepare attestation for release
* removed references to RetrieveResponseValidationCollateral from docs
* Added C++ SDK team as owners of attestation SDK
* Added Ahmad from attestation team to attestation owners
* Removed dead API; switched attestation back to beta-3
* Split out attestation client factory into separate class
* Updated readme; clang-format
* Final set of API review changes
* Replaced () constructors with {} constructors
* binary cache and manifest
* update packages
* udpate
* add cache to private pipeline
* next try
* aver
* other
* again
* aver quick
* more
* this way
* again
* one more
* print
* test
* use depend
* more deps
* dep
* Apply suggestions from code review
Co-authored-by: Daniel Jurek <djurek@microsoft.com>
* updated to use group variable
* update identity
* end line
* Update vcpkg.json
* Apply suggestions from code review
* cspell
* remove comment
* updates
* make cache mode depend on SAS env var
* map env var only for internal pipelines
* other approach
* what about this
* and this
* try
* amd
* another
* extra step
* typo
* override for internal
* use default succeded
* azure core update manifest
* a
* run cmake-generate nightly as well
* check for SAS
* check cache
* no secret
* fix is secret
* pass explicit
* use secret all the time
* char
* One more
* export
* echos
* last
* array
* remove question
* ok
* weird
* use account key
* substring
* VCPKG_BINARY_SOURCES
* Add module installation
* task:
* Correct pathing for module
* update source gen
* format
* update spelling
* IsWindoows
* Use pwsh
* Cannot clobber with PSModule-Helpers. Attempt plain install
* Attempt plain install
* Revert unnecessary change to Update-DocsMsToc.ps1
* template ready
* curl is required on Windows as well for some CI gates
* attestation
* fix format
Co-authored-by: Daniel Jurek <djurek@microsoft.com>
* Moved samples around to meet new recommendations; added a couple of additional tests.
* Reworked attestation to include RetrieveAttestationValidationCollateral
* Attestation sample readme updates
* TPM doesn't need to retrieve response validation collateral
* Added cautionary warning about the dangers of overriding the TearDown method from inside a test case
* Added attestation team members to codeowners for attestation SDK
* Remove CODEOWNERS from cspell checks
* Don't hold a lock across retrieving the signers over the network
* Updated snippets in readme; clang-format
* changes
* vs generator
* use macro
* release and samples
* add target to build smoke test
* revert
* reuse macro for samples
* extra legs for samples only
* storage connection string
* format
* order and macro
* update
* samples
* missing
* Added GetPolicyManagementCertificates
* Added Add and Remove policy certificate
* Added support for Tpm attestation
* Moved TPM to attestation service specific section
* Added support for validation callback
* Updated RequestFailedException documentation
* Created extendableenumeration class
* Convert attestation to use extendable enumeration; added test for extendable enumerations
Co-authored-by: Casey Carter <cartec69@gmail.com>
* SetPolicy and ResetPolicy APIs
* Added tests for attestation result; cleaned up attest test suite a bit; fixed up attestation sample to be more real
* Further refactored test collateral to reduce wasted test cases
* Reformat with clang-format-11
* Removed use of Nullable::HasValue() and Nullable::Value() in favor of operator bool and operator *
* Get policy sample to run in CI pipeline
* Added AZURE_ variables to test-resources.json
* Set WinHTTP_OPTION_CLIENT_CERT_CONTEXT to enable connections to attestation service
* Set serviceDirectory from environment variables
Co-authored-by: Victor Vazquez <victor.vazquez@microsoft.com>
