fix!: return latest workflow for retried-persisted workflows (cherry-pick #15030 for 3.7) (#15032)

Signed-off-by: Elliot Gunton <[email protected]>
Signed-off-by: Alan Clucas <[email protected]>
Co-authored-by: Elliot Gunton <[email protected]>

Author: Joibel

persist/sqldb/workflow_archive.go

@@ -384,14 +384,19 @@ func (r *workflowArchive) GetWorkflow(uid string, namespace string, name string)
 			}
 			num := int64(total.Total)
 			if num > 1 {
-				return nil, fmt.Errorf("found %d archived workflows with namespace/name: %s/%s", num, namespace, name)
+				log.WithFields(log.Fields{
+					"namespace": namespace,
+					"name":      name,
+					"num":       num,
+				}).Debug("returning latest of archived workflows")
 			}
 			err = r.session.SQL().
 				Select("workflow").
 				From(archiveTableName).
 				Where(r.clusterManagedNamespaceAndInstanceID()).
 				And(namespaceEqual(namespace)).
 				And(nameEqual(name)).
+				OrderBy("-startedat").
 				One(archivedWf)
 		} else {
 			return nil, sutils.ToStatusError(fmt.Errorf("both name and namespace are required if uid is not specified"), codes.InvalidArgument)