feat(redis): add cluster connect command #5211
Conversation
github-actions
bot
added
the
redis
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #5211 +/- ##
==========================================
- Coverage 58.36% 58.30% -0.06%
==========================================
Files 311 311
Lines 68512 68637 +125
==========================================
+ Hits 39990 40022 +32
- Misses 27037 27129 +92
- Partials 1485 1486 +1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
jremy42
force-pushed
the
feat/redis-connect
branch
from
1b1777f to
68afb9d
Compare
| scw redis cluster connect | ||
|
|
||
| Connect to a Redis cluster via private network | ||
| scw redis cluster connect |
There was a problem hiding this comment.
Is it normal that the private network is not mentioned here?
There was a problem hiding this comment.
Fixed! I've updated the examples to use Raw instead of ArgsJSON so they explicitly show the private-network=true flag.
| Short: "Command line tool to use, default to redis-cli", | ||
| }, | ||
| core.ZoneArgSpec( | ||
| scw.ZoneFrPar1, |
There was a problem hiding this comment.
Could it be defined with a zones coming from the SDK instead of explicitly?
| cliRedis, | ||
| "-h", hostStr, | ||
| "-p", strconv.FormatUint(uint64(port), 10), | ||
| "-a", password, |
There was a problem hiding this comment.
Is there a way to have an argument passed through the command line?
There was a problem hiding this comment.
Yes! The cli-args parameter allows passing additional arguments to redis-cli. Example: scw redis cluster connect <cluster-id> cli-args.0=--raw
close #2961
Community Note
Relates OR Closes #0000
Release note for CHANGELOG: