make runtime port and host configurable

ocis-pkg/config/config.go

@@ -68,6 +68,12 @@ const (
 
 type Mode int
 
+// Runtime configures the oCIS runtime when running in supervised mode.
+type Runtime struct {
+	Port string
+	Host string
+}
+
 // Config combines all available configuration parts.
 type Config struct {
 	Mode Mode
@@ -80,6 +86,7 @@ type Config struct {
 	GRPC         GRPC
 	Tracing      Tracing
 	TokenManager TokenManager
+	Runtime      Runtime
 
 	Accounts      *accounts.Config
 	GLAuth        *glauth.Config

ocis/pkg/command/kill.go

@@ -20,18 +20,20 @@ func KillCommand(cfg *config.Config) *cli.Command {
 		Category: "Runtime",
 		Flags: []cli.Flag{
 			&cli.StringFlag{
-				Name:    "hostname",
+				Name:        "hostname",
-				Value:   "localhost",
+				Value:       "localhost",
-				EnvVars: []string{"OCIS_RUNTIME_HOSTNAME"},
+				EnvVars:     []string{"OCIS_RUNTIME_HOSTNAME"},
+				Destination: &cfg.Runtime.Host,
 			},
 			&cli.StringFlag{
-				Name:    "port",
+				Name:        "port",
-				Value:   "6060",
+				Value:       "6060",
-				EnvVars: []string{"OCIS_RUNTIME_PORT"},
+				EnvVars:     []string{"OCIS_RUNTIME_PORT"},
+				Destination: &cfg.Runtime.Port,
 			},
 		},
 		Action: func(c *cli.Context) error {
-			client, err := rpc.DialHTTP("tcp", net.JoinHostPort("localhost", "6060"))
+			client, err := rpc.DialHTTP("tcp", net.JoinHostPort(cfg.Runtime.Host, cfg.Runtime.Port))
 			if err != nil {
 				log.Fatal("dialing:", err)
 			}

ocis/pkg/command/list.go

@@ -19,18 +19,20 @@ func ListCommand(cfg *config.Config) *cli.Command {
 		Category: "Runtime",
 		Flags: []cli.Flag{
 			&cli.StringFlag{
-				Name:    "hostname",
+				Name:        "hostname",
-				Value:   "localhost",
+				Value:       "localhost",
-				EnvVars: []string{"OCIS_RUNTIME_HOSTNAME"},
+				EnvVars:     []string{"OCIS_RUNTIME_HOSTNAME"},
+				Destination: &cfg.Runtime.Host,
 			},
 			&cli.StringFlag{
-				Name:    "port",
+				Name:        "port",
-				Value:   "6060",
+				Value:       "6060",
-				EnvVars: []string{"OCIS_RUNTIME_PORT"},
+				EnvVars:     []string{"OCIS_RUNTIME_PORT"},
+				Destination: &cfg.Runtime.Port,
 			},
 		},
 		Action: func(c *cli.Context) error {
-			client, err := rpc.DialHTTP("tcp", net.JoinHostPort("localhost", "6060"))
+			client, err := rpc.DialHTTP("tcp", net.JoinHostPort(cfg.Runtime.Host, cfg.Runtime.Port))
 			if err != nil {
 				log.Fatal("dialing:", err)
 			}

ocis/pkg/command/run.go

@@ -21,18 +21,20 @@ func RunCommand(cfg *config.Config) *cli.Command {
 		Category: "Runtime",
 		Flags: []cli.Flag{
 			&cli.StringFlag{
-				Name:    "hostname",
+				Name:        "hostname",
-				Value:   "localhost",
+				Value:       "localhost",
-				EnvVars: []string{"OCIS_RUNTIME_HOSTNAME"},
+				EnvVars:     []string{"OCIS_RUNTIME_HOSTNAME"},
+				Destination: &cfg.Runtime.Host,
 			},
 			&cli.StringFlag{
-				Name:    "port",
+				Name:        "port",
-				Value:   "6060",
+				Value:       "6060",
-				EnvVars: []string{"OCIS_RUNTIME_PORT"},
+				EnvVars:     []string{"OCIS_RUNTIME_PORT"},
+				Destination: &cfg.Runtime.Port,
 			},
 		},
 		Action: func(c *cli.Context) error {
-			client, err := rpc.DialHTTP("tcp", net.JoinHostPort("localhost", "6060"))
+			client, err := rpc.DialHTTP("tcp", net.JoinHostPort(cfg.Runtime.Host, cfg.Runtime.Port))
 			if err != nil {
 				log.Fatal("dialing:", err)
 			}

ocis/pkg/flagset/flagset.go

@@ -82,6 +82,20 @@ func RootWithConfig(cfg *config.Config) []cli.Flag {
 			EnvVars:     []string{"OCIS_JWT_SECRET"},
 			Destination: &cfg.TokenManager.JWTSecret,
 		},
+		&cli.StringFlag{
+			Name:        "runtime-port",
+			Value:       "6060",
+			Usage:       "Configures which port the runtime starts",
+			EnvVars:     []string{"OCIS_RUNTIME_PORT"},
+			Destination: &cfg.Runtime.Port,
+		},
+		&cli.StringFlag{
+			Name:        "runtime-host",
+			Value:       "localhost",
+			Usage:       "Configures the host where the runtime process is running",
+			EnvVars:     []string{"OCIS_RUNTIME_HOST"},
+			Destination: &cfg.Runtime.Host,
+		},
 	}
 }
 

ocis/pkg/runtime/service/service.go

@@ -149,8 +149,7 @@ func Start(o ...Option) error {
 	halt := make(chan os.Signal, 1)
 	signal.Notify(halt, syscall.SIGINT, syscall.SIGTERM, syscall.SIGQUIT, syscall.SIGHUP)
 
-	// TODO(refs) change default port
+	l, err := net.Listen("tcp", net.JoinHostPort(s.cfg.Runtime.Host, s.cfg.Runtime.Port))
-	l, err := net.Listen("tcp", fmt.Sprintf("%v:%v", "localhost", "6060"))
 	if err != nil {
 		s.Log.Fatal().Err(err)
 	}
@@ -158,8 +157,7 @@ func Start(o ...Option) error {
 	defer func() {
 		if r := recover(); r != nil {
 			reason := strings.Builder{}
-			// TODO(refs) change default port
+			if _, err := net.Dial("tcp", net.JoinHostPort(s.cfg.Runtime.Host, s.cfg.Runtime.Port)); err != nil {
-			if _, err := net.Dial("localhost", "6060"); err != nil {
 				reason.WriteString("runtime address already in use")
 			}