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

Upgrade to integrations to new configuration API #544

Merged
merged 23 commits into from
Oct 2, 2018

Conversation

delner
Copy link
Contributor

@delner delner commented Sep 17, 2018

In order to get consistent behavior, and access to new functions, we should upgrade all of our integrations to the new integration API added in #450 .

Upgrades should be backwards compatible, and users should not need to make any changes to their existing configurations in order to use this newly upgraded API.

Here's a list of all the integrations that need to be upgraded:

After all of these have been updated, we should merge this PR.

@delner delner added core Involves Datadog core libraries integrations Involves tracing integrations do-not-merge/WIP Not ready for merge dev/refactor Involves refactoring existing components labels Sep 17, 2018
@delner delner self-assigned this Sep 17, 2018
@delner delner force-pushed the refactor/upgrade_to_integration_configuration branch 2 times, most recently from 966f7e4 to bb6b171 Compare September 21, 2018 15:08
@delner delner changed the base branch from 0.16-dev to 0.17-dev September 21, 2018 15:09
@delner delner added this to the 0.17.0 milestone Sep 21, 2018
@delner delner force-pushed the refactor/upgrade_to_integration_configuration branch 5 times, most recently from e7f8cca to e772421 Compare September 21, 2018 21:11
@delner delner force-pushed the refactor/upgrade_to_integration_configuration branch from e772421 to 16d9782 Compare September 24, 2018 15:46
@delner delner removed the do-not-merge/WIP Not ready for merge label Oct 1, 2018
@delner delner requested a review from pawelchcki October 1, 2018 21:22
Copy link
Contributor

@pawelchcki pawelchcki left a comment

Choose a reason for hiding this comment

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

:shipit:

@delner delner merged commit 11985ec into 0.17-dev Oct 2, 2018
@delner delner deleted the refactor/upgrade_to_integration_configuration branch October 2, 2018 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Involves Datadog core libraries dev/refactor Involves refactoring existing components integrations Involves tracing integrations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants