Skip to content

[WIP] Respect cron locks when running crons #38

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
Closed
schlessera wants to merge 7,314 commits into from
Closed

[WIP] Respect cron locks when running crons #38

schlessera wants to merge 7,314 commits into from

Conversation

@schlessera
Copy link
Member

@schlessera schlessera commented Nov 25, 2018

Fixes #27

balbuf and others added 30 commits February 1, 2017 09:47
@balbuf
Fix test - look for the correct err message
82e91b7
@danielbachhuber
Bump working version to 1.2.0-alpha
0e0278c
@lichtscheu
Update LICENSE year to 2017
be7df99
@danielbachhuber
Merge pull request #3791 from wp-cli/bump-working-1-2-0-alpha
71da692 
Bump working version to 1.2.0-alpha
@danielbachhuber
Merge pull request #3792 from lichtscheu/master
4850e6c 
Update LICENSE year to 2017
@danielbachhuber
Merge pull request #3755 from balbuf/3721/media-import-exit
cd130ac 
Improve media import error handling
@timdream
HTTPS everywhere on README.md
6854be1 
This avoids partial insecure warning on https://wp-cli.org/ .

I've updated all the outbound links to HTTPS as well.
@danielbachhuber
Merge pull request #3795 from timdream/patch-1
8505f3a 
HTTPS everywhere on README.md
@danielbachhuber
README edits
c4b764b 
* Change email updates link
* Drop link to runcommand
@danielbachhuber
Merge pull request #3797 from wp-cli/readme-links
76a4ee7 
README edits
@danielbachhuber
Order Travis matrix by fastest -> slowest
ff9e78e
@danielbachhuber
Ensure version_tags() always returns array, which is expected
02576d9
@danielbachhuber
Use private key delivered by Travis, instead of stored in project
fe4b0fd
@danielbachhuber
Disable test suite for purposes of testing deployment
53f80e8
@danielbachhuber
Disable branch verification too for
a9078b1 
53f80e8
@danielbachhuber
Use Travis' SSH key instead of our own
d79db82
@danielbachhuber
See what's in SSH dir
d85bac9
@danielbachhuber
Revert "See what's in SSH dir"
f8631e3 
This reverts commit d85bac9.
@danielbachhuber
Revert "Use Travis' SSH key instead of our own"
ec7041c 
This reverts commit d79db82.
@danielbachhuber
Test the private key so we can see what's up
a09a463
@danielbachhuber
Ensure the environment variable is evaluated
4ee305b
@danielbachhuber
Let's see what this variable is
f2047cf
@danielbachhuber
Use base64-encoded ssh key
041a9ed
@danielbachhuber
Properly pass environment variable to file
9821928
@danielbachhuber
Remove code that exposes private key
0653ac3
@danielbachhuber
Revert debugging comments
f985ef3
@danielbachhuber
DEPLOY_BRANCH needs to be 'master'
bd0d8cb
@danielbachhuber
Merge pull request #3799 from wp-cli/test-improvements
509732c 
General improvements to the build process
@danielbachhuber
Failing test case for #3794
3ff6abf
@danielbachhuber
Escape prefix when listing all tables with prefix
85a5125
schlessera and others added 22 commits May 5, 2021 07:23
@schlessera
Use version range for wp-cli-tests dependency
aceb034
@schlessera
Merge pull request #74 from wp-cli/fix/use-version-range-for-tests-de…
f16b03e 
…pendency

Use version range for `wp-cli/wp-cli-tests` dependency
@schlessera
Update file(s) from wp-cli/.github
62f0eb0
@schlessera
Update file(s) from wp-cli/.github
d7bcd06
@schlessera
Regenerate README file - 2021-05-10
754a2b9
@schlessera
Merge pull request #76 from wp-cli/regenerate-readme
f0d493f 
Regenerate README file
@schlessera
Update file(s) from wp-cli/.github
3322aff
@schlessera
Update file(s) from wp-cli/.github
649e373
@schlessera
Update file(s) from wp-cli/.github
cd62ef9
@schlessera
Update file(s) from wp-cli/.github
dc89f49
@schlessera
Update file(s) from wp-cli/.github
bc64b04
@schlessera
Update file(s) from wp-cli/.github
c12a684
@schlessera
Update file(s) from wp-cli/.github
81da826
@schlessera
Update file(s) from wp-cli/.github
3bb8d3b
@wojsmol
Add php 8.1 support
2f2d6e6
@schlessera
Merge pull request #77 from wojsmol/fix/php8.1-compat
bfa120b
@schlessera
Update file(s) from wp-cli/.github
6e98d33
@schlessera
Adapt feature test to use HTTPS with example.com
970817d
@schlessera
Merge pull request #78 from wp-cli/fix/use-https-example-com
829586b
@schlessera
Update file(s) from wp-cli/.github
f9e965e
@schlessera
Update file(s) from wp-cli/.github
f0f8537
@schlessera
First try at fixing cron locks
4800f1a
@schlessera schlessera force-pushed the 27-respect-doing-cron-transient branch from cc9f454 to 4800f1a Compare January 12, 2022 00:29
@schlessera schlessera requested a review from a team as a code owner January 12, 2022 00:29
@johnbillion johnbillion mentioned this pull request Aug 11, 2022
Closed
@danielbachhuber
Copy link
Member

danielbachhuber commented Nov 18, 2022

Proceeding with wp-cli/wp-cli#5594 for this repository. I've captured this PR to https://gist.github.com/danielbachhuber/e76c5c5e1479c385863cc86f5f21eb87 in case this PR is auto-closed or broken in some way.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

bug command:cron-event-run

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

wp cron event run --due-now doesn't respect doing_cron transient