Skip to content

Add SSHMaster::Connection::trySetBufferSize (backport #10765)#12485

Merged
Ericson2314 merged 2 commits into2.26-maintenancefrom
mergify/bp/2.26-maintenance/pr-10765
Feb 17, 2025
Merged

Add SSHMaster::Connection::trySetBufferSize (backport #10765)#12485
Ericson2314 merged 2 commits into2.26-maintenancefrom
mergify/bp/2.26-maintenance/pr-10765

Conversation

@mergify
Copy link
Contributor

@mergify mergify bot commented Feb 17, 2025

Motivation

It is unused in Nix currently, but will be used in Hydra. This reflects what Hydra does in NixOS/hydra#1387.

Context

We may probably to use it more widely for better SSH store performance, but this needs to be subject to more testing before we do that.

Priorities and Process

Add 👍 to pull requests you find important.

The Nix maintainer team uses a GitHub project board to schedule and track reviews.


This is an automatic backport of pull request #10765 done by [Mergify](https://mergify.com).

It is unused in Nix currently, but will be used in Hydra. This reflects
what Hydra does in NixOS/hydra#1387.

We may probably to use it more widely for better SSH store performance,
but this needs to be subject to more testing before we do that.

(cherry picked from commit 0d25cc6)
Exposed for Hydra. We could make it fancier but with (a) new store
settings (b) switch to `ssh-ng://` both in the works, it doesn't seem
worth it.

(cherry picked from commit 94a7c34)
@mergify mergify bot requested a review from Ericson2314 as a code owner February 17, 2025 16:56
@mergify mergify bot added automatic backport This PR is a backport produced by automation (does not trigger backporting) merge-queue labels Feb 17, 2025
@github-actions github-actions bot added the store Issues and pull requests concerning the Nix store label Feb 17, 2025
@Ericson2314 Ericson2314 merged commit bcbfdc1 into 2.26-maintenance Feb 17, 2025
21 checks passed
@Ericson2314 Ericson2314 deleted the mergify/bp/2.26-maintenance/pr-10765 branch February 17, 2025 17:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automatic backport This PR is a backport produced by automation (does not trigger backporting) merge-queue store Issues and pull requests concerning the Nix store

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant