-
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
[BUG]: AzureFunctionApp Task may result in the WEBSITE_RUN_FROM_PACKAGE AppSetting missing on SCM\Kudu and Validation failures #19909
Comments
This should be resolved in #20191 |
@komjera I was not able to reproduce the issue myself so I implemented additional tracing and the update logic from AzureRMWebAppDeployment@4 which should have helped. I need to see new deployment logs with this error and debug enabled so I can see what is missing. People can either share them here or contact me offline or open a support case. Thanks, Finbar |
@FinVamp1 It's hard to tell which id's and what information in debug logs are safe to share. Please let me know if you need any more information or if i have redacted any information you need. I would probably be able to share more logs "offline", how would I do that? Selected parts from debug logs earlier today below
|
@FinVamp1 were these logs enough to troubleshoot? |
@FinVamp1 This issue seems to have been occurring more frequently lately. |
@FinVamp1 This problem is occuring everyday and causes us to manually monitor our deploys, is there any way we can assist in resolving the issue besides providing logs? |
@FinVamp1 @v-schhabra any news? We are still experiencing this bug every day. Is there anything we can do on our side to mitigate the issue? |
I suppose that being unable to deploy to Azure isn't all that critical. |
@komjera Can you please send me the unredacted logs to finbarr At Microsoft.com and I will take a look at this. I'll need to understand which SKU you are targeting to see why this is still happening and see what we can do to fix this? |
@FinVamp1 I'll look into if there are any parts of the logs that will need to remain redacted. The storage account is LRS and the sku of the App Service Plan is
Is that enough information or do you still need the unredacted logs? |
@komjera If possible I would like to have a DevOps Pipeline log ran in debug mode with the site not redacted so that I can track the journey through our control plane. "Setting System.Debug to true configures verbose logs for all runs. You can also configure verbose logs for a single run with the Enable system diagnostics checkbox." |
@FinVamp1 Is this issue is resolved? |
New issue checklist
Task name
AzureFunctionApp
Task version
2.239.1
Issue Description
Validating deployment package for functions app before Zip Deploy
##[warning]"ZipDeploy Validation WARNING: It is recommended to set app setting WEBSITE_RUN_FROM_PACKAGE = 1 unless you are targeting one of the following scenarios: 1. Using portal editing. 2. Running post deployment scripts. 3. Need write permission in wwwroot. 4. Using custom handler with special requirements. NOTE: If you decide to update app setting WEBSITE_RUN_FROM_PACKAGE = 1, you will have to re-deploy your code. "
Package deployment using ZIP Deploy initiated.
it looks like the Web App Task knew that Kudu could be out of sync and had some code to wait.
azure-pipelines-tasks/Tasks/AzureRmWebAppDeploymentV4/operations/AzureAppServiceUtility.ts
Line 171 in 259e0d3
tl.debug('retrieving values from Kudu service to check if new values are updated');
This work item is to add this code to the Functions task.
Environment type (Please select at least one enviroment where you face this issue)
Azure DevOps Server type
dev.azure.com (formerly visualstudio.com)
Azure DevOps Server Version (if applicable)
No response
Operation system
Windows\Ubuntu
Relevant log output
Full task logs with system.debug enabled
N/A
Repro steps
The text was updated successfully, but these errors were encountered: