mirror of
				https://github.com/optim-enterprises-bv/kubernetes.git
				synced 2025-11-04 04:08:16 +00:00 
			
		
		
		
	Merge pull request #105615 from MikeSpreitzer/correct-linger-logging
Unconfuse logging wrt additional latency
This commit is contained in:
		@@ -877,11 +877,11 @@ func (qs *queueSet) finishRequestLocked(r *request) {
 | 
				
			|||||||
		additionalLatency := r.workEstimate.AdditionalLatency
 | 
							additionalLatency := r.workEstimate.AdditionalLatency
 | 
				
			||||||
		if !klog.V(6).Enabled() {
 | 
							if !klog.V(6).Enabled() {
 | 
				
			||||||
		} else if r.queue != nil {
 | 
							} else if r.queue != nil {
 | 
				
			||||||
			klog.Infof("QS(%s) at t=%s R=%v: request %#+v %#+v finished main use but lingering on %d seats for %v seconds, adjusted queue %d start R to %v due to service time %.9fs, queue will have %d requests with %#v waiting & %d requests occupying %d seats",
 | 
								klog.Infof("QS(%s) at t=%s R=%v: request %#+v %#+v finished main use of %d seats but lingering on %d seats for %v seconds, adjusted queue %d start R to %v due to service time %.9fs, queue will have %d requests with %#v waiting & %d requests occupying %d seats",
 | 
				
			||||||
				qs.qCfg.Name, now.Format(nsTimeFmt), qs.currentR, r.descr1, r.descr2, r.workEstimate.InitialSeats, additionalLatency.Seconds(), r.queue.index,
 | 
									qs.qCfg.Name, now.Format(nsTimeFmt), qs.currentR, r.descr1, r.descr2, r.workEstimate.InitialSeats, r.workEstimate.FinalSeats, additionalLatency.Seconds(), r.queue.index,
 | 
				
			||||||
				r.queue.nextDispatchR, actualServiceDuration.Seconds(), r.queue.requests.Length(), r.queue.requests.QueueSum(), r.queue.requestsExecuting, r.queue.seatsInUse)
 | 
									r.queue.nextDispatchR, actualServiceDuration.Seconds(), r.queue.requests.Length(), r.queue.requests.QueueSum(), r.queue.requestsExecuting, r.queue.seatsInUse)
 | 
				
			||||||
		} else {
 | 
							} else {
 | 
				
			||||||
			klog.Infof("QS(%s) at t=%s R=%v: request %#+v %#+v finished main use but lingering on %d seats for %v seconds, qs will have %d requests occupying %d seats", qs.qCfg.Name, now.Format(nsTimeFmt), qs.currentR, r.descr1, r.descr2, r.workEstimate.InitialSeats, additionalLatency.Seconds(), qs.totRequestsExecuting, qs.totSeatsInUse)
 | 
								klog.Infof("QS(%s) at t=%s R=%v: request %#+v %#+v finished main use of %d seats but lingering on %d seats for %v seconds, qs will have %d requests occupying %d seats", qs.qCfg.Name, now.Format(nsTimeFmt), qs.currentR, r.descr1, r.descr2, r.workEstimate.InitialSeats, r.workEstimate.FinalSeats, additionalLatency.Seconds(), qs.totRequestsExecuting, qs.totSeatsInUse)
 | 
				
			||||||
		}
 | 
							}
 | 
				
			||||||
		// EventAfterDuration will execute the event func in a new goroutine,
 | 
							// EventAfterDuration will execute the event func in a new goroutine,
 | 
				
			||||||
		// so the seats allocated to this request will be released after
 | 
							// so the seats allocated to this request will be released after
 | 
				
			||||||
@@ -895,10 +895,10 @@ func (qs *queueSet) finishRequestLocked(r *request) {
 | 
				
			|||||||
			if !klog.V(6).Enabled() {
 | 
								if !klog.V(6).Enabled() {
 | 
				
			||||||
			} else if r.queue != nil {
 | 
								} else if r.queue != nil {
 | 
				
			||||||
				klog.Infof("QS(%s) at t=%s R=%v: request %#+v %#+v finished lingering on %d seats, queue %d will have %d requests with %#v waiting & %d requests occupying %d seats",
 | 
									klog.Infof("QS(%s) at t=%s R=%v: request %#+v %#+v finished lingering on %d seats, queue %d will have %d requests with %#v waiting & %d requests occupying %d seats",
 | 
				
			||||||
					qs.qCfg.Name, now.Format(nsTimeFmt), qs.currentR, r.descr1, r.descr2, r.workEstimate.InitialSeats, r.queue.index,
 | 
										qs.qCfg.Name, now.Format(nsTimeFmt), qs.currentR, r.descr1, r.descr2, r.workEstimate.FinalSeats, r.queue.index,
 | 
				
			||||||
					r.queue.requests.Length(), r.queue.requests.QueueSum(), r.queue.requestsExecuting, r.queue.seatsInUse)
 | 
										r.queue.requests.Length(), r.queue.requests.QueueSum(), r.queue.requestsExecuting, r.queue.seatsInUse)
 | 
				
			||||||
			} else {
 | 
								} else {
 | 
				
			||||||
				klog.Infof("QS(%s) at t=%s R=%v: request %#+v %#+v finished lingering on %d seats, qs will have %d requests occupying %d seats", qs.qCfg.Name, now.Format(nsTimeFmt), qs.currentR, r.descr1, r.descr2, r.workEstimate.InitialSeats, qs.totRequestsExecuting, qs.totSeatsInUse)
 | 
									klog.Infof("QS(%s) at t=%s R=%v: request %#+v %#+v finished lingering on %d seats, qs will have %d requests occupying %d seats", qs.qCfg.Name, now.Format(nsTimeFmt), qs.currentR, r.descr1, r.descr2, r.workEstimate.FinalSeats, qs.totRequestsExecuting, qs.totSeatsInUse)
 | 
				
			||||||
			}
 | 
								}
 | 
				
			||||||
			qs.dispatchAsMuchAsPossibleLocked()
 | 
								qs.dispatchAsMuchAsPossibleLocked()
 | 
				
			||||||
		}, additionalLatency)
 | 
							}, additionalLatency)
 | 
				
			||||||
 
 | 
				
			|||||||
		Reference in New Issue
	
	Block a user