/trooper says "Last updated: 12/31/1969, 4:00:00 PM PST" |
||
Issue descriptionBased on initial debugging it looks like <som-app>_computeLastUpdated seems to get a different parameter list than expected when called for /trooper. The "Last updated" value for regular sheriff trees is fine though, so long as the initial page load was for one of those trees rather than trooper. If you navigate straight to /trooper for initial page load, the bad update timestamp will stick around through subsequent client-side navigations.
,
Dec 13 2016
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/b410602af66f5115947c91ff9ec57ac6d4a9ff54 commit b410602af66f5115947c91ff9ec57ac6d4a9ff54 Author: Tiffany Zhang <zhangtiff@google.com> Date: Tue Dec 13 01:48:44 2016 SoM: Fix trooper timestamp. BUG= 672974 Change-Id: I32402cd04a395ca4b93df6e5dbca162f127dd080 Reviewed-on: https://chromium-review.googlesource.com/419118 Commit-Queue: Tiffany Zhang <zhangtiff@chromium.org> Reviewed-by: Sean McCullough <seanmccullough@chromium.org> [modify] https://crrev.com/b410602af66f5115947c91ff9ec57ac6d4a9ff54/go/src/infra/appengine/sheriff-o-matic/som/main_test.go [modify] https://crrev.com/b410602af66f5115947c91ff9ec57ac6d4a9ff54/go/src/infra/appengine/sheriff-o-matic/som/trooper.go
,
Dec 13 2016
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/b410602af66f5115947c91ff9ec57ac6d4a9ff54 commit b410602af66f5115947c91ff9ec57ac6d4a9ff54 Author: Tiffany Zhang <zhangtiff@google.com> Date: Tue Dec 13 01:48:44 2016 SoM: Fix trooper timestamp. BUG= 672974 Change-Id: I32402cd04a395ca4b93df6e5dbca162f127dd080 Reviewed-on: https://chromium-review.googlesource.com/419118 Commit-Queue: Tiffany Zhang <zhangtiff@chromium.org> Reviewed-by: Sean McCullough <seanmccullough@chromium.org> [modify] https://crrev.com/b410602af66f5115947c91ff9ec57ac6d4a9ff54/go/src/infra/appengine/sheriff-o-matic/som/main_test.go [modify] https://crrev.com/b410602af66f5115947c91ff9ec57ac6d4a9ff54/go/src/infra/appengine/sheriff-o-matic/som/trooper.go
,
Dec 13 2016
This will go out in the next release. |
||
►
Sign in to add a comment |
||
Comment 1 by zhangtiff@chromium.org
, Dec 12 2016