Update portMappings to include containerPortRange

[asmithdt]

what

Allow specifying containerPortRange in portMappings

why

To support containerPortRange

references

[joe-niland]

/terratest

[aknysh]

/terratest

[joe-niland]

Hi @asmithdt The tests will need updating to allow the Terratest check to pass. Are you happy to take a look?

[asmithdt]

Apologies, I missed these messages. I have made a change which I hope will resolve the tests, I can't seem to run locally to check though; not sure if that's something I need to setup with the build-harness? Let me know if that doesn't get it. Cheers!

[mergify[bot]]

💥 This pull request now has conflicts. Could you fix it @asmithdt? 🙏

[mergify[bot]]

This PR was closed due to inactivity and merge conflicts. 😭 Please resolve the conflicts and reopen if necessary.

[YeomansIII]

@asmithdt Anything I can do to help on this? Honestly not sure how to get containerPortRange to work in terraform at all, even though it supported on ECS. If anyone has any workarounds in the meantime, like having terraform forgo any validation of the container_definition (I can't think of another way to get containerPortRange into AWS...)

[mergify[bot]]

💥 This pull request now has conflicts. Could you fix it @asmithdt? 🙏

[joe-niland]

Hi @asmithdt

Sorry for the delay.

Could you please run the following and commit the result?

make init
make readme

[mergify[bot]]

This PR was closed due to inactivity and merge conflicts. 😭 Please resolve the conflicts and reopen if necessary.