1004185 - Do a second HTTP cache v2 trial on Firefox Desktop

Status: RESOLVED FIXED

(Core :: Networking: Cache, defect)

Description

[Honza Bambas (:mayhemer)]

Despite we need to fix talos and one last test (according the runs on Gum we have now) we could do another public test in the meantime.

We can use again the patches from bug 967693.  I will merge them and request review.

I don't remember how/who we've announced this the last time, jason?

Comment 1

[Honza Bambas (:mayhemer)]

Attachment: v1

It's that easy..!

Comment 2

[Jason Duell]

Comment on attachment 8415594 [details] [diff] [review]
v1

Review of attachment 8415594 [details] [diff] [review]:
-----------------------------------------------------------------

Just to be clear: this is turning cache2 on just for desktop, and not android/b2g, and also not our buildbots, right?

Comment 3

[Honza Bambas (:mayhemer)]

(In reply to Jason Duell (:jduell) from comment #2)
> Just to be clear: this is turning cache2 on just for desktop, and not
> android/b2g, and also not our buildbots, right?

Exactly.  It does the same thing as the last time.  The "temp on" pref C++ handling code was left in, the talos specific preference to disable this "temp on" pref was left in too.  So, yes - this only enables cache2 for Firefox Desktop users.

Comment 4

[Justin Dolske [:Dolske]]

When is the intended date to revert this temporary" trial? Sounds like a few days?

Comment 5

[Jason Duell]

Yes, just a few days.  With a little luck we'll actually be able to fully land the cache immediately/shortly after that.

Comment 6

[Jason Duell]

Comment on attachment 8415594 [details] [diff] [review]
v1

Review of attachment 8415594 [details] [diff] [review]:
-----------------------------------------------------------------

I think we're good to land the trial on m-c for a few days.

Comment 7

[Jason Duell]

Bug 913819 comment 18 makes it sound like we want to land that bug before we turn this on.

Comment 8

[Honza Bambas (:mayhemer)]

(In reply to Jason Duell (:jduell) from comment #7)
> Bug 913819 comment 18 makes it sound like we want to land that bug before we
> turn this on.

Michal seems to be offline today.  I will land the patch for him.

Comment 9

[Honza Bambas (:mayhemer)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/acbdfd79dbb5

Comment 10

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/acbdfd79dbb5