Add 2 Required Log Analytics Solutions for SQL Assessments for MDFC

[jtracey93]

See upstream issue Azure/Enterprise-Scale#979

Add 2 Required Log Analytics Solutions for SQL Assessments for MDFC.

Missing Solutions:

• SQLVulnerabilityAssessment
• SQLAdvancedThreatProtection

"product": "OMSGallery/SQLAdvancedThreatProtection",
"publisher": "Microsoft"

&

"product": "OMSGallery/SQLVulnerabilityAssessment",
"publisher": "Microsoft"

[krowlandson]

@jtracey93 Do you think these should be covered under the existing enable_solution_for_sql_assessment input variable, or need dedicated ones?

terraform-azurerm-caf-enterprise-scale/variables.tf

Lines 75 to 95 in 246cf7d

	variable "configure_management_resources" {
	type = object({
	settings = object({
	log_analytics = object({
	enabled = bool
	config = object({
	retention_in_days = number
	enable_monitoring_for_arc = bool
	enable_monitoring_for_vm = bool
	enable_monitoring_for_vmss = bool
	enable_solution_for_agent_health_assessment = bool
	enable_solution_for_anti_malware = bool
	enable_solution_for_azure_activity = bool
	enable_solution_for_change_tracking = bool
	enable_solution_for_service_map = bool
	enable_solution_for_sql_assessment = bool
	enable_solution_for_updates = bool
	enable_solution_for_vm_insights = bool
	enable_sentinel = bool
	})
	})

[jtracey93]

@krowlandson whilst I agree they could fit under the enable_solution_for_sql_assessment input variable I think they should be separate in an effort to provide as much felxibility as possible, just in case someone needs to not have one of the solutions but does want the other.

TLDR: separate 👍