From 02a8e6c2ad04741885b375985942a286ee39956e Mon Sep 17 00:00:00 2001 From: Charlie Voiselle <464492+angrycub@users.noreply.github.com> Date: Mon, 12 Apr 2021 09:27:04 -0400 Subject: [PATCH] Fix parameterized <-> non-parameterized job error (#10357) The error messages are reversed from tests performed above them. The test uses the `validateJobUpdate()` function, but ignores the text of the error message itself. --- nomad/job_endpoint.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/nomad/job_endpoint.go b/nomad/job_endpoint.go index 7212db8ab..3015808f6 100644 --- a/nomad/job_endpoint.go +++ b/nomad/job_endpoint.go @@ -1830,10 +1830,10 @@ func validateJobUpdate(old, new *structs.Job) error { // Transitioning to/from parameterized is disallowed if old.IsParameterized() && !new.IsParameterized() { - return fmt.Errorf("cannot update non-parameterized job to being parameterized") + return fmt.Errorf("cannot update parameterized job to being non-parameterized") } if new.IsParameterized() && !old.IsParameterized() { - return fmt.Errorf("cannot update parameterized job to being non-parameterized") + return fmt.Errorf("cannot update non-parameterized job to being parameterized") } if old.Dispatched != new.Dispatched {