Add Host IP to port response for v4 container response

[mythri-garaga]

Summary

Currently there is no way to identify ipv4 ports vs ipv6 ports from Task metadata endpoint v4. This PR includes changes to add Host IP to port response for Task metadata endpoint v4

Related to issue - #3096

Implementation details

• agent/handlers/v1/response.go : updated PortResponse struct to include HostIp field.
• agent/handlers/v2/response.go : added HostIp to container response if includeV4Metadata flag is set
• agent/handlers/v2/response_test.go: updated TestContainerResponseMarshal() test to check if containerResponse for includes HostIp when includeV4Metadata is set

Testing

Tested this change by running a task with container port mappings configured on a IPv6 enabled instance and see the following response for curl ${ECS_CONTAINER_METADATA_URI_V4:-}/task | jq

{
  "Cluster": "default",
  ...
  "Containers": [
    {
      "DockerId": "502ec633cc68f7e7ebdc00ae9b0c1f7ef730a0a9b3dc747107e5cfd91a42fb11",
      "Name": "server",
      "DockerName": "ecs-nginx-server-3-server-84f9f8dbe4879a8e8801",
      "Image": "nginx:latest",
      "ImageID": "sha256:c316d5a335a5cf324b0dc83b3da82d7608724769f6454f6d9a621f3ec2534a5a",
      "Ports": [
        {
          "ContainerPort": 8888,
          "Protocol": "tcp",
          "HostPort": 49825,
          "HostIp": "0.0.0.0"
        },
        {
          "ContainerPort": 8888,
          "Protocol": "tcp",
          "HostPort": 49825,
          "HostIp": "::"
        },
        {
          "ContainerPort": 8888,
          "Protocol": "tcp",
          "HostPort": 49832,
          "HostIp": "0.0.0.0"
        },
        {
          "ContainerPort": 8888,
          "Protocol": "tcp",
          "HostPort": 49832,
          "HostIp": "::"
        }
      ],
      ...
}

Container response with v2 curl ${ECS_CONTAINER_METADATA_URI:-}/task | jq

{
  "Cluster": "default",
   ...
  "Containers": [
    {
      "DockerId": "502ec633cc68f7e7ebdc00ae9b0c1f7ef730a0a9b3dc747107e5cfd91a42fb11",
      "Name": "server",
      "DockerName": "ecs-nginx-server-3-server-84f9f8dbe4879a8e8801",
      "Image": "nginx:latest",
      "ImageID": "sha256:c316d5a335a5cf324b0dc83b3da82d7608724769f6454f6d9a621f3ec2534a5a",
      "Ports": [
        {
          "ContainerPort": 8888,
          "Protocol": "tcp",
          "HostPort": 49825
        },
        {
          "ContainerPort": 8888,
          "Protocol": "tcp",
          "HostPort": 49825
        },
        {
          "ContainerPort": 8888,
          "Protocol": "tcp",
          "HostPort": 49832
        },
        {
          "ContainerPort": 8888,
          "Protocol": "tcp",
          "HostPort": 49832
        }
      ],
...
}

New tests cover the changes: yes

Description for the changelog

Licensing

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.