Merge preview to release/preview/v8 for 8.0.0-preview.3 #581
Conversation
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
* target .NET 8 and update other testing packages * use version of dependency injection equal to net version * remove test comment * fix json content test * revert change to flattener * change back version of json package * add back versions * fix test to work for new versions of microsoft.extensions.configuration.json * remove conditional packages * fix spacing, test with ado pipeline * fix spacing * update other examples * allow informationalversionattribute format in test for useragentheader
* some progress * update variables and methods related to cache expiration * update configstoredemo * change cacheexpirationinterval to a set method * allow either property or method for feature flag options for now * fix summaries for refreshoptions * PR revisions * fix method name reference in obsolete message * fix incorrect refreshinterval name
Bumps [Azure.Identity](https://github.com/Azure/azure-sdk-for-net) from 1.10.2 to 1.11.0. - [Release notes](https://github.com/Azure/azure-sdk-for-net/releases) - [Commits](Azure/azure-sdk-for-net@Azure.Identity_1.10.2...Azure.Identity_1.11.0) --- updated-dependencies: - dependency-name: Azure.Identity dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [Azure.Identity](https://github.com/Azure/azure-sdk-for-net) from 1.10.2 to 1.11.0. - [Release notes](https://github.com/Azure/azure-sdk-for-net/releases) - [Commits](Azure/azure-sdk-for-net@Azure.Identity_1.10.2...Azure.Identity_1.11.0) --- updated-dependencies: - dependency-name: Azure.Identity dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Amer Jusupovic <32405726+amerjusupovic@users.noreply.github.com>
* in progress shuffle clients * first draft load balancing, need tests * WIP logic for client shuffling - unsure how to incorporate priority * WIP * shuffle all clients together, fix logic for order of clients used * WIP * WIP store shuffle order for combined list * WIP shuffle logic * WIP new design * clean up logic/leftover code * move tests, check if dynamic clients are available in getclients * remove unused code * fix syntax issues, extend test * fix logic to increment client index * add clarifying comment * remove tests for now * WIP tests * add some tests, will add more * add to last test * remove unused usings * add extra verify statement to check client isnt used * edit logic to treat passed in clients as highest priority * PR comment revisions * check for more than one client in load balancing logic * set clients equal to new copied list before finding next available client * remove convert list to clients
* WIP adding utf8jsonreader * WIP jsondocument * replace all json adapters with jsondocument * combine code to serialize feature flag with code for adding key values to configuration * remove unused classes from jsonserializer approach * remove jsonserializer from testhelpers * WIP add handling and error messages for invalid values * WIP adding error messages * use method to create formatexception * validate for key vault uri and throw exception, fix logic in featuremanagement adapter * update error message, fix expected type for enabled * WIP * WIP testing aot compatible * small logic updates * improve variable names * edit error message * add isaotcompatible property to packages * fix error message for bool * WIP use utf8jsonreader * WIP * use utf8jsonreader for feature flags * use utf8jsonreader for keyvault secret reference * PR comment revision * WIP adding tests, PR revisions * WIP add some tests, PR revisions * key vault tests, use keyvaultreferenceexception for all scenarios because it's the established pattern * fix missing setup * PR revisions * update tests again, PR revisions * don't handle invalidoperationexception * remove unused exception var
…ity in example projects (#560) * update package * update package
…on parsing (#556) * WIP change constants to microsoft version, edit properties in fmkvadapter * index feature flags within new microsoft section feature_flags * fix all tests, use reset state * add warning log for old feature management libraries with new provider package * WIP adding deserialization changes, updating for variants/telemetry * WIP adding exceptions for incorrect types within arrays, allocation properties * WIP allocations mostly done * update remaining properties, fix tests * improve metadata error message * update tests, add variant and telemetry testing * remove alwayson * fix warning version check * Add tests for invalid cases * test boolean allows string * fix tests * remove unused using * add tests from original main PR for json parsing * revisions * run dotnet format on tests file * Update src/Microsoft.Extensions.Configuration.AzureAppConfiguration/Constants/ErrorMessages.cs Co-authored-by: Zhiyuan Liang <141655842+zhiyuanliang-ms@users.noreply.github.com> * some PR comment revisions * make from and to nullable in percentile to check if any values were set * Update src/Microsoft.Extensions.Configuration.AzureAppConfiguration/Constants/LoggingConstants.cs Co-authored-by: Avani Gupta <avanigupta@users.noreply.github.com> * Update src/Microsoft.Extensions.Configuration.AzureAppConfiguration/Constants/LoggingConstants.cs Co-authored-by: Avani Gupta <avanigupta@users.noreply.github.com> * PR comment revisions * combine if statements * change feature flag index state method * use on configuration refresh and updated as adapter events * change method names * removed unused methods --------- Co-authored-by: Zhiyuan Liang <141655842+zhiyuanliang-ms@users.noreply.github.com> Co-authored-by: Avani Gupta <avanigupta@users.noreply.github.com>
…551) * add log for formatexception from invalid feature flag json * remove capitalized json * remove unused using * fix comments * fix constant reference
…etProvider into merge-main-into-preview
Merge main into preview
* get full informational version name * use getcustomattribute * comment revision
* remove net7 references and target frameworks * update install script
* adding new private methods for dotnet versus microsoft schema, fixing tests * remove unused method * remove schema warning, PR comments * unify process feature flag approaches with id, fix schema check * fix tests to match changes * check for empty variants when deciding schema
* collect usage of SignalR * fix typo * fix typo * rename signalR
…ing (#540) * WIP create featurevariantstracing class * WIP class structure * add first working draft, combine tracing for fm and add allocation/telemetry * remove unused string * fix naming, add variant present tag * change constants names * update highest variants key and usesSeed tag * variant configuration suggestion * change variant config info to tag for ref only * reformatting tracing * add method for new features key in request tracing * fix tracing reference * PR comments * adjust features to have flag/other features lists * remove features key * Update src/Microsoft.Extensions.Configuration.AzureAppConfiguration/FeatureManagement/FeatureManagementKeyValueAdapter.cs Co-authored-by: Avani Gupta <avanigupta@users.noreply.github.com> * rename variables and methods --------- Co-authored-by: Avani Gupta <avanigupta@users.noreply.github.com>
…etProvider into main-to-preview
Merge main to preview
avanigupta
approved these changes
Jul 18, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.