Call available_apps with distroseries=any, arch=any
Bug #973821 reported by
Anthony Lenton
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Ratings and Reviews server |
Fix Released
|
Low
|
Danny Tamez | ||
Bug Description
sca's api is called from within reviewsapp.
Within that method it calls sca's api for each distroseries, and caches that separately using the sca_<distroseri
If we call sca's available_apps method with distroseries=any, arch=any, we can store a single cache key for all applications, and check the returned series in the json to decide which distro/arch series an app should be available on.
Related branches
lp:~zematynnad/rnr-server/api_check_856808
- Michael Nelson (community): Approve
-
Diff: 203 lines (+57/-61)4 files modified.bzrignore (+2/-1)
fabtasks/bootstrap.py (+8/-1)
src/reviewsapp/tests/test_utilities.py (+32/-24)
src/reviewsapp/utilities.py (+15/-35)
| Changed in rnr-server: | |
| status: | New → Confirmed |
| importance: | Undecided → Low |
Revision history for this message
| Michael Nelson (michael.nelson) wrote | #1 |
| Changed in rnr-server: | |
| status: | Confirmed → In Progress |
| assignee: | nobody → Danny Tamez (zematynnad) |
Revision history for this message
| Anthony Lenton (elachuni) wrote | #2 |
| Changed in rnr-server: | |
| status: | In Progress → Fix Committed |
| Changed in rnr-server: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
This seems to be a reason *not* to fix bug 972565?