Skip to content

backwards compatibility fixes#7

Merged
seetadev merged 5 commits intomasterfrom
dhuseby/fix/backwards-compatibility
Feb 22, 2026
Merged

backwards compatibility fixes#7
seetadev merged 5 commits intomasterfrom
dhuseby/fix/backwards-compatibility

Conversation

@dhuseby
Copy link
Copy Markdown
Contributor

@dhuseby dhuseby commented Feb 20, 2026

This PR adds backwards compatiblity to the transport interop tests while preserving most of the optimizations made in the refactor. This adds the following:

  • Implementations using the old interface are marked with "legacy: true" to signal that it uses lower case environment variables, does not isolate Redis keys, and outputs JSON results.
  • Adds a tiny Redis proxy whenever the dialer or listener is "legacy: true". The Redis proxy takes the TEST_KEY and translates the Redis key from the legacy implementation into a properly isolated Redis key and request to the global Redis server

The net effect is that the only failing tests are due to upstream bugs. There are 13 different upstream bugs that cause test failures.

Signed-off-by: Dave Grantham dwg@linuxprogrammer.org

Signed-off-by: Dave Grantham <dwg@linuxprogrammer.org>
@dhuseby dhuseby requested a review from acul71 February 20, 2026 02:48
Dave Grantham added 4 commits February 20, 2026 15:37
Signed-off-by: Dave Grantham <dwg@linuxprogrammer.org>
Signed-off-by: Dave Grantham <dwg@linuxprogrammer.org>
Signed-off-by: Dave Grantham <dwg@linuxprogrammer.org>
Signed-off-by: Dave Grantham <dwg@linuxprogrammer.org>
@dhuseby
Copy link
Copy Markdown
Contributor Author

dhuseby commented Feb 22, 2026

@seetadev @acul71 this is ready to go. 63 tests are failing but they are all related to the upstream bugs in implementations.

@seetadev
Copy link
Copy Markdown
Collaborator

@seetadev @acul71 this is ready to go. 63 tests are failing but they are all related to the upstream bugs in implementations.

@dhuseby : Hi Dave. Thank you for sharing. Appreciate it.

@acul71 : Lets catch up today or tomorrow morning and arrive at a workplan on these 63 tests that are failing. We can report them to the maintainers of the respective implementations to coordinate with us to arrive at a good conclusion on these issues.

@seetadev seetadev merged commit 4775a72 into master Feb 22, 2026
5 of 6 checks passed
@seetadev seetadev deleted the dhuseby/fix/backwards-compatibility branch February 22, 2026 08:01
@seetadev seetadev restored the dhuseby/fix/backwards-compatibility branch February 22, 2026 08:03
Keerthivasan-Venkitajalam added a commit to Keerthivasan-Venkitajalam/unified-testing that referenced this pull request Feb 22, 2026
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