Skip to content

Refactor Azure App Insights#7550

Merged
wozniakjan merged 3 commits into
kedacore:mainfrom
rickbrouwer:azureappinsights
Apr 8, 2026
Merged

Refactor Azure App Insights#7550
wozniakjan merged 3 commits into
kedacore:mainfrom
rickbrouwer:azureappinsights

Conversation

@rickbrouwer

Copy link
Copy Markdown
Member

Refactor Azure App Insights

Checklist

  • I have verified that my change is according to the deprecations & breaking changes policy
  • Tests have been added (if applicable)
  • Ensure make generate-scalers-schema has been run to update any outdated generated files
  • Commits are signed with Developer Certificate of Origin (DCO - learn more)

Relates to #5797

@rickbrouwer rickbrouwer requested a review from a team as a code owner March 16, 2026 16:30
@keda-automation keda-automation requested a review from a team March 16, 2026 16:30
@github-actions

Copy link
Copy Markdown

Thank you for your contribution! 🙏

Please understand that we will do our best to review your PR and give you feedback as soon as possible, but please bear with us if it takes a little longer as expected.

While you are waiting, make sure to:

  • Add an entry in our changelog in alphabetical order and link related issue
  • Update the documentation, if needed
  • Add unit & e2e tests for your changes
  • GitHub checks are passing
  • Is the DCO check failing? Here is how you can fix DCO issues

Once the initial tests are successful, a KEDA member will ensure that the e2e tests are run. Once the e2e tests have been successfully completed, the PR may be merged at a later date. Please be patient.

Learn more about our contribution guide.

@snyk-io

snyk-io Bot commented Mar 16, 2026

Copy link
Copy Markdown

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@rickbrouwer

rickbrouwer commented Mar 16, 2026

Copy link
Copy Markdown
Member Author

/run-e2e azure*
Update: You can check the progress here

Signed-off-by: Rick Brouwer <[email protected]>
@rickbrouwer

rickbrouwer commented Mar 16, 2026

Copy link
Copy Markdown
Member Author

/run-e2e azure*
Update: You can check the progress here

Signed-off-by: Rick Brouwer <[email protected]>
Signed-off-by: Rick Brouwer <[email protected]>
@rickbrouwer

rickbrouwer commented Mar 17, 2026

Copy link
Copy Markdown
Member Author

/run-e2e azure*
Update: You can check the progress here

@rickbrouwer rickbrouwer added the Awaiting/2nd-approval This PR needs one more approval review label Mar 23, 2026
@wozniakjan wozniakjan merged commit 95f2011 into kedacore:main Apr 8, 2026
26 checks passed
@wozniakjan wozniakjan removed the Awaiting/2nd-approval This PR needs one more approval review label Apr 8, 2026
@rickbrouwer rickbrouwer deleted the azureappinsights branch April 8, 2026 19:42
jansworld pushed a commit to jansworld/keda that referenced this pull request Apr 10, 2026
* Refactor Azure App Insights

Signed-off-by: Rick Brouwer <[email protected]>

* remove logger

Signed-off-by: Rick Brouwer <[email protected]>

* remove logr

Signed-off-by: Rick Brouwer <[email protected]>

---------

Signed-off-by: Rick Brouwer <[email protected]>
Signed-off-by: jansworld <[email protected]>
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.

3 participants