Fix aux_address crash

[dubo-dubon-duponey]

@mavogel fixes #7

I'm not completely sure about this so... PTAL and let me know if this is the right approach.

[suzuki-shunsuke]

Basically, it looks good.
I add a small suggestion. #8 (comment)

[dubo-dubon-duponey]

Basically, it looks good.
I add a small suggestion. #8 (comment)

Fixed.
Thanks a lot for the help.

[suzuki-shunsuke]

CI failed, but it seems to have nothing to do with this pull request.

https://github.com/kreuzwerker/terraform-provider-docker/pull/8/checks?check_run_id=1484103804

/go/src/github.com/hashicorp/terraform-website/content/scripts/check-links.sh "http://127.0.0.1:4567/docs/providers/docker/"
Crawling site...
http://127.0.0.1:4567/docs/providers/docker/:
Remote file does not exist -- broken link!!!
Found 1 broken link.

http://127.0.0.1:4567/docs/providers/docker/

make[1]: *** [website-provider-test] Error 8
Makefile:85: recipe for target 'website-provider-test' failed
make[1]: Leaving directory '/go/src/github.com/hashicorp/terraform-website'
GNUmakefile:64: recipe for target 'website-test' failed
make: *** [website-test] Error 2
Error: Process completed with exit code 2.

[dubo-dubon-duponey]

CI failed, but it seems to have nothing to do with this pull request.

Yeah, looks like it's about docs / website.

@suzuki-shunsuke @mavogel is there a discord / slack for this project?

[suzuki-shunsuke]

suzuki-shunsuke mavogel is there a discord / slack for this project?

Sorry but I don't know because recently I became a collaborator.

---

I have confirmed the crash is solved by this pull request.

$ terraform apply                                                                                                                                    

An execution plan has been generated and is shown below.
Resource actions are indicated with the following symbols:
  + create

Terraform will perform the following actions:

  # docker_network.vlan will be created
  + resource "docker_network" "vlan" {
      + driver      = (known after apply)
      + id          = (known after apply)
      + internal    = (known after apply)
      + ipam_driver = "default"
      + name        = "pr-8"
      + options     = (known after apply)
      + scope       = (known after apply)

      + ipam_config {
          + aux_address = {
              + "foo" = "192.168.1.5"
            }
          + subnet      = "192.168.0.0/16"
        }
    }

Plan: 1 to add, 0 to change, 0 to destroy.

Do you want to perform these actions?
  Terraform will perform the actions described above.
  Only 'yes' will be accepted to approve.

  Enter a value: yes

docker_network.vlan: Creating...
docker_network.vlan: Creation complete after 2s [id=227b69944c4ddb549245f305206a7e3a76ba18af8f1eaa0077f893f3850a4020]

Apply complete! Resources: 1 added, 0 changed, 0 destroyed.

[suzuki-shunsuke]

@mavogel Can I merge this pull request?
I became a collaborator and have a permission to merge the pull request,
but I don't know the role which I'm expected and the rule of this repository yet.

[mavogel]

@dubo-dubon-duponey I just created one at gophers.slack.com -> #terraform-provider-docker
Maybe you're also interested in maintaining and helping us distribute the work on more shoulders. That would be awesome.

@suzuki-shunsuke

I became a collaborator and have a permission to merge the pull request,
but I don't know the role which I'm expected and the rule of this repository yet.

Feel free to merge yes. If you think you need an additional review add me.

[dubo-dubon-duponey]

@dubo-dubon-duponey I just created one at gophers.slack.com -> #terraform-provider-docker
Maybe you're also interested in maintaining and helping us distribute the work on more shoulders. That would be awesome.

I will happily continue reporting and triaging issues, and submitting small fixes for my own pet bugs ;-).

It's hard to formerly commit being available beyond that though, but yeah, happy to chip in...

[suzuki-shunsuke]

Thank you for your contribution!