Always log host metadata payload, even when too large (DataDog#2899)

pkg/serializer/serializer.go

@@ -231,16 +231,19 @@ func (s *Serializer) SendMetadata(m marshaler.Marshaler) error {
 	smallEnough, payload, err := split.CheckSizeAndSerialize(m, false, split.MarshalJSON)
 	if err != nil {
 		return fmt.Errorf("could not determine size of metadata payload: %s", err)
-	} else if !smallEnough {
-		return fmt.Errorf("metadata payload was too big to send, metadata payloads cannot be split")
+	}
+
+	log.Debugf("Sending host metadata payload, content: %v", apiKeyRegExp.ReplaceAllString(string(payload), apiKeyReplacement))
+
+	if !smallEnough {
+		return fmt.Errorf("metadata payload was too big to send (%d bytes), metadata payloads cannot be split", len(payload))
 	}
 
 	if err := s.Forwarder.SubmitV1Intake(forwarder.Payloads{&payload}, jsonExtraHeaders); err != nil {
 		return err
 	}
 
 	log.Infof("Sent host metadata payload, size: %d bytes.", len(payload))
-	log.Debugf("Sent host metadata payload, content: %v", apiKeyRegExp.ReplaceAllString(string(payload), apiKeyReplacement))
 	return nil
 }
 

releasenotes/notes/log_host_meta-161ea1937da7bb2a.yaml

@@ -0,0 +1,10 @@
+# Each section from every releasenote are combined when the
+# CHANGELOG.rst is rendered. So the text needs to be worded so that
+# it does not depend on any information only available in another
+# section. This may mean repeating some details, but each section
+# must be readable independently of the other.
+#
+# Each section note must be formatted as reStructuredText.
+---
+other:
+  - Log host metadata at debug level regardless of its size.