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: switch to official redis client and redis-cache/v9 #1345

Merged
merged 2 commits into from
Feb 19, 2023
Merged

Conversation

markphelps
Copy link
Collaborator

@markphelps markphelps commented Feb 17, 2023

go-redis has moved under the official 'redis' org: https://redis.com/blog/go-redis-official-redis-client/

This PR updates to use this new version (v9) and also updates the redis/cache lib to also be v9.

This change will allow us to add tracing/metrics to our cache since v9 now supports otel: https://redis.uptrace.dev/guide/go-redis-monitoring.html

ex: https://github.com/redis/go-redis/blob/master/example/otel/client.go#L35-L40

@markphelps markphelps requested a review from a team as a code owner February 17, 2023 19:20
@codecov-commenter
Copy link

codecov-commenter commented Feb 17, 2023

Codecov Report

Merging #1345 (4d92e2b) into main (b196f96) will increase coverage by 0.17%.
The diff coverage is n/a.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff             @@
##             main    #1345      +/-   ##
==========================================
+ Coverage   80.47%   80.65%   +0.17%     
==========================================
  Files          43       43              
  Lines        3375     3375              
==========================================
+ Hits         2716     2722       +6     
+ Misses        527      523       -4     
+ Partials      132      130       -2     
Impacted Files Coverage Δ
internal/server/cache/redis/cache.go 63.63% <ø> (ø)
internal/storage/oplock/sql/sql.go 96.33% <0.00%> (+5.50%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Member

@GeorgeMac GeorgeMac left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lush

@markphelps markphelps enabled auto-merge (squash) February 19, 2023 12:32
@markphelps markphelps merged commit 19e25c3 into main Feb 19, 2023
@markphelps markphelps deleted the redis-v9 branch February 19, 2023 12:37
markphelps added a commit that referenced this pull request Feb 22, 2023
* main:
  feat: enable ability to specify bootstrap token (#1350)
  feat(auth/k8s): kubernetes auth method service implementation (#1344)
  feat(rpc/auth): define AuthenticationMethodKubernetesService proto (#1341)
  chore: fix dependencies after testcontainers-go bump (#1349)
  chore(deps): bump github.com/testcontainers/testcontainers-go (#1348)
  chore(deps): bump golang.org/x/net from 0.6.0 to 0.7.0 (#1347)
  chore: switch to official redis client and redis-cache/v9 (#1345)
  chore(deps): bump golang.org/x/net from 0.6.0 to 0.7.0 in /_tools (#1346)
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.

3 participants