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

New Log Analytics Sku - PerGB2018 #1513

Closed
wants to merge 1 commit into from
Closed

Conversation

MattMencel
Copy link

@MattMencel MattMencel commented Apr 5, 2018

  • I'm not making changes to Auto-Generated files which will just get erased next time there's a release.
  • I've tested my changes, adding unit tests where applicable.
  • I'm submitting this PR to the dev branch, or I'm fixing a bug that warrants its own release and I'm targeting the master branch.

On April 3rd, 2018 Microsoft changed the pricing tiers for Log Analytics.

https://azure.microsoft.com/en-us/blog/introducing-a-new-way-to-purchase-azure-monitoring-services/

I did some research and was able to find the new Sku name, PerGB2018. This is not officially documented anywhere yet that I can find. I successfully tested a deployment with the Azure RM Terraform provider using the new Sku with my forked code.

I believe the old Skus will still work if you have an older Azure subscription and have deployed Log Analytics services prior to April 3, 2018. I am unable to test this in my subscription.

@azuresdkci
Copy link

Can one of the admins verify this patch?

@msftclas
Copy link

msftclas commented Apr 5, 2018

CLA assistant check
All CLA requirements met.

@MattMencel MattMencel changed the title New Log Analytics Sku for Log Analytics New Log Analytics Sku - PerGB2018 Apr 5, 2018
@MattMencel
Copy link
Author

I was informed I should have updated the azure-rest-api-specs instead. I've done that now. If this PR just needs to be closed let me know.

@jhendrixMSFT
Copy link
Member

Closing as this should be fixed in the specs repo. Once the change has been merged into the specs repo we'll get it into the SDK.

@MattMencel MattMencel deleted the dev branch April 19, 2018 18:30
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.

4 participants