Updated error messages to be facility focused.

Signed-off-by: Chris Privitere <[email protected]>

Author: cprivitere

api/v1beta1/packetcluster_webhook.go

@@ -92,15 +92,15 @@ func (c *PacketCluster) ValidateUpdate(oldRaw runtime.Object) error {
 	if len(c.Spec.Facility) == 0 && len(c.Spec.Metro) == 0 {
 		allErrs = append(allErrs,
 			field.Invalid(field.NewPath("spec", "Metro"),
-				c.Spec.Metro, "field is required when Facility is not set"),
+				c.Spec.Metro, "Metro is required when Facility is not set"),
 		)
 	}
 
 	// Must have only one of Metro or Facility
 	if len(c.Spec.Facility) > 0 && len(c.Spec.Metro) > 0 {
 		allErrs = append(allErrs,
-			field.Invalid(field.NewPath("spec", "Metro"),
-				c.Spec.Metro, "field and Facility field are mutually exclusive"),
+			field.Invalid(field.NewPath("spec", "Facility"),
+				c.Spec.Facility, "Metro and Facility are mutually exclusive"),
 		)
 	}
 

api/v1beta1/packetmachine_webhook.go

@@ -63,8 +63,8 @@ func (m *PacketMachine) ValidateUpdate(old runtime.Object) error {
 	// Must have only one of Metro or Facility specified
 	if len(m.Spec.Facility) > 0 && len(m.Spec.Metro) > 0 {
 		allErrs = append(allErrs,
-			field.Invalid(field.NewPath("spec", "Metro"),
-				m.Spec.Metro, "field and Facility field are mutually exclusive"),
+			field.Invalid(field.NewPath("spec", "Facility"),
+				m.Spec.Facility, "Metro and Facility field are mutually exclusive"),
 		)
 	}