Removing the ignore on start-node and minor bug fix (flyteorg#176)

Signed-off-by: Prafulla Mahindrakar <[email protected]>

flytectl/.gitignore

@@ -5,4 +5,5 @@ bin
 .DS_Store
 _test
 ./config.yaml
-docs/build/*
+docs/build/*
+cmd/upgrade/flyte.ext

flytectl/cmd/get/node_execution.go

@@ -5,7 +5,6 @@ import (
 	"context"
 	"sort"
 	"strconv"
-	"strings"
 
 	cmdCore "github.com/flyteorg/flytectl/cmd/core"
 	"github.com/flyteorg/flytectl/pkg/printer"
@@ -149,10 +148,6 @@ func getNodeExecDetailsInt(ctx context.Context, project, domain, execName, nodeN
 				return nil, err
 			}
 		} else {
-			// Bug in admin https://github.com/flyteorg/flyte/issues/1221
-			if strings.HasSuffix(nodeExec.Id.NodeId, "start-node") {
-				continue
-			}
 			taskExecList, err := cmdCtx.AdminFetcherExt().FetchTaskExecutionsOnNode(ctx,
 				nodeExec.Id.NodeId, execName, project, domain)
 			if err != nil {
@@ -260,6 +255,9 @@ func createNodeDetailsTreeView(rootView gotree.Tree, nodeExecutionClosures []*No
 
 func extractLiteralMap(literalMap *core.LiteralMap) (map[string]interface{}, error) {
 	m := make(map[string]interface{})
+	if literalMap == nil || literalMap.Literals == nil {
+		return m, nil
+	}
 	for key, literalVal := range literalMap.Literals {
 		extractedLiteralVal, err := coreutils.ExtractFromLiteral(literalVal)
 		if err != nil {

flytectl/cmd/get/node_execution_test.go

@@ -312,3 +312,13 @@ func TestGetExecutionDetails(t *testing.T) {
 		assert.Equal(t, fmt.Errorf("unable to fetch task exec details"), err)
 	})
 }
+
+func TestExtractLiteralMapError(t *testing.T) {
+	literalMap, err := extractLiteralMap(nil)
+	assert.Nil(t, err)
+	assert.Equal(t, len(literalMap), 0)
+
+	literalMap, err = extractLiteralMap(&core.LiteralMap{})
+	assert.Nil(t, err)
+	assert.Equal(t, len(literalMap), 0)
+}