panic: keep lease panic

[bufferflies]

Bug Report

CampaignLeader maybe pass if the retry time has reached campaignLeaderFrequencyTimes, but the lease maybe not success ,so the lease time is zero .

pd/server/server.go

Lines 1645 to 1655 in cf718a7

	if err := s.member.CampaignLeader(s.ctx, s.cfg.LeaderLease); err != nil {
	if err.Error() == errs.ErrEtcdTxnConflict.Error() {
	log.Info(fmt.Sprintf("campaign %s leader meets error due to txn conflict, another PD/API server may campaign successfully", s.mode),
	zap.String("campaign-leader-name", s.Name()))
	} else {
	log.Error(fmt.Sprintf("campaign %s leader meets error due to etcd error", s.mode),
	zap.String("campaign-leader-name", s.Name()),
	errs.ZapError(err))
	}
	return
	}

What did you do?

What did you expect to see?

normal

What did you see instead?

"[log.go:87] [panic] [recover=\"\\\"non-positive interval for NewTicker\\\"\"] [stack=\"[github.com/tikv/pd/pkg/utils/logutil.LogPanic\\n\\t/home/jenkins/agent/workspace/build-common/go/src/github.com/pingcap/pd/pkg/utils/logutil/log.go:87\\nruntime.gopanic\\n\\t/usr/local/go/src/runtime/panic.go:914\\ntime.NewTicker\\n\\t/usr/local/go/src/time/tick.go:22\\ngithub.com/tikv/pd/pkg/election.(*lease).keepAliveWorker.func1\\n\\t/home/jenkins/agent/workspace/build-common/go/src/github.com/pingcap/pd/pkg/election/lease.go:152](http://github.com/tikv/pd/pkg/utils/logutil.LogPanic//n//t/home/jenkins/agent/workspace/build-common/go/src/github.com/pingcap/pd/pkg/utils/logutil/log.go:87//nruntime.gopanic//n//t/usr/local/go/src/runtime/panic.go:914//ntime.NewTicker//n//t/usr/local/go/src/time/tick.go:22//ngithub.com/tikv/pd/pkg/election.(*lease).keepAliveWorker.func1//n//t/home/jenkins/agent/workspace/build-common/go/src/github.com/pingcap/pd/pkg/election/lease.go:152)\"]"

What version of PD are you using (pd-server -V)?

8196d84

[CabinfeverB]

also see https://github.com/tikv/pd/actions/runs/7273239531/job/19816841865?pr=7597