Skip to content

Commit 615e485

Browse files
authored
Merge pull request #99 from Azure/patelchandni/fix-deployment
deployement issue for manifest file
2 parents a07a1b1 + 326514f commit 615e485

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

src/publishers/zipDeploy.ts

+3-3
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ export class ZipDeploy {
3232
if (isDeploymentSucceeded) {
3333
await context.kuduServiceUtil.postZipDeployOperation(deploymentId, deploymentId);
3434
}
35-
await context.kuduServiceUtil.updateDeploymentStatus(isDeploymentSucceeded, null, {
35+
await context.kuduServiceUtil.updateDeploymentStatus(isDeploymentSucceeded, deploymentId, {
3636
'type': 'Deployment',
3737
'slotName': context.appService ? context.appService.getSlot() : 'production'
3838
});
@@ -52,9 +52,9 @@ export class ZipDeploy {
5252
}
5353

5454
if (appSettings.WEBSITE_RUN_FROM_PACKAGE !== undefined &&
55-
appSettings.WEBSITE_RUN_FROM_PACKAGE.trim() === '1' &&
55+
appSettings.WEBSITE_RUN_FROM_PACKAGE.trim() === '0' &&
5656
(context.os === undefined || context.os === RuntimeStackConstant.Linux)) {
57-
Logger.Warn("Detected WEBSITE_RUN_FROM_PACKAGE is set to '1'. If you are deploying to a Linux " +
57+
Logger.Warn("Detected WEBSITE_RUN_FROM_PACKAGE is set to '0'. If you are deploying to a Linux " +
5858
"function app, you may need to remove this app setting.");
5959
return;
6060
}

0 commit comments

Comments
 (0)