Skip to content
This repository has been archived by the owner on Dec 9, 2024. It is now read-only.

feat: Dynamically generate ARM templates #174

Merged
merged 17 commits into from
Jun 24, 2019
Merged

Conversation

wbreza
Copy link
Contributor

@wbreza wbreza commented Jun 20, 2019

Based on various well-known skus we can generate ARM templates with the correct resources and expose parameters.

Well Known variations:

  1. Consumption
  2. Premium
  3. AppServiceEnvironment

Consolidate ARM template deployment strategy into ArmService

@wbreza wbreza changed the base branch from master to dev June 20, 2019 01:03
@wbreza wbreza changed the base branch from dev to wabrez/cli-options June 20, 2019 01:03
@wbreza wbreza force-pushed the wabrez/split-arm-templates branch from ac07475 to 4fb18be Compare June 20, 2019 01:07
src/armTemplates/ase.ts Outdated Show resolved Hide resolved
src/services/armService.ts Outdated Show resolved Hide resolved
Copy link
Contributor

@tbarlow12 tbarlow12 left a comment

Choose a reason for hiding this comment

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

Looks great. Good idea

@pjlittle
Copy link
Contributor

Good stuff -- like the flexibility of options.

@wbreza wbreza force-pushed the wabrez/split-arm-templates branch from dc5028e to 86c3bc7 Compare June 21, 2019 19:48
@wbreza wbreza changed the base branch from wabrez/cli-options to dev June 21, 2019 19:48
@wbreza wbreza force-pushed the wabrez/split-arm-templates branch from 7293a1d to c49b66c Compare June 21, 2019 22:42
@wbreza wbreza changed the title feat: WIP: Dynamically generate ARM templates feat: Dynamically generate ARM templates Jun 21, 2019
Copy link
Contributor

@mydiemho mydiemho left a comment

Choose a reason for hiding this comment

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

this PR is way too big. it's really hard to focus and do a good review

src/armTemplates/resources/appServicePlan.ts Show resolved Hide resolved
src/armTemplates/premium.ts Outdated Show resolved Hide resolved
src/armTemplates/consumption.ts Outdated Show resolved Hide resolved
src/armTemplates/ase.ts Show resolved Hide resolved
Copy link
Contributor

@tbarlow12 tbarlow12 left a comment

Choose a reason for hiding this comment

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

Had a few questions/suggestions, otherwise looks great :shipit:

src/armTemplates/consumption.ts Outdated Show resolved Hide resolved
src/armTemplates/premium.ts Outdated Show resolved Hide resolved
src/armTemplates/resources/apim.ts Outdated Show resolved Hide resolved
src/armTemplates/resources/appInsights.ts Outdated Show resolved Hide resolved
src/models/serverless.ts Show resolved Hide resolved
src/plugins/deploy/azureDeployPlugin.test.ts Show resolved Hide resolved
src/services/armService.ts Outdated Show resolved Hide resolved
src/services/baseService.ts Show resolved Hide resolved
src/services/baseService.ts Outdated Show resolved Hide resolved
@@ -33,7 +35,7 @@ describe("Function App Service", () => {

beforeAll(() => {

// TODO: How to spy on defaul exported function?
// TODO: How to spy on default exported function?
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove this. Pretty sure I added it, but I don't think it's needed

@wbreza wbreza merged commit 09d7d05 into dev Jun 24, 2019
@wbreza wbreza deleted the wabrez/split-arm-templates branch June 25, 2019 18:34
tbarlow12 pushed a commit that referenced this pull request Sep 13, 2019
Based on various well-known skus we can generate ARM templates with the correct resources and expose parameters.

Well Known variations:

- Consumption
- Premium
- AppServiceEnvironment

Consolidate ARM template deployment strategy into ArmService
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants