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

[TestBot][Linux] Deploy on Azure using Azure Pipelines #48

Closed
wants to merge 25 commits into from

Conversation

ParadoxARG
Copy link

@ParadoxARG ParadoxARG commented May 30, 2019

This PR is based on #47 and #44. We will rebase it after they are merged.

Description

Create deployment template and scripts to deploy TestBot on an Azure Linux Host.

Steps to configure the Pipeline

We are going to use the Windows deployment Pipeline created on #44.

  1. Clone the Windows deployment Pipeline.
  2. Modify the Create Bot Resource Group task to point to the linux template. Replace the BotName, AppId and AppSecret parameters with the pipeline variables.

image

  1. Remove the Compress Bot Source Code Powershell task and replace it with a Command Line task with the following code:
Move $(System.DefaultWorkingDirectory)\libraries\testbot\deployment-scripts\linux\* $(System.DefaultWorkingDirectory)\libraries\testbot

git config --global user.name "BotBuilderDotNetPipeline"

git config --global user.email BotBuilderDotNet@Pipeline.com

git init

git add .

git commit -m "Add bot source code"

git remote add azure https://$(AzureDeploymentUser):$(AzureDeploymentPassword)@$(TestBotName).scm.azurewebsites.net:443/$(TestBotName).git

git push azure master

Also, set the Working Directory on the advanced tab to libraries/testbot.
image

  1. Remove the zip-push deploy line from Create Directline Azure CLI task.
    image

Testing

image

image

Bill7zz and others added 25 commits May 1, 2019 02:26
- Update AppServicePlan Region
- Update ZipUrl template to get an echo bot
- Separate the bot logic from the index.js file
- Update the package-lock.json
@ParadoxARG ParadoxARG requested a review from Bill7zz May 30, 2019 19:48
@denscollo
Copy link

I think that the steps' numbers are wrong. Last step should be number 4 instead of number 1

@gasper-az
Copy link

Can you blur the Azure subscription in the first image?

@ParadoxARG
Copy link
Author

Feedback applied @denscollo @GasparAcevedoZainSouthworks

Copy link

@gasper-az gasper-az left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link

@Bill7zz Bill7zz left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link

@denscollo denscollo left a comment

Choose a reason for hiding this comment

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

LGTM!

@ParadoxARG
Copy link
Author

Promoted to #961

@ParadoxARG ParadoxARG closed this May 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants