Closed madhuravius closed 1 year ago
Note - I used caddy as it has multiple exposed ports on their base docker image https://hub.docker.com/_/caddy for integration tests
caddy
This correctly will validate (https not allowed this, container_port and container_ports being mutually exclusive and/or only usable for tcp/tls):
https
container_port
container_ports
It will also correctly create:
Integration tests and unit tests also pass:
This is needed for merge: https://github.com/aptible/go-deploy/pull/63
This pull request has been linked to Shortcut Story #16403: Update Terraform Provider to properly support TCP/TLS endpoints.
After most recent changes, tests continue to pass:
ports
port
db
Note - I used
caddy
as it has multiple exposed ports on their base docker image https://hub.docker.com/_/caddy for integration testsThis correctly will validate (
https
not allowed this,container_port
andcontainer_ports
being mutually exclusive and/or only usable for tcp/tls):It will also correctly create:
Integration tests and unit tests also pass:
This is needed for merge: https://github.com/aptible/go-deploy/pull/63