Project

Profile

Help

Issue #2615

closed

Scheduled calls support broken for Celery4+Kombu4

Added by bmbouter over 6 years ago. Updated over 4 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
Category:
-
Sprint/Milestone:
-
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
Platform Release:
2.12.2
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
Sprint:
Sprint 17
Quarter:

Description

This issue only affects Pulp when used with the Celery4+Kombu4. For example in a Fedora26 environment or if you manually install the stack via pip.

You can test this with pulp smash by running:

workon pulp-smash
python3 -m unittest pulp_smash.tests.rpm.api_v2.test_schedule_publish.ScheduledPublishTestCase.test_total_run_count

To reproduce with hand testing:

1. Sync a repo
2. Configure it to publish ever N minutes (e.g. the pulp smash does it every 2 minutes)
3. Observe that waiting N minutes no scheduled publishes occur
Actions #1

Updated by bmbouter over 6 years ago

  • Sprint/Milestone set to 34

Adding to Sprint since this will be broken in F26.

Actions #2

Updated by dalley over 6 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to dalley
Actions #3

Updated by bizhang over 6 years ago

  • Triaged changed from No to Yes
Actions #6

Updated by bmbouter over 6 years ago

To do some hand testing on this I do the following:

1. I start with a Pulp system that is brand new and has the zoo repo on it
2. Configure the zoo repo to publish every 30 seconds.

phttp POST https://localhost/pulp/api/v2/repositories/zoo/distributors/export_distributor/schedules/publish/ <<< '{"schedule": "PT30S"}'

3. Observe the task run via the logs every 30 seconds.

4. Read the schedule call to observe its total_run_count is increasing. I do this with:

phttp GET https://localhost/pulp/api/v2/repositories/zoo/distributors/export_distributor/schedules/publish/
Actions #7

Updated by dalley over 6 years ago

  • Status changed from ASSIGNED to MODIFIED
Actions #8

Updated by dalley over 6 years ago

  • Status changed from MODIFIED to POST

https://github.com/pulp/pulp/pull/2970

I am not sure what happened, I did not modify this issue nor push code, only create a pull request

Added by dalley over 6 years ago

Revision 619fc12f

Fixes scheduled calls with Celery 4.x.

Celery 4.x changes a number of implentation details which breaks our assumptions about how the scheduling system works. Therefore, we want to check for the celery version and take actions to trigger the desired behavior depending on that celery version.

closes #2615 https://pulp.plan.io/issues/2615

Actions #9

Updated by dalley over 6 years ago

  • Status changed from POST to MODIFIED

Added by dalley over 6 years ago

Revision baf6f1cf

Fixes scheduled calls with Celery 4.x.

Celery 4.x changes a number of implentation details which breaks our assumptions about how the scheduling system works. Therefore, we want to check for the celery version and take actions to trigger the desired behavior depending on that celery version.

closes #2615 https://pulp.plan.io/issues/2615

Added by dalley over 6 years ago

Revision baf6f1cf

Fixes scheduled calls with Celery 4.x.

Celery 4.x changes a number of implentation details which breaks our assumptions about how the scheduling system works. Therefore, we want to check for the celery version and take actions to trigger the desired behavior depending on that celery version.

closes #2615 https://pulp.plan.io/issues/2615

Actions #10

Updated by dalley over 6 years ago

  • Status changed from MODIFIED to POST
Actions #11

Updated by mhrivnak over 6 years ago

  • Sprint/Milestone changed from 34 to 36
Actions #12

Updated by dalley over 6 years ago

  • Status changed from POST to MODIFIED
Actions #14

Updated by bizhang over 6 years ago

  • Platform Release set to 2.12.2
Actions #15

Updated by bizhang over 6 years ago

  • Status changed from MODIFIED to 5
Actions #16

Updated by bizhang over 6 years ago

  • Status changed from 5 to CLOSED - CURRENTRELEASE
Actions #17

Updated by bmbouter over 5 years ago

  • Sprint set to Sprint 17
Actions #18

Updated by bmbouter over 5 years ago

  • Sprint/Milestone deleted (36)
Actions #19

Updated by bmbouter over 4 years ago

  • Tags Pulp 2 added

Also available in: Atom PDF