]> Cypherpunks repositories - gostls13.git/commitdiff
cmd/vendor/golang.org/x/telemetry: update to ac8fed8
authorAlan Donovan <adonovan@google.com>
Wed, 15 May 2024 19:06:14 +0000 (15:06 -0400)
committerAlan Donovan <adonovan@google.com>
Wed, 15 May 2024 21:12:15 +0000 (21:12 +0000)
Updates #67182

Change-Id: I14f6a35491e3a58fff2f33285bd13ac706668df6
Reviewed-on: https://go-review.googlesource.com/c/go/+/585818
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Austin Clements <austin@google.com>
src/cmd/go.mod
src/cmd/go.sum
src/cmd/vendor/golang.org/x/telemetry/internal/crashmonitor/crash_go123.go
src/cmd/vendor/golang.org/x/telemetry/internal/upload/date.go
src/cmd/vendor/golang.org/x/telemetry/internal/upload/findwork.go
src/cmd/vendor/golang.org/x/telemetry/internal/upload/reports.go
src/cmd/vendor/golang.org/x/telemetry/internal/upload/run.go
src/cmd/vendor/golang.org/x/telemetry/internal/upload/upload.go
src/cmd/vendor/golang.org/x/telemetry/start.go
src/cmd/vendor/golang.org/x/telemetry/upload/upload.go [deleted file]
src/cmd/vendor/modules.txt

index cb485826767c69b4f48cabe0bbc06bb3cd844285..684191a6e40646419b68e83d6ca15eab57d0422f 100644 (file)
@@ -9,7 +9,7 @@ require (
        golang.org/x/mod v0.17.1-0.20240514174713-c0bdc7bd01c9
        golang.org/x/sync v0.7.0
        golang.org/x/sys v0.20.0
-       golang.org/x/telemetry v0.0.0-20240514182607-7d78a974cc44
+       golang.org/x/telemetry v0.0.0-20240515190011-ac8fed89e775
        golang.org/x/term v0.18.0
        golang.org/x/tools v0.20.1-0.20240429173604-74c9cfe4d22f
 )
index 48561550ab660a63f71935a5485eeb0e4652c39f..551fcdf4c57820d6d9ca534a70e4eda15ac09feb 100644 (file)
@@ -32,8 +32,8 @@ golang.org/x/sync v0.7.0 h1:YsImfSBoP9QPYL0xyKJPq0gcaJdG3rInoqxTWbfQu9M=
 golang.org/x/sync v0.7.0/go.mod h1:Czt+wKu1gCyEFDUtn0jG5QVvpJ6rzVqr5aXyt9drQfk=
 golang.org/x/sys v0.20.0 h1:Od9JTbYCk261bKm4M/mw7AklTlFYIa0bIp9BgSm1S8Y=
 golang.org/x/sys v0.20.0/go.mod h1:/VUhepiaJMQUp4+oa/7Zr1D23ma6VTLIYjOOTFZPUcA=
-golang.org/x/telemetry v0.0.0-20240514182607-7d78a974cc44 h1:aVJH+bdTb68otvUnahQ3CwIZTrJdYAW/gji9t6wuqcE=
-golang.org/x/telemetry v0.0.0-20240514182607-7d78a974cc44/go.mod h1:pRgIJT+bRLFKnoM1ldnzKoxTIn14Yxz928LQRYYgIN0=
+golang.org/x/telemetry v0.0.0-20240515190011-ac8fed89e775 h1:rWPDGnFE+SjKc7S5CrkYqx8I7hiwWV9oYcnZhmHAcm0=
+golang.org/x/telemetry v0.0.0-20240515190011-ac8fed89e775/go.mod h1:pRgIJT+bRLFKnoM1ldnzKoxTIn14Yxz928LQRYYgIN0=
 golang.org/x/term v0.18.0 h1:FcHjZXDMxI8mM3nwhX9HlKop4C0YQvCVCdwYl2wOtE8=
 golang.org/x/term v0.18.0/go.mod h1:ILwASektA3OnRv7amZ1xhE/KTR+u50pbXfZ03+6Nx58=
 golang.org/x/text v0.14.0 h1:ScX5w1eTa3QqT8oi6+ziP7dTV1S2+ALU0bI+0zXKWiQ=
index 2e0c1b32e3a9b839aee304433101dc9e3a1a9ba2..a2706d3abcdaf2d4271404fe32d2722516d03ffe 100644 (file)
@@ -7,8 +7,11 @@
 
 package crashmonitor
 
-import "runtime/debug"
+import (
+       "os"
+       "runtime/debug"
+)
 
 func init() {
-       setCrashOutput = debug.SetCrashOutput
+       setCrashOutput = func(f *os.File) error { return debug.SetCrashOutput(f) }
 }
index 26e65bd9bf6753b2b670373710b016bafeb1061f..22e4e8aa30c8b2e6d5be401b829241d5679695ab 100644 (file)
@@ -18,7 +18,7 @@ import (
 var distantPast = 21 * 24 * time.Hour
 
 // reports that are too old (21 days) are not uploaded
-func (u *Uploader) tooOld(date string, uploadStartTime time.Time) bool {
+func (u *uploader) tooOld(date string, uploadStartTime time.Time) bool {
        t, err := time.Parse("2006-01-02", date)
        if err != nil {
                u.logger.Printf("tooOld: %v", err)
@@ -31,7 +31,7 @@ func (u *Uploader) tooOld(date string, uploadStartTime time.Time) bool {
 // counterDateSpan parses the counter file named fname and returns the (begin,
 // end) span recorded in its metadata, or an error if this data could not be
 // extracted.
-func (u *Uploader) counterDateSpan(fname string) (begin, end time.Time, _ error) {
+func (u *uploader) counterDateSpan(fname string) (begin, end time.Time, _ error) {
        parsed, err := u.parseCountFile(fname)
        if err != nil {
                return time.Time{}, time.Time{}, err
@@ -61,7 +61,7 @@ type parsedCache struct {
        m  map[string]*counter.File
 }
 
-func (u *Uploader) parseCountFile(fname string) (*counter.File, error) {
+func (u *uploader) parseCountFile(fname string) (*counter.File, error) {
        u.cache.mu.Lock()
        defer u.cache.mu.Unlock()
        if u.cache.m == nil {
index 6bd559a8412c452a71b7bb9feec1ce2fe5e184e9..f1490be3a57b8508ca634afd6384bb2d99dc6f64 100644 (file)
@@ -22,7 +22,7 @@ type work struct {
 // find all the files that look like counter files or reports
 // that need to be uploaded. (There may be unexpected leftover files
 // and uploading is supposed to be idempotent.)
-func (u *Uploader) findWork() work {
+func (u *uploader) findWork() work {
        localdir, uploaddir := u.dir.LocalDir(), u.dir.UploadDir()
        var ans work
        fis, err := os.ReadDir(localdir)
index a335132602ca0ced908cb4804bdce109df80b555..41757897ea447de2bfe823f1a4ffa0b4f3df07a8 100644 (file)
@@ -21,7 +21,7 @@ import (
 )
 
 // reports generates reports from inactive count files
-func (u *Uploader) reports(todo *work) ([]string, error) {
+func (u *uploader) reports(todo *work) ([]string, error) {
        if mode, _ := u.dir.Mode(); mode == "off" {
                return nil, nil // no reports
        }
@@ -104,7 +104,7 @@ func notNeeded(date string, todo work) bool {
        return false
 }
 
-func (u *Uploader) deleteFiles(files []string) {
+func (u *uploader) deleteFiles(files []string) {
        for _, f := range files {
                if err := os.Remove(f); err != nil {
                        // this could be a race condition.
@@ -117,7 +117,7 @@ func (u *Uploader) deleteFiles(files []string) {
 
 // createReport for all the count files for the same date.
 // returns the absolute path name of the file containing the report
-func (u *Uploader) createReport(start time.Time, expiryDate string, countFiles []string, lastWeek string) (string, error) {
+func (u *uploader) createReport(start time.Time, expiryDate string, countFiles []string, lastWeek string) (string, error) {
        uploadOK := true
        mode, asof := u.dir.Mode()
        if mode != "on" {
index 714617a1968a62188c276402b9361bc24549bd30..eba13b1a573560d3a4316df4512c735626c7b66c 100644 (file)
@@ -30,9 +30,24 @@ type RunConfig struct {
        StartTime    time.Time // if set, overrides the upload start time
 }
 
-// Uploader encapsulates a single upload operation, carrying parameters and
+// Run generates and uploads reports, as allowed by the mode file.
+func Run(config RunConfig) error {
+       defer func() {
+               if err := recover(); err != nil {
+                       log.Printf("upload recover: %v", err)
+               }
+       }()
+       uploader, err := newUploader(config)
+       if err != nil {
+               return err
+       }
+       defer uploader.Close()
+       return uploader.Run()
+}
+
+// uploader encapsulates a single upload operation, carrying parameters and
 // shared state.
-type Uploader struct {
+type uploader struct {
        // config is used to select counters to upload.
        config        *telemetry.UploadConfig //
        configVersion string                  // version of the config
@@ -47,11 +62,11 @@ type Uploader struct {
        logger  *log.Logger
 }
 
-// NewUploader creates a new uploader to use for running the upload for the
+// newUploader creates a new uploader to use for running the upload for the
 // given config.
 //
-// Uploaders should only be used for one call to [Run].
-func NewUploader(rcfg RunConfig) (*Uploader, error) {
+// Uploaders should only be used for one call to [uploader.Run].
+func newUploader(rcfg RunConfig) (*uploader, error) {
        // Determine the upload directory.
        var dir telemetry.Dir
        if rcfg.TelemetryDir != "" {
@@ -108,7 +123,7 @@ func NewUploader(rcfg RunConfig) (*Uploader, error) {
                startTime = rcfg.StartTime
        }
 
-       return &Uploader{
+       return &uploader{
                config:          config,
                configVersion:   configVersion,
                dir:             dir,
@@ -121,7 +136,7 @@ func NewUploader(rcfg RunConfig) (*Uploader, error) {
 }
 
 // Close cleans up any resources associated with the uploader.
-func (u *Uploader) Close() error {
+func (u *uploader) Close() error {
        if u.logFile == nil {
                return nil
        }
@@ -129,7 +144,7 @@ func (u *Uploader) Close() error {
 }
 
 // Run generates and uploads reports
-func (u *Uploader) Run() error {
+func (u *uploader) Run() error {
        if telemetry.DisabledOnPlatform {
                return nil
        }
index bec2230837e923737f67293943a70293a8131937..0e75d09c017d9adee34c9b765c8764bdad2428f9 100644 (file)
@@ -22,7 +22,7 @@ var (
 
 // uploadReportDate returns the date component of the upload file name, or "" if the
 // date was unmatched.
-func (u *Uploader) uploadReportDate(fname string) time.Time {
+func (u *uploader) uploadReportDate(fname string) time.Time {
        match := dateRE.FindStringSubmatch(fname)
        if match == nil || len(match) < 2 {
                u.logger.Printf("malformed report name: missing date: %q", filepath.Base(fname))
@@ -36,7 +36,7 @@ func (u *Uploader) uploadReportDate(fname string) time.Time {
        return d
 }
 
-func (u *Uploader) uploadReport(fname string) {
+func (u *uploader) uploadReport(fname string) {
        thisInstant := u.startTime
        // TODO(rfindley): use uploadReportDate here, once we've done a gopls release.
 
@@ -60,7 +60,7 @@ func (u *Uploader) uploadReport(fname string) {
 }
 
 // try to upload the report, 'true' if successful
-func (u *Uploader) uploadReportContents(fname string, buf []byte) bool {
+func (u *uploader) uploadReportContents(fname string, buf []byte) bool {
        b := bytes.NewReader(buf)
        fdate := strings.TrimSuffix(filepath.Base(fname), ".json")
        fdate = fdate[len(fdate)-len("2006-01-02"):]
index 2b6b15be5c5bb08a1ae730b1a0b4bed8a4b9caf5..414c9fc5c9168fb80042ccc8372d315c379a3a16 100644 (file)
@@ -17,7 +17,7 @@ import (
        "golang.org/x/telemetry/counter"
        "golang.org/x/telemetry/internal/crashmonitor"
        "golang.org/x/telemetry/internal/telemetry"
-       "golang.org/x/telemetry/upload"
+       "golang.org/x/telemetry/internal/upload"
 )
 
 // Config controls the behavior of [Start].
diff --git a/src/cmd/vendor/golang.org/x/telemetry/upload/upload.go b/src/cmd/vendor/golang.org/x/telemetry/upload/upload.go
deleted file mode 100644 (file)
index 0e2fb45..0000000
+++ /dev/null
@@ -1,33 +0,0 @@
-// Copyright 2023 The Go Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style
-// license that can be found in the LICENSE file.
-
-package upload
-
-import (
-       "log"
-
-       "golang.org/x/telemetry/internal/upload"
-)
-
-// TODO(rfindley): remove, in favor of all callers using Start.
-
-// A RunConfig controls the behavior of Run.
-// The zero value RunConfig is the default behavior; fields may be set to
-// override various reporting and uploading choices.
-type RunConfig = upload.RunConfig
-
-// Run generates and uploads reports, as allowed by the mode file.
-func Run(config RunConfig) error {
-       defer func() {
-               if err := recover(); err != nil {
-                       log.Printf("upload recover: %v", err)
-               }
-       }()
-       uploader, err := upload.NewUploader(config)
-       if err != nil {
-               return err
-       }
-       defer uploader.Close()
-       return uploader.Run()
-}
index 17d0ed0fe190a485d0cdf2da3f34440bb230b6e7..240a26aab3e44afbaf74de48e3cce41c60dcbbb7 100644 (file)
@@ -45,7 +45,7 @@ golang.org/x/sync/semaphore
 golang.org/x/sys/plan9
 golang.org/x/sys/unix
 golang.org/x/sys/windows
-# golang.org/x/telemetry v0.0.0-20240514182607-7d78a974cc44
+# golang.org/x/telemetry v0.0.0-20240515190011-ac8fed89e775
 ## explicit; go 1.20
 golang.org/x/telemetry
 golang.org/x/telemetry/counter
@@ -57,7 +57,6 @@ golang.org/x/telemetry/internal/crashmonitor
 golang.org/x/telemetry/internal/mmap
 golang.org/x/telemetry/internal/telemetry
 golang.org/x/telemetry/internal/upload
-golang.org/x/telemetry/upload
 # golang.org/x/term v0.18.0
 ## explicit; go 1.18
 golang.org/x/term