snap: Record the snapshot save duration on success rather than only o…

…n error.

It makes more sense to record the latency of successes (or all attempts)
than of only a particular failure case.

snap/snapshotter.go

@@ -75,7 +75,7 @@ func (s *Snapshotter) save(snapshot *raftpb.Snapshot) error {
 		return err
 	}
 	err = ioutil.WriteFile(path.Join(s.dir, fname), d, 0666)
-	if err != nil {
+	if err == nil {
 		saveDurations.Observe(float64(time.Since(start).Nanoseconds() / int64(time.Microsecond)))
 	}
 	return err