Azure / azure-powershell

Microsoft Azure PowerShell
Other
4.19k stars 3.8k forks source link

Fixing the names for Operational Insights cmdlets #2334

Open alexandair opened 8 years ago

alexandair commented 8 years ago

IMO, it would make sense to create aliases that will use "LogAnalytics" instead of "OperationalInsights".

"IntelligentPack" should be also changed to "Solution". For example:

Set-AzureRmOperationalInsightsIntelligencePack  > Set-AzureRmLogAnalyticsSolution
Get-AzureRmOperationalInsightsIntelligencePacks > Get-AzureRmLogAnalyticsSolution

Also, there is a bunch of wrongly named cmdlets that use a plural nouns instead of singular nouns:

Get-AzureRmOperationalInsightsIntelligencePacks
Get-AzureRmOperationalInsightsLinkTargets
Get-AzureRmOperationalInsightsSavedSearchResults
Get-AzureRmOperationalInsightsSearchResults
Get-AzureRmOperationalInsightsWorkspaceManagementGroups
Get-AzureRmOperationalInsightsWorkspaceSharedKeys
jeffpatton1971 commented 8 years ago

I like that idea @alexandair but maybe instead of operational insight (old name as I recall) perhaps replace all that with just OMS.

Set-AzureRMOMSIntelligencePack

I like the intelligence pack more than your suggestion of log analytics, because not all of the IP's are log analytics. but I could totally be wrong.

ghost commented 4 years ago

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @AzmonLogA.

jeffpatton1971 commented 4 years ago

Cleaning house I suppose