commit ac73c7c9e457dfe9ace16468fb9db965195872c4
author Bert Muthalaly <stijlist@google.com> Wed Jul 08 18:47:39 2020 +0000
committer CQ bot account: commit-bot@chromium.org <commit-bot@chromium.org> Wed Jul 08 18:47:39 2020 +0000
tree dcc5b87d5f702c6619c5ebcadf8ad9983ffbbbff
parent 691ddccb91eb8f070e952a5f33f7879b51354d16

[netstack] cobalt client: log transport errors

Change-Id: Idcf649e2292fa95b89ee65368e87f36e4083b2c7
Reviewed-on: https://fuchsia-review.googlesource.com/c/fuchsia/+/405285
Reviewed-by: Tamir Duberstein <tamird@google.com>
Testability-Review: Tamir Duberstein <tamird@google.com>
Commit-Queue: Bert Muthalaly <stijlist@google.com>

src/connectivity/network/netstack/netstack.go

diff --git a/src/connectivity/network/netstack/netstack.go b/src/connectivity/network/netstack/netstack.go
index 9a7eda8..e54bbb4 100644
--- a/src/connectivity/network/netstack/netstack.go
+++ b/src/connectivity/network/netstack/netstack.go
@@ -129,8 +129,10 @@
 			return ctx.Err()
 		case <-ticker.C:
 			events := c.Collect()
-			if status, err := cobaltLogger.LogCobaltEvents(context.Background(), events); err != nil || status != cobalt.StatusOk {
-				syslog.Warnf("cobaltLogger.LogCobaltEvents(_, %+v) = %s, %s", events, status, err)
+			if status, err := cobaltLogger.LogCobaltEvents(context.Background(), events); err != nil {
+				syslog.Warnf("cobaltLogger.LogCobaltEvents(_, %+v) failed: %s", events, err)
+			} else if status != cobalt.StatusOk {
+				syslog.Warnf("cobaltLogger.LogCobaltEvents(_, %+v) rejected: %s", events, status)
 			}
 		}
 	}