Skip to content

Recording Redis Version in the executor#514

Closed
vaishalitvv wants to merge 0 commit into
microsoft:mainfrom
vaishalitvv:Recording-Redis-Version
Closed

Recording Redis Version in the executor#514
vaishalitvv wants to merge 0 commit into
microsoft:mainfrom
vaishalitvv:Recording-Redis-Version

Conversation

@vaishalitvv

Copy link
Copy Markdown
Contributor

Summary:
This update enables the recording of the Redis version on the server side using the RedisServerExecutor.

Testing:
The functionality has been validated by recording the Redis version in both manual tests and in the Juno PPE environment.

@vaishalitvv vaishalitvv marked this pull request as draft May 20, 2025 20:39
@vaishalitvv vaishalitvv marked this pull request as ready for review May 20, 2025 20:43
@RakeshwarK

Copy link
Copy Markdown
Contributor

Please add unit tests as well.

@vaishalitvv vaishalitvv force-pushed the Recording-Redis-Version branch 2 times, most recently from e0bec11 to 901256c Compare May 27, 2025 19:02
@vaishalitvv

Copy link
Copy Markdown
Contributor Author

Please add unit tests as well.
Yes, I've added the unit tests as well.

@vaishalitvv vaishalitvv reopened this May 27, 2025
@vaishalitvv vaishalitvv force-pushed the Recording-Redis-Version branch from 8e3f86f to 6545def Compare May 29, 2025 19:55
@vaishalitvv

Copy link
Copy Markdown
Contributor Author

@microsoft-github-policy-service agree company="Microsoft"

@vaishalitvv vaishalitvv force-pushed the Recording-Redis-Version branch 2 times, most recently from 98222d2 to cb629ee Compare June 4, 2025 22:58
@vaishalitvv vaishalitvv closed this Jun 4, 2025
@vaishalitvv vaishalitvv force-pushed the Recording-Redis-Version branch from cb629ee to 5035150 Compare June 4, 2025 23:01
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.

2 participants