Improve Service Discovery flag help text

SoManyHs · SoManyHs · commit f6aa876d1664 · 2019-01-25T15:53:25.000-08:00
diff --git a/ecs-cli/modules/commands/compose/service/compose_service_command.go b/ecs-cli/modules/commands/compose/service/compose_service_command.go
@@ -137,15 +137,15 @@ func serviceDiscoveryFlags() []cli.Flag {
 	return []cli.Flag{
 		cli.BoolFlag{
 			Name:  flags.EnableServiceDiscoveryFlag,
-			Usage: "[Service Discovery] Enable Service Discovery for your ECS Service",
+			Usage: "[Service Discovery] Enable Service Discovery for your ECS Service.",
 		},
 		cli.StringFlag{
 			Name:  flags.VpcIdFlag,
-			Usage: "[Service Discovery] The VPC that will be attached to the private DNS namespace",
+			Usage: "[Service Discovery] The VPC that will be attached to the private DNS namespace.",
 		},
 		cli.StringFlag{
 			Name:  flags.PrivateDNSNamespaceNameFlag,
-			Usage: "[Service Discovery] The name of the private DNS namespace to use with Service Discovery. The CLI creates the namespace if it doesn't already exist. For example, if the name is 'corp' than a service 'foo' will be reachable via DNS at 'foo.corp'",
+			Usage: "[Service Discovery] The name of the private DNS namespace to use with Service Discovery. The CLI creates the namespace if it doesn't already exist. For example, if the name is 'corp' than a service 'foo' will be reachable via DNS at 'foo.corp'.",
 		},
 		cli.StringFlag{
 			Name:  flags.PrivateDNSNamespaceIDFlag,
@@ -157,27 +157,27 @@ func serviceDiscoveryFlags() []cli.Flag {
 		},
 		cli.StringFlag{
 			Name:  flags.PublicDNSNamespaceNameFlag,
-			Usage: "[Service Discovery] The name of an existing public DNS namespace to use with Service Discovery. For example, if the name is 'corp' than a service 'foo' will be reachable via DNS at 'foo.corp'",
+			Usage: "[Service Discovery] The name of an existing public DNS namespace to use with Service Discovery. For example, if the name is 'corp' than a service 'foo' will be reachable via DNS at 'foo.corp'.",
 		},
 		cli.StringFlag{
 			Name:  flags.ServiceDiscoveryContainerNameFlag,
-			Usage: "[Service Discovery] The name of the container (service name in compose) that will use Service Discovery",
+			Usage: "[Service Discovery] The name of the container (service name in compose) that will use Service Discovery.",
 		},
 		cli.StringFlag{
 			Name:  flags.ServiceDiscoveryContainerPortFlag,
-			Usage: "[Service Discovery] The port on the container used for Service Discovery",
+			Usage: "[Service Discovery] The port on the container used for Service Discovery.",
 		},
 		cli.StringFlag{
 			Name:  flags.DNSTTLFlag,
-			Usage: "[Service Discovery] The TTL of the DNS Records used with the Route53 Service Discovery Resource",
+			Usage: "[Service Discovery] The TTL of the DNS Records used with the Route53 Service Discovery Resource. Default value is 60 seconds.",
 		},
 		cli.StringFlag{
 			Name:  flags.DNSTypeFlag,
-			Usage: "[Service Discovery] The type of the DNS Records used with the Route53 Service Discovery Resource (A or SRV). Note that SRV records require container name and container port",
+			Usage: "[Service Discovery] The type of the DNS Records used with the Route53 Service Discovery Resource (A or SRV). Note that SRV records require container name and container port.",
 		},
 		cli.StringFlag{
 			Name:  flags.HealthcheckCustomConfigFailureThresholdFlag,
-			Usage: "[Service Discovery] The number of 30-second intervals that you want service discovery service to wait after receiving an UpdateInstanceCustomHealthStatus request before it changes the health status of a service instance",
+			Usage: "[Service Discovery] The number of 30-second intervals that you want service discovery service to wait after receiving an UpdateInstanceCustomHealthStatus request before it changes the health status of a service instance. Default value is 1.",
 		},
 	}
 }
