addon list for table write package and profile list

addons list and profile list and making pass the test

Author: pavansaikrishna78

cmd/minikube/cmd/config/addons_list.go

@@ -34,6 +34,7 @@ import (
 	"k8s.io/minikube/pkg/minikube/out"
 	"k8s.io/minikube/pkg/minikube/reason"
 	"k8s.io/minikube/pkg/minikube/style"
+	"k8s.io/minikube/pkg/minikube/constants"
 )
 
 var addonListOutput string
@@ -107,7 +108,7 @@ var printAddonsList = func(cc *config.ClusterConfig, printDocs bool) {
 	if cc == nil {
 		tHeader = []string{"Addon Name", "Maintainer"}
 	} else {
-		tHeader = []string{"Addon Name", "Profile", "Status", "Maintainer"}
+		tHeader = []string{"Addon Name", "Enabled", "Maintainer"}
 	}
 	if printDocs {
 		tHeader = append(tHeader, "Docs")
@@ -127,14 +128,80 @@ var printAddonsList = func(cc *config.ClusterConfig, printDocs bool) {
 		if docs == "" {
 			docs = "n/a"
 		}
+		
+		// Determine base row structure and colors
+		var enabled bool
+		var colorCode string
+		var applyColor bool
+		
+		if cc != nil {
+			enabled = addonBundle.IsEnabled(cc)
+			if enabled {
+				colorCode = constants.Enabled
+				applyColor = true
+			}
+		}
+		
+		// Build base row data
+		var rawValues []interface{}
 		if cc == nil {
-			temp = []string{addonName, maintainer}
+			rawValues = []interface{}{addonName, maintainer}
 		} else {
-			enabled := addonBundle.IsEnabled(cc)
-			temp = []string{addonName, cc.Name, fmt.Sprintf("%s %s", stringFromStatus(enabled), iconFromStatus(enabled)), maintainer}
+			status := ""
+			if enabled {
+				status = iconFromStatus(enabled)
+			}
+			rawValues = []interface{}{addonName, status, maintainer}
 		}
+		
+		// Add docs if needed
 		if printDocs {
-			temp = append(temp, docs)
+			rawValues = append(rawValues, docs)
+		}
+		
+		// Apply colors using loop
+		temp = make([]string, len(rawValues))
+		for i, value := range rawValues {
+			valueStr := fmt.Sprintf("%v", value)
+			
+			// Apply color based on context
+			shouldColorValue := false
+			var valueColorCode string
+			
+			if cc == nil {
+				// No coloring for null cluster config
+				temp[i] = valueStr
+				continue
+			}
+			
+			switch i {
+			case 0: // addonName
+				shouldColorValue = applyColor
+				valueColorCode = colorCode
+			case 1: // status (only for non-null cc)
+				shouldColorValue = applyColor
+				valueColorCode = colorCode
+			case 2: // maintainer
+				shouldColorValue = applyColor
+				valueColorCode = colorCode
+			default: // docs or other columns
+				if printDocs && i == len(rawValues)-1 {
+					// This is the docs column
+					if enabled {
+						shouldColorValue = true
+						valueColorCode = constants.Enabled
+					} else {
+						shouldColorValue = true
+						valueColorCode = constants.Disabled
+					}
+				}
+			}
+			
+			if shouldColorValue {
+				temp[i] = fmt.Sprintf("%s%s%s", valueColorCode, valueStr, constants.Default)
+			} else {
+				temp[i] = valueStr
+			}
 		}
 		tData = append(tData, temp)
 	}

cmd/minikube/cmd/config/profile_list.go

@@ -1,5 +1,5 @@
 /*
-Copyright 2019 The Kubernetes Authors All rights reserved.
+Copyright 2025 The Kubernetes Authors All rights reserved.
 
 Licensed under the Apache License, Version 2.0 (the "License");
 you may not use this file except in compliance with the License.
@@ -14,6 +14,21 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
+/*
+Copyright 2019 The Kubernetes Authors All rights reserved.
+
+Licensed under the Apache License, Version 2.0 (the "License");
+you may not use this file except in compliance with the License.
+You may obtain a copy of the License at
+
+    http://www.apache.org/licenses/LICENSE-2.0
+
+Unless required by applicable law or agreed to in writing, software
+distributed under the License is distributed on an "AS IS" BASIS,
+WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+See the License for the specific language governing permissions and
+limitations under the License.
+*/
 package config
 
 import (
@@ -23,6 +38,7 @@ import (
 	"strconv"
 	"strings"
 
+	"k8s.io/klog/v2"
 	"k8s.io/minikube/pkg/minikube/cluster"
 	"k8s.io/minikube/pkg/minikube/config"
 	"k8s.io/minikube/pkg/minikube/constants"
@@ -37,8 +53,6 @@ import (
 	"github.com/olekukonko/tablewriter"
 	"github.com/olekukonko/tablewriter/tw"
 	"github.com/spf13/cobra"
-
-	"k8s.io/klog/v2"
 )
 
 var (
@@ -73,21 +87,17 @@ func listProfiles() (validProfiles, invalidProfiles []*config.Profile, err error
 	} else {
 		validProfiles, invalidProfiles, err = config.ListProfiles()
 	}
-
 	return validProfiles, invalidProfiles, err
 }
 
 func printProfilesTable() {
 	validProfiles, invalidProfiles, err := listProfiles()
-
 	if err != nil {
 		klog.Warningf("error loading profiles: %v", err)
 	}
-
 	if len(validProfiles) == 0 {
 		exit.Message(reason.UsageNoProfileRunning, "No minikube profile was found.")
 	}
-
 	updateProfilesStatus(validProfiles)
 	renderProfilesTable(profilesToTableData(validProfiles))
 	warnInvalidProfiles(invalidProfiles)
@@ -100,13 +110,11 @@ func updateProfilesStatus(profiles []*config.Profile) {
 		}
 		return
 	}
-
 	api, err := machine.NewAPIClient()
 	if err != nil {
 		klog.Errorf("failed to get machine api client %v", err)
 	}
 	defer api.Close()
-
 	for _, p := range profiles {
 		p.Status = profileStatus(p, api).StatusName
 	}
@@ -127,9 +135,7 @@ func profileStatus(p *config.Profile, api libmachine.API) cluster.State {
 			},
 		}
 	}
-	clusterStatus := cluster.GetState(statuses, ClusterFlagValue(), p.Config)
-
-	return clusterStatus
+	return cluster.GetState(statuses, ClusterFlagValue(), p.Config)
 }
 
 func renderProfilesTable(ps [][]string) {
@@ -166,9 +172,8 @@ func profilesToTableData(profiles []*config.Profile) [][]string {
 			cpIP = cp.IP
 			cpPort = cp.Port
 		}
-
 		k8sVersion := p.Config.KubernetesConfig.KubernetesVersion
-		if k8sVersion == constants.NoKubernetesVersion { // for --no-kubernetes flag
+		if k8sVersion == constants.NoKubernetesVersion {
 			k8sVersion = "N/A"
 		}
 		var c, k string
@@ -179,11 +184,90 @@ func profilesToTableData(profiles []*config.Profile) [][]string {
 			k = "*"
 		}
 		if isDetailed {
-			data = append(data, []string{p.Name, p.Config.Driver, p.Config.KubernetesConfig.ContainerRuntime,
-				cpIP, strconv.Itoa(cpPort), k8sVersion, p.Status, strconv.Itoa(len(p.Config.Nodes)), c, k})
+			var colorCode string
+			switch p.Status {
+			case "OK":
+				colorCode = constants.Enabled
+			case "Broken", "Error":
+				colorCode = constants.Error
+			default:
+				colorCode = constants.Default
+			}
+			
+			rawValues := []interface{}{
+				p.Name,
+				p.Config.Driver,
+				p.Config.KubernetesConfig.ContainerRuntime,
+				cpIP,
+				cpPort,
+				k8sVersion,
+				p.Status,
+				len(p.Config.Nodes),
+				c,
+				k,
+			}
+			
+			row := make([]string, len(rawValues))
+			for i, value := range rawValues {
+				switch v := value.(type) {
+				case string:
+					row[i] = fmt.Sprintf("%s%s%s", colorCode, v, constants.Default)
+				case int:
+					row[i] = fmt.Sprintf("%s%d%s", colorCode, v, constants.Default)
+				default:
+					row[i] = fmt.Sprintf("%s%v%s", colorCode, v, constants.Default)
+				}
+			}
+			data = append(data, row)
 		} else {
-			data = append(data, []string{p.Name, p.Config.Driver, p.Config.KubernetesConfig.ContainerRuntime,
-				cpIP, k8sVersion, p.Status, strconv.Itoa(len(p.Config.Nodes)), c, k})
+			var colorCode string
+			var applyColor bool
+			switch p.Status {
+			case "OK":
+				colorCode = constants.Enabled
+				applyColor = true
+			case "Broken", "Error":
+				colorCode = constants.Error
+				applyColor = true
+			default:
+				applyColor = false
+			}
+			
+			rawValues := []interface{}{
+				p.Name,
+				p.Config.Driver,
+				p.Config.KubernetesConfig.ContainerRuntime,
+				cpIP,
+				k8sVersion,
+				p.Status,
+				len(p.Config.Nodes),
+				c,
+				k,
+			}
+			
+			row := make([]string, len(rawValues))
+			for i, value := range rawValues {
+				if applyColor {
+					switch v := value.(type) {
+					case string:
+						row[i] = fmt.Sprintf("%s%s%s", colorCode, v, constants.Default)
+					case int:
+						row[i] = fmt.Sprintf("%s%d%s", colorCode, v, constants.Default)
+					default:
+						row[i] = fmt.Sprintf("%s%v%s", colorCode, v, constants.Default)
+					}
+				} else {
+					switch v := value.(type) {
+					case string:
+						row[i] = v
+					case int:
+						row[i] = strconv.Itoa(v)
+					default:
+						row[i] = fmt.Sprintf("%v", v)
+					}
+				}
+			}
+			data = append(data, row)
 		}
 	}
 	return data
@@ -193,12 +277,10 @@ func warnInvalidProfiles(invalidProfiles []*config.Profile) {
 	if invalidProfiles == nil {
 		return
 	}
-
 	out.WarningT("Found {{.number}} invalid profile(s) ! ", out.V{"number": len(invalidProfiles)})
 	for _, p := range invalidProfiles {
 		out.ErrT(style.Empty, "\t "+p.Name)
 	}
-
 	out.ErrT(style.Tip, "You can delete them using the following command(s): ")
 	for _, p := range invalidProfiles {
 		out.Errf("\t $ minikube delete -p %s \n", p.Name)
@@ -208,8 +290,7 @@ func warnInvalidProfiles(invalidProfiles []*config.Profile) {
 func printProfilesJSON() {
 	validProfiles, invalidProfiles, err := listProfiles()
 	updateProfilesStatus(validProfiles)
-
-	var body = map[string]interface{}{}
+	body := map[string]interface{}{}
 	if err == nil || config.IsNotExist(err) {
 		body["valid"] = profilesOrDefault(validProfiles)
 		body["invalid"] = profilesOrDefault(invalidProfiles)

pkg/minikube/constants/constants.go

@@ -163,6 +163,12 @@ const (
 
 	// Mirror CN
 	AliyunMirror = "registry.cn-hangzhou.aliyuncs.com/google_containers"
+
+	//TABLE WRITER COLORS
+	Enabled = "\033[32m"
+	Default = "\033[0m"
+	Disabled = "\033[0m"
+	Error = "\033[31m"  
 )
 
 var (