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

feat: add vultr container registry resource

Open im6h opened this issue 6 months ago • 4 comments

Description

  • Based on Vultr API, add resource container registry and data container registry
  • Add docs and examples to use the new resource

Checklist:

  • [x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • [x] Have you linted your code locally prior to submission?
  • [x] Have you successfully ran tests with your changes locally?

im6h avatar Dec 16 '23 01:12 im6h

I fixed golint and added resources to terrafrom code @optik-aper Would you like to check it again? Thanks

im6h avatar Dec 16 '23 02:12 im6h

I'd really like to see this merged! I'm in the process of writing TF manifests for all my stuff on Vultr, and container registry is one of the first things I need to have in order to bootstrap everything else.

I'm no Go expert, but I can hack on this and try to resolve the conflicts if needed.

Stealthmate avatar Feb 02 '24 12:02 Stealthmate

Sorry for the delay! I'll test this out and merge it this afternoon.

optik-aper avatar Feb 02 '24 19:02 optik-aper

I'd really like to see this merged! I'm in the process of writing TF manifests for all my stuff on Vultr, and container registry is one of the first things I need to have in order to bootstrap everything else.

I'm no Go expert, but I can hack on this and try to resolve the conflicts if needed.

Sure. I think we can make PR better before merging into the main

im6h avatar Feb 04 '24 03:02 im6h