Skip to content
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

Instrument redis.asyncio clients #1076

Merged
merged 4 commits into from
May 5, 2022

Conversation

sd2k
Copy link
Contributor

@sd2k sd2k commented Apr 29, 2022

Description

This PR adds instrumentation to async Redis clients in the Redis library.

As of redis-py 4.2.0, async support has been merged into the main redis-py library (basically taken from aioredis, I believe). Instrumentation should work in the same way as the sync Redis clients.

No dependency changes are required in the actual package; we only activate async instrumentation if the Redis version is high enough. The docker-tests testenv has its Redis version bumped to one that include async support, however.

Should supersede #569 since aioredis is no longer recommended.

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

  • Functional tests added to test_redis_functional.py

Does This PR Require a Core Repo Change?

  • Yes. - Link to PR:
  • No.

Checklist:

See contributing.md for styleguide, changelog guidelines, and more.

  • Followed the style guidelines of this project
  • Changelogs have been updated
  • Unit tests have been added
  • Documentation has been updated

@sd2k sd2k requested a review from a team April 29, 2022 10:20
@sd2k sd2k force-pushed the instrument-redis-asyncio branch from d8629ee to b191b0b Compare April 29, 2022 10:39
@sd2k sd2k force-pushed the instrument-redis-asyncio branch from b191b0b to 20c01f2 Compare April 29, 2022 10:40
@sd2k sd2k force-pushed the instrument-redis-asyncio branch from 20c01f2 to 0765def Compare May 1, 2022 14:37
@sd2k sd2k force-pushed the instrument-redis-asyncio branch from 9b8d8f4 to 0498d3e Compare May 3, 2022 08:56
@srikanthccv srikanthccv merged commit fa56c6c into open-telemetry:main May 5, 2022
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.

3 participants