-
Notifications
You must be signed in to change notification settings - Fork 142
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
Add ability to terraform fastly users #214
Conversation
confused about why travis is failing... since go-fastly was updated to latest in an earlier PR :-( |
re: travis... looks like it's because some of the dependencies now require go 1.12 and 1.13 |
I think maybe |
isn't this provider also using 1.13? |
@chenrui333 everything was working locally (where I was using 1.13)... but CI uses testing locally using 1.11 I get...
|
Let me help with the CI a little bit, this travis config is already out of sync with the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good @sbfaulkner, thank you for your contribution. A few minor changes requested. Would also be good to see the acceptance tests output.
@sbfaulkner I have now merged @chenrui333's PR #215 so hopefully your tests may pass now if you want to rebase and re-run them. |
@phamann thanks... it does in fact pass now, and I think I've addressed all points from your review :-) |
glad it works out! |
I re-ran the acceptance tests with your new changes and all are passing.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 This is looking great @sbfaulkner , I added one minor suggestion to the website documentation, but other than that I am happy to merge.
Co-Authored-By: Patrick Hamann <[email protected]>
thanks @phamann - if you can ping me once you cut a new release, we'll start using it right away :-) |
related: #134
(I think) this PR adds support for basic user crud operations -- and updates the docs
a separate PR would be needed add support for configuring service access for the "engineer" role (and probably requires additional support in go-fastly)
please let me know what else might be needed to get this minimum viable support merged @philippschulte :-)
acceptance tests...