Closed Bug 1105507 Opened 10 years ago Closed 8 years ago

Move reviewer approve logic to celery task

Categories

(Marketplace Graveyard :: Reviewer Tools, defect, P4)

Avenir
x86
macOS
defect

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: robhudson, Unassigned)

References

Details

(Whiteboard: [reviewer-tools-rewrite][marketplace-transition])

I propose we move all the logic that happens when a reviewer approves an app to a celery task. That's roughly all of this code:
https://github.com/mozilla/zamboni/blob/bbff4145/mkt/reviewers/utils.py#L206-L304

Essentially the new task would do the following:
* Update the app status and file status
* Put timestamps where necessary (reviewed datetime)
* Update any info from the updated package
* Sign the app
* Reindex the app info
* Issues reviewer incentive points
* Tell IARC about this app
* etc.

I'm creating this as a blocker for bug 936444 so a new package update doesn't trigger all the above plus require the db transaction stuff we used in reviewer tools. The app developer may have to wait a couple minutes for the task to be completed but that seems reasonable.
Priority: -- → P4
Whiteboard: [reviewer-tools-rewrite]
No longer blocks: 1122263
I added bug 1122263 as blocker because it seemed to be nearly the same use-case as bug 936444 - i.e. auto-approving of a package leading to all the stuff mentioned in #c0 (sans IARC).  Should it be a blocker?  If not, is it a blocker on 936444 still?
Flags: needinfo?(robhudson.mozbugs)
(In reply to Andrew Williamson [:eviljeff] from comment #1)
> I added bug 1122263 as blocker because it seemed to be nearly the same
> use-case as bug 936444 - i.e. auto-approving of a package leading to all the
> stuff mentioned in #c0 (sans IARC).  Should it be a blocker?  If not, is it
> a blocker on 936444 still?

I'm going to leave this up to mat. If he wants it to be a blocker and needs/wants help I'm happy to jump in. We talked on IRC previously and he was going to decide on this soon.
Flags: needinfo?(robhudson.mozbugs) → needinfo?(mpillard)
It could block bug 1122277, but I'm not sure yet, I'll look into that later this week. Langpacks don't have statuses, they are active or not and that's it, so they wouldn't necessarily share a lot of code with the reviewer stuff we have for apps.
Flags: needinfo?(mpillard)
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WONTFIX
Whiteboard: [reviewer-tools-rewrite] → [reviewer-tools-rewrite][marketplace-transition]
You need to log in before you can comment on or make changes to this bug.