Summary

Add a local_site parameter to SCMClients.

Review Request #14910 — Created March 17, 2026 and updated March 17, 2026, 9:14 a.m.

Information

Owner

chipx86

Repository

Review Board

Branch

release-7.1.x

Bugs

Depends On

Reviewers

Groups

reviewboard

People

Description

Many SCMClients need to manage state tied to a Local Site, and do so by
passing in a client-specific local_site_name. With the upcoming cert
work, this is going to be needed more broadly, and we'll need an
instance.

We always have access to an instance in places where we initialize a
client. This happens in both an SCMTool's __init__() and in
check_repository(). This makes it easy to pass this in.

This change updates the base SCMClient to take and store this,
updates all subclasses to pass it to the parent, and updates call sites
to pass it to the client. In many cases, surrounding code has been
cleaned up to avoid duplicate effort or to bail early when there are
problems.

There were a couple of clients that didn't inherit from SCMClient.
This change updates most of them. A notable exception is the Subversion
clients, which have a lot more complexity going on and aren't worth
tackling in this change.

Testing Done

Unit tests pass.

Commits

Summary	ID
Add a local_site parameter to SCMClients. Many SCMClients need to manage state tied to a Local Site, and do so by passing in a client-specific `local_site_name`. With the upcoming cert work, this is going to be needed more broadly, and we'll need an instance. We always have access to an instance in places where we initialize a client. This happens in both an SCMTool's `__init__()` and in `check_repository()`. This makes it easy to pass this in. This change updates the base `SCMClient` to take and store this, updates all subclasses to pass it to the parent, and updates call sites to pass it to the client. In many cases, surrounding code has been cleaned up to avoid duplicate effort or to bail early when there are problems. There were a couple of clients that didn't inherit from `SCMClient`. This change updates most of them. A notable exception is the Subversion clients, which have a lot more complexity going on and aren't worth tackling in this change.	44faab9b3c5435949d662cf2e08c58349fd4fb8b

Summary

Add a local_site parameter to SCMClients.

Many SCMClients need to manage state tied to a Local Site, and do so by passing in a client-specific `local_site_name`. With the upcoming cert work, this is going to be needed more broadly, and we'll need an instance. We always have access to an instance in places where we initialize a client. This happens in both an SCMTool's `__init__()` and in `check_repository()`. This makes it easy to pass this in. This change updates the base `SCMClient` to take and store this, updates all subclasses to pass it to the parent, and updates call sites to pass it to the client. In many cases, surrounding code has been cleaned up to avoid duplicate effort or to bail early when there are problems. There were a couple of clients that didn't inherit from `SCMClient`. This change updates most of them. A notable exception is the Subversion clients, which have a lot more complexity going on and aren't worth tackling in this change.

44faab9b3c5435949d662cf2e08c58349fd4fb8b

Issues

Description	From	Last Updated
A bunch of the clients also take local_site_name and then store it as an attribute. Perhaps we should plumb that …	david	March 17, 2026, 9:14 a.m.
'reviewboard.site.models.LocalSite' imported but unused Column: 5 Error code: F401	reviewbot	March 17, 2026, 2:55 a.m.
Typo: local_site -> local_site_name	david	March 17, 2026, 9:14 a.m.

flake8 failed.

JSHint passed.

flake8

reviewboard/scmtools/cvs.py (Diff revision 1)

An issue was opened. Show all issues

'reviewboard.site.models.LocalSite' imported but unused

Column: 5
Error code: F401

Fix it!

An issue was opened. Show all issues

A bunch of the clients also take local_site_name and then store it as an attribute. Perhaps we should plumb that through to the base class as well, and then we could derive local_site or local_site_name as needed if one or the other is provided?

reviewboard/scmtools/git.py (Diff revision 1)
An issue was opened. Show all issues
```
Typo: local_site -> local_site_name
```