Skip to content

Commit 0f5e0cc

Browse files
fix(ci): include ogx-open-client in unified release pipeline (#6205)
# What does this PR do? ogx-open-client was built and published ia a separate workflow, publish-openapi-sdk.yml, with no version coupling to the main ogx release. ogx-open-client version was taken from the fallback version of ogx, so it was always ended with `devX`. This change adds ogx-open-client to the pypi.yml build/test/publish matrix so every ogx release automatically publishes a same-version client SDK. Fixes the standalone publish-openapi-sdk.yml to also derive its version from the trigger tag or an explicit input. To do the above it changes the version assignment in Makefile. --------- Signed-off-by: E Geiger <egeiger@redhat.com> Signed-off-by: Francisco Javier Arceo <farceo@redhat.com> Co-authored-by: Francisco Javier Arceo <farceo@redhat.com>
1 parent 9ebdb57 commit 0f5e0cc

10 files changed

Lines changed: 143 additions & 533 deletions

File tree

.github/CODEOWNERS

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,4 +7,3 @@
77
# OpenAPI SDK generation and publishing
88
/client-sdks/openapi/ @ashwinb @leseb @bbrowning
99
/.github/workflows/openapi-generator-validation.yml @ashwinb @leseb @bbrowning
10-
/.github/workflows/publish-openapi-sdk.yml @ashwinb @leseb @bbrowning

.github/workflows/README.md

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,6 @@ OGX uses GitHub Actions for Continuous Integration (CI). Below is a table detail
2828
| Pre-commit | [pre-commit.yml](pre-commit.yml) | Run pre-commit checks |
2929
| Prepare release | [prepare-release.yml](prepare-release.yml) | Prepare release |
3030
| Test OGX Build | [providers-build.yml](providers-build.yml) | Test ogx build and list-deps |
31-
| Publish OpenAPI SDK to PyPI | [publish-openapi-sdk.yml](publish-openapi-sdk.yml) | Publish ogx-open-client to PyPI |
3231
| Build, test, and publish packages | [pypi.yml](pypi.yml) | Build, test, and publish packages |
3332
| Integration Tests (Record) | [record-integration-tests.yml](record-integration-tests.yml) | Auto-record missing test recordings for PR |
3433
| vLLM GPU Recording | [record-vllm-gpu-tests.yml](record-vllm-gpu-tests.yml) | GPU recording for gpt-oss:20b (${{ inputs.suite }} suite) |

.github/workflows/openapi-generator-validation.yml

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -187,7 +187,7 @@ jobs:
187187
188188
- name: Generate Python SDK
189189
working-directory: client-sdks/openapi
190-
run: make sdk OPEN=1
190+
run: make sdk OPEN=0
191191

192192
- name: Validate generated SDK
193193
working-directory: client-sdks/openapi
@@ -217,14 +217,14 @@ jobs:
217217
218218
- name: Install generated SDK
219219
run: |
220-
echo "Reinstalling OpenAPI-generated SDK (ogx_open_client)..."
221-
uv pip uninstall ogx-open-client || true
220+
echo "Reinstalling OpenAPI-generated SDK (ogx_client)..."
221+
uv pip uninstall ogx-client || true
222222
223223
# Install SDK using uv pip
224224
uv pip install -e client-sdks/openapi/sdks/python
225225
226226
echo "Verifying installation..."
227-
uv run python -c "import ogx_open_client; print(f'Installed: {ogx_open_client.__name__}')"
227+
uv run python -c "import ogx_client; print(f'Installed: {ogx_client.__name__}')"
228228
229229
- name: Setup Ollama (for integration tests)
230230
if: runner.os == 'Linux'
@@ -242,7 +242,7 @@ jobs:
242242
|| echo "::warning::Some integration tests failed - this may indicate SDK compatibility issues"
243243
244244
# Show which SDK is actually being used
245-
uv run python -c "import ogx_open_client; import inspect; print(f'SDK location: {inspect.getfile(ogx_open_client)}')"
245+
uv run python -c "import ogx_client; import inspect; print(f'SDK location: {inspect.getfile(ogx_client)}')"
246246
247247
- name: Summary
248248
if: runner.os == 'Linux'
@@ -255,5 +255,5 @@ jobs:
255255
echo "✅ Integration tests executed (check logs for results)"
256256
echo ""
257257
echo "**Platform**: ${{ matrix.os }}"
258-
echo "**Package**: ogx_open_client (OpenAPI-generated)"
258+
echo "**Package**: ogx_client (OpenAPI-generated)"
259259
} >> "$GITHUB_STEP_SUMMARY"

.github/workflows/publish-openapi-sdk.yml

Lines changed: 0 additions & 176 deletions
This file was deleted.

0 commit comments

Comments
 (0)