Skip to content

Conversation

@navba-MSFT
Copy link
Contributor

@navba-MSFT navba-MSFT commented May 19, 2022

The CLI command to update the Redis cache shardCount using --set parameter doesnt require double quote.
If you run the command with --set parameter in double quotes like "shardCount"="1" it will fail with below error:

usage error: Empty key in --set. Correct syntax: --set KEY=VALUE [KEY=VALUE ...]

Fixes #22498

Related command

Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature


This checklist is used to make sure that common guidelines for a pull request are followed.

The CLI command to update the Redis cache shardCount using --set parameter doesnt require double quote.
If you run the command with --set parameter in double quotes like "shardCount"="1" it will fail with below error:

usage error: Empty key in --set. Correct syntax: --set KEY=VALUE [KEY=VALUE ...]

Fixes #22498
@ghost ghost added Redis Cache Auto-Assign Auto assign by bot labels May 19, 2022
@ghost ghost requested review from evelyn-ys and yonzhan May 19, 2022 08:43
@ghost ghost assigned evelyn-ys May 19, 2022
@ghost ghost added this to the May 2022 (2022-05-24) - For Build milestone May 19, 2022
@ghost ghost requested a review from zhoxing-ms May 19, 2022 08:43
@yonzhan
Copy link
Collaborator

yonzhan commented May 19, 2022

AzureRedisCache

@evelyn-ys evelyn-ys changed the base branch from main to dev May 20, 2022 02:00
@navba-MSFT
Copy link
Contributor Author

@evelyn-ys Thanks for approving this. Since all the tests have been passed, could you merge this once you get a chance ? Thanks in advance.

@navba-MSFT
Copy link
Contributor Author

@kairu-ms Could you please merge this once you get a chance ? Thanks in advance.

@evelyn-ys evelyn-ys merged commit 0c6bd3e into Azure:dev May 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Quotes in --set parameter of redis update command given in the doc looks inaccurate

4 participants