terraform-provider-github icon indicating copy to clipboard operation
terraform-provider-github copied to clipboard

Migrate creation of repos from a template to the GraphQL API

Open pmjacinto opened this issue 4 years ago • 3 comments

Fixes #836.

pmjacinto avatar Jun 27 '21 15:06 pmjacinto

Hey @jcudit can I get a review on this PR?

We've had to change to our forked version because of this issue, but we'd like to get back to the upstream version.

thanks!

pmjacinto avatar Aug 07 '21 09:08 pmjacinto

A couple of concerns with this one:

  • We've had a large maintenance burden come out of switching to the v4 API for the branch protection resource and am concerned about introducing it further with such a popular resource.
  • Our testing suite is not well equipped to exercise this change given the usage scenario of non-administrative access. Ideally, we'd grow that capability and revisit here with a stronger testing foundation.

We have an issue open to fix how our rate limiting operates with the v4 API. I'd like to revisit this feature once that is cleared up so we have better confidence with relying on the v4 API for more of our resources.

jcudit avatar Aug 31 '21 19:08 jcudit

hey, @jcudit thanks for the feedback, I see your point.

Just an update from my side: we've finally managed to test the provider with a GitHub app that has R&W permissions on everything and I can confirm that (as of 4.17.0) it is not possible to create repos from a template.

Error: PATCH https://api.github.com/repos/my-org/test-test-test3: 422 Visibility can't be changed by this user. []

pmjacinto avatar Nov 03 '21 11:11 pmjacinto

👋 Hey Friends, this issue has been automatically marked as stale because it has no recent activity. It will be closed if no further activity occurs. Please add the Status: Pinned label if you feel that this issue needs to remain open/active. Thank you for your contributions and help in keeping things tidy!

nickfloyd avatar Nov 30 '22 16:11 nickfloyd

👋 Hey Friends, this pull request has been automatically marked as stale because it has no recent activity. It will be closed if no further activity occurs. Please add the Status: Pinned label if you feel that this issue needs to remain open/active. Thank you for your contributions and help in keeping things tidy!

github-actions[bot] avatar Aug 30 '23 01:08 github-actions[bot]