Some fixes

Signed-off-by: Prafulla Mahindrakar <[email protected]>
flyteorg · May 14, 2021 · 57ceb9f · 57ceb9f
1 parent 7c519c9
commit 57ceb9f
Show file tree

Hide file tree

Showing 4 changed files with 3 additions and 3 deletions.
diff --git a/cmd/delete/delete.go b/cmd/delete/delete.go
@@ -37,7 +37,7 @@ func RemoteDeleteCommand() *cobra.Command {
 		"cluster-resource-attribute": {CmdFunc: deleteClusterResourceAttributes, Aliases: []string{"cluster-resource-attributes"},
 			Short: clusterResourceAttributesShort,
 			Long:  clusterResourceAttributesLong, PFlagProvider: clusterresourceattribute.DefaultDelConfig, ProjectDomainNotRequired: true},
-		"execution-cluster-label": {CmdFunc: deleteExecutionQueueAttributes, Aliases: []string{"execution-cluster-labels"},
+		"execution-cluster-label": {CmdFunc: deleteExecutionClusterLabel, Aliases: []string{"execution-cluster-labels"},
 			Short: executionClusterLabelShort,
 			Long:  executionClusterLabelLong, PFlagProvider: executionclusterlabel.DefaultDelConfig, ProjectDomainNotRequired: true},
 		"execution-queue-attribute": {CmdFunc: deleteExecutionQueueAttributes, Aliases: []string{"execution-queue-attributes"},

diff --git a/...ble_execution_queue_cluster_label_test.go → ...matchable_execution_cluster_label_test.go b/...ble_execution_queue_cluster_label_test.go → ...matchable_execution_cluster_label_test.go
diff --git a/cmd/update/matchable_execution_cluster_label.go b/cmd/update/matchable_execution_cluster_label.go
@@ -54,7 +54,7 @@ Usage
 func updateExecutionClusterLabelFunc(ctx context.Context, args []string, cmdCtx cmdCore.CommandContext) error {
 	updateConfig := executionclusterlabel.DefaultUpdateConfig
 	if len(updateConfig.AttrFile) == 0 {
-		return fmt.Errorf("attrFile is mandatory while calling update for execution queue attribute")
+		return fmt.Errorf("attrFile is mandatory while calling update for execution cluster label")
 	}
 
 	executionClusterLabelFileConfig := executionclusterlabel.FileConfig{}

diff --git a/cmd/update/matchable_execution_cluster_label_test.go b/cmd/update/matchable_execution_cluster_label_test.go
@@ -24,7 +24,7 @@ func TestExecutionClusterLabel(t *testing.T) {
 		updateExecutionClusterLabelSetup()
 		err = updateExecutionClusterLabelFunc(ctx, args, cmdCtx)
 		assert.NotNil(t, err)
-		assert.Equal(t, fmt.Errorf("attrFile is mandatory while calling update for execution queue attribute"), err)
+		assert.Equal(t, fmt.Errorf("attrFile is mandatory while calling update for execution cluster label"), err)
 		tearDownAndVerify(t, ``)
 	})
 	t.Run("successful update project domain attribute", func(t *testing.T) {