Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Create AI ARM 2.1 for Functions and App Service #1200

Merged
merged 23 commits into from
Jul 21, 2022
Merged

Create AI ARM 2.1 for Functions and App Service #1200

merged 23 commits into from
Jul 21, 2022

Conversation

nturinski
Copy link
Member

Related to microsoft/vscode-azurefunctions#2835

  • Upgrade AppInsights to T2 which supports workspaceId property
  • Include LogAnalyticsWorkspaceCreateStep
    • Workspace is a requirement for new AI 2.1
    • Default to using a workspace that is in the same location and rg, if neither conditions are true, create a new one

nturinski added 13 commits March 8, 2022 15:13
@nturinski
Workaround for checkNameAvailability bug for site and slots
ae9d0dc
@nturinski
Merge main
6c41179
@nturinski
Merge main
dce8def
@nturinski
Merge branch 'main' of /~https://github.com/microsoft/vscode-azuretools
693e6af
@nturinski
Merge branch 'main' of /~https://github.com/microsoft/vscode-azuretools
8e1b77a
@nturinski
Merge branch 'main' of /~https://github.com/microsoft/vscode-azuretools
edffce8
@nturinski
Merge branch 'main' of /~https://github.com/microsoft/vscode-azuretools
4ab7a74
@nturinski
Merge branch 'main' of /~https://github.com/microsoft/vscode-azuretools
e94dbfd
@nturinski
Merge branch 'main' of /~https://github.com/microsoft/vscode-azuretools
7edafc1
@nturinski
Merge branch 'main' of /~https://github.com/microsoft/vscode-azuretools
de4d5ed
@nturinski
Merge branch 'main' of /~https://github.com/microsoft/vscode-azuretools
0684d29
@nturinski
Merge branch 'main' of /~https://github.com/microsoft/vscode-azuretools
45a74d7
@nturinski
Create log analytics workspace to use for AI 2.1
7fafe84
@nturinski nturinski requested a review from a team as a code owner July 20, 2022 00:32
alexweininger
alexweininger previously approved these changes Jul 20, 2022
View reviewed changes
@nturinski @bwateratmsft
Update appservice/src/utils/azureClients.ts
ddf04e5 
Co-authored-by: Brandon Waterloo [MSFT] <36966225+bwateratmsft@users.noreply.github.com>
nturinski and others added 6 commits July 20, 2022 10:43
@nturinski @bwateratmsft
Update appservice/src/createAppService/IAppServiceWizardContext.ts
87614e1 
Co-authored-by: Brandon Waterloo [MSFT] <36966225+bwateratmsft@users.noreply.github.com>
@nturinski @bwateratmsft
Update appservice/src/createAppService/LogAnalyticsCreateStep.ts
744d31f 
Co-authored-by: Brandon Waterloo [MSFT] <36966225+bwateratmsft@users.noreply.github.com>
@nturinski @bwateratmsft
Update appservice/src/createAppService/LogAnalyticsCreateStep.ts
7f9bcfb 
Co-authored-by: Brandon Waterloo [MSFT] <36966225+bwateratmsft@users.noreply.github.com>
@nturinski
Refactor out getAppInsightsSupprotedLocation and use it in LogAnalyti…
82b5f16 
…csCreateStep
@nturinski
Bump version
f9a65f5
@nturinski
Minor fixes
87458e4
@nturinski
Copy link
Member Author

Sorry for all the stupid styling problems. I didn't realize my auto-formatter wasn't working

bwateratmsft
bwateratmsft previously approved these changes Jul 21, 2022
View reviewed changes
@nturinski nturinski merged commit 7df6b17 into main Jul 21, 2022
@nturinski nturinski deleted the nat/newAI branch July 21, 2022 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bwateratmsft bwateratmsft bwateratmsft approved these changes

@alexweininger alexweininger alexweininger left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@nturinski @alexweininger @bwateratmsft