-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
AzureFunctionAppV2 Changes for Flex and fix race condition. #20191
Conversation
…-tasks into users/FinVamp1/FunctionsSprint240
azp /rerun |
azp /run |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please bump the task version
Tasks/AzureFunctionAppV2/deploymentProvider/BuiltInLinuxWebAppDeploymentProvider.ts
Show resolved
Hide resolved
Hi @FinVamp1 ! I stumbled across this PR after troubleshooting deployment of a Function app to new Flex Consumption. I am aware that Flex Consumption is in preview mode and the documentation says "Continuous deployment using Azure DevOps Tasks (AzureFunctionApp@2) is not supported". So what was this PR for? When I run it, I see the following in the logs:
But the Function app did not have any functions. I cannot find any documentation how to use it also, do you have it somewhere? |
Hello, @e-karlsson the way the task is designed is that the appType should be set to functionAppLinux and then the task will detect that the App is running on Flex Consumption and deploy to app using the Flex Consumption deployment endpoint. The aim of this PR was to add support for this while Flex Consumption is on Preview. Thanks for pointing out that the docs have not been updated and we will work on this soon. If you can share your app name or full deployment logs to me using finbarr AT microsoft.com and I will look at this for you. |
Task name: AzureFunctionAppv2
Description: Added Support for Flex Consumption and resolved race condition on Run From Package App Setting.
Documentation changes required: (Y) This adds support for a New Sku and has a new Parameter.
Added unit tests: (N)
Attached related issue: (Y) #19909
Checklist: