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

Add node function app examples. #113

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

xiaxyi
Copy link
Contributor

@xiaxyi xiaxyi commented Sep 23, 2022

No description provided.

Copy link
Member

@lonegunmanb lonegunmanb left a comment

Choose a reason for hiding this comment

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

Thanks @xiaxyi for opening this pr. I've met errors when I tried to apply 201-function-app-node module, and I think 201-function-app-key-vault-ref is duplicated with #114 .

@@ -0,0 +1,114 @@
data "azurerm_subscription" "current" {}
Copy link
Member

Choose a reason for hiding this comment

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

I've seen this 201-function-app-key-vault-ref in #114, is this by intention or by mistake?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

removed the folder, this is my mistake.

}

resource "azurerm_service_plan" "default_linux" {
name = "${var.name_prefix}-sp"
Copy link
Member

Choose a reason for hiding this comment

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

I've met a name collision due to declaring two azurerm_service_plan with the same name when I try to apply this module, please assign different names for these two plans, thanks.

}

resource "azurerm_service_plan" "default_windows" {
name = "${var.name_prefix}-sp"
Copy link
Member

Choose a reason for hiding this comment

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

I've met a name collision due to declaring two azurerm_service_plan with the same name when I try to apply this module, please assign different names for these two plans, thanks.


# Please use application_stack in site_config to set the node version, instead of specifying the WEBSITE_NODE_DEFAULT_VERSION under app_setting block. Terraform provider will handle the app_setting and set the necessary node runtime keys such as FUNCTIONS_EXTENSION_VERSION and WEBSITE_NODE_DEFAULT_VERSION.
resource "azurerm_linux_function_app" "default" {
name = "${var.name_prefix}-fa"
Copy link
Member

Choose a reason for hiding this comment

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

Same issue as the azurerm_service_plan, we'll meet name collision even these two function apps have different types. Please assign different names for these two function apps, thanks.

}

resource "azurerm_windows_function_app" "default" {
name = "${var.name_prefix}-fa"
Copy link
Member

Choose a reason for hiding this comment

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

Same here.

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.

None yet

2 participants