Skip to content

Conversation

@juntakata
Copy link
Contributor

@juntakata juntakata commented Jan 29, 2019

it should be -RegenerationPeriod ([System.Timespan]::FromDays(180))

Description

-RegenerationPeriod 180 will result in a bad request.

Checklist

it should be -RegenerationPeriod ([System.Timespan]::FromDays(180))
@adxsdkps
Copy link
Collaborator

Can one of the admins verify this patch?

@cormacpayne
Copy link
Member

@juntakata Hey Jun, thanks for making this fix! In December we moved from the AzureRM module to our new Az module, so currently, all of our development efforts are towards Az. The change you're proposing is for the AzureRM documentation, which we aren't actively shipping, so I went ahead and made your change for the corresponding Az cmdlet in our master branch: #8433

I'm going to close this PR in favor of the one above; please let me know if you have any questions or concerns!

@juntakata
Copy link
Contributor Author

Thanks! Yes you are right. AzureRM is obsolete.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants