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.

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

[FunctionApp] Updated stacks api json to add PowerShell on Linux in Functions #17678

Merged

Conversation

stefanushinardi
Copy link
Contributor

Description

The PR is to add Powershell on linux offerings #17657
Testing Guide

Verified that now the command does not throw validation error when deploying with Linux os type. And also verified creation of both consumption and elastic premium plan

sample command:

az functionapp create --name powershell-linux-premium-cli-create-shinardi --os-type Linux --runtime powershell --resource-group powershell-linux-cli-shinardi --functions-version 3 --storage-account powershelllinuxcli --plan ASP-pythonscaling-8a67

Application Insights "powershell-linux-premium-cli-create-shinardi" was created for this Function App. You can visit https://portal.azure.com/#resource/subscriptions/df6d3c16-321b-4f5b-a9de-c4ed04950b97/resourceGroups/powershell-linux-cli-shinardi/providers/microsoft.insights/components/powershell-linux-premium-cli-create-shinardi/overview to view your Application Insights component

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.


This checklist is used to make sure that common guidelines for a pull request are followed.

@stefanushinardi
Copy link
Contributor Author

cc: @Francisco-Gamino @AnatoliB @pragnagopa @gzuber Please review this PR. Thanks!

@yungezz yungezz added the Functions az functionapp label Apr 14, 2021
},
"isPreview": false,
"isDeprecated": false,
"isHidden": false
Copy link
Member

Choose a reason for hiding this comment

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

Please use: https://github.com/pragnagopa/azure-functions-supported-runtime-stacks/blob/master/LinuxFunctionsStacks.json

We want this stack to be hidden until we roll out functions host changes to support linux consumption

Copy link
Member

@pragnagopa pragnagopa left a comment

Choose a reason for hiding this comment

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

Please add stack as hidden. Also, need to add tests that check powershell stack

@gzuber
Copy link
Member

gzuber commented Apr 14, 2021

Please add stack as hidden. Also, need to add tests that check powershell stack

Hey @stefanushinardi I sent you a message about the tests, let me know if I can help out! 😄

@stefanushinardi
Copy link
Contributor Author

@pragnagopa @gzuber Added that tests for powershell. Thanks @Hazhzeng to help fix the local setip

Copy link
Member

@gzuber gzuber left a comment

Choose a reason for hiding this comment

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

Looks good!

@qwordy
Copy link
Member

qwordy commented Apr 16, 2021

@stefanushinardi Are we ready to merge?

Copy link
Member

@pragnagopa pragnagopa left a comment

Choose a reason for hiding this comment

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

💯 Thanks @stefanushinardi

@stefanushinardi
Copy link
Contributor Author

@qwordy yup, thanks!

@qwordy qwordy merged commit 9362079 into Azure:dev Apr 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Functions az functionapp
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants