Project

Profile

Help

Task #4500

closed

Collect test coverage from rq workers

Added by daviddavis over 5 years ago. Updated over 3 years ago.

Status:
CLOSED - WONTFIX
Priority:
Normal
Assignee:
-
Category:
-
Sprint/Milestone:
-
Start date:
Due date:
% Done:

0%

Estimated time:
Platform Release:
Groomed:
No
Sprint Candidate:
No
Tags:
Sprint:
Quarter:

Description

This is a task to revisit this PR:

https://github.com/pulp/pulpcore/pull/10

We had to revert these changes because it doubled the build time from 20 to 40 min.


Related issues

Related to Pulp - Issue #4499: Pulpcore Travis builds are taking twice as long (40 min vs 20 min)CLOSED - CURRENTRELEASEdaviddavisActions
Related to Pulp - Refactor #4416: Gather coverage from Content app for test purposesCLOSED - WONTFIX

Actions
Actions #1

Updated by daviddavis over 5 years ago

  • Related to Issue #4499: Pulpcore Travis builds are taking twice as long (40 min vs 20 min) added
Actions #2

Updated by amacdona@redhat.com over 5 years ago

I like the idea of running this nightly instead of each PR. That will do less for encouraging high coverage, but we were mostly ignoring that anyway.

Actions #3

Updated by daviddavis over 5 years ago

  • Related to Refactor #4416: Gather coverage from Content app for test purposes added
Actions #4

Updated by bmbouter over 5 years ago

I asked about this to the RQ community in their issue tracker here: https://github.com/rq/rq/issues/1051

Their suggestion is to use the SimpleWorker which will run the RQ tasks in the webserver itself synchronously. We could try to have a new worker type pulpcore.tests.functional.worker_with_coverage.CoveragePulpWorker that would use SimpleWorker with our custom worker behaviors from PulpWorker.

Actions #5

Updated by bmbouter over 5 years ago

  • Tags deleted (Pulp 3)
Actions #6

Updated by dalley over 3 years ago

Should we close this?

Actions #7

Updated by daviddavis over 3 years ago

  • Status changed from NEW to CLOSED - WONTFIX

Also available in: Atom PDF