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 Rust crate tough to 0.15 #4477

Merged
merged 8 commits into from
Nov 20, 2023
Merged

Update Rust crate tough to 0.15 #4477

merged 8 commits into from
Nov 20, 2023

Conversation

oxide-renovate[bot]
Copy link
Contributor

@oxide-renovate oxide-renovate bot commented Nov 9, 2023

This PR contains the following updates:

Package Type Update Change
tough workspace.dependencies minor 0.14 -> 0.15

Release Notes

awslabs/tough (tough)

v0.15.0

Compare Source

Changes
  • ❗Breaking Change❗: Most public functions and traits have been changed to async, thanks @​phu-cinemo #​687

Configuration

📅 Schedule: Branch creation - "after 8pm,before 6am" in timezone America/Los_Angeles, Automerge - "after 8pm,before 6am" in timezone America/Los_Angeles.

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

@oxide-renovate oxide-renovate bot added the dependencies Pull requests that update a dependency file label Nov 9, 2023
@oxide-renovate oxide-renovate bot force-pushed the renovate/tough-0.x branch 2 times, most recently from c7b57cb to 96d749c Compare November 14, 2023 04:07
@oxide-renovate oxide-renovate bot changed the title chore(deps): update rust crate tough to 0.15 Update Rust crate tough to 0.15 Nov 17, 2023
@oxide-renovate
Copy link
Contributor Author

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.

You can manually request rebase by checking the rebase/retry box above.

Warning: custom changes will be lost.

@sunshowers
Copy link
Contributor

Ah crap, some failures that I need to address.

nexus/src/updates.rs Outdated Show resolved Hide resolved
Copy link
Contributor

@jgallagher jgallagher left a comment

Choose a reason for hiding this comment

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

Thanks for going through this - looks pretty tedious, and I apologize for some of the additionally-tedious comments I left. :-/

tufaceous-lib/src/artifact.rs Show resolved Hide resolved
tufaceous-lib/src/repository.rs Outdated Show resolved Hide resolved
.map_err(RepositoryError::LoadRepository)?;

let artifacts = repository
.read_artifacts()
.await
.map_err(RepositoryError::ReadArtifactsDocument)?;

// Create another temporary directory where we'll "permanently" (as long
Copy link
Contributor

Choose a reason for hiding this comment

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

UpdatePlanBuilder::new() uses camino_tempfile to create a temporary directory - should we spawn_blocking it? (Feels like maybe "technically yes since it's I/O, but practically probably doesn't matter"?)

Copy link
Contributor

Choose a reason for hiding this comment

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

Ehhhh I really don't think it matters to be honest. This isn't an operation that takes a long time.

wicketd/src/artifacts/artifacts_with_plan.rs Show resolved Hide resolved
wicketd/src/artifacts/extracted_artifacts.rs Show resolved Hide resolved
wicketd/src/artifacts/update_plan.rs Show resolved Hide resolved
wicketd/src/artifacts/update_plan.rs Show resolved Hide resolved
@sunshowers sunshowers enabled auto-merge (squash) November 20, 2023 21:08
@sunshowers sunshowers merged commit cd2d23b into main Nov 20, 2023
21 of 22 checks passed
@sunshowers sunshowers deleted the renovate/tough-0.x branch November 20, 2023 22:20
sunshowers added a commit that referenced this pull request Dec 7, 2023
The switch to a multithreaded executor for this test in #4477 appears to have
exposed a race within this test (quite ironic given the name!) Fix the race by
being explicit about the ordering of events.

Fixes #4590.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants