Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

Migrate AppInsights Azure resources from classic to work book based #1291

Merged
merged 1 commit into from
Sep 29, 2021

Conversation

stishkin
Copy link
Contributor

@stishkin stishkin commented Sep 27, 2021

Summary of the Pull Request

What is this about?

PR Checklist

Info on Pull Request

What does this include?
Updated azuredeploy.json ARM template to deploy newer version of app insights

Validation Steps Performed

How does someone test & validate?

Run deploy.py script to deploy fresh instance of OneFuzz or deploy on existing deployment to upgrade App Insights. Deploying on existing OneFuzz instance preserved app insights data.

@bmc-msft
Copy link
Contributor

The PR title doesn't match what is going on. Can you rename the PR to summarize the changes you are making?

@stishkin stishkin changed the title Deployment fails in westus3 due to "Region doesn’t support Classic resource mode for Application Insights resources." #1273 Migrate AppInsights Azure resources fror classic to work book based #1273 Sep 27, 2021
@stishkin stishkin changed the title Migrate AppInsights Azure resources fror classic to work book based #1273 Migrate AppInsights Azure resources from classic to work book based #1273 Sep 27, 2021
@bmc-msft bmc-msft changed the title Migrate AppInsights Azure resources from classic to work book based #1273 Migrate AppInsights Azure resources from classic to work book based Sep 28, 2021
Copy link
Contributor

@bmc-msft bmc-msft left a comment

Choose a reason for hiding this comment

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

I thought this came about because deployments in westus3 failed? This fails to deploy in westus3.

I let the deployment run for an hour before stopping it. From the script, users will see:

INFO:deploy:app password created
INFO:deploy:deploying arm template: azuredeploy.json

Then no further progress.

Looking into portal, I see the "Update hosting plan" fails many times with the statement "This subscription is not allowed for the region. Please try deploying to a different region."

@bmc-msft
Copy link
Contributor

bmc-msft commented Sep 28, 2021

Digging into the exception, it looks like there are other issues than just work book application insights.

This was deploying Microsoft.Web/serverFarms, which got the error:

{
 "Code": "Forbidden",
 "Message": "This subscription is not allowed for the region. Please try deploying to a different region.",
 "Target": null,
 "Details": [
  {
   "Message": "This subscription is not allowed for the region. Please try deploying to a different region."
  },
  {
   "Code": "Forbidden"
  },
  {
   "ErrorEntity": {
    "ExtendedCode": "52032",
    "MessageTemplate": "This subscription is not allowed for the region. Please try deploying to a different region.",
    "Parameters": [],
    "Code": "Forbidden",
    "Message": "This subscription is not allowed for the region. Please try deploying to a different region."
   }
  }
 ],
 "Innererror": null
}

@stishkin
Copy link
Contributor Author

Digging into the exception, it looks like there are other issues than just work book application insights.

This was deploying Microsoft.Web/serverFarms, which got the error:

{
 "Code": "Forbidden",
 "Message": "This subscription is not allowed for the region. Please try deploying to a different region.",
 "Target": null,
 "Details": [
  {
   "Message": "This subscription is not allowed for the region. Please try deploying to a different region."
  },
  {
   "Code": "Forbidden"
  },
  {
   "ErrorEntity": {
    "ExtendedCode": "52032",
    "MessageTemplate": "This subscription is not allowed for the region. Please try deploying to a different region.",
    "Parameters": [],
    "Code": "Forbidden",
    "Message": "This subscription is not allowed for the region. Please try deploying to a different region."
   }
  }
 ],
 "Innererror": null
}

This error is specific to that one subscription and it is a failure to deploy ServerFarms resource to West US 3. It is not related to AppInsights Classic issue that this PR is fixing.

I am following up on this error internally.

@stishkin
Copy link
Contributor Author

@bmc-msft I opened #1298 to investigate failure that you reported.

@stishkin stishkin merged commit c4133be into microsoft:main Sep 29, 2021
@stishkin stishkin deleted the 1273 branch September 29, 2021 15:09
@ghost ghost locked as resolved and limited conversation to collaborators Nov 3, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
2 participants