Merge pull request #19390 from k8s-infra-cherrypick-robot/cherry-pick-19367-to-release-3.6

[release-3.6] Fix a log formatting bug in the e2e downgrade tests

Author: ahrtr

tests/e2e/cluster_downgrade_test.go

@@ -24,7 +24,6 @@ import (
 	"github.com/coreos/go-semver/semver"
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"
-	"go.uber.org/zap"
 
 	"go.etcd.io/etcd/api/v3/version"
 	"go.etcd.io/etcd/client/pkg/v3/fileutil"
@@ -160,7 +159,7 @@ func testDowngradeUpgrade(t *testing.T, numberOfMembersToDowngrade int, clusterS
 	}
 
 	membersToChange := rand.Perm(len(epc.Procs))[:numberOfMembersToDowngrade]
-	t.Logf(fmt.Sprintln("Elect members for operations"), zap.Any("members", membersToChange))
+	t.Logf("Elect members for operations on members: %v", membersToChange)
 
 	t.Logf("Starting downgrade process to %q", lastVersionStr)
 	err = e2e.DowngradeUpgradeMembersByID(t, nil, epc, membersToChange, currentVersion, lastClusterVersion)

tests/framework/e2e/downgrade.go

@@ -84,7 +84,7 @@ func DowngradeCancel(t *testing.T, epc *EtcdProcessCluster) {
 
 func DowngradeUpgradeMembers(t *testing.T, lg *zap.Logger, clus *EtcdProcessCluster, numberOfMembersToChange int, currentVersion, targetVersion *semver.Version) error {
 	membersToChange := rand.Perm(len(clus.Procs))[:numberOfMembersToChange]
-	t.Logf(fmt.Sprintln("Elect members for operations"), zap.Any("members", membersToChange))
+	t.Logf("Elect members for operations on members: %v", membersToChange)
 
 	return DowngradeUpgradeMembersByID(t, lg, clus, membersToChange, currentVersion, targetVersion)
 }