-
Notifications
You must be signed in to change notification settings - Fork 4.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
There are a couple of issues with the Logic apps Standard terraform template shared at https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/logic_app_standard #18351
Comments
Amier3
added
question
documentation
service/logic
good first issue
and removed
question
labels
Sep 14, 2022
Hey @NitashaV Thanks for pointing that out, would you be willing to PR a change to the example config? It'd need to look something like this ( we'd appreciate you testing this out on your end if you take this up ):
|
Hi, I can work on this and submit a PR later if no one take this out. ;-) |
xyq175com
added a commit
to fidelity-contributions/hashicorp-terraform-provider-azurerm
that referenced
this issue
Nov 23, 2022
xyq175com
added a commit
to fidelity-contributions/hashicorp-terraform-provider-azurerm
that referenced
this issue
Nov 23, 2022
stephybun
pushed a commit
that referenced
this issue
Nov 28, 2022
* fix issue #18351: update a standard logic app with correct stack settings * fix issue #18351: update a standard logic app with correct stack settings * Update website/docs/r/logic_app_standard.html.markdown Co-authored-by: jackofallops <11830746+jackofallops@users.noreply.github.com> * fix format Signed-off-by: yan xu <58940428+xyq175com@users.noreply.github.com> * fix format by using terrafmt Signed-off-by: yan xu <58940428+xyq175com@users.noreply.github.com> Co-authored-by: Xu, Yan <yan.xu@fmr.com> Co-authored-by: jackofallops <11830746+jackofallops@users.noreply.github.com>
favoretti
pushed a commit
to favoretti/terraform-provider-azurerm
that referenced
this issue
Jan 12, 2023
) * fix issue hashicorp#18351: update a standard logic app with correct stack settings * fix issue hashicorp#18351: update a standard logic app with correct stack settings * Update website/docs/r/logic_app_standard.html.markdown Co-authored-by: jackofallops <11830746+jackofallops@users.noreply.github.com> * fix format Signed-off-by: yan xu <58940428+xyq175com@users.noreply.github.com> * fix format by using terrafmt Signed-off-by: yan xu <58940428+xyq175com@users.noreply.github.com> Co-authored-by: Xu, Yan <yan.xu@fmr.com> Co-authored-by: jackofallops <11830746+jackofallops@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Is there an existing issue for this?
Community Note
Terraform Version
1.0.0
AzureRM Provider Version
3.0.0
Affected Resource(s)/Data Source(s)
azurerm_logic_app_standard
Terraform Configuration Files
Debug Output/Panic Output
Expected Behaviour
The following settings need to be added to the template to ensure workflows work after the deployment
{
"name": "FUNCTIONS_WORKER_RUNTIME",
"value": "node",
"slotSetting": false
},
{
"name": "WEBSITE_NODE_DEFAULT_VERSION",
"value": "~14",
"slotSetting": false
}
Actual Behaviour
The current template deploys fine but key appsettings are missing from the configuration, workflows wont work.
Steps to Reproduce
No response
Important Factoids
No response
References
No response
The text was updated successfully, but these errors were encountered: