* 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>
* Update relative links to sections in the contributing guide doc to fix link verification
* Make some source code change modifications to trigger GenerateRelease CI steps
* Don't use relative paths in links, instead use the full path from master to be consistent with other places
* Get static/dynamic CRT link options in sync
* Update docs
* PR feedback
* en-us
* Another 'en-us' in a link
* MSVC_USE_STATIC_CRT
* Update README.md
Co-authored-by: Anton Kolesnyk <antkmsft@users.noreply.github.com>
It is generally useful for PR authors to go through their own PR first to make sure that commonly occurring feedback and issues have already been addressed. This checklist should help remind contributors what to look out for and is a living doc that we can modify and add to.
I ask these questions to myself before submitting a PR, when looking through my changes in the "pre-PR-submit-view".
Where possible leverage linter tools and CI gates to help address some of these. For example, for spell-check, here are some solutions to make things easier within the developer inner-loop:
- VS Code extension: [Code Spell Checker](https://marketplace.visualstudio.com/items?itemName=streetsidesoftware.code-spell-checker)
- VS Code extension: [Markdown linter](https://github.com/DavidAnson/vscode-markdownlint)
- Browser extension: [Site Spell](https://www.site-spell.com/)
Adding CMake module to enable/disable transport adapters
TRANSPORT ADAPTER BUILD
Default: If no option is explicitly added, curl will be used for POSIX and WIN HTTP for WIndows
Windows: Both CURL and WIN_HTTP can be build to be used.
POSIX: Only CURL is acceptable. If WIN_HTTP is set, generate step will fail for user
Defines `BUILD_WIN_HTTP_TRANSPORT_ADAPTER` and `BUILD_CURL_HTTP_TRANSPORT_ADAPTER` for source code
Fixes#350
Added build arguments to pass to cmake for running tests. If these
options are not passed during cmake build and someone tries to run test,
it will throw error "No tests were found!!!"
Cmake fails to find packages(curl, etc) if VCPKG_DEFAULT_TRIPLET is not set to x64-windows-static. If not set, cmake sets VCPKG_TARGET_TRIPLET to x64-windows instead of x64-windows-static. Since curl was installed using x64-windows-static in vcpkg, cmake fails to find the package. Adding set VCPKG_DEFAULT_TRIPLET to x64-windows-static explicitly in build instructions so that it is helpful to avoid cmake error(could not find CURL)
