generateEvent is also used in Event() with metav1.Now().  if the timestamp is ignored there too, does this mean that there is a bug in generateEvent()?    generateEvent(), calls makeEvent() which always uses the Now() from the recorder clock:  https://github.com/kubernetes/kubernetes/blob/master/staging/src/k8s.io/client-go/tools/record/event.go#L328    does this mean that makeEvent() should optionally support a custom timestamp for past events?  