use T.Equal method instead of reflect.DeepEqual

nomad/structs/workload_id.go

@@ -21,3 +21,19 @@ func (wi *WorkloadIdentity) Copy() *WorkloadIdentity {
 		File: wi.File,
 	}
 }
+
+func (wi *WorkloadIdentity) Equal(other *WorkloadIdentity) bool {
+	if wi == nil || other == nil {
+		return wi == other
+	}
+
+	if wi.Env != other.Env {
+		return false
+	}
+
+	if wi.File != other.File {
+		return false
+	}
+
+	return true
+}

nomad/structs/workload_id_test.go

@@ -0,0 +1,32 @@
+package structs
+
+import (
+	"testing"
+
+	"github.com/hashicorp/nomad/ci"
+	"github.com/shoenig/test/must"
+)
+
+func TestWorkloadIdentity_Equal(t *testing.T) {
+	ci.Parallel(t)
+
+	var orig *WorkloadIdentity
+
+	newWI := orig.Copy()
+	must.Equal(t, orig, newWI)
+
+	orig = &WorkloadIdentity{}
+	must.NotEqual(t, orig, newWI)
+
+	newWI = &WorkloadIdentity{}
+	must.Equal(t, orig, newWI)
+
+	orig.Env = true
+	must.NotEqual(t, orig, newWI)
+
+	newWI.Env = true
+	must.Equal(t, orig, newWI)
+
+	newWI.File = true
+	must.NotEqual(t, orig, newWI)
+}

scheduler/util.go

@@ -600,7 +600,7 @@ func tasksUpdated(jobA, jobB *structs.Job, taskGroup string) bool {
 		}
 
 		// Inspect Identity being exposed
-		if !reflect.DeepEqual(at.Identity, bt.Identity) {
+		if !at.Identity.Equal(bt.Identity) {
 			return true
 		}
 	}