aboutsummaryrefslogtreecommitdiff
path: root/devtools
AgeCommit message (Collapse)Author
2026-03-26all: run go fix -stringsbuilderHana Kim
Change-Id: I839e47c2b39ee592909f7ecb03603d4fb1d1954b Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/753430 Reviewed-by: Jonathan Amsterdam <jba@google.com> kokoro-CI: kokoro <noreply+kokoro@google.com> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
2026-03-24all: pgx/v4 to v5 and upgrade pqEthan Lee
- In preparation for upgrade to PostgreSQL 17, upgrade stale dependencies within pkgsite. Change-Id: Id54cbb46182711b9808ee3bbbe2f0d5d9a04583d Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/758121 Auto-Submit: Ethan Lee <ethanalee@google.com> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> Reviewed-by: Jonathan Amsterdam <jba@google.com> kokoro-CI: kokoro <noreply+kokoro@google.com>
2026-03-13all: fix forvarHana Kim
Change-Id: I02d2be535d9e3ac1788306adcc78d997d0caf951 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/753422 LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> kokoro-CI: kokoro <noreply+kokoro@google.com> Reviewed-by: Ethan Lee <ethanalee@google.com>
2026-03-12devtools/cmd/seeddb, internal/database: reduce contention and increase retriesHana Kim
Lower the concurrency limit of seeddb from 10 to 5 to reduce database contention. Increase the maximum number of transaction retries on serialization failure from 10 to 20 to improve reliability for large module insertions. Change-Id: Id57eea3bf7b7cf5554d4e99fbe68ac89bb355904 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/754540 kokoro-CI: kokoro <noreply+kokoro@google.com> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> Reviewed-by: Ethan Lee <ethanalee@google.com>
2026-03-12all: upgrade Node.js to version 20Hana Kim
Change-Id: Id9f71f9e14b72675c74cdd60be1fb1ba0d843791 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/752186 LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> kokoro-CI: kokoro <noreply+kokoro@google.com> Reviewed-by: Alexis Eduardo Avendaño Mijangos <punker308989@gmail.com> Reviewed-by: Ethan Lee <ethanalee@google.com>
2026-03-09all: update PostgreSQL version to 14Ethan Lee
Update Docker configurations and documentation to use PostgreSQL 14, aligning local development and testing environments with production. There has been a change in how PostgreSQL 14 tokenizes paths. This has caused an alteration in how multi word searches are resolved. Update test case to reflect this. See golang/go#78036 for more context. To test: `tests/search/run.sh` Fixes golang/go#77903 Change-Id: I28eaa722914b2163fe962a1b4a1b8681d60b18b3 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/750740 Auto-Submit: Ethan Lee <ethanalee@google.com> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> Reviewed-by: Hyang-Ah Hana Kim <hyangah@gmail.com> kokoro-CI: kokoro <noreply+kokoro@google.com>
2025-12-30deploy: bump Go version to 1.25.5 and update depsEthan Lee
Change-Id: I811776287454ba74b0ec8c1796c20907bf562150 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/733341 Reviewed-by: Nicholas Husin <husin@google.com> kokoro-CI: kokoro <noreply+kokoro@google.com> Reviewed-by: Nicholas Husin <nsh@golang.org> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> Auto-Submit: Ethan Lee <ethanalee@google.com>
2025-11-20devtools: ensure that copyright year is up to date in create_migrationEthan Lee
Change-Id: I6a3fe4d964269c1132dd5c8f5a26fa6d8abb8b80 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/721061 Reviewed-by: Robert Findley <rfindley@google.com> kokoro-CI: kokoro <noreply+kokoro@google.com> Auto-Submit: Ethan Lee <ethanalee@google.com> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
2025-03-21devtools: add script for Cloud ArmorJonathan Amsterdam
Add a script that displays our Cloud Armor policies and explains how to change them. Change-Id: I1e662bd91f886ae25288d114de120917c92ec40b Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/659896 LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> kokoro-CI: kokoro <noreply+kokoro@google.com> Reviewed-by: Robert Findley <rfindley@google.com>
2025-03-11go.mod, misc: update to go 1.24Jonathan Amsterdam
Change-Id: Iba3d15376214048113314150944d3b0df187a8cf Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/656475 LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> kokoro-CI: kokoro <noreply+kokoro@google.com> Reviewed-by: Robert Findley <rfindley@google.com>
2025-01-15devtools/cmd/seeddb: apply -keep_going to versions checkJonathan Amsterdam
Some module proxies may return invalid responses for versions for some modules. This change helps skip through those, when the majority of other modules would succeed. Concrete example: given a GitHub repo with no tagged releases, when querying the official Go module proxy for versions at /v@/list, it returns 200 OK and no content (since there are no versions). The jFrog Artifactory go module proxy's behaviour, in contrast, is to return 404 NOT FOUND if there are no versions. That means that if you're using seeddb and any of your seed.txt modules does not have tagged versions, we fail-fast when we try to list that module's versions, despite using -keep_going. This change applies -keep_going to that section of code, so that we skip over the 404 errors. Original author: Jean Barkhuysen <jean.barkhuysen@gmail.com> Fixes #71140 Change-Id: I2ccbcc356c322deed81860ee92274fba04a079b2 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/641675 LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> Reviewed-by: Michael Knyszek <mknyszek@google.com> Reviewed-by: Jonathan Amsterdam <jba@google.com> kokoro-CI: kokoro <noreply+kokoro@google.com> Reviewed-by: Jean Barkhuysen <jean.barkhuysen@gmail.com>
2024-11-26tests/screentest: update goldens to latest chromedp versionJonathan Amsterdam
Use a more recent version of chromedp and screentest. The newer screentest takes arguments differently, so update its invocations. The newer chromedp displays things slightly differently, so update the goldens. Also use the suffix "want" instead of "a", as the new screentest requires. Comment out buggy or flaky tests. These will be handled in followup CLs. Also, improve and enhance run.sh, notably adding the -run flag to run a subset of the tests. Change-Id: Ia46bf02f8cae15a763c80c6d6744f4ac9c005b9b Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/631780 kokoro-CI: kokoro <noreply+kokoro@google.com> Reviewed-by: Robert Findley <rfindley@google.com> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
2024-11-13devtools/cmd/seeddb: improve loggingJonathan Amsterdam
- Use the stdlib log package, to distinguish seeddb logs from pkgsite logs. - Log events as they happen, instead of at the end. Change-Id: Ia3877e821def72d89144ad50d2b6706bb531eb1c Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/626857 Reviewed-by: Hyang-Ah Hana Kim <hyangah@gmail.com> Reviewed-by: Robert Findley <rfindley@google.com> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> kokoro-CI: kokoro <noreply+kokoro@google.com>
2024-11-13tests/screentest: improve run.shJonathan Amsterdam
- Use single-hyphen flags to conform to Go style. - Remove unnecessary quotation. - Minor shell style nits. - Change the "local" case to run all components, instead of assuming that some are already running. The last change makes the "local" case much more useful, and faster than the "ci" case for local testing. Change-Id: Id3412fac8bd4a29539563e3cc568a3c7edd06c2e Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/626855 Reviewed-by: Robert Findley <rfindley@google.com> Reviewed-by: Hyang-Ah Hana Kim <hyangah@gmail.com> kokoro-CI: kokoro <noreply+kokoro@google.com> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
2024-08-24Revert "static/frontend: made cancel button in search field keyboard accessible"Jonathan Amsterdam
This reverts commit a224e846c93566e132672c606d14b07ef659553c. Reason for revert: breaks the UI (#68538) Fixes #68538. Change-Id: I23a6acaf741e7cb863c304363ee9b7a4fdf0bbe2 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/608036 LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> kokoro-CI: kokoro <noreply+kokoro@google.com> Reviewed-by: Hyang-Ah Hana Kim <hyangah@gmail.com>
2024-08-22devtools/cmd/seeddb: limit concurrencyJonathan Amsterdam
Limit the number of goroutines that are fetching modules at once, to reduce the chance of transactions timing out. We've been noticing kokoro failures due to that recently. I chose a limit of 10 arbitrarily. We'll see if it helps. I tried testing locally, but I don't get the same behavior as on kokoro. Also, use an errgroup with a context to clean up seeddb failures faster. Also, remove the seeddb flag permanently. Change-Id: Ib0244c76572c9f9e34064f99a05b26d70cc55aea Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/607755 Reviewed-by: Robert Findley <rfindley@google.com> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> kokoro-CI: kokoro <noreply+kokoro@google.com> Reviewed-by: Hyang-Ah Hana Kim <hyangah@gmail.com>
2024-08-20various: upgrade to go 1.23 containersJonathan Amsterdam
Upgrade to use go 1.23 images in various files that specify docker containers to run. Change-Id: I6331a07c4d0f00dc7513a5286d315dd0f64cd362 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/606778 Reviewed-by: Robert Findley <rfindley@google.com> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> kokoro-CI: kokoro <noreply+kokoro@google.com>
2024-07-15tests/screentest: add test for vuln with custom_rangesTatiana Bradley
Change-Id: Idf26fba7712eeb98b84c5d87aa31da86e2ea0a7a Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/597615 kokoro-CI: kokoro <noreply+kokoro@google.com> Reviewed-by: Damien Neil <dneil@google.com> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
2024-05-22tests: update vuln testdata to pull in review_statusTatiana Bradley
Update the script that pulls in the latest vulns to include an UNREVIEWED report. Run the script to update the test corpus to bring in this new vuln, plus the review_status for all vulns. Update the screentests accordingly. Change-Id: I45aa815ac9a33d186bac196220cce169b8523dae Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/586100 kokoro-CI: kokoro <noreply+kokoro@google.com> TryBot-Result: Gopher Robot <gobot@golang.org> Reviewed-by: Damien Neil <dneil@google.com> Run-TryBot: Tatiana Bradley <tatianabradley@google.com> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
2024-04-24tests/screentest: add a test for a vuln with no affected packagesTatiana Bradley
Adds a test for a vulnerability report that only lists affected module(s), not affected packages. Package site currently does not display this correctly (it displays an empty section instead of listing the modules). An upcoming CL will fix. Change-Id: Ia9b73ab8ad3f2ec71061bd329ab7497eccb91ee2 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/581179 Reviewed-by: Damien Neil <dneil@google.com> Run-TryBot: Tatiana Bradley <tatianabradley@google.com> kokoro-CI: kokoro <noreply+kokoro@google.com> TryBot-Result: Gopher Robot <gobot@golang.org> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
2024-04-18test/screentest: add withdrawn vuln and update screentestsTatiana Bradley
Add a "withdrawn" vulnerability example to the screentests so we can more easily test upcoming functionality that will handle withdrawn vulns. Change-Id: I3c373927055e3ce4a0f93fc1f1608596c9889c0a Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/579916 TryBot-Result: Gopher Robot <gobot@golang.org> Reviewed-by: Tatiana Bradley <tatianabradley@google.com> Reviewed-by: Jonathan Amsterdam <jba@google.com> Run-TryBot: Tatiana Bradley <tatianabradley@google.com> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> Reviewed-by: Hyang-Ah Hana Kim <hyangah@gmail.com> kokoro-CI: kokoro <noreply+kokoro@google.com>
2024-03-04tests/screentest: fix bug in scriptJonathan Amsterdam
Missing backslash was causing a panic. Change-Id: I630acd72a56a2d6242f64b17e6a4f944cb5b1c75 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/568239 LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> kokoro-CI: kokoro <noreply+kokoro@google.com> Reviewed-by: Michael Matloob <matloob@golang.org>
2024-02-26devtools/cmd/dumpdoc: add symbol docJonathan Amsterdam
Add the documentation for all the exported symbols to the output file. Change-Id: I25270765d81f78c3ff558243fea726fa68da65a3 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/566495 kokoro-CI: kokoro <noreply+kokoro@google.com> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> Reviewed-by: Tatiana Bradley <tatianabradley@google.com>
2024-02-23devtools/cmd/dumpdoc: tool for writing out package documentationJonathan Amsterdam
This is a tool for an experiment. It reads package documentation from the database and writes it to a file in gob format. Change-Id: I3b6cfde24ce155a647c3c43caff68be63832f80f Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/566416 kokoro-CI: kokoro <noreply+kokoro@google.com> Reviewed-by: Tatiana Bradley <tatianabradley@google.com> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
2024-01-18devtools/cmd/db: show waiting queriesJonathan Amsterdam
Add a subcommand to show which queries are waiting for others. Change-Id: Ia5b9f40c4ca45e81b63fe24a4e987cceaf8267ce Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/555460 TryBot-Result: Gopher Robot <gobot@golang.org> TryBot-Bypass: Jonathan Amsterdam <jba@google.com> Reviewed-by: Michael Matloob <matloob@golang.org> Run-TryBot: Jonathan Amsterdam <jba@google.com>
2024-01-17deploy,devtools: update go versionHana (Hyang-Ah) Kim
use go1.22rc1 Change-Id: I57dfddb84b551eca9b08bd988411dcd161745611 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/555196 kokoro-CI: kokoro <noreply+kokoro@google.com> Run-TryBot: Hyang-Ah Hana Kim <hyangah@gmail.com> Auto-Submit: Hyang-Ah Hana Kim <hyangah@gmail.com> TryBot-Result: Gopher Robot <gobot@golang.org> Reviewed-by: Jonathan Amsterdam <jba@google.com> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
2024-01-17deploy: use `docker compose` (V2)Hana (Hyang-Ah) Kim
It also replaces the custom docker-compose container image with the gcr.io/cloud-builders/docker, that includes docker compose and maintained. $ docker run -it gcr.io/cloud-builders/docker compose version Docker Compose version v2.21.0 Change-Id: I44900df9fb9adeda0b7a7646007b61d52d1db394 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/555997 kokoro-CI: kokoro <noreply+kokoro@google.com> Reviewed-by: Michael Matloob <matloob@golang.org> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
2024-01-03devtools/cmd/wait_available: add timeout flagMichael Matloob
And properly set the timeout flag in devtools/docker/compose.yaml and tests/screentest/run.sh instead of dropping it. Change-Id: I6aeee276b5fe1d01bef2b809b017a570c4f73368 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/553535 TryBot-Result: Gopher Robot <gobot@golang.org> Reviewed-by: Jonathan Amsterdam <jba@google.com> Run-TryBot: Michael Matloob <matloob@golang.org> kokoro-CI: kokoro <noreply+kokoro@google.com> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> Reviewed-by: Dan Peterson <danp@danp.net>
2023-12-24third_party/wait-for-it: replace with a simple go programMichael Matloob
The program doesn't have all the features of wait-for-it, but it implements the core functionality we need. For golang/go#61399 Change-Id: Ia5498523e44b74dcd5af1c984521f1a46208d2c5 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/552295 kokoro-CI: kokoro <noreply+kokoro@google.com> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> Reviewed-by: Jonathan Amsterdam <jba@google.com>
2023-10-09devtools/ci: reenable the screentestsJamal Carvalho
The screentests were temporarily disabled to speed up the GAR4 review process. That work is coming to a close so we can turn them back on. Change-Id: I424e97c1ce17250460fcc611ff4853f611684ff8 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/524796 LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> Reviewed-by: Hyang-Ah Hana Kim <hyangah@gmail.com> kokoro-CI: kokoro <noreply+kokoro@google.com> TryBot-Result: Gopher Robot <gobot@golang.org> Run-TryBot: Hyang-Ah Hana Kim <hyangah@gmail.com> Reviewed-by: Tatiana Bradley <tatianabradley@google.com>
2023-08-29internal/proxy: inject transport in proxy.NewMichael Matloob
This removes the dependency from internal/proxy on ochttp. For golang/go#61399 Change-Id: Ib2a350151752dc18c9a6e03b38b2d423ed55f4a1 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/523513 LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> Run-TryBot: Michael Matloob <matloob@golang.org> TryBot-Result: Gopher Robot <gobot@golang.org> kokoro-CI: kokoro <noreply+kokoro@google.com> Reviewed-by: Robert Findley <rfindley@google.com>
2023-08-29internal/source: inject *http.Client into source.NewClientMichael Matloob
This removes the dependency from package source onto ochttp. The users of source.NewClient that want an ochttp.Transport can set the transport on the *http.Client. For golang/go#61399 Change-Id: Ifb7126c482f664ee5a359f594d9324f0fd90f8b2 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/523510 LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> Reviewed-by: Robert Findley <rfindley@google.com> TryBot-Result: Gopher Robot <gobot@golang.org> Run-TryBot: Michael Matloob <matloob@golang.org> kokoro-CI: kokoro <noreply+kokoro@google.com>
2023-08-25internal/config: separate config initialization into serverconfigMichael Matloob
This change creates a new package that does config initialization and other GCP-specific operations that were previously done in package config, so that config can have no cloud dependencies. For golang/go#61399 Change-Id: I8d78294834e325b47d838892a1cef87003a4b90a Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/522516 Run-TryBot: Michael Matloob <matloob@golang.org> TryBot-Result: Gopher Robot <gobot@golang.org> Reviewed-by: Robert Findley <rfindley@google.com> kokoro-CI: kokoro <noreply+kokoro@google.com>
2023-08-24devtools/ci: temporarily disable screentestsJamal Carvalho
To speed up GAR 4 code review process, we'll temporarily disable the screentests. Change-Id: Ide96e8544cbf56db4dee7f4f2924f76a60af5f6c Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/522736 kokoro-CI: kokoro <noreply+kokoro@google.com> Reviewed-by: Robert Findley <rfindley@google.com> TryBot-Result: Gopher Robot <gobot@golang.org> Run-TryBot: Jamal Carvalho <jamal@golang.org> Auto-Submit: Jamal Carvalho <jamal@golang.org>
2023-08-04all: switch from github.com/ghodss/yaml to gopkg.in/yaml.v3Michael Matloob
yaml.v3 is the more widely used yaml. The main difference between the two is that ghodss converts yaml to JSON as an intermediate to marshalling and unmarshalling. That means we need to add yaml struct tags to use yaml.v3. Add yaml tags to ConfigOverride and QuotaSettings because yaml.v3 lowercases the names of fields by default and the configs have used uppercase names. Change a test experiment config to use lowercase names since that's what's used in most other configs. For golang/go#61399 Change-Id: Id7f09f2635ee013506b1573bfe555ec0348e60e4 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/514522 TryBot-Result: Gopher Robot <gobot@golang.org> Run-TryBot: Michael Matloob <matloob@golang.org> Reviewed-by: Hyang-Ah Hana Kim <hyangah@gmail.com> Reviewed-by: Jamal Carvalho <jamal@golang.org> kokoro-CI: kokoro <noreply+kokoro@google.com>
2023-07-18deploy: use go1.21rc3Jamal Carvalho
Change-Id: I36ecd3b393027717fb971284756d3636dcd9c916 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/509975 kokoro-CI: kokoro <noreply+kokoro@google.com> TryBot-Result: Gopher Robot <gobot@golang.org> Reviewed-by: Hyang-Ah Hana Kim <hyangah@gmail.com> Run-TryBot: Jamal Carvalho <jamal@golang.org>
2023-05-19Fix content.yaml syntaxAlex Hays
Current content.yaml format errors with docker-compose v2.17+. Running snapshot tests currently errors with: unmarshal errors: line 37: mapping key “<<” already defined at line 36 line 52: mapping key “<<” already defined at line 51 line 83: mapping key “<<” already defined at line 82 This CR fixes that error. Context: https://github.com/docker/compose/issues/10407#issuecomment-1484787932 Change-Id: I15d9e1d7bb799819a783904cf9dc0bb5a26efcae Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/496189 Run-TryBot: Jamal Carvalho <jamal@golang.org> TryBot-Result: kokoro <noreply+kokoro@google.com> Reviewed-by: Jamal Carvalho <jamal@golang.org> Reviewed-by: Hyang-Ah Hana Kim <hyangah@gmail.com>
2023-05-10x/pkgsite: expose Postgres port to loopbackSteven Hartland
Expose the Postgres port to loopback so that it works under WSL, enabling the tests run by all.bash to complete successfully based on the setup instructions in doc/postgres.md. Fixes golang/go#60009 Change-Id: I091d6db6319ac504c77279ada67c9a6e5a5bef84 GitHub-Last-Rev: 2dcf9d22deaa13cb6f2e567c6e2e2d699c1b8be0 GitHub-Pull-Request: golang/pkgsite#65 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/493076 Reviewed-by: Robert Findley <rfindley@google.com> Run-TryBot: Robert Findley <rfindley@google.com> Reviewed-by: Cherry Mui <cherryyz@google.com> TryBot-Result: kokoro <noreply+kokoro@google.com>
2023-03-31devtools,tests/screentest: use vulndb v1 for screentestsTatiana Bradley
Update screentests to run against the vulndb v1 schema instead of the legacy schema by turning on the v1 experiment in the screentest config. Add a folder, vulndb-v1, containing test data for the v1 schema and point to it for screentests. Also, add a script that can be used to update this testdata. Change-Id: Ia94ae4d125233e0c275b5abbfb22c2043e34f05f Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/479656 TryBot-Result: kokoro <noreply+kokoro@google.com> Run-TryBot: Tatiana Bradley <tatianabradley@google.com> Reviewed-by: Julie Qiu <julieqiu@google.com>
2023-03-31screentest/data: add additional reports to screentest dataTatiana Bradley
This allows us to test that the /vuln/list page shows all the reports, sorted by Go ID, and the /vuln page shows only the 5 most recent reports. (Previously, there were only 5 test reports). Change-Id: I820b8d856dc58e04c418cfc9301d4f2efad56d02 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/480355 TryBot-Result: kokoro <noreply+kokoro@google.com> Run-TryBot: Tatiana Bradley <tatianabradley@google.com> Reviewed-by: Jamal Carvalho <jamal@golang.org>
2023-03-29tests/screentest: update vulndb testdata to latestTatiana Bradley
Update vulndb test data to latest, and update expected screenshots accordingly. Change-Id: I53d3ac4d88015f0d3650cf2def20b2817fbbf212 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/479655 Reviewed-by: Jamal Carvalho <jamal@golang.org> Run-TryBot: Tatiana Bradley <tatianabradley@google.com> TryBot-Result: kokoro <noreply+kokoro@google.com>
2023-02-22internal,static: upgrade esbuildJamal Carvalho
The previous version of esbuild would sometimes out non-deterministic versions of minified files. This would generate large diffs when make small changes to script code on different machines. Change-Id: I30540088ad2a259e4c086b18505016a95937dc2f Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/469855 Run-TryBot: Jamal Carvalho <jamal@golang.org> Reviewed-by: Hyang-Ah Hana Kim <hyangah@gmail.com> TryBot-Result: kokoro <noreply+kokoro@google.com>
2023-02-21deploy,devtools: use golang:1.20 imageHana (Hyang-Ah) Kim
This will make pkgsite pick up the latest 1.20.x. And - reenable staticcheck. - mark the working directory as safe when running for ci. (this workaround is needed after the security patch https://release.debian.org/proposed-updates/bullseye_diffs/git_2.30.2-1+deb11u1.debdiff) Updates golang/go#53532 Change-Id: I31c5fc838821d8bd6ade81dc439eb46fabb75fe1 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/468917 Run-TryBot: Hyang-Ah Hana Kim <hyangah@gmail.com> Reviewed-by: Jamal Carvalho <jamal@golang.org> TryBot-Result: kokoro <noreply+kokoro@google.com>
2023-02-01deploy,devtools: upgrade to go1.20rc3Jamal Carvalho
- Disabled staticcheck while go1.20 is unsupported. - Fixed out of bounds error in doc html parsing. - Fixed test issue related to rand package changes. Change-Id: Ib7b30ed9c40505a5a51d986426f0e41e53a298b1 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/464296 TryBot-Result: kokoro <noreply+kokoro@google.com> Reviewed-by: Hyang-Ah Hana Kim <hyangah@gmail.com> Run-TryBot: Jamal Carvalho <jamal@golang.org>
2022-11-10tests/screentest: create a tool to generate testdata for pkgsiteJamal Carvalho
Added a script to create a snapshot of selected data from vulndb.go.dev for use in screenshot tests. Updated the screenshots for the latest vuln data. Change-Id: Iecbdbc6528221cd58b0049a900b1ae2b5e67304c Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/448416 TryBot-Result: kokoro <noreply+kokoro@google.com> Run-TryBot: Jamal Carvalho <jamal@golang.org> Reviewed-by: Hyang-Ah Hana Kim <hyangah@gmail.com>
2022-09-22pkgsite: screentest improvementsJamal Carvalho
The screentest have been stable for some time now. Unsetting the concurrency config to use the tools default for local test runs. Also setting seeddb to run by default to reduce confusion around first test runs. Speed improvements were made to seeddb so it adds minimal time for a previously seeded database. Change-Id: I2d5eeabfeb0b7f57782aa1a854ebad1c3e3ebaa3 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/432419 Run-TryBot: Jamal Carvalho <jamal@golang.org> TryBot-Result: kokoro <noreply+kokoro@google.com> Reviewed-by: Hyang-Ah Hana Kim <hyangah@gmail.com>
2022-09-22tests: add screentests for vuln pagesJamal Carvalho
Using a vulndb snapshot as testdata, added screentests that run only in CI for vuln pages. Because regular updates are made to the vulndb, these tests are not deterministic for non CI runs. Change-Id: I5aed1b350a13065cd7e45e100de1a9ba5123c2a5 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/431175 TryBot-Result: kokoro <noreply+kokoro@google.com> Run-TryBot: Jamal Carvalho <jamal@golang.org> Reviewed-by: Jonathan Amsterdam <jba@google.com> Reviewed-by: Hyang-Ah Hana Kim <hyangah@gmail.com>
2022-08-04pkgsite: update to go1.19Jamal Carvalho
Change-Id: Icf9ea86e7cffbd79295cd5713cb28d3d3af820f2 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/420957 TryBot-Result: kokoro <noreply+kokoro@google.com> Run-TryBot: Jamal Carvalho <jamal@golang.org> Reviewed-by: Robert Findley <rfindley@google.com>
2022-06-17many: re-enable staticcheck and fix problemsJonathan Amsterdam
Most of the checks were about io/ioutil. There were a couple of other minor ones. I didn't address the check for strings.Title; instead, I turned off that check globally with a staticcheck.conf file. Change-Id: I286a6894fb1fd891818ab9e451c891f52a3828fc Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/412675 Reviewed-by: Jamal Carvalho <jamal@golang.org> Run-TryBot: Jonathan Amsterdam <jba@google.com> TryBot-Result: kokoro <noreply+kokoro@google.com>
2022-06-17all: upgrade to go 1.19beta1Jonathan Amsterdam
This is a prerequisite for using the new go/doc/comment package. Change-Id: Id92e4db359a0d57e945264078b5e58d86f6aa64b Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/412674 Reviewed-by: Jamal Carvalho <jamal@golang.org> Run-TryBot: Jonathan Amsterdam <jba@google.com>