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

Users/suaggar/warcustomname #11181

Merged
merged 10 commits into from
Aug 27, 2019
Merged

Users/suaggar/warcustomname #11181

merged 10 commits into from
Aug 27, 2019

Conversation

SumiranAgg
Copy link
Contributor

No description provided.

@SumiranAgg SumiranAgg added Area: Release Area: AzureAppService Label to monitor Azure App Service issues labels Aug 22, 2019
@SumiranAgg SumiranAgg requested a review from vincent1173 August 22, 2019 06:46
@SumiranAgg SumiranAgg requested a review from kmkumaran as a code owner August 22, 2019 06:46
@@ -18,6 +18,8 @@
"loc.input.help.slotName": "Enter or Select an existing Slot other than the Production slot.",
"loc.input.label.package": "Package or folder",
"loc.input.help.package": "File path to the package or a folder containing app service contents generated by MSBuild or a compressed zip or war file.<br />Variables ( [Build](https://docs.microsoft.com/vsts/pipelines/build/variables) | [Release](https://docs.microsoft.com/vsts/pipelines/release/variables#default-variables)), wildcards are supported. <br/> For example, $(System.DefaultWorkingDirectory)/\\*\\*/\\*.zip or $(System.DefaultWorkingDirectory)/\\*\\*/\\*.war.",
"loc.input.label.customWarName": "Custom war name",
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think function app takes WAR package. Can you please verify it once?

@@ -45,7 +45,7 @@ export class BuiltInLinuxWebAppDeploymentProvider extends AzureRmWebAppDeploymen

case PackageType.war:
tl.debug("Initiated deployment via kudu service for webapp war package : "+ this.taskParams.Package.getPath());
let warName = webCommonUtility.getFileNameFromPath(this.taskParams.Package.getPath(), ".war");
let warName = this.taskParams.CustomWarName || webCommonUtility.getFileNameFromPath(this.taskParams.Package.getPath(), ".war");
Copy link
Contributor

Choose a reason for hiding this comment

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

can we print this value in debug logs?

@SumiranAgg SumiranAgg merged commit 32e52fb into master Aug 27, 2019
@SumiranAgg SumiranAgg deleted the users/suaggar/warcustomname branch December 23, 2019 06:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: AzureAppService Label to monitor Azure App Service issues Area: Release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants