Closed
Bug 1184208
Opened 9 years ago
Closed 9 years ago
Update automated tests for onyx/splice to include 'hello' channel
Categories
(Content Services Graveyard :: Tiles: Ops, defect)
Content Services Graveyard
Tiles: Ops
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: kthiessen, Assigned: nanj)
References
Details
From IRC: [09:46:27] <relud> oyiptong: it occurs to me that testing does not cover this new channel [09:46:54] <oyiptong> aha hmm [09:47:14] <oyiptong> there is new information [09:47:17] <relud> it would be nice if we could get a splice and onyx release that updates the default channels for the geoip_api_test.py and tile_index_crawl.py, but which will only be deployed to the monitor server [09:48:07] <oyiptong> we want the data in the monitoring script to make sure Access-Control-Allow-Origin is set on the CDN front [09:48:24] <oyiptong> there is ongoing investigation work there [09:48:34] <relud> oyiptong: none of the monitoring scripts download anything from the cdn [09:48:36] <oyiptong> ah [09:48:38] <oyiptong> just on S3? [09:48:39] <oyiptong> cool [09:48:41] <relud> yep [09:48:49] <oyiptong> that would be easy to do [09:49:06] <relud> yeah, possibly a good test to have [09:49:27] <oyiptong> maybe nanj could work on it? it would be a good way to get familiar with onyx/splice [09:49:28] <relud> perhaps configure the tile_index_crawl.py to download everything except the tile index from the cdn [09:49:44] <oyiptong> i mean onyx [09:49:45] <oyiptong> lol [09:52:45] <kthiessen> relud, oyiptong: I am willing to file a bug to get those tests modified/updated. Olivier can assign it to Nan if that seems reasonable. [09:52:58] <oyiptong> awesome [09:53:07] <relud> kthiessen: that would be great
Comment 1•9 years ago
|
||
to clarify: tile_index_crawl.py is a splice test geoip_api_test.py is an onyx also https://github.com/mozilla/onyx/blob/master/onyx/api/v3.py#L14 needs to add support for the new channel and onyx configuration needs to add support for the new channel
Updated•9 years ago
|
Assignee: oyiptong → najiang
Updated•9 years ago
|
Summary: Update automated tests for onyx to include 'hello' channel → Update automated tests for onyx/splice to include 'hello' channel
Comment 2•9 years ago
|
||
Commit pushed to master at https://github.com/mozilla/onyx https://github.com/mozilla/onyx/commit/a23634bf7b30ff52dd20fcb82a1a9ba2d6691f4c Close Bug 1184208 - Update automated tests for onyx/splice to include 'hello' channel commit 97a691353bc7b53424b1efd730b76ad0cac62628 Author: Nan Jiang <njiang028@gmail.com> Date: Fri Jul 17 13:55:31 2015 -0400 Add support for the 'hello' channel Close #29
Updated•9 years ago
|
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•