Collect more system info

[asmacdo]

Summary

Adds OS and Linux-distribution provenance to the system block of info.json, so downstream consumers (plot, gallery, future tooling) can tell what host a run came from without parsing free-text strings.

Schema bumped 0.2.2 → 0.2.3. Old logs read by con-duct ls are backfilled with empty strings for the new fields, so consumers can rely on the fields being present.

What's added (14 fields, all under system)

From platform.uname():
os_name, os_release, os_version, arch, processor

From platform.freedesktop_os_release() (Python 3.10+; reads /etc/os-release, falls back to "" on systems without the file):
distro_id, distro_id_like, distro_name, distro_version, distro_version_id, distro_codename, distro_variant_id, distro_pretty_name, distro_build_id

Sample system block:

 'system': {'arch': 'x86_64',
            'cpu_total': 20,
            'distro_build_id': '',
            'distro_codename': '',
            'distro_id': 'fedora',
            'distro_id_like': '',
            'distro_name': 'Fedora Linux',
            'distro_pretty_name': 'Fedora Linux 42 (Workstation Edition)',
            'distro_variant_id': 'workstation',
            'distro_version': '42 (Workstation Edition)',
            'distro_version_id': '42',
            'hostname': 'fancy',
            'memory_total': 33330028544,
            'os_name': 'Linux',
            'os_release': '6.19.12-100.fc42.x86_64',
            'os_version': '#1 SMP PREEMPT_DYNAMIC Sun Apr 12 15:27:03 UTC 2026',
            'processor': '',
            'uid': 1000,
            'user': 'austin'},

Why so many fields?

The motivation here is provenance (because everything matters), not any single consumer's branching needs. Plot logic flagged the gap (it needs to branch between Linux and macOS for pcpu), but capturing the standard os-release fields now means future readers — gallery, repro tooling, archeology — can answer questions we haven't anticipated (variant, codename, build id) without us having to re-collect.

Compatibility

• Schema version bumped to 0.2.3.
• ls.ensure_compliant_schema migrates old logs by adding empty strings for all 14 fields, matching the convention set by the 0.2.1 (working_directory) and 0.2.2 (message) migrations.
• LS_FIELD_CHOICES extended so the new fields are queryable via con-duct ls -F and --eval-filter.
• POSIX-only, matching the existing constraint imposed by os.sysconf calls in get_system_info(). On systems without /etc/os-release (macOS), the nine distro_* fields fall back to "".

TODOs

• [ ] fixup for the failing osx run one way or another (just flakey dont let it block.)

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.17%. Comparing base (dff0654) to head (1164563).

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #425   +/-   ##
=======================================
  Coverage   91.16%   91.17%           
=======================================
  Files          15       15           
  Lines        1257     1280   +23     
  Branches      171      173    +2     
=======================================
+ Hits         1146     1167   +21     
- Misses         77       78    +1     
- Partials       34       35    +1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[yarikoptic]

failing test on intel osx

______________________ test_spawn_children[duct-1-plain] _______________________

temp_output_dir = '/private/var/folders/x7/_nsk_h4s1kng41z5pgfbx7sh0000gn/T/pytest-of-runner/pytest-0/test_spawn_children_duct_1_pla3/'
duct_cmd = 'duct', mode = 'plain', num_children = 1

    @pytest.mark.flaky(reruns=3)
    @pytest.mark.parametrize("mode", ["plain", "subshell", "nohup", "setsid"])
    @pytest.mark.parametrize("num_children", [1, 2, 10])
    def test_spawn_children(
        temp_output_dir: str, duct_cmd: str, mode: str, num_children: int
    ) -> None:
        duct_prefix = f"{temp_output_dir}log_"
        script_path = TEST_SCRIPT_DIR / "spawn_children.sh"
        dur = "0.3"
        command = (
            f"{duct_cmd} -q --s-i 0.001 --r-i 0.01 "
            f"-p {duct_prefix} {script_path} {mode} {num_children} {dur}"
        )
        subprocess.check_output(command, shell=True)
    
        with open(f"{duct_prefix}{SUFFIXES['usage']}") as usage_file:
            all_samples = [json.loads(line) for line in usage_file]
    
        # Only count the child sleep processes
        all_child_pids = set(
            pid
            for sample in all_samples
            for pid, proc in sample["processes"].items()
            if "sleep" in proc["cmd"]
        )
        # Add one pid for the hold-the-door process, see spawn_children.sh line 7
        if mode == "setsid":
            assert len(all_child_pids) == 1
        else:
>           assert len(all_child_pids) == num_children + 1
E           assert 0 == (1 + 1)
E            +  where 0 = len(set())

test/duct_main/test_e2e.py:60: AssertionError
=========================== short test summary info ============================
FAILED test/duct_main/test_e2e.py::test_spawn_children[duct-1-plain] - assert 0 == (1 + 1)
 +  where 0 = len(set())

[Copilot]

Pull request overview

This PR extends the info.json system block with OS + Linux distribution provenance (from platform.uname() and /etc/os-release) so downstream consumers can reliably branch on host metadata, and bumps the schema version from 0.2.2 to 0.2.3.

Changes:

• Add os_*, arch, processor, and distro_* fields to SystemInfo and collect them in the tracker.
• Backfill new system fields when reading older logs via ls.ensure_compliant_schema, and expose them as selectable con-duct ls fields.
• Update tests to include system and validate the new fields’ presence/types.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
test/test_ls.py	Updates fixtures to include system and asserts migration backfills new fields.
test/duct_main/test_report.py	Extends system-info sanity test to validate new OS/distro provenance fields.
src/con_duct/ls.py	Adds new fields to ls field choices and migrates pre-0.2.3 logs by backfilling system fields.
src/con_duct/_tracker.py	Collects uname + freedesktop OS-release metadata into SystemInfo.
src/con_duct/_models.py	Extends SystemInfo dataclass with OS/distro provenance fields.
src/con_duct/_constants.py	Bumps __schema_version__ to 0.2.3.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[yarikoptic]

Good that it uses built in library , so let's proceed

[github-actions]

🚀 PR was released in v0.22.0 🚀