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

Update dependencies before Agent release #16154

Merged
merged 9 commits into from
Nov 8, 2023
Merged

Update dependencies before Agent release #16154

merged 9 commits into from
Nov 8, 2023

Conversation

iliakur
Copy link
Contributor

@iliakur iliakur commented Nov 6, 2023

What does this PR do?

Motivation

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 review from a team as code owners November 6, 2023 13:05
@iliakur iliakur requested a review from a team November 6, 2023 13:05
@github-actions GitHub Actions
Copy link

github-actions bot commented Nov 6, 2023

The validations job has failed; please review the Files changed tab for possible suggestions to resolve.

@codecov Codecov
Copy link

codecov bot commented Nov 6, 2023 •
edited
Loading

Codecov Report

Merging #16154 (aa6d792) into master (8426dd1) will increase coverage by 0.09%.
Report is 9 commits behind head on master.
The diff coverage is n/a.

Flag Coverage Δ
activemq ?
amazon_msk 89.07% <ø> (ø)
cassandra ?
cloud_foundry_api 96.35% <ø> (+0.12%) ⬆️
datadog_checks_base 89.62% <ø> (+0.36%) ⬆️
datadog_checks_downloader 81.65% <ø> (ø)
hive ?
hivemq ?
hudi ?
ignite ?
jboss_wildfly ?
kafka ?
pgbouncer 91.33% <ø> (ø)
postgres 91.79% <ø> (-0.05%) ⬇️
presto ?
snowflake 96.66% <ø> (ø)
solr ?
sqlserver 86.10% <ø> (-1.44%) ⬇️
tomcat ?
vertica 98.34% <ø> (ø)
weblogic ?

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

@github-actions GitHub Actions
Copy link

github-actions bot commented Nov 6, 2023

The validations job has failed; please review the Files changed tab for possible suggestions to resolve.

@iliakur iliakur mentioned this pull request Nov 6, 2023
Merged
10 tasks
@github-actions GitHub Actions
Copy link

github-actions bot commented Nov 6, 2023 •
edited
Loading

Test Results

       84 files       84 suites   1h 31m 2s ⏱️
  1 957 tests 1 925 ✔️      32 💤 0 ❌
10 771 runs  9 687 ✔️ 1 084 💤 0 ❌

Results for commit aa6d792.

♻️ This comment has been updated with latest results.

lu-zhengda
lu-zhengda reviewed Nov 6, 2023
View reviewed changes
postgres/pyproject.toml Outdated Show resolved Hide resolved
sqlserver/pyproject.toml Outdated Show resolved Hide resolved
@github-actions GitHub Actions
Copy link

github-actions bot commented Nov 6, 2023

The validations job has failed; please review the Files changed tab for possible suggestions to resolve.

lu-zhengda
lu-zhengda previously approved these changes Nov 6, 2023
View reviewed changes
sarah-witt
sarah-witt previously approved these changes Nov 6, 2023
View reviewed changes
Copy link
Contributor

@sarah-witt sarah-witt left a comment

Choose a reason for hiding this comment

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

looks good for platform integrations!

@iliakur iliakur dismissed stale reviews from sarah-witt and lu-zhengda via 628972f November 7, 2023 13:04
@github-actions GitHub Actions
Copy link

github-actions bot commented Nov 7, 2023

The validations job has failed; please review the Files changed tab for possible suggestions to resolve.

@iliakur iliakur mentioned this pull request Nov 7, 2023
Merged
3 tasks
@iliakur iliakur requested a review from a team as a code owner November 7, 2023 13:39
@ghost ghost added the downloader label Nov 7, 2023
FlorentClarret
FlorentClarret previously approved these changes Nov 8, 2023
View reviewed changes
Copy link
Member

@FlorentClarret FlorentClarret left a comment

Choose a reason for hiding this comment

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

Looks good to me but you'll need to fix the conflict

FlorentClarret
FlorentClarret previously approved these changes Nov 8, 2023
View reviewed changes
@github-actions GitHub Actions
Copy link

github-actions bot commented Nov 8, 2023

The validations job has failed; please review the Files changed tab for possible suggestions to resolve.

FlorentClarret
FlorentClarret approved these changes Nov 8, 2023
View reviewed changes
Copy link
Member

@FlorentClarret FlorentClarret left a comment

Choose a reason for hiding this comment

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

Thanks Ilia

@iliakur iliakur merged commit 42480b9 into master Nov 8, 2023
51 checks passed
@iliakur iliakur deleted the ik/update-deps branch November 8, 2023 10:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@FlorentClarret FlorentClarret FlorentClarret approved these changes

@sarah-witt sarah-witt sarah-witt left review comments

@lu-zhengda lu-zhengda lu-zhengda left review comments

Assignees
No one assigned
Labels
base_package dependencies downloader integration/amazon_msk integration/cloud_foundry_api integration/mysql integration/oracle integration/pgbouncer integration/postgres integration/snowflake integration/sqlserver integration/vertica
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@iliakur @FlorentClarret @lu-zhengda @sarah-witt