foreman_katello_product is broken

terraform-coop / terraform-provider-foreman

Terraform provider for Foreman

https://registry.terraform.io/providers/terraform-coop/foreman

Mozilla Public License 2.0

33 stars 31 forks source link

foreman_katello_product is broken #54

Closed agriffit79 closed 5 months ago

agriffit79 commented 2 years ago

It seems recent versions of Katello have made organization_id a required parameter. The provider needs to be updated to handle this.

GET /katello/api/products

I've started investigating this. I'm thinking to use the ID in the provider config rather than make it an explicit parameter of the resource.

lhw commented 2 years ago

Lovely. Another version of the API call which doesn't match the same paradigm they already used elsewhere. The Foreman API might be one of the worst, or at least most inconsistent ones I have ever seen

bitkeks commented 5 months ago

Fixed in bc544ceddb86f143bf72d14929419623d2811812, merged via PR #142