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

Update template-oms.parameters.json #7

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Carl-Hugo
Copy link

Applied the Log Analytics pricing model change of April 2018 fix.

@slapointe
Copy link
Member

We know this changed recently, it would be a good thing to be this value by default in the parameters file. It would be more future proof IMO.

This requires a change in the Step1 documentation for those still running on the old pricing model (because you're not forced to switch).

Please also update the documentation from PerGB2018 to Free, adapt the screen shots and error message and we will let this change go through.

@slapointe
Copy link
Member

slapointe commented Jun 29, 2018

Do you want to perform the changes @Carl-Hugo ? If not, we'll leave it this way and reject the PR.

@Carl-Hugo
Copy link
Author

This is on my todo list. I will do my best to complete it before next year Bootcamp.

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