[s1s2] Run tests with and without cache enabled

RESOLVED FIXED

Status

Testing
Autophone
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: mcote, Assigned: bc)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

5 years ago
For comparison, we should do two runs of each s1s2 test, one with cache enabled and one without, remembering that the first iteration of the cache-enabled run is a special case.
(Assignee)

Updated

5 years ago
Assignee: nobody → bclary
Status: NEW → ASSIGNED
(Assignee)

Comment 1

5 years ago
Created attachment 726170 [details] [diff] [review]
cache autophone patch
Attachment #726170 - Flags: review?(mcote)
(Assignee)

Comment 2

5 years ago
Created attachment 726171 [details] [diff] [review]
cache phonedash patch
Attachment #726171 - Flags: review?(mcote)
(Assignee)

Comment 3

5 years ago
The autophone patch is applied on top of the patch in bug 851288
(Assignee)

Updated

5 years ago
Duplicate of this bug: 849269
(Reporter)

Comment 5

5 years ago
Comment on attachment 726170 [details] [diff] [review]
cache autophone patch

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

Looks good and seems to work fine now.  Just try to keep lines < 80 characters.  (Something I admittedly sometimes bypass :)
Attachment #726170 - Flags: review?(mcote) → review+
(Reporter)

Comment 6

5 years ago
Comment on attachment 726171 [details] [diff] [review]
cache phonedash patch

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

::: html/index.html
@@ +50,3 @@
>          Show error bars <input name="errorbars" id="errorbars" type="checkbox"  />
>          <br />
>          Show initial only <input name="initialonly" id="initialonly" type="checkbox" />

Could you document this in one of the included text doc files, if it isn't already?  This is assuming the initial cached value is still fairly different from the uncached values--something we should probably check.

::: html/scripts/app.js
@@ +148,5 @@
>    if (hash != document.location.hash) {
>      document.location.hash = hash;
>      return false;
>    }
> +  $.getJSON('api/s1s2/data/?product=' + params.product + '&metric=' + params.metric + '&test=' + params.test + '&start=' + $('#startdate').attr('value') + '&end=' + $('#enddate').attr('value') + '&cached=' + ($('#cached').attr('checked')?'cached':'notcached') + '&errorbars=' + ($('#errorbars').attr('checked')?'errorbars':'noerrorbars') + '&initialonly=' + ($('#initialonly').attr('checked')?'initialonly':'notinitialonly'), function(data) {

Man oh man I gotta clean this up at some point. :)

::: server/handlers.py
@@ +85,5 @@
>      @templeton.handlers.json_response
>      def GET(self):
>          query, body = templeton.handlers.get_request_parms()
> +        # bc:
> +        print 'GET: query %s, body %s' % (query, body)

I think you left this in by accident.
Attachment #726171 - Flags: review?(mcote) → review+
(Assignee)

Comment 7

5 years ago
https://github.com/markrcote/phonedash/commit/f864b4737924bdba186fb419e7c9ad615bb27d38
https://github.com/mozilla/autophone/commit/33564b1af2ee196d01e83508d5c092fe65ea81a8

After you land your build server patch, I'll go throw s1s2 and try to clean up the line lengths.

Had some git issues with the nested paths and rebase. Hopefully I fixed it properly. :/
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.