-
Notifications
You must be signed in to change notification settings - Fork 37
Db retries #146
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.
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
Merged
+878
−160
Merged
Db retries #146
Changes from all commits
Commits
Show all changes
57 commits
Select commit
Hold shift + click to select a range
61ef375
chaos tests
maxdml 214ffd7
cleanup errors
maxdml 1885024
retrier
maxdml 74d8919
use retrier
maxdml 7224dfc
fix defer in test
maxdml 0d03ad4
Revert "cleanup errors"
maxdml f4121d5
imports
maxdml 7fb6a62
more wrapping
maxdml 35ff511
tests
maxdml dd1c15e
no need to install docker in gha
maxdml 5f01217
cli readme @ latest
maxdml 0500445
missing folder
maxdml 941d9b3
QueryEscape
maxdml d6c422e
on the password, and do not pass env var
maxdml 6d39fa4
no -race for chaos
maxdml be3829a
fix
maxdml 7874bc9
double escape
maxdml f7f417f
cleanup
maxdml 0c4bb5c
fix
maxdml a7bdb98
info
maxdml 756eae8
cleanup
maxdml 816f1df
support Unwrap in DBOSError + have newWorkflowExecutionError receive …
maxdml e79bf4c
handle pgx.ErrTxClosed as retryable but wrap entire RunWorkflow in a …
maxdml a5b9f5f
explicit warning for pgx.ErrTxClosed
maxdml 60a36dc
cleanup
maxdml acd5524
remove postgres from gha
maxdml 9085bee
start postgres before running the test
maxdml 8fd09ad
CLI must escape urls too
maxdml 3f4d01e
remove cache for now + shorter timeout
maxdml 28640f1
debug in tests...
maxdml 38d6e62
more debug
maxdml 066c137
handle lost wake-up
maxdml a888735
typedErasedWorkflow does not need to handle errors: it is always call…
maxdml a14ac95
must run retried function at least once
maxdml dd806fd
die less often
maxdml 102a8df
cache does not seem very useful
maxdml ecab120
check interface before using logger -- fixes mock path
maxdml da38b95
handle 'container is marked for removal and cannot be started' and th…
maxdml 7062c15
retoggle debug
maxdml 0adbbea
some clues
maxdml 9aef64a
a long timeout
maxdml 34f0268
more timeout
maxdml c54c692
+1
maxdml 6c527dc
log starts too
maxdml e61ae86
specify persistent volume when creating postgres container
maxdml 569bc32
manually handle locks in getEvent
maxdml 74ffbe0
fix
maxdml b87b0c7
wrap tx only in retry in RunWorkflow
maxdml 7df56b6
client enqueue retry
maxdml c875ec0
should not use uncancellable contexts for retry itself
maxdml 8505f14
less chatty logs
maxdml d795789
less chatty x2
maxdml 6f9ec3e
move cancel function setup out of RunWorkflow transaction
maxdml dbf7b2c
do less
maxdml 713e6d2
comment
maxdml 7cef987
test scheduled workflow
maxdml 97685fd
revert template change to not conflict with #155
maxdml File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,44 @@ | ||
name: Run Chaos Tests | ||
on: | ||
schedule: | ||
# Runs every hour on the hour | ||
- cron: '0 * * * *' | ||
push: | ||
branches: | ||
- main | ||
pull_request: | ||
branches: | ||
types: | ||
- ready_for_review | ||
- opened | ||
- reopened | ||
- synchronize | ||
workflow_dispatch: | ||
|
||
jobs: | ||
chaos-test: | ||
runs-on: ubuntu-latest | ||
timeout-minutes: 60 | ||
steps: | ||
- uses: actions/checkout@v5 | ||
with: | ||
fetch-depth: 0 | ||
fetch-tags: true | ||
|
||
- name: Setup Go | ||
uses: actions/setup-go@v5 | ||
with: | ||
go-version: '1.25.x' | ||
|
||
- name: Download dependencies | ||
run: go mod download | ||
|
||
- name: Install gotestsum | ||
run: go install gotest.tools/gotestsum@latest | ||
|
||
- name: Run chaos tests | ||
run: go vet ./... && go test -v -race -timeout 60m -count=1 ./... | ||
working-directory: ./chaos_tests | ||
env: | ||
PGPASSWORD: a!b@c$d()e*_,/:;=?@ff[]22 | ||
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doesn't seem very useful