-
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
App Service Deploy 4.* Task Version (Preview) #6422
Changes from all commits
e8cac40
5271132
c3f560f
e036c9e
74f259b
ee241c1
d1d3f4b
1bab242
b456d17
6a2029d
56ba5e2
5b56a2e
2aa7310
123e2a6
16bacba
8f4accf
d917bc7
bf8804f
c263ce8
1e0553f
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -22,7 +22,7 @@ export class ContainerBasedDeploymentUtility { | |
} | ||
|
||
public async deployWebAppImage(taskParameters: TaskParameters): Promise<void> { | ||
var imageName = this._getImageName(); | ||
let imageName: string = this._getDockerHubImageName(); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Since we are jumping task major version here. There is an opportunity for us to bring docker tasks and azure web app task in sync with how they take image name as input. in Docker tasks there is only one input image name which includes repository name and tag while azure weapp task takes them separately. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I thought the reason these are separate is because at some point we want to enable drop down for each of these inputs, which will not be possible if we take them as a single input. |
||
tl.debug("Deploying an image " + imageName + " to the webapp " + this._appService.getName()); | ||
|
||
tl.debug("Updating the webapp configuration."); | ||
|
@@ -154,7 +154,7 @@ export class ContainerBasedDeploymentUtility { | |
} | ||
|
||
private async _getContainerRegistrySettings(imageName, endPoint): Promise<string> { | ||
var containerRegistryType: string = tl.getInput('ImageSource', true); | ||
var containerRegistryType: string = 'Registry'; | ||
var containerRegistrySettings: string = "-DOCKER_CUSTOM_IMAGE_NAME " + imageName; | ||
var containerRegistryAuthParamsFormatString: string = "-DOCKER_REGISTRY_SERVER_URL %s -DOCKER_REGISTRY_SERVER_USERNAME %s -DOCKER_REGISTRY_SERVER_PASSWORD %s"; | ||
|
||
|
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.
@jikuma - Can you review the changes related container deployment.