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

Revise the fix for the random container ports

Open mavogel opened this issue 4 years ago • 6 comments

This issue was originally opened by @mavogel as https://github.com/hashicorp/terraform-provider-docker/issues/138. It was migrated here as a result of the community provider takeover from @kreuzwerker. The original body of the issue is below.


Current State

  • the ordering of ports numerically ascending of https://github.com/terraform-providers/terraform-provider-docker/issues/103 introduced the bug of https://github.com/terraform-providers/terraform-provider-docker/issues/110
  • converting back from list to set does not work because it would break https://github.com/terraform-providers/terraform-provider-docker/issues/103
  • https://github.com/terraform-providers/terraform-provider-docker/issues/115 fixed the issue but it does not seem to be clean
  • also the solution of https://github.com/terraform-providers/terraform-provider-docker/issues/128 is not clean

Desired state

  • convert the list back to a set and fix https://github.com/terraform-providers/terraform-provider-docker/issues/103
  • test the setup in windows. https://github.com/terraform-providers/terraform-provider-docker/issues/110 should set up the base line
  • also investigate if it is an internal SDK bug
  • consider the changer for the docker service resource https://github.com/terraform-providers/terraform-provider-docker/issues/222 as well

mavogel avatar Dec 25 '20 19:12 mavogel

This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 7 days. If you don't want this issue to be closed, please set the label pinned.

github-actions[bot] avatar Mar 29 '21 10:03 github-actions[bot]

Any update on this issue?

vipradwivedi avatar Mar 31 '21 06:03 vipradwivedi

Hi @vipradwivedi, we will tackle it in the next major, because this issue bugs me as well. I want it clean

mavogel avatar May 07 '21 06:05 mavogel

Related to #188

mavogel avatar May 13 '21 05:05 mavogel

This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 7 days. If you don't want this issue to be closed, please set the label pinned.

github-actions[bot] avatar Jul 12 '21 10:07 github-actions[bot]

Hi Any updated on fix for issue - Allow range of ports for containers and services

https://github.com/kreuzwerker/terraform-provider-docker/issues/73

pranshuverma avatar Apr 24 '23 04:04 pranshuverma