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.

Sign up for GitHub

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

Jump to bottom

Exclude psycopg2 from automatic upgrades #15864

Merged
merged 2 commits into from
Sep 19, 2023
Merged

Exclude psycopg2 from automatic upgrades #15864

merged 2 commits into from
Sep 19, 2023

Conversation

iliakur
Copy link
Contributor

@iliakur iliakur commented Sep 19, 2023

What does this PR do?

Prevent us from upgrading psycopg with all other dependencies.

Motivation

#15859

Additional Notes

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Changelog entries must be created for modifications to shipped code
  • Add the qa/skip-qa label if the PR doesn't need to be tested during QA.

@iliakur iliakur requested a review from a team as a code owner September 19, 2023 14:09
@iliakur iliakur requested a review from a team as a code owner September 19, 2023 14:12
@ghost ghost added the documentation label Sep 19, 2023
@codecov
Copy link

codecov bot commented Sep 19, 2023

Codecov Report

Merging #15864 (a9be6ac) into master (1118352) will decrease coverage by 0.01%.
The diff coverage is n/a.

Flag Coverage Δ
activemq ?
cassandra ?
datadog_checks_dev 79.96% <ø> (-2.49%) ⬇️
hive ?
hivemq ?
hudi ?
ignite ?
jboss_wildfly ?
kafka ?
presto ?
solr ?
tomcat ?
weblogic ?

Flags with carried forward coverage won't be shown. Click here to find out more.

@github-actions
Copy link

Test Results

    4 files      4 suites   5m 4s ⏱️
418 tests 418 ✔️   0 💤 0 ❌
936 runs  903 ✔️ 33 💤 0 ❌

Results for commit a9be6ac.

@iliakur iliakur merged commit 1e80680 into master Sep 19, 2023
40 checks passed
@iliakur iliakur deleted the ik/pin-psycopg branch September 19, 2023 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yzhan289 yzhan289 yzhan289 approved these changes

@vivek-datadog vivek-datadog vivek-datadog approved these changes

Assignees
No one assigned
Labels
dev_package dev/tooling documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@iliakur @yzhan289 @vivek-datadog