awx_inventory_source doesn't allow empty credentials ID about terraform-provider-awx HOT 7 CLOSED

Changing the type inside the resource to "TypeString" (and the casts as well) I get the following error:

resource "awx_inventory_source" "dynamic_inventory_source" {
│
│ Unable to create Inventory Source got Errors:
│ - credential: [A valid integer is required.]

So I think there's work to do inside the https://github.com/denouche/goawx.git repository as well @denouche

from terraform-provider-awx.

Hello @fciava , I just pushed a PR trying to fix this bug.
Can you test if this branch of the provider resolve the issue on your side?

from terraform-provider-awx.

Hello @fciava

Requirements : go >= 1.14

You can clone the repo in your local directory and next install the terraform provider locally :

git clone ...
git checkout fix-inventory-source-no-credential
make install

And next point your terraform requirements to this repo:

terraform {
  required_providers {
    awx = {
      source  = "~/.terraform.d/plugins/github.com/denouche/awx/0.1/linux_amd64"
      version = "0.1"
    }
  }
}

from terraform-provider-awx.

Hello @denouche and @Tioborto,
thank you very much for your help.

I can confirm you that's working with both no credentials, with credential id set to 0 and to "0".

e.g.:

resource "awx_inventory_source" "dynamic_inventory_source" {
  name              = "dynamic_inventory_source"
  inventory_id      = awx_inventory.dynamic_inventory.id
  credential_id     = "0"
  overwrite         = true
  source            = "scm"
  source_project_id = awx_project.dynamic_inventory_project.id
  source_path       = "inventories/aws/all_running_hosts.aws_ec2.yml"
}

from terraform-provider-awx.

I am seeing the same problem with source_project_id, not allowing a 0 value when not necessary, such as for ec2 sources.

from terraform-provider-awx.

I created a new pull request to the same branch with the fix for source_project_id. (#29)

from terraform-provider-awx.

Hey @denouche! I've tested the PR locally as well, and confirmed it worked. Is there anything pending that PR being merged? I'd love to be able to point at the "official" release rather than a local build.

from terraform-provider-awx.