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: Update redis to v9 in x/go.mod #795

Closed

Conversation

amaury1093
Copy link

@amaury1093 amaury1093 commented Dec 6, 2023

When using asynq v0.24 with x/rate (master), I get an incompatibility issue due to redis go.mod version (v8 vs v9) at this type cast:

Fixes #790

  • Update redis v8 to v9 (to match root asynq)
  • Update x/go.mod's asynq version from 0.21 to 0.24

@kamikazechaser
Copy link
Collaborator

CI is not passing. Looks like we need to remove everything in the CI build matrix and support only 1.20 and 1.21.

@kamikazechaser kamikazechaser changed the base branch from master to sohail/go-update December 7, 2023 07:57
kamikazechaser added a commit that referenced this pull request Dec 7, 2023
Squashed commit of the following:

commit 6e3656db222a3f9347ee4806ef065a1b9b01a214
Author: Mohammed Sohail <[email protected]>
Date:   Thu Dec 7 11:12:41 2023 +0300

    pkg(x): go version update -> 1.20

commit 2931df3
Author: Amaury <[email protected]>
Date:   Wed Dec 6 17:47:03 2023 +0100

    fix tests

commit 1122780
Author: Amaury <[email protected]>
Date:   Wed Dec 6 16:40:32 2023 +0100

    chore: Update redis to v9 in x/go.mod
@kamikazechaser
Copy link
Collaborator

I can't push to your fork, so I'll push your changes on top of mine which upgrades the go version directive in all modules.

8f79693

@amaury1093
Copy link
Author

Sounds good, I'll close my PR then.

@amaury1093 amaury1093 closed this Dec 7, 2023
hibiken pushed a commit that referenced this pull request Dec 10, 2023
Squashed commit of the following:

commit 6e3656db222a3f9347ee4806ef065a1b9b01a214
Author: Mohammed Sohail <[email protected]>
Date:   Thu Dec 7 11:12:41 2023 +0300

    pkg(x): go version update -> 1.20

commit 2931df3
Author: Amaury <[email protected]>
Date:   Wed Dec 6 17:47:03 2023 +0100

    fix tests

commit 1122780
Author: Amaury <[email protected]>
Date:   Wed Dec 6 16:40:32 2023 +0100

    chore: Update redis to v9 in x/go.mod
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] rate.NewSemaphore: type conversion failure (MakeRedisClient error?)
2 participants