support OpenOCD bootstrap without environment

[ozan956]

The bootstrap fallback path was too narrow for OpenOCD-based setupswithout a local environment.

So far, labgrid-client bootstrap only auto-created OpenOCDDriver for NetworkAlteraUSBBlaster. If a place exposed only NetworkUSBDebugger, the client failed with "target has no compatible resource available" even though OpenOCDDriver can bind to that resource type.

In addition, the existing bootstrap_args path only handled raw 'key=valuestrings. This was not sufficient forOpenOCDDriverarguments such asload_commands, search, or config`, which often need structured values like lists.

Extend the bootstrap fallback to NetworkUSBDebugger and parse bootstrap driver arguments as YAML values. This allows CLI-only OpenOCD bootstrap flows to pass structured driver arguments without requiring a local environment file.

[github-actions]

LLM review

This series extends labgrid-client bootstrap's driver fallback to also cover NetworkUSBDebugger resources (matching OpenOCDDriver's existing bindings and documentation), and switches bootstrap_args parsing from raw key=value strings to YAML-parsed values so structured OpenOCDDriver arguments (e.g. load_commands, search) can be passed from the CLI.

run: 31177809422

ebd8e7d - remote/client: support OpenOCD bootstrap without environment

Ordering bug: args = self._parse_driver_args(self.args.bootstrap_args) was moved to the top of bootstrap(), executing unconditionally before checking whether an existing BootstrapProtocol driver can be used. Previously this parsing only happened inside the NetworkAlteraUSBBlaster branch, i.e. only when an OpenOCDDriver was actually being constructed.

As a result, any leftover/malformed token in bootstrap_args now breaks bootstrap for IMXUSBDriver/MXSUSBDriver/RKUSBDriver, or even an environment-configured OpenOCDDriver found via get_driver("BootstrapProtocol"), even though those args are never consumed on that path.

Reproduced with a script instantiating an IMXUSBDriver + NetworkIMXUSBLoader and calling bootstrap() with bootstrap_args=["--verbose"]:

• at the parent commit (46569aa): succeeds.
• at ebd8e7d: raises UserError: invalid bootstrap argument '--verbose', expected key=value.

Fix: move the _parse_driver_args() call back into the OpenOCD branch, right before constructing a new OpenOCDDriver. Verified the repro succeeds again and all existing tests in tests/test_openocd_client.py still pass.

CI warnings

No CI logs were available for this run (CI notes input was empty). pyflakes and pycodestyle (110 cols) on labgrid/remote/client.py show no new issues introduced by this series.

Verification data

Reproduced the regression directly by checking out labgrid/remote/client.py at both 46569aa and ebd8e7d and calling ClientSession.bootstrap() against a mocked Target/NetworkIMXUSBLoader/IMXUSBDriver. Also added a regression test (test_bootstrap_ignores_bad_args_for_existing_driver) confirmed to fail without the fix and pass with it. Cross-checked OpenOCDDriver.bindings in labgrid/driver/openocddriver.py and doc/configuration.rst to confirm NetworkUSBDebugger support is consistent with existing driver capability/documentation (no doc changes needed). Confirmed PyYAML is already a declared dependency in pyproject.toml, so import yaml adds no new dependency.

Suggested patches

Apply the suggested patches with:

cd path/to/repository
export GITHUB_TOKEN=ghp_***
apply-patches --repo=analogdevicesinc/labgrid 31177809422

Install instructions

The following one-liner installs the script if not present already:

curl -fSsL "https://raw.githubusercontent.com/analogdevicesinc/doctools/refs/heads/main/ci/scripts/apply-patches.sh"      -o ~/.local/bin/apply-patches.sh &&   grep -q "/apply-patches.sh" ~/.bashrc || echo "source ~/.local/bin/apply-patches.sh" >> $_ ; . $_

More information at AI Usage.