Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge preview into release/preview/v8 #605

Merged
merged 25 commits into from
Oct 23, 2024
Merged

Conversation

rossgrambo
Copy link
Member

Preparing for v8.1.0-preview release

amerjusupovic and others added 25 commits July 25, 2024 10:22
* add ci.yml for testing

* remove comments

* fix pull request syntax

* fix codeql version

* fix codeql, test failing pipeline

* revert test change

* add security events write permission

* update publish tests path

* update task versions

* update task versions

* remove exit 0 for tests

* remove unused pipelines
* add linting rule

* remove unused package

* fix format issue & enforce format check

* fix format issue

* fix format issue

* add directory props

* merge

* update
* add secret client option

* update comment

* rename to configure

* use callback configuration pattern

* remove unused package
* add failover telemetry

* add load balancing enabled tracing

* use Features=xxx+xxx pattern

* revert change

* add comment

* update context string & move helper method to TracingUtil

* add comments

* update

* revert change

* add checks

* remove unused package

* fix typo

---------

Co-authored-by: Amer Jusupovic <[email protected]>
* increase default maxretries for key vault client to max value

* update minimum secret refresh interval to 1 minute, enforce for refresh all operations as well

* PR comments

* PR comments

* update max retries default for key vault

* update to use sourceid as key in cached secret dictionary

* move retries number to const int

* call invalidate cache with previous setting if current one is null

* add comment explaining number of key vault retries

* pull out utcnow as a variable in for loop in clearcache
Add allocation id when telemetry is enabled for feature flag
Merge main to preview with resolved conflicts
Copy link
Member

@avanigupta avanigupta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merge commit

@rossgrambo rossgrambo merged commit 6e9971a into release/preview/v8 Oct 23, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants