-
Notifications
You must be signed in to change notification settings - Fork 319
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
Added external Wiki service #1003
Conversation
…al wiki files vs windows
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.
Thanks. Just found a test bug.
internal/provider/resource_gitlab_service_external_wiki_test.go
Outdated
Show resolved
Hide resolved
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.
lgtm!
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.
Test failed
Error running post-test destroy, there may be dangling resources: [ERROR] External Wiki Service 89 still exists
@armsnyder I wonder if this is the downstream API as looking at the code nothing is apparent. I will check the GitLab API and see what it returns after some testing. |
Yup it appears there is an active flag that is set to false on delete. I will refactor the resource to include this computed field |
@armsnyder After looking at the other integrations code I believe most if not all of them need to have their resource + unit tests updated to include the active flag + some computed fields. The unit tests for destruction verification seem to rely on the project being marked for deletion vs the integration being deactivated on top of a 404 being impossible to get after an integration is activated the first time. |
Thanks for updating the code to match the other services. There's a new test failure now:
It looks like in the test you are comparing testCheckResourceAttrLazy(externalWikiResourceName, "updated_at", func() string { return externalWikiService.CreatedAt.Format(time.RFC3339) }), |
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.
lgtm thanks!
This functionality has been released in v3.14.0 of the Terraform GitLab Provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. For further feature requests or bug reports with this functionality, please create a new GitHub issue. Thank you! |
Resolves #996