go-x-pkgsite/internal/log/log.go, branch main

internal/api: test ServePackage with db

2026-04-14T16:19:25Z

Test the ServePackage function against a real database, as well as the fake data source. - Modules and packages must be redistributable, or the DB strips the documentation. - Documentation must contain source code. We must temporarily skip the dependency tests, to reduce the diffs on this change. The test is still important, because it asserts that the pkgsite command has only a few dependencies. To re-establish it, we will move the API tests to internal/testing/api. Change-Id: I67243ea2345c5e735edbdd6ee5bf2349a6b1eb60 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/765502 kokoro-CI: kokoro LUCI-TryBot-Result: golang-scoped@luci-project-accounts.iam.gserviceaccount.com Reviewed-by: Ethan Lee Reviewed-by: Hyang-Ah Hana Kim

cmd/log: move stackdriver logging to its own package

2023-08-07T18:54:25Z

This change breaks out the stackdriver logging to its own package. It also only does logging of trace ids and labels to the stackdriver logger, to keep the logging interface simple. For golang/go#61399 Change-Id: I9a25d0c6391d1667fe476e5fdc30fc057f07c40f Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/515375 TryBot-Result: Gopher Robot kokoro-CI: kokoro Run-TryBot: Michael Matloob Reviewed-by: Jamal Carvalho

all: convert interface{} to any

2022-11-15T20:10:34Z

Change-Id: I1f3b7cc8899c7707abb01e3d14807c37c3451382 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/449695 TryBot-Result: kokoro Reviewed-by: Jamal Carvalho Run-TryBot: Hyang-Ah Hana Kim

cmd,internal: refactor UseStackdriverLogger

2021-07-14T13:36:10Z

Because UseStackdriverLogger currently uses the config package, the config package uses the standard library logger instead of internal/log from the pkgsite repo. Since UseStackdriverLogger only requires a few pieces of information from cfg, it is refactored to accept those args explicitly. Change-Id: I80bbbb1e8de7b1d21b685f201f55e948c3946b75 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/333930 Trust: Julie Qiu Run-TryBot: Julie Qiu TryBot-Result: kokoro Reviewed-by: Jonathan Amsterdam

internal/log: add warning level

2020-09-15T21:10:25Z

Change-Id: I76977535302b20df174d219a722fc5d839cd83b7 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/255057 Trust: Jonathan Amsterdam Run-TryBot: Jonathan Amsterdam TryBot-Result: kokoro Reviewed-by: Julie Qiu

internal/{log,worker}: use new config functions

2020-09-09T15:05:43Z

Use Config.DeploymentEnvironment and Config.Application instead of existing methods for extracting that information from the ServiceID. Change-Id: I6d8927297e51a71568e28e06067aea8756fdb3ab Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/253138 Reviewed-by: Julie Qiu

internal/{config,log}: configure logging for GKE

2020-09-03T17:01:32Z

Set the monitored resource and log labels so that when we run on GKE, the logs will show up where we expect them to, under the "Kubernetes Container" resource in the log viewer. Clicking on the logs from the GKE console (e.g. https://pantheon.corp.google.com/kubernetes/workload?project=go-discovery-exp) will also work. Also, log to stdout on GKE because logs to stderr have Error severity, which is distracting. Change-Id: I3a60800587cfefc6e158e76e4606b932d48ddeae Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/252898 Reviewed-by: Julie Qiu

internal: rename AppMonitoredResource to MonitoredResource

2020-09-03T16:06:22Z

We'll be supporting GKE monitored resources shortly in addtiion to AppEngine, so the "App" is wrong. Change-Id: I6225225732db24edcfc5c140e1efe58efe657168 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/252897 Reviewed-by: Julie Qiu

internal/log: add log level support

2020-08-06T13:27:19Z

This change introduces minimum log level support. GO_DISCOVERY_LOG_LEVEL config can be used to set the log level. When set, no logs below GO_DISCOVERY_LOG_LEVEL will be printed. Possible log levels are debug, info, error, fatal. Log severity sequence is debug < info < error < fatal. In case of invalid/empty log level, it print all logs. Fixes golang/go#40339 Change-Id: I66116e6778eb6386bf1e8df6947078a78fece691 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/245797 Reviewed-by: Julie Qiu

internal/{log,worker}: label fetch log messages with module@version

2020-06-04T00:01:50Z

- Make it possible to add a label to a context, so it appears in all log messages. - Add the module and version as a label when fetching. This will make it easy to find all log messages related to the fetch of a single module. Change-Id: I6c564b6aa58a494180d61168e739ef62dd67a64c Reviewed-on: https://team-review.git.corp.google.com/c/golang/discovery/+/760101 CI-Result: Cloud Build Reviewed-by: Julie Qiu