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.

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

chore: allow pydantic v2 #14

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

JonasKs
Copy link

@JonasKs JonasKs commented Apr 25, 2024

Creating as draft, as I were not able to run tests locally.
This also caused errors for my pre-commit.

Closes #13

Errors when trying to set up local env: I've tried a bunch of things, but I couldn't get it to work. This is on a clean setup on my Macbook. I won't spend too much time on this, in case someone already encountered the issue and has a fix, or the pipelines gives me enough context to keep working on this PR.
❯ just setup
just teardown
docker compose down --volumes
[+] Running 9/9
 ✔ Container redis-rate-limiters-redis-leader-3-1    Removed                                                                                                                                                                                   0.4s
 ✔ Container redis-rate-limiters-redis-follower-3-1  Removed                                                                                                                                                                                   0.4s
 ✔ Container redis-rate-limiters-redis-leader-1-1    Removed                                                                                                                                                                                   0.4s
 ✔ Container redis-rate-limiters-redis-standalone-1  Removed                                                                                                                                                                                   0.0s
 ✔ Container redis-rate-limiters-redis-follower-2-1  Removed                                                                                                                                                                                   0.4s
 ✔ Container redis-rate-limiters-redis-leader-2-1    Removed                                                                                                                                                                                   0.4s
 ✔ Container redis-rate-limiters-redis-follower-1-1  Removed                                                                                                                                                                                   0.4s
 ✔ Network redis-rate-limiters_redis                 Removed                                                                                                                                                                                   0.1s
 ✔ Network redis-rate-limiters_default               Removed                                                                                                                                                                                   0.1s
docker compose up -d
[+] Running 9/9
 ✔ Network redis-rate-limiters_redis                 Created                                                                                                                                                                                   0.1s
 ✔ Network redis-rate-limiters_default               Created                                                                                                                                                                                   0.1s
 ✔ Container redis-rate-limiters-redis-follower-1-1  Started                                                                                                                                                                                   1.2s
 ✔ Container redis-rate-limiters-redis-leader-3-1    Started                                                                                                                                                                                   1.0s
 ✔ Container redis-rate-limiters-redis-follower-2-1  Started                                                                                                                                                                                   1.1s
 ✔ Container redis-rate-limiters-redis-leader-2-1    Started                                                                                                                                                                                   1.1s
 ✔ Container redis-rate-limiters-redis-standalone-1  Started                                                                                                                                                                                   0.7s
 ✔ Container redis-rate-limiters-redis-follower-3-1  Started                                                                                                                                                                                   0.6s
 ✔ Container redis-rate-limiters-redis-leader-1-1    Started                                                                                                                                                                                   0.8s
./setup-cluster.sh


Could not connect to Redis at 10.5.0.2:6379: Operation timed out
>>> Adding node 10.5.0.5:6379 to cluster 10.5.0.2:6379
Could not connect to Redis at 10.5.0.2:6379: Operation timed out
Unrecognized option or bad number of args for: '--cluster-master-id'
Unrecognized option or bad number of args for: '--cluster-master-id'
error: Recipe `setup` failed on line 10 with exit code 1

@JonasKs JonasKs closed this Jun 11, 2024
@klette klette reopened this Nov 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Pydantic 2 support
2 participants