Bug: 12263565:[UR2][bugbash]Function monitor page is broken showing l…

[exc3eed]

RDBug 12263565:[UR2][bugbash]Function monitor page is broken (showing loading message for ever)

[nertim]

There is already a ScenarioIds.enableAppInsights available. Can you please update the code to use that appropriately? If not, open a bug on me and I can update it. No need to introduce a new scenarioId.

[exc3eed]

There are difference between ScenarioIds.enableAppInsights and ScenarioIds.appInsightsConfiguration
enableAppInsights can be enabled or disabled based on AppInsights configuration i.e in azure we can enable Appinsights. in this case it will return enabled.
On the other hand ScenarioIds.appInsightsConfiguration tells whether we need to show AppInsight configuration page/componenet/message to user. In some environment like onprem AppInsights is not supported yet that means you cannot configure it and there is no point to show AppInsight configuration page, it should simply default to classic view in this case
So ScenarioIds.enableAppInsights = disabled doesn’t necessarily mean that we have to disable configuration component/message/page. We still need to allow user to configure AppInsights if they want in azure scenario.

[nertim]

I have sent #2615 with the fix we discussed earlier.