-
Notifications
You must be signed in to change notification settings - Fork 9.6k
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
provider/fastly Adds healthcheck service #11709
Conversation
ca06e7a
to
fe4e4d8
Compare
Thanks for the work here - unfortunately, the test is broken
Can you have a look and see? Thanks Paul |
Hey @stack72 I believe I fixed the issue there. |
53f79c5
to
fe07dcd
Compare
I also rebased to get rid of a merge conflict, let me know if you all would rather merging master than rebasing. |
* Adds schema for fastly healthcheck * Handles changes to the fastly healthcheck * Flattens and refreshed fastly healthchecks * Adds testing for fastly healthcheck * Adds website documentation for fastly healthcheck * Fixes terraform syntax in test examples
Thanks for fixing this up @travertischio This now LGTM and all tests pass :)
P. |
* Adds schema for fastly healthcheck * Handles changes to the fastly healthcheck * Flattens and refreshed fastly healthchecks * Adds testing for fastly healthcheck * Adds website documentation for fastly healthcheck * Fixes terraform syntax in test examples
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 have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
Allows the user to set up healthchecks for fastly
Fastly API
Closes #6944