microsoft / Docker-Provider

Azure Monitor for Containers
Other
139 stars 106 forks source link

remove adal go package and disable ADX log route #1294

Closed pfrcks closed 2 months ago

pfrcks commented 2 months ago

This pull request includes changes to remove the ADX (Azure Data Explorer) related code and dependencies from the project, as well as a change to the .trivyignore file to remove a critical telegraf vulnerability. The most important changes include removing the ADX related code from the oms.go and utils.go files, removing the ADX related dependencies from the go.mod file, and removing the critical telegraf vulnerability from the .trivyignore file.

Removal of ADX related code:

Removal of ADX related dependencies:

Removal of critical telegraf vulnerability:

Other changes:

pfrcks commented 2 months ago

validated that logs are flowing and no crashes. ADX mode is now a NOOP and logs error in the fluent-bit log files