Merge pull request #104790 from sttts/sttts-ssa-should-not-happen-name
server-side-apply: print object name in SHOULD NOT HAPPEN message Kubernetes-commit: 0ec2aaee473ef6256361bfa1aa044909dd0132fe
This commit is contained in:
		
						commit
						552db9ed90
					
				| 
						 | 
				
			
			@ -194,8 +194,15 @@ func (f *FieldManager) UpdateNoErrors(liveObj, newObj runtime.Object, manager st
 | 
			
		|||
	obj, err := f.Update(liveObj, newObj, manager)
 | 
			
		||||
	if err != nil {
 | 
			
		||||
		atMostEverySecond.Do(func() {
 | 
			
		||||
			ns, name := "unknown", "unknown"
 | 
			
		||||
			accessor, err := meta.Accessor(newObj)
 | 
			
		||||
			if err == nil {
 | 
			
		||||
				ns = accessor.GetNamespace()
 | 
			
		||||
				name = accessor.GetName()
 | 
			
		||||
			}
 | 
			
		||||
 | 
			
		||||
			klog.ErrorS(err, "[SHOULD NOT HAPPEN] failed to update managedFields", "VersionKind",
 | 
			
		||||
				newObj.GetObjectKind().GroupVersionKind())
 | 
			
		||||
				newObj.GetObjectKind().GroupVersionKind(), "namespace", ns, "name", name)
 | 
			
		||||
		})
 | 
			
		||||
		// Explicitly remove managedFields on failure, so that
 | 
			
		||||
		// we can't have garbage in it.
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
		Reference in New Issue