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

Support older pika versions #837

Merged
merged 7 commits into from
Dec 25, 2021

Conversation

nozik
Copy link
Contributor

@nozik nozik commented Dec 19, 2021

Description

Currently, opentelemetry-instrumentation-pika supports only pika versions above 1.0.0. This PR adds support to pika >= 0.12.0

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)
  • This change requires a documentation update

How Has This Been Tested?

I've updated tox.ini to include tests for the older pika versions.

Does This PR Require a Core Repo Change?

  • 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

@nozik nozik requested a review from a team December 19, 2021 20:13
@github-actions github-actions bot requested review from nikosokolik and oxeye-nikolay and removed request for a team December 19, 2021 20:13
@nozik nozik force-pushed the support_older_pika_versions branch from 52e6330 to fdba606 Compare December 20, 2021 15:58
@nozik nozik force-pushed the support_older_pika_versions branch 2 times, most recently from bc01feb to 26399c3 Compare December 20, 2021 18:08
@nozik nozik force-pushed the support_older_pika_versions branch from 26399c3 to 29fbd93 Compare December 20, 2021 18:17
@nozik nozik force-pushed the support_older_pika_versions branch from 29fbd93 to e85a2c8 Compare December 20, 2021 18:21
@owais owais merged commit 26aa17f into open-telemetry:main Dec 25, 2021
@nozik nozik deleted the support_older_pika_versions branch December 28, 2021 10:49
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.

4 participants