Fix #194

Added check for broker-cluster-context parameter

Signed-off-by: Aswin Surayanarayanan <[email protected]>

pkg/subctl/cmd/join.go

@@ -96,6 +96,10 @@ var joinCmd = &cobra.Command{
 		err := checkArgumentPassed(args)
 		exitOnError("Argument missing", err)
 		subctlData, err := datafile.NewFromFile(args[0])
+		if subctlData.ServiceDiscovery {
+			err = checkBrokerContextPassed(brokerClusterContext)
+		}
+		exitOnError("Argument missing", err)
 		exitOnError("Error loading the broker information from the given file", err)
 		fmt.Printf("* %s says broker is at: %s\n", args[0], subctlData.BrokerURL)
 		exitOnError("Error connecting to broker cluster", err)
@@ -112,6 +116,13 @@ func checkArgumentPassed(args []string) error {
 	return nil
 }
 
+func checkBrokerContextPassed(brokerClusterContext string) error {
+	if brokerClusterContext == "" {
+		return errors.New("brokerClusterContext is not passed")
+	}
+	return nil
+}
+
 func joinSubmarinerCluster(config *rest.Config, subctlData *datafile.SubctlData) {
 
 	// Missing information