feat: configure built-in logging

[kov117]

Checklist

• I've updated both the azurerm_linux_function_app and azurerm_windows_function_app resources.

[kov117]

Test passed 🎆

[hknutsen]

It's recommended for built-in logging to be disabled by default: https://learn.microsoft.com/en-us/azure/azure-functions/configure-monitoring?tabs=v2#disable-built-in-logging

[kov117]

will change the configuration to disabled 🫡

[hknutsen]

It's recommended for built-in logging to be disabled by default: https://learn.microsoft.com/en-us/azure/azure-functions/configure-monitoring?tabs=v2#disable-built-in-logging

Could even consider hard-coding it to be disabled, and recommend using App Insights instead. A variable for enabling built-in logging can always be added at a later date if absolutely needed.

[kov117]

i don't see a problem with that, if the rest of the team @equinor/terraform-baseline-maintainers agree then i will implement the suggested change 👍🏼

[kov117]

Ran a new test and it passed 👍🏼

[hknutsen]

It's recommended for built-in logging to be disabled by default: https://learn.microsoft.com/en-us/azure/azure-functions/configure-monitoring?tabs=v2#disable-built-in-logging

Could even consider hard-coding it to be disabled, and recommend using App Insights instead. A variable for enabling built-in logging can always be added at a later date if absolutely needed.

Waiting for feedback from @equinor/terraform-baseline-maintainers.

[hknutsen]

@kov117 Hardcode built-in logging to be disabled for now, and update tests accordingly. We can always add a variable later if absolutely necessary.