azurerm_monitor_action_group - add support for the use_common_alert_schema webhook property

[phires]

Updated Azure SDK API for insights to 2019-06-01 to be able to use the option 'UseCommonAlertSchema'

(fixs #3363)

[phires]

Build seems to fail, because travis does not know / import the updated dependency https://github.com/Azure/azure-sdk-for-go/tree/master/services/preview/monitor/mgmt/2019-06-01/insights
Any way for me to fix this?

[mbfrahry]

Hey @phires, dependencies are updated manually so you'll have to do this yourself. The below should do the trick

go get github.com/Azure/azure-sdk-for-go/services/preview/monitor/mgmt/2019-06-01/insights
go mod tidy
go mod vendor 

[phires]

Fix is commited, anythint I can do to resolve the conflicts?

[ghost]

Fix is commited, anythint I can do to resolve the conflicts?

You can merge the latest master to your branch to resolve these conflicts: https://stackoverflow.com/questions/26241428/this-pull-request-contains-merge-conflicts-that-must-be-resolved
Let me know if you need more help on this.

I am looking forward to seeing this feature in v1.36.0.

[phires]

Finally :) Thank you all, I'm learning something new every day!

[katbyte]

@phires, It seems like there are an excessive number of changes in this PR? many vendored in packages are being removed.. i'm not sure that is accurate? i tried to fix things locally but i can't push to your branch

[phires]

@katbyte Well, I guess I have to read up on all those "mod" stuff...
I've added you as a collaborator to my repository, your help is highly appreciated :)

[ghost]

@phires Can you also add me as a collaborator to your repo? I will try to sort it out with the missing files.

[katbyte]

Thanks @phires!

I've merged master & that fixed up the diff, LGTM now 👍

[ghost]

This has been released in version 1.36.0 of the provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. As an example:

provider "azurerm" {
    version = "~> 1.36.0"
}
# ... other configuration ...

[ghost]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 hashibot-feedback@hashicorp.com. Thanks!