Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Job status check cron #552

Draft
wants to merge 50 commits into
base: feature/regenerate-existing-images
Choose a base branch
from
Draft

Job status check cron #552

wants to merge 50 commits into from

Conversation

ankitrox
Copy link

@ankitrox ankitrox commented Oct 6, 2022

Summary

Create a cron which will do the following

  1. Runs hourly.
  2. Resume any halted/incomplete jobs.
  3. Remove old completed jobs from the queue (background_job taxonomy).

Fixes #484

Relevant technical choices

Checklist

  • PR has either [Focus] or Infrastructure label.
  • PR has a [Type] label.
  • PR has a milestone or the no milestone label.

Ankit Gade added 30 commits September 5, 2022 17:32
Background process runner class
ae86c66
Memory check for process
ca2b628
Fix: Action name for non-logged in user
dc08469
Tests: constant and hook
7c10ad2
Code commenting
076a5bb
Fix: phpcs
d1f0f63
Tests: handle request
e3c7381
Merge branch 'experiment/background-job-class' into experiment/bg-pro…
bf2bc3d 
…cess-runner
Change job status when ran successfully
a7c0150 
To-do: Add partial status in job class
Merge branch 'experiment/background-job-class' into experiment/bg-pro…
fbbeb2e 
…cess-runner
Merge branch 'experiment/background-job-class' into experiment/bg-pro…
f5a5f9b 
…cess-runner
Set partial status for successful batch
8d85858
Merge branch 'experiment/background-job-class' into experiment/bg-pro…
6aca9d2 
…cess-runner
Fix: tests
a0b78d7
Update method docs
6f4682f
Merge branch 'experiment/background-job-class' into experiment/bg-pro…
96aa1f1 
…cess-runner
Fix: tests
439930b
Method to call next batch of jobs
6c50fe2
Merge branch 'experiment/background-job-class' into experiment/bg-pro…
a15a608 
…cess-runner
Fix failing test
31d75f4
Merge branch 'experiment/background-job-class' into experiment/bg-pro…
0630872 
…cess-runner
Execute job till completed
3dec70e
Fix: while loop condition
230c694
Merge branch 'experiment/background-job-class' into experiment/bg-pro…
5b103f6 
…cess-runner
Adapt job class changes
c5c25fe 
Fix: tests
Refactor next batch method
589c29b
Refactor dispatch method
6d27758
Status check cron
1bc07fe
Resolve: Background job class conflicts
6eb0079
Fix typo
b0f9e1d
Ankit Gade added 20 commits September 16, 2022 13:47
Move dispatch function to helper
0dbbf91
Merge branch 'experiment/background-job-class' into experiment/bg-pro…
ee83559 
…cess-runner
Use get_identifier to trigger custom action
8c7accf
Docblock updates
fc1dfdf
Merge branch 'experiment/background-job-class' into experiment/bg-pro…
dfa4cad 
…cess-runner
Merge branch 'experiment/background-job-class' into experiment/bg-pro…
b0fa309 
…cess-runner
Fix: method name for job action
0f9f459
Merge branch 'experiment/background-job-class' into experiment/bg-pro…
b50a436 
…cess-runner
Docblock updates
9fa957c 
Filter to update expiry days for job cleanup
Merge branch 'experiment/background-job-class' into experiment/bg-pro…
d0dbc86 
…cess-runner
Merge branch 'feature/regenerate-existing-images' into experiment/bg-…
c7558f2 
…process-runner
Remove irrelevant code
8aee2e2
Helper method to get job instance
923142b
Tests: perflab_get_background_job
38d1596
Tests: get background job
8029a90
Refactor handle_request method
e0213ed
Refactor request triggering function
8c5dbab
Tests: start process and next batch process
f63e59f
Tests: helper functions for batch call
d517093
Status check cron
220f9e0
@ankitrox ankitrox added [Focus] Images Issues related to the Images focus area no milestone PRs that do not have a defined milestone for release [Type] Epic A high-level project / epic that will encompass several sub-issues [Module] Regenerate Existing Images Issues for the Regenerate Existing Images module labels Oct 6, 2022
@ankitrox ankitrox self-assigned this Oct 6, 2022
@ankitrox ankitrox changed the base branch from trunk to feature/regenerate-existing-images October 7, 2022 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jjgrainger jjgrainger Awaiting requested review from jjgrainger jjgrainger will be requested when the pull request is marked ready for review jjgrainger is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@ankitrox ankitrox

Labels
[Focus] Images Issues related to the Images focus area [Module] Regenerate Existing Images Issues for the Regenerate Existing Images module no milestone PRs that do not have a defined milestone for release [Type] Epic A high-level project / epic that will encompass several sub-issues
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Create Background Processing Infrastructure: Create Status Cron Event
1 participant
@ankitrox