Increase default connect timeout to improve reliability in CI

Signed-off-by: Huan Wang <[email protected]>
pivotal-cf · Dec 31, 2018 · bc458e1 · bc458e1
1 parent d408b54
commit bc458e1
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/acceptance/help_test.go b/acceptance/help_test.go
@@ -15,7 +15,7 @@ om helps you interact with an Ops Manager
 Usage: om [options] <command> [<args>]
   --client-id, -c, OM_CLIENT_ID                          string  Client ID for the Ops Manager VM (not required for unauthenticated commands)
   --client-secret, -s, OM_CLIENT_SECRET                  string  Client Secret for the Ops Manager VM (not required for unauthenticated commands)
-  --connect-timeout, -o                                  int     timeout in seconds to make TCP connections (default: 5)
+  --connect-timeout, -o                                  int     timeout in seconds to make TCP connections (default: 10)
   --decryption-passphrase, -d, OM_DECRYPTION_PASSPHRASE  string  Passphrase to decrypt the installation if the Ops Manager VM has been rebooted (optional for most commands)
   --env, -e                                              string  env file with login credentials
   --help, -h                                             bool    prints this usage information (default: false)
@@ -86,7 +86,7 @@ This unauthenticated command helps setup the internal userstore authentication m
 Usage: om [options] configure-authentication [<args>]
   --client-id, -c, OM_CLIENT_ID                          string  Client ID for the Ops Manager VM (not required for unauthenticated commands)
   --client-secret, -s, OM_CLIENT_SECRET                  string  Client Secret for the Ops Manager VM (not required for unauthenticated commands)
-  --connect-timeout, -o                                  int     timeout in seconds to make TCP connections (default: 5)
+  --connect-timeout, -o                                  int     timeout in seconds to make TCP connections (default: 10)
   --decryption-passphrase, -d, OM_DECRYPTION_PASSPHRASE  string  Passphrase to decrypt the installation if the Ops Manager VM has been rebooted (optional for most commands)
   --env, -e                                              string  env file with login credentials
   --help, -h                                             bool    prints this usage information (default: false)

diff --git a/main.go b/main.go
@@ -40,7 +40,7 @@ type options struct {
 	ClientSecret         string `yaml:"client-secret"        short:"s"  long:"client-secret"       env:"OM_CLIENT_SECRET"                 description:"Client Secret for the Ops Manager VM (not required for unauthenticated commands)"`
 	Help                 bool   `                            short:"h"  long:"help"                                       default:"false" description:"prints this usage information"`
 	Password             string `yaml:"password"             short:"p"  long:"password"            env:"OM_PASSWORD"                      description:"admin password for the Ops Manager VM (not required for unauthenticated commands)"`
-	ConnectTimeout       int    `yaml:"connect-timeout"      short:"o"  long:"connect-timeout"                            default:"5"     description:"timeout in seconds to make TCP connections"`
+	ConnectTimeout       int    `yaml:"connect-timeout"      short:"o"  long:"connect-timeout"                            default:"10"    description:"timeout in seconds to make TCP connections"`
 	RequestTimeout       int    `yaml:"request-timeout"      short:"r"  long:"request-timeout"                            default:"1800"  description:"timeout in seconds for HTTP requests to Ops Manager"`
 	SkipSSLValidation    bool   `yaml:"skip-ssl-validation"  short:"k"  long:"skip-ssl-validation"                        default:"false" description:"skip ssl certificate validation during http requests"`
 	Target               string `yaml:"target"               short:"t"  long:"target"              env:"OM_TARGET"                        description:"location of the Ops Manager VM"`