Skip to content

fix(deps): make OCI dependencies optional on release 0.6 - #6192

Merged
skamenan7 merged 6 commits into
ogx-ai:release-0.6.xfrom
skamenan7:fix/oci-default-deps-release-0.6
Jun 26, 2026
Merged

fix(deps): make OCI dependencies optional on release 0.6#6192
skamenan7 merged 6 commits into
ogx-ai:release-0.6.xfrom
skamenan7:fix/oci-default-deps-release-0.6

Conversation

@skamenan7

@skamenan7 skamenan7 commented Jun 26, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • remove OCI-related packages from default package dependencies on release-0.6.x
  • expose them through the explicit oci extra instead
  • add tests that guard default/starter installs while preserving explicit OCI provider deps

Before / after proof

Built wheel metadata from v0.6.0 before the fix advertised OCI dependencies unconditionally:

Requires-Dist: oracledb>=3.4.1
Requires-Dist: oci>=2.165.0
Requires-Dist: numpy>=2.3.2

Built wheel metadata after this fix advertises them only behind the oci extra:

Provides-Extra: oci
Requires-Dist: numpy>=2.3.2; extra == "oci"
Requires-Dist: oci>=2.165.0; extra == "oci"
Requires-Dist: oracledb>=3.4.1; extra == "oci"

Test plan

  • uv run --locked --group unit pytest tests/unit/distribution/test_list_deps_output.py -q
  • env UV_CACHE_DIR=/tmp/uv-cache-ogx-oci-main UV_PYTHON_INSTALL_DIR=/tmp/uv-python-ogx-oci-main uv build --wheel

Signed-off-by: Sumanth Kamenani <skamenan@redhat.com>
Signed-off-by: Sumanth Kamenani <skamenan@redhat.com>
Signed-off-by: Sumanth Kamenani <skamenan@redhat.com>
Signed-off-by: Sumanth Kamenani <skamenan@redhat.com>
Signed-off-by: Sumanth Kamenani <skamenan@redhat.com>
Signed-off-by: Sumanth Kamenani <skamenan@redhat.com>
@skamenan7
skamenan7 marked this pull request as ready for review June 26, 2026 16:36
@skamenan7
skamenan7 merged commit 261ef61 into ogx-ai:release-0.6.x Jun 26, 2026
93 checks passed
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.

2 participants