CI: stand up Gitea Actions (TST-03) + fix the defects it caught #123

Merged

dohertj2 merged 9 commits from fix/ci-selfhosted-tooling into main 2026-07-10 05:11:03 -04:00

Conversation 0 Commits 9 Files Changed 7

+274 -66

dohertj2 commented 2026-07-10 04:28:51 -04:00

Owner

Copy Link

Copy Source

Stand up working Gitea Actions CI (TST-03) + fix the defects it caught

The CI pipeline was authored under TST-03 (P1) but had never executed on a runner. This branch makes it run and go green (portable + java jobs), and fixes five real latent defects the first real run surfaced.

Why it never ran

The co-located gitea-runner on the docker host spawned job containers on an isolated network (container.network: ""), so actions/checkout couldn't resolve Gitea's internal clone URL http://gitea:3000. Fixed on the host (container.network: "traefik"), not in this branch.

Workflow portability (the self-hosted catthehacker act image lacks GitHub-hosted tooling)

• Install PowerShell as a .NET global tool (for check-codegen.ps1's shell: pwsh).
• Install Gradle 9.5.1 directly (act can't resolve the gradle/actions monorepo action; repo has no gradle wrapper).

Real defects the run caught and this branch fixes

1. check-codegen.ps1 called .Trim() on $null when the tree is clean — Check 2 could never report success. → | Out-String.
2. clients/rust/protos/mxaccess_worker.proto was stale (missing canonical max_frame_bytes). → refreshed.
3. OrphanWorkerTerminatorTests hard-coded a C:\ path that Linux Path.GetFullPath won't root, so the matcher found nothing (Expected 2/1 kills, Actual 0). → OS-neutral normalized path (5/5 pass).
4. clients/java generated MxaccessWorker.java was stale (missing max_frame_bytes). → gradle generateProto (protobuf 4.33.1, no version churn); ci.yml churn-revert broadened to both aggregates.
5. A sync python test built a grpc.aio.Channel with no current event loop on py3.12 (pytest-asyncio 1.x). → autouse conftest fixture ensuring a loop.

Result

portable ✅ (NonWindows build + codegen freshness + 808/808 gateway tests + .NET/Go/Rust/Python) · java ✅. windows + live-mxaccess remain queued pending a self-hosted windev runner (separate follow-up). Tracker: TST-03 → Done.

https://claude.ai/code/session_01DMXXvNuPekkkrTEyPNxEkW

## Stand up working Gitea Actions CI (TST-03) + fix the defects it caught The CI pipeline was authored under TST-03 (P1) but had never executed on a runner. This branch makes it run and go **green** (`portable` + `java` jobs), and fixes five real latent defects the first real run surfaced. ### Why it never ran The co-located `gitea-runner` on the docker host spawned job containers on an isolated network (`container.network: ""`), so `actions/checkout` couldn't resolve Gitea's internal clone URL `http://gitea:3000`. Fixed on the host (`container.network: "traefik"`), not in this branch. ### Workflow portability (the self-hosted `catthehacker` act image lacks GitHub-hosted tooling) - Install PowerShell as a .NET global tool (for `check-codegen.ps1`'s `shell: pwsh`). - Install Gradle 9.5.1 directly (act can't resolve the `gradle/actions` monorepo action; repo has no gradle wrapper). ### Real defects the run caught and this branch fixes 1. `check-codegen.ps1` called `.Trim()` on `$null` when the tree is clean — Check 2 could never report success. → `| Out-String`. 2. `clients/rust/protos/mxaccess_worker.proto` was stale (missing canonical `max_frame_bytes`). → refreshed. 3. `OrphanWorkerTerminatorTests` hard-coded a `C:\` path that Linux `Path.GetFullPath` won't root, so the matcher found nothing (Expected 2/1 kills, Actual 0). → OS-neutral normalized path (5/5 pass). 4. `clients/java` generated `MxaccessWorker.java` was stale (missing `max_frame_bytes`). → `gradle generateProto` (protobuf 4.33.1, no version churn); ci.yml churn-revert broadened to both aggregates. 5. A sync python test built a `grpc.aio.Channel` with no current event loop on py3.12 (pytest-asyncio 1.x). → autouse conftest fixture ensuring a loop. ### Result `portable` ✅ (NonWindows build + codegen freshness + 808/808 gateway tests + .NET/Go/Rust/Python) · `java` ✅. `windows` + `live-mxaccess` remain queued pending a self-hosted windev runner (separate follow-up). Tracker: TST-03 → Done. https://claude.ai/code/session_01DMXXvNuPekkkrTEyPNxEkW

dohertj2 added 5 commits 2026-07-10 04:28:52 -04:00

ci(tst-03): provision pwsh + gradle for the self-hosted act runner ... a36844d59d

The portable and java jobs assumed tools GitHub-hosted runners preinstall but the
self-hosted catthehacker act image lacks: `pwsh` (the codegen freshness check uses
`shell: pwsh`) and `gradle` (the java job runs `gradle test`; the repo has no
wrapper). Install PowerShell as a .NET global tool (the SDK is already set up) and
provision gradle via gradle/actions/setup-gradle. No behavior change on
GitHub-hosted runners, which already have both.

Claude-Session: https://claude.ai/code/session_01DMXXvNuPekkkrTEyPNxEkW

ci(tst-03): fix codegen-check null bug, refresh rust proto, gradle direct install ... 866d053ec2

Surfaced by the first real CI run on the self-hosted runner:

- scripts/check-codegen.ps1: `git status --porcelain` returns $null on a clean
  tree, so `.Trim()` threw "cannot call a method on a null-valued expression" —
  i.e. Check 2 could never report success. Pipe through Out-String (null -> '').
- clients/rust/protos/mxaccess_worker.proto: genuinely stale — missing the
  canonical `max_frame_bytes` field. Refreshed from
  src/ZB.MOM.WW.MxGateway.Contracts/Protos (Check 3 drift guard, added in P1).
- ci.yml java job: act cannot resolve the gradle/actions monorepo action
  (`unsupported object type` on the v4 tag); install Gradle 9.5.1 directly instead
  (matches the local build) since the repo has no gradle wrapper.

Claude-Session: https://claude.ai/code/session_01DMXXvNuPekkkrTEyPNxEkW

ci(tst-03): fix orphan-terminator test on Linux + refresh stale java worker codegen ... 576179bd3f

Two real defects the CI run surfaced (driving to fully green):

- OrphanWorkerTerminatorTests hard-coded a Windows path (C:\...); on the Linux
  runner Path.GetFullPath does not root it, so the terminator's exact-path match
  found nothing (Expected 2/1 kills, Actual 0). Use an OS-appropriate normalized
  path so the matching logic is validated on every platform (5/5 pass locally).
- clients/java generated MxaccessWorker.java was stale — missing the canonical
  max_frame_bytes field. Refreshed via `gradle generateProto` (protobuf 4.33.1,
  the pinned version — the diff is purely the field, no version churn). Broaden
  the ci.yml churn-revert to both generated aggregates so verify-clean still
  guards real drift elsewhere.

Claude-Session: https://claude.ai/code/session_01DMXXvNuPekkkrTEyPNxEkW

ci(tst-03): ensure a current event loop for sync python tests (py3.12) ... 70ef87aea2

The last portable failure: pytest-asyncio 1.x does not install a current event
loop for synchronous tests and clears it after each async test, so a later sync
test that builds a grpc.aio.Channel (create_channel) hit
"RuntimeError: There is no current event loop" on Python 3.12 depending on test
order (CI hit it; local order did not). Add an autouse conftest fixture that
ensures a usable current loop per test. Local: 127 passed, 1 skipped.

Claude-Session: https://claude.ai/code/session_01DMXXvNuPekkkrTEyPNxEkW

docs(archreview): TST-03 -> Done (CI live and green on the co-located runner) ... 18f6905eab

CI now runs end-to-end: portable + java jobs green on fix/ci-selfhosted-tooling.
Flip the tracker status and record the root cause (runner container.network) and
the five real defects the run surfaced and fixed. windows/live-mxaccess jobs
remain queued pending a self-hosted windev runner.

Claude-Session: https://claude.ai/code/session_01DMXXvNuPekkkrTEyPNxEkW

dohertj2 added 1 commit 2026-07-10 04:49:25 -04:00

ci: re-trigger to exercise windev windows runner ... c58a593cc6

Claude-Session: https://claude.ai/code/session_01DMXXvNuPekkkrTEyPNxEkW

dohertj2 added 1 commit 2026-07-10 04:52:12 -04:00

ci(tst-03): manual checkout for Windows host-mode jobs (act can't stage JS actions) ... 4af2a6b859

act host-mode on Windows does not materialize a JS action's dist/ (actions/checkout
fails with "Cannot find module .../dist/index.js"), so the windows + live-mxaccess
jobs check the repo out with plain git instead (public repo, anonymous https clone
of GITHUB_SHA). The self-hosted windev runner otherwise works (host mode, persistent).

Claude-Session: https://claude.ai/code/session_01DMXXvNuPekkkrTEyPNxEkW

dohertj2 added 1 commit 2026-07-10 04:54:11 -04:00

ci(tst-03): run Windows host-mode jobs via cmd (act pwsh shim is broken) ... 7b223d8f39

Claude-Session: https://claude.ai/code/session_01DMXXvNuPekkkrTEyPNxEkW

dohertj2 added 1 commit 2026-07-10 04:56:34 -04:00

ci(tst-03): disable Windows jobs (act host-mode broken); gate on WINDOWS_CI_ENABLED ... 843f5fe868

act_runner v0.6.1 host-mode on Windows cannot run steps (unresolvable step-script
path) or stage JS actions; Windows containers are impractical for net48/x86/MXAccess.
Skip the windows + live-mxaccess jobs until a working Windows runner exists, so
push/PR checks are the green portable + java jobs. x86 Worker verification stays on
the manual windev worktree process. Re-enable via repo var WINDOWS_CI_ENABLED=true.

Claude-Session: https://claude.ai/code/session_01DMXXvNuPekkkrTEyPNxEkW

dohertj2 merged commit d6c2ca33f1 into main 2026-07-10 05:11:03 -04:00

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

area:auth

Authentication and authorization

area:client-dotnet

.NET client library

area:client-go

Go client library

area:client-java

Java client library

area:client-python

Python client library

area:client-rust

Rust client library

area:contracts

Protocol and generated contract work

area:dashboard

Blazor dashboard work

area:docs

Documentation

area:gateway

Gateway server work

area:tests

Automated and live tests

area:worker

MXAccess worker process work

blocked

Blocked by dependency

priority:p0

Must-have / blocking

priority:p1

High priority

priority:p2

Normal priority

type:docs

Documentation work

type:feature

Feature implementation

type:infra

Infrastructure and scaffolding

type:test

Test work

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

dohertj2 (dohertj2)

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: dohertj2/mxaccessgw#123