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

Unlink log analytics workspace and automation account #555

Merged
merged 7 commits into from
Jun 14, 2023

Conversation

picccard
Copy link
Contributor

Overview/Summary

Adds option to not link the log analytics workspace with the automation account.

If the centralized workspace got the solution 'Updates', then the automation account will charge $6 for each non-azure VM (on-prem VM) that sends logs to the workspace. The non-azure VMs that shows up in the Configuration Management -> Inventory section of the automation account will result in cost.
https://azure.microsoft.com/en-us/pricing/details/automation/

This PR fixes/adds/changes/removes

  1. Adds param parLogAnalyticsWorkspaceLinkAutomationAccount to the logging module

Breaking Changes

  1. Parameter is optional, default behavior in the module is not changed.

Testing Evidence

None

As part of this Pull Request I have

@ghost ghost added the Needs: Triage 🔍 Needs triaging by the team label Jun 14, 2023
@picccard picccard changed the title Unlink automationaccount workspace Unlink log analytics workspace and automation account Jun 14, 2023
@jtracey93 jtracey93 added Area: Logging & Automation and removed Needs: Triage 🔍 Needs triaging by the team labels Jun 14, 2023
@jtracey93
Copy link
Collaborator

/azp run validateazcloud

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jtracey93 jtracey93 merged commit 48a1bfd into Azure:main Jun 14, 2023
@picccard picccard deleted the unlink-automationaccount-workspace branch June 14, 2023 14:21
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.

2 participants