Skip to content

Add moqx client role#66

Open
gmarzot wants to merge 1 commit into
englishm:mainfrom
gmarzot:devops/moqx-client-role
Open

Add moqx client role#66
gmarzot wants to merge 1 commit into
englishm:mainfrom
gmarzot:devops/moqx-client-role

Conversation

@gmarzot
Copy link
Copy Markdown
Contributor

@gmarzot gmarzot commented Apr 21, 2026

Summary

Adds a `client` role block to moqx's existing entry in `implementations.json`. Mirrors the pattern moxygen already uses.

moqx is currently registered relay-only, so it only appears in the relay column of the interop matrix. The moqx project already publishes an interop-client image (built alongside the relay image from the same tarball via multi-stage `FROM debian:bookworm AS builder`), so wiring it in here closes the loop.

Diff

```json
"moqx": {
...
"roles": {
"relay": { ... },
"client": {
"docker": {
"image": "ghcr.io/openmoq/moqx-interop-client:latest"
}
}
}
}
```

Image details

  • Image: `ghcr.io/openmoq/moqx-interop-client:latest` (multi-arch: amd64 + arm64 manifest list)
  • Also tagged: `:` per-build; `:main-latest` rolling; `:latest` back-compat alias on main pushes
  • Base: `debian:bookworm-slim`
  • Source: openmoq/moqx docker/Dockerfile.interop-client
  • CI: auto-built + pushed on every push to `main` / `release/**` by ci-main.yml publish job

Runner conventions

The binary honors the runner's env-var conventions natively:

  • `RELAY_URL` — target URL
  • `TESTCASE` — per-test dispatch
  • `TLS_DISABLE_VERIFY` — insecure mode for dev certs
  • `VERBOSE` — GLOG verbosity

Output is TAP v14 to stdout. Same interface as moxygen's moq-interop-client since both link the same moxygen client library.

Test plan

  • `generate-report.sh` renders moqx in both relay and client columns
  • moqx-as-client × moxygen-as-relay passes setup + announce suites
  • moqx-as-client × moq-dev-js-as-relay passes setup suite

moqx's interop-client image is published by moqx ci-main alongside the
relay image (multi-arch, FROM debian:bookworm). Adding the client role
makes moqx appear in both relay and client columns of the interop
matrix — full NxN coverage rather than relay-only.

Mirrors moxygen's existing client entry. Image accepts the runner's
standard RELAY_URL / TESTCASE / TLS_DISABLE_VERIFY env convention and
emits TAP v14 to stdout.
@gmarzot gmarzot force-pushed the devops/moqx-client-role branch from 91267ad to 09c5c32 Compare April 21, 2026 19:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant