fix(helm): Fixed migrate not being consistent in name for hooks

This makes it so that the yaml being created for the migrate job is
consistent when `useHook` is `true` or `false`. At the moment if use
hook is false, the the name on the yaml and in the metadata match. But
if use hook is true then the name on the object itself is different from
the one in the metadata.

helm/oncall/templates/engine/job-migrate.yaml

@@ -26,7 +26,11 @@ spec:
   {{- end }}
   template:
     metadata:
+      {{- if .Values.migrate.useHook }}
+      name: {{ printf "%s-migrate" (include "oncall.engine.fullname" .) }}
+      {{- else }}
       name: {{ printf "%s-migrate-%s" (include "oncall.engine.fullname" .) (now | date "2006-01-02-15-04-05") }}
+      {{- end }}
       {{- with .Values.podAnnotations }}
       annotations:
         random-annotation: {{ randAlphaNum 10 | lower }}