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

update to reqwest 0.12 #7561

Merged
merged 5 commits into from
May 2, 2024
Merged

update to reqwest 0.12 #7561

merged 5 commits into from
May 2, 2024

Conversation

conradludgate
Copy link
Contributor

Problem

#7557

Summary of changes

Checklist before requesting a review

  • I have performed a self-review of my code.
  • If it is a core feature, I have added thorough tests.
  • Do we need to implement analytics? if so did you add the relevant metrics to the dashboard?
  • If this PR requires public announcement, mark it with /release-notes label and add several sentences in this section.

Checklist before merging

  • Do not forget to reformat commit message to not include the above checklist

@conradludgate
Copy link
Contributor Author

conradludgate commented Apr 30, 2024

azure and opentelemetry need updating when they are supported

$ cargo tree -i [email protected]
reqwest v0.11.19
├── azure_core v0.19.0
│   ├── azure_identity v0.19.0
│   │   └── remote_storage v0.1.0 (/Users/conrad/Documents/neon/neon/libs/remote_storage)
│   │       ├── compute_api v0.1.0 (/Users/conrad/Documents/neon/neon/libs/compute_api)
│   │       │   ├── compute_tools v0.1.0 (/Users/conrad/Documents/neon/neon/compute_tools)
│   │       │   └── control_plane v0.1.0 (/Users/conrad/Documents/neon/neon/control_plane)
│   │       │       └── storage_controller v0.1.0 (/Users/conrad/Documents/neon/neon/storage_controller)
│   │       ├── compute_tools v0.1.0 (/Users/conrad/Documents/neon/neon/compute_tools)
│   │       ├── pagectl v0.1.0 (/Users/conrad/Documents/neon/neon/pageserver/ctl)
│   │       ├── pageserver v0.1.0 (/Users/conrad/Documents/neon/neon/pageserver)
│   │       │   ├── pagectl v0.1.0 (/Users/conrad/Documents/neon/neon/pageserver/ctl)
│   │       │   └── s3_scrubber v0.1.0 (/Users/conrad/Documents/neon/neon/s3_scrubber)
│   │       ├── proxy v0.1.0 (/Users/conrad/Documents/neon/neon/proxy)
│   │       ├── s3_scrubber v0.1.0 (/Users/conrad/Documents/neon/neon/s3_scrubber)
│   │       └── safekeeper v0.1.0 (/Users/conrad/Documents/neon/neon/safekeeper)
│   ├── azure_storage v0.19.0
│   │   ├── azure_storage_blobs v0.19.0
│   │   │   └── remote_storage v0.1.0 (/Users/conrad/Documents/neon/neon/libs/remote_storage) (*)
│   │   └── remote_storage v0.1.0 (/Users/conrad/Documents/neon/neon/libs/remote_storage) (*)
│   ├── azure_storage_blobs v0.19.0 (*)
│   ├── azure_svc_blobstorage v0.19.0
│   │   └── azure_storage_blobs v0.19.0 (*)
│   └── remote_storage v0.1.0 (/Users/conrad/Documents/neon/neon/libs/remote_storage) (*)
├── opentelemetry-http v0.9.0
│   └── opentelemetry-otlp v0.13.0
│       └── tracing-utils v0.1.0 (/Users/conrad/Documents/neon/neon/libs/tracing-utils)
│           ├── compute_tools v0.1.0 (/Users/conrad/Documents/neon/neon/compute_tools)
│           └── proxy v0.1.0 (/Users/conrad/Documents/neon/neon/proxy)
└── opentelemetry-otlp v0.13.0 (*)

Copy link

github-actions bot commented Apr 30, 2024

2856 tests run: 2735 passed, 0 failed, 121 skipped (full report)


Flaky tests (2)

Postgres 15

  • test_vm_bit_clear_on_heap_lock: debug

Postgres 14

  • test_vm_bit_clear_on_heap_lock: debug

Code coverage* (full report)

  • functions: 28.1% (6600 of 23487 functions)
  • lines: 46.7% (46860 of 100251 lines)

* collected from Rust tests only


The comment gets automatically updated with the latest test results
8d89c75 at 2024-05-01T16:34:38.708Z :recycle:

@conradludgate conradludgate force-pushed the reqwest0.12 branch 2 times, most recently from dcdfe85 to 9d2f94f Compare May 1, 2024 08:21
@conradludgate conradludgate marked this pull request as ready for review May 1, 2024 15:52
@conradludgate conradludgate requested review from a team as code owners May 1, 2024 15:52
@conradludgate conradludgate requested review from khanova and problame May 1, 2024 15:52
@conradludgate conradludgate enabled auto-merge (squash) May 1, 2024 15:53
@arpad-m
Copy link
Member

arpad-m commented May 1, 2024

Needs proxy review, otherwise this is ready for merge. Thanks Conrad!

@conradludgate conradludgate merged commit cb4b475 into main May 2, 2024
54 checks passed
@conradludgate conradludgate deleted the reqwest0.12 branch May 2, 2024 09:16
@arpad-m arpad-m mentioned this pull request May 2, 2024
conradludgate added a commit that referenced this pull request May 8, 2024
## Problem

#7557

## Summary of changes
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