Configure appinsights full connection string

Arturo Hernandez [EPAM]requested to merge
az/ah_fullappinsights_conn into master
  • Created new secret in secretclassprovider common object.
  • Got rid of unneeded dependency in common-base Chart.
  • Refactor values of Charts which involves APPLICATIONINSIGHTS_CONNECTION_STRING env to use secret.

As for now the APPLICATIONINSIGHTS_CONNECTION_STRING env var it is composed by:

"InstrumentationKey=$(appinsights_key)"

However, this can lead to non-predictive behavior.

I noticed there are some docs about this and requirement parameters, as for now, looks like current implementation only relies on InstrumentationKey, per docs, recommend to overwrite Endpoints schemas, makes sense to me from multiregion perspective.

The connection string secret was implemented a while back on the infrastructure side on infra-azure-provisioning#316 (closed)

Related to:

Merge request reports