Detector fails to link Slack notification silently about terraform-provider-signalform HOT 6 OPEN

Hi @blkperl actually we never implemented the slack integration in this provider (L263). I'll make a quick patch.

from terraform-provider-signalform.

#9 out for review. I'll update you as soon as it gets merged. I tested it locally. If it keeps failing with an error that looks like the following, the problem is related to invalid credentials:

• signalform_detector. site_capacity.0: For the resource SignalFx returned status 400:

HTTP ERROR 400

Problem accessing /v2/detector. Reason:

    Bad Request

from terraform-provider-signalform.

Tested #9 and confirmed that it works!

from terraform-provider-signalform.

This could go in a follow up story but I think it should throw an error if the notification type is not defined.

from terraform-provider-signalform.

Hi it's a great idea but unfortunately ValidateFunc only works for primitive types so it makes kinda hard to do validations on lists. thanks for the feedback and for taking some time to test my change. really appreciated that ;)

from terraform-provider-signalform.

It might be possible to add the validation check to https://github.com/Yelp/signalform-tools, since we have already a command to perform more advanced validation of terraform resources: https://github.com/Yelp/signalform-tools/blob/master/signalform_tools/validate.py

We run this validation as git pre-commit hook (https://pre-commit.com/) for our internal repo at Yelp.

@blkperl If this could work for you and you want to take a stab at it, feel free to go ahead.

from terraform-provider-signalform.