-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
[Redis] Add support for 2023-08-01 #27599
Conversation
️✔️AzureCLI-FullTest
|
|
rule | cmd_name | rule_message | suggest_message |
---|---|---|---|
redis access-policy | sub group redis access-policy added |
||
redis access-policy-assignment | sub group redis access-policy-assignment added |
||
redis create | cmd redis create added parameter update_channel |
||
redis flush | cmd redis flush added |
Redis |
Please resolve the conflicts |
2415961
to
99527c9
Compare
Please also re-record the related tests in network module |
src/azure-cli/azure/cli/command_modules/redis/tests/latest/test_redis_scenario.py
Outdated
Show resolved
Hide resolved
@@ -127,7 +236,7 @@ def test_redis_cache_list_keys(self, resource_group): | |||
result = self.cmd('az redis list-keys -n {name} -g {rg}').get_output_in_json() | |||
self.assertTrue(result['primaryKey'] is not None) | |||
self.assertTrue(result['secondaryKey'] is not None) | |||
self.cmd('az redis update -n {name} -g {rg} --set "tags.mytag=mytagval"') | |||
# TODO: self.cmd('az redis update -n {name} -g {rg} --set "tags.mytag=mytagval"') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
May I know why you comment out this update command? Is there anything wrong?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is a known bug with update, the fix is rolling out. Once roll out completes I will create another PR to uncomment the TODOs.
Co-authored-by: Yishi Wang <[email protected]>
…/azure-cli into kj/release_2023-08-01
Added in this PR
Description
New api version to introduce the following changes:
cache aad enablement

access policy
access policy assignment
History Notes
[Redis] Add support for flush operation
[Redis] Add support for Microsoft Entra Authentication
[Redis] Add support for update channels
This checklist is used to make sure that common guidelines for a pull request are followed.
The PR title and description has followed the guideline in Submitting Pull Requests.
I adhere to the Command Guidelines.
I adhere to the Error Handling Guidelines.