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

Make AzureWebJobsDashboard optional for Function Apps #1979

Closed
scottambroseio opened this issue Sep 26, 2018 · 1 comment · Fixed by #2268
Closed

Make AzureWebJobsDashboard optional for Function Apps #1979

scottambroseio opened this issue Sep 26, 2018 · 1 comment · Fixed by #2268
Assignees
Milestone

Comments

@scottambroseio
Copy link

Hey all,

Based on the documentation here https://docs.microsoft.com/en-us/azure/azure-functions/functions-monitoring#disable-built-in-logging it would be nice for the AzureWebJobsDashboard app setting to be optional.

If you enable Application Insights, we recommend that you disable the built-in logging that uses Azure storage. The built-in logging is useful for testing with light workloads but is not intended for high-load production use

To disable built-in logging, delete the AzureWebJobsDashboard app setting

I'm note sure what the ideal API would look like, but something along the lines of the following might work (with disable_inbuilt_logging defaulting to false to keep backwards compatibility)?

resource "azurerm_function_app" "fa" {
  name                                   = "scott-fnapp-dev-eun-func"
  location                               = "${azurerm_resource_group.rg.location}"
  resource_group_name        = "${azurerm_resource_group.rg.name}"
  app_service_plan_id            = "${azurerm_app_service_plan.sp.id}"
  storage_connection_string = "${azurerm_storage_account.sa.primary_connection_string}"
  disable_inbuilt_logging      = true
}

Cheers!

@ghost
Copy link

ghost commented Mar 6, 2019

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 [email protected]. Thanks!

@ghost ghost locked and limited conversation to collaborators Mar 6, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants