Skip to content

[persist] Configure timeouts in azure and wire them to LD#31453

Merged
bkirwi merged 2 commits intoMaterializeInc:mainfrom
bkirwi:debug-azure
Feb 12, 2025
Merged

[persist] Configure timeouts in azure and wire them to LD#31453
bkirwi merged 2 commits intoMaterializeInc:mainfrom
bkirwi:debug-azure

Conversation

@bkirwi
Copy link
Contributor

@bkirwi bkirwi commented Feb 10, 2025

Motivation

Worked this up as part of: https://github.com/MaterializeInc/database-issues/issues/8892

This didn't end up being useful for working around the Azurite issue so far, but it seems useful in general...

Checklist

  • This PR has adequate test coverage / QA involvement has been duly considered. (trigger-ci for additional test/nightly runs)
  • This PR has an associated up-to-date design doc, is a design doc (template), or is sufficiently small to not require a design.
  • If this PR evolves an existing $T ⇔ Proto$T mapping (possibly in a backwards-incompatible way), then it is tagged with a T-proto label.
  • If this PR will require changes to cloud orchestration or tests, there is a companion cloud PR to account for those changes that is tagged with the release-blocker label (example).
  • If this PR includes major user-facing behavior changes, I have pinged the relevant PM to schedule a changelog post.

@bkirwi bkirwi marked this pull request as ready for review February 11, 2025 22:14
@bkirwi bkirwi requested review from a team as code owners February 11, 2025 22:14
Copy link
Contributor

@ParkMyCar ParkMyCar left a comment

Choose a reason for hiding this comment

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

Nice!

@bkirwi bkirwi merged commit f0ce5d0 into MaterializeInc:main Feb 12, 2025
80 checks passed
@bkirwi bkirwi added the self-managed-backport-v25.1 Needs to be backported into the v25.1 self-managed release label Feb 12, 2025
def- added a commit to def-/materialize that referenced this pull request Feb 14, 2025
Follow-up to MaterializeInc#31453

Seeing build failures in the 0dt bump test: https://buildkite.com/materialize/nightly/builds/11138#019500e5-e622-409a-97d3-bfc1a23179f3

We already have many reqwest versions in our code, maybe we should not add more
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

self-managed-backport-v25.1-done self-managed-backport-v25.1 Needs to be backported into the v25.1 self-managed release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants