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

Move api to separate repo

Open mikelsid opened this issue 2 years ago • 4 comments

Is your feature request related to a problem? Please describe. Hello, I wanted to consider moving the proxmox API to a separate repo It has a much higher coverage than Telmate's and I think it could be easily added to the Proxmox wiki, hence attract more contributors It's also currently impossible to import it to other projects, since the api uses custom types that are placed in internal package

mikelsid avatar Jul 11 '23 15:07 mikelsid

I think this is a good idea, but i'm bit worried about extra complexity of managing changes across 2 repos.

Started playing with it in

  • https://github.com/bpg/terraform-provider-proxmox/tree/move-api-client-out
  • https://github.com/bpg/proxmox-api/tree/move-api-client

bpg avatar Jul 16 '23 18:07 bpg

Marking this issue as stale due to inactivity in the past 180 days. This helps us focus on the active issues. If this issue is reproducible with the latest version of the provider, please comment. If this issue receives no comments in the next 30 days it will automatically be closed. If this issue was automatically closed and you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thank you!

github-actions[bot] avatar Jan 13 '24 00:01 github-actions[bot]

Hey! I would love to help out here. I've been playing around with an autogenerated go api library for Proxmox. I run into some slight hiccups here & there (you can see from my last two commits) due to Proxmox's idea of a jsonschema not really being the best implementation of one, nor is the API. But it generally works pretty well.

I'd love more contribution if folks would like to help out. https://pkg.go.dev/github.com/FreekingDean/proxmox-api-go https://github.com/FreekingDean/proxmox-api-go

Would be nice if folks using proxmox in go could use a single API that got all the love instead of everyone having to re-implement themselves via resty!

FreekingDean avatar Jan 17 '24 14:01 FreekingDean

Hey @FreekingDean! 👋🏼

I'm contemplating your proposal for several days now, but can't commit myself to switching the underlying API client to a different one, it seems too much undertaking from my side, but with relatively low return value for the provider users.

If we have a bigger team of contributors here I would consider the switch, but for now my main long-term goal is to migrate from Terraform SDK to Terraform Plugin Framework, fixing long list of provider's state management issues along the road.

bpg avatar Jan 31 '24 03:01 bpg

Marking this issue as stale due to inactivity in the past 180 days. This helps us focus on the active issues. If this issue is reproducible with the latest version of the provider, please comment. If this issue receives no comments in the next 30 days it will automatically be closed. If this issue was automatically closed and you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thank you!

bpg-autobot[bot] avatar Jul 30 '24 00:07 bpg-autobot[bot]

At this stage I don't think separating Proxmox API client from the provider will be beneficial. We keep changing the API layer, and separation only adds to the maintenance headache.

bpg avatar Jul 30 '24 02:07 bpg