Repository navigation
Conversation
playb0t
requested review from
AlaaShaker,
dustin-sale,
gebhardtr and
krisrice
as code owners
November 2, 2026 19:02
|
Thank you for your pull request and welcome to our community! To contribute, please sign the Oracle Contributor Agreement (OCA).
To sign the OCA, please create an Oracle account and sign the OCA in Oracle's Contributor Agreement Application. When signing the OCA, please provide your GitHub username. After signing the OCA and getting an OCA approval from Oracle, this PR will be automatically updated. If you are an Oracle employee, please make sure that you are a member of the main Oracle GitHub organization, and your membership in this organization is public. |
Generated-by: Codex:gpt-6.1-sol Signed-off-by: Alex Gercog <w0rldlogic.0point@proton.me>
Generated-by: Codex:gpt-6.1-sol Signed-off-by: Alex Gercog <w0rldlogic.0point@proton.me>
playb0t
force-pushed
the
540-container-installed-entrypoint
branch
2 times, most recently
from
November 5, 2026 11:42
6dcd7cc to
d32ef5e
Compare
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Each container's entry point runs
uv run, which readstool.uv.sourcesagain at startup and stops with "oracle-mcp-commonreferences a workspace intool.uv.sources... but is not a workspace member", because the workspace member../commonis outside the build context. The build already installs the application into/app/.venvwithuv sync --no-sources --all-extras --no-editable(#421), so the entry point now runs that installed console script directly.Changes:
src/oci-api-mcp-server/Containerfile:ENTRYPOINT ["/app/.venv/bin/oracle.oci-api-mcp-server"]and its comment linesrc/oci-cloud-mcp-server/Containerfile:ENTRYPOINT ["/app/.venv/bin/oracle.oci-cloud-mcp-server"]and its comment lineCHANGELOG.mdNo Python code changes. No new dependencies. These are the only two images with both a
uv runentry point and a workspace source.Fixes #540
Type of change
How Has This Been Tested?
Two stages, both on
maine75013f9.The commands as written, default log level, no settings beyond those shown:
Containerfile:podman run --rm -i --network=none oci-api-mcpexits 2 with the uv error quoted in OCI API and OCI Cloud MCP server containers exit before initialize:uv runre-reads the workspace source at startup #540; stdout stays empty.podman run --rm -i --network=none -e OCI_CLI_AUTH=api_key oci-api-mcpwaits on stdin; the MCP Python SDK 1.30.0 client getsinitializeandlist_tools(2 tools); on EOF the container exits 0 and--rmremoves it.Containerfile:podman run --rm -i --network=none oci-cloud-mcpexits 2 with the same error fororacle-oci-cloud-mcp-server==2.2.3.initializeandlist_tools(5 tools); exit 0 on EOF, removed by--rm.One image, both entry points, so the result does not depend on a rebuild picking up different packages. Every run used
--network=none, no host mounts and no OCI profile, with these settings:-e FASTMCP_CHECK_FOR_UPDATES=off -e FASTMCP_LOG_LEVEL=ERROR -e OCI_CLI_AUTH=api_key -e OCI_MCP_AUTH_TYPE=api_key -e OCI_CONFIG_FILE=<a path that does not exist>. WithOCI_CLI_AUTHset, the server passes no auth arguments to the CLI, so--versionruns without a profile.uv runentry point (--entrypointset back to it on this image): the uv workspace error quoted in OCI API and OCI Cloud MCP server containers exit before initialize:uv runre-reads the workspace source at startup #540, beforeinitialize.--entrypointoverride, noUV_*variables): MCPinitializeandlist_toolssucceed with the MCP Python SDK 1.30.0 client over stdio; theoci-api-commandsresource returns the CLI help;run_oci_commandwith--versionreturns OCI CLI 3.92.0;get_oci_command_helpfor a service that does not exist returns a handled error, and the next call succeeds.oracle(UID 1000) and is removed by--rmwhen the client disconnects.Containerfilewith this change,--network=none, no host mounts,-e FASTMCP_CHECK_FOR_UPDATES=off -e FASTMCP_LOG_LEVEL=ERROR -e OCI_CONFIG_FILE=<a path that does not exist>: the original entry point stops with the same uv error beforeinitialize; the new default entry point passesinitialize, lists all 5 tools, answerslist_client_operations(a metadata call, no OCI client is created), returns a handled error forlimit=0, and repeats the first call with the same result. Both containers are removed by--rm.To verify (for Cloud:
src/oci-cloud-mcp-server, tagoci-cloud-mcp, no-eneeded):On startup the server prints an
AuthlibDeprecationWarningfrom authlib's httpx client. It comes from the dependency, not from this change.Test Configuration:
ghcr.io/oracle/oraclelinux:9-slim@sha256:1950389c...(unchanged)oracle-mcp-common0.1.4; the Cloud image contains Python 3.13.16, Cloud 2.2.3, FastMCP 3.4.10, MCP 1.30.0, OCI SDK 2.185.1 andoracle-mcp-common0.1.4Not run: the Python unit suites and the Moon wrapper (no Python code changed); rootless Podman; Docker.
Checklist:
Containerfilecomment states why)