Skip to content

mgmt redis , fix live test failures #47602

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.

Sign up for GitHub

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

Jump to bottom
Open
v-huizhu2 wants to merge 12 commits into
base: main
Choose a base branch
from
Open

mgmt redis , fix live test failures #47602

v-huizhu2 wants to merge 12 commits into from

Conversation

@v-huizhu2
Copy link
Contributor

@v-huizhu2 v-huizhu2 commented Dec 25, 2025

  1. This request was denied due to internal policy. Local authentication methods are not allowed.
    Disable local authentication in Redis Cache creation.
  2. Microsoft Entra Authentication cannot be used cache is configured to allow non-SSL connections.
  3. Use managed identity for AOF & RDB backup configuration.

#47372

Description

Please add an informative description that covers that changes made by the pull request and link all relevant issues.

If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.

All SDK Contribution checklist:

  • The pull request does not introduce [breaking changes]
  • CHANGELOG is updated for new features, bug fixes or other significant changes.
  • I have read the contribution guidelines.

General Guidelines and Best Practices

  • Title of the pull request is clear and informative.
  • There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

  • Pull request includes test coverage for the included changes.

@v-huizhu2
feat(redis): add support to disable local authentication
0d5f233 
- Implemented `localAuthDisabled()` method to check if local auth is disabled
- Added `disableLocalAuth()` method to disable SAS authentication
- Updated Redis cache definition and update stages to include local auth settings
- Modified tests to verify local auth disable functionality
- Extended Redis cache models to support local authentication configuration
- Integrated local auth settings into Redis cache creation and update flows
@v-huizhu2
test(redis): remove disableLocalAuth calls in tests
f6ee95e 
- Removed disableLocalAuth() from Redis cache update operations
- Commented out Redis configuration backup settings and assertions
- Simplified test cases by removing redundant local auth disabling
- Updated public network access test to match new behavior
- Maintained core Redis cache functionality verification
@v-huizhu2
fix(redis): correct disableLocalAuth implementation for create and up…
f5955be 
…date modes

- Updated disableLocalAuth to properly set authentication flag during create mode
- Updated disableLocalAuth to properly set authentication flag during update mode
- Removed redundant disableLocalAuth call in test updates
- Added assertions to verify local auth disabled state after creation
- Enabled AAD configuration in multiple test scenarios for comprehensive coverage
@v-huizhu2
fix(redis): remove non-SSL port configuration from tests
32c29f3 
- Removed deprecated `withNonSslPort()` method calls in Redis cache setup
- Updated test configurations to align with current SSL-only requirements
- Ensured firewall rules and other configurations remain unchanged
- Verified that all existing test cases pass without non-SSL port dependency
@v-huizhu2
fix(redis): update Redis cache AOF backup configuration
f145c96 
- Replace hardcoded connection string with storage account blob endpoint
- Add managed identity authentication for data persistence
- Disable secondary AOF storage connection string validation
- Maintain AOF backup enabled configuration
- Keep local auth disabled and AAD enabled settings
@v-huizhu2
test(redis): update RedisCacheOperationsTests to use managed identity…
ed2dfa9 
… for RDB backup configuration
@v-huizhu2
test(redis): simplify Redis configuration in RedisCacheOperationsTests
ea6f550
@v-huizhu2
chore(redis): update assets tag for azure-resourcemanager-redis
e2d50e1 
Update the assets tag from 87b73bc96e to 4829c91ebd in the
assets.json file for the redis resource manager package.
@v-huizhu2
Merge branch 'Azure:main' into mgmt_livetests_redis_resourcehealth
99dace4
Copilot AI review requested due to automatic review settings December 25, 2025 08:29
@github-actions github-actions bot added the Mgmt This issue is related to a management-plane library. label Dec 25, 2025
Copilot AI reviewed Dec 25, 2025
View reviewed changes
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes live test failures in the Azure Redis Cache management library by addressing Azure policy requirements that disallow local authentication. The changes enable Azure Active Directory (AAD) authentication, disable local/access key authentication, and configure managed identity for backup operations.

Key Changes:

  • Adds support for disabling local authentication via new disableLocalAuth() API and localAuthDisabled() getter
  • Updates all Redis Cache test creations to enable AAD authentication and disable local auth
  • Removes non-SSL port configuration (incompatible with AAD authentication)
  • Switches backup configuration from connection strings to managed identity authentication

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 8 comments.

File Description
RedisCacheOperationsTests.java Updates test Redis Cache creations to enable AAD, disable local auth, remove non-SSL ports, and use managed identity for RDB/AOF backup configurations
RedisCache.java Adds localAuthDisabled() method and WithLocalAuth interfaces to definition and update stages to support disabling local authentication
RedisCacheImpl.java Implements localAuthDisabled() getter and disableLocalAuth() method that sets the disableAccessKeyAuthentication property

@v-huizhu2
```
41ff0c8 
docs(redis): update Redis cache documentation to use correct terminology

- change "Redis namespace" to "Redis cache" in interface documentation
- update method descriptions to reference Redis cache instead of namespace
- correct return type descriptions to use "Redis cache definition" terminology
- maintain consistent documentation across both creation and update stages
```
@v-huizhu2
Merge branch 'mgmt_livetests_redis_resourcehealth' of https://github.…
de574f7 
…com/v-huizhu2/azure-sdk-for-java into mgmt_livetests_redis_resourcehealth
@v-huizhu2
refactor(test): remove unused storage connection string code in Redis…
6b9f30c 
… cache tests

- Remove commented out storage connection string generation code
- Remove unused connectionString variable declaration
- Remove commented out Redis configuration lines for aof-storage-connection-string-0 and
  aof-storage-connection-string-1
- Remove commented out assertion for aofStorageConnectionString1
- Clean up redundant test code that was not being used
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@weidongxu-microsoft weidongxu-microsoft Awaiting requested review from weidongxu-microsoft weidongxu-microsoft is a code owner

@haolingdong-msft haolingdong-msft Awaiting requested review from haolingdong-msft haolingdong-msft is a code owner

@XiaofeiCao XiaofeiCao Awaiting requested review from XiaofeiCao XiaofeiCao is a code owner

@ArthurMa1978 ArthurMa1978 Awaiting requested review from ArthurMa1978 ArthurMa1978 is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

Mgmt This issue is related to a management-plane library.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@v-huizhu2