Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(196)

Issue 1842463002: Try setting cache-snapshot-write-cold-duration to 1m as suggested in https://github.com/influxdata/… (Closed)

Created:
4 years, 8 months ago by dogben
Modified:
4 years, 8 months ago
Reviewers:
borenet
CC:
reviews_skia.org, jcgregorio
Base URL:
https://skia.googlesource.com/buildbot@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Try setting cache-snapshot-write-cold-duration to 1m as suggested in https://github.com/influxdata/influxdb/issues/5685. BUG=skia: Committed: https://skia.googlesource.com/buildbot/+/4fd9f61bfee15f4426cd957e51c650dce76b9ac5

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M influxdb/influxdb-config.toml View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 13 (5 generated)
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1842463002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1842463002/1
4 years, 8 months ago (2016-03-28 19:45:03 UTC) #2
commit-bot: I haz the power
Dry run: This issue passed the CQ dry run.
4 years, 8 months ago (2016-03-28 19:47:34 UTC) #4
dogben
If this doesn't seem to help, I'll revert.
4 years, 8 months ago (2016-03-28 19:47:35 UTC) #6
borenet
LGTM. We should also restart all of our apps if possible. I suspect they've all ...
4 years, 8 months ago (2016-03-28 19:49:51 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1842463002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1842463002/1
4 years, 8 months ago (2016-03-28 19:52:13 UTC) #9
commit-bot: I haz the power
Committed patchset #1 (id:1) as https://skia.googlesource.com/buildbot/+/4fd9f61bfee15f4426cd957e51c650dce76b9ac5
4 years, 8 months ago (2016-03-28 19:52:28 UTC) #11
dogben
On 2016/03/28 at 19:49:51, borenet wrote: > LGTM. We should also restart all of our ...
4 years, 8 months ago (2016-03-28 19:53:39 UTC) #12
borenet
4 years, 8 months ago (2016-03-28 19:56:03 UTC) #13
Message was sent while issue was closed.
On 2016/03/28 19:53:39, Ben Wagner wrote:
> On 2016/03/28 at 19:49:51, borenet wrote:
> > LGTM. We should also restart all of our apps if possible. I suspect they've
> all been accumulating data which hasn't been sent.
> 
> You mean before pushing the new config?

Yeah, just before, I think.  I'm guessing all of our apps are attempting to push
hours worth of data all at once, which is going to prevent it from working. 
Restarting the apps will empty out the queues.

Powered by Google App Engine
This is Rietveld 408576698