Skip to content

ODSC-88491: Fix conda environment resolution for slug versus full OCI path inputs#1393

Open
cheryloracle wants to merge 2 commits into
oracle:mainfrom
cheryloracle:csa/odsc-88491
Open

ODSC-88491: Fix conda environment resolution for slug versus full OCI path inputs#1393
cheryloracle wants to merge 2 commits into
oracle:mainfrom
cheryloracle:csa/odsc-88491

Conversation

@cheryloracle

Copy link
Copy Markdown
Member

Summary

Fixed ADS conda environment resolution so slug inputs and full OCI conda paths follow separate resolution paths. Short service conda slugs resolve through the service conda index and now fail with explicit slug-specific guidance to use the full Environment Explorer path when unresolved. Full oci:// conda paths are treated as authoritative: they validate object accessibility directly, avoid service_pack/index.json lookup, preserve the original runtime path metadata, and classify custom published paths as published unless the path itself is an explicit service conda bucket/path. Regression tests now assert slug guidance, full-path published metadata, no service-index lookup for full paths, and path-specific validation failures for both training and inference env info.

Tests Run

/tmp/ads-py314-core-venv/bin/python -m pytest tests/unitary/with_extras/model/test_env_info.py::TestTrainingEnvInfo::test_from_slug_not_exist tests/unitary/with_extras/model/test_env_info.py::TestTrainingEnvInfo::test_from_slug_service_pack_list_not_extracted tests/unitary/with_extras/model/test_env_info.py::TestInferenceEnvInfo::test_from_slug_not_exist tests/unitary/with_extras/model/test_env_info.py::TestInferenceEnvInfo::test_from_slug_service_pack_list_not_extracted -q (passed: 4 tests)
/tmp/ads-py314-core-venv/bin/python -m pytest tests/unitary/with_extras/model/test_env_info.py::TestTrainingEnvInfo::test_from_path_cp tests/unitary/with_extras/model/test_env_info.py::TestTrainingEnvInfo::test_from_path_not_accessible tests/unitary/with_extras/model/test_env_info.py::TestInferenceEnvInfo::test_from_path_custom_pack tests/unitary/with_extras/model/test_env_info.py::TestInferenceEnvInfo::test_from_path_not_accessible -q (passed: 4 tests)
/tmp/ads-py314-core-venv/bin/python -m pytest tests/unitary/with_extras/model/test_env_info.py::TestTrainingEnvInfo::test_from_slug_dev_sp tests/unitary/with_extras/model/test_env_info.py::TestTrainingEnvInfo::test_from_slug_prod_sp tests/unitary/with_extras/model/test_env_info.py::TestTrainingEnvInfo::test_from_slug_not_exist tests/unitary/with_extras/model/test_env_info.py::TestTrainingEnvInfo::test_from_slug_service_pack_list_not_extracted tests/unitary/with_extras/model/test_env_info.py::TestTrainingEnvInfo::test_from_path_cp tests/unitary/with_extras/model/test_env_info.py::TestTrainingEnvInfo::test_from_path_not_accessible tests/unitary/with_extras/model/test_env_info.py::TestInferenceEnvInfo::test_from_slug_dev_sp tests/unitary/with_extras/model/test_env_info.py::TestInferenceEnvInfo::test_from_slug_prod_sp tests/unitary/with_extras/model/test_env_info.py::TestInferenceEnvInfo::test_from_slug_not_exist tests/unitary/with_extras/model/test_env_info.py::TestInferenceEnvInfo::test_from_slug_service_pack_list_not_extracted tests/unitary/with_extras/model/test_env_info.py::TestInferenceEnvInfo::test_from_path_custom_pack tests/unitary/with_extras/model/test_env_info.py::TestInferenceEnvInfo::test_from_path_not_accessible -q (passed: 12 tests)
/tmp/ads-py314-core-venv/bin/python -m pytest tests/unitary/with_extras/model/test_env_info.py -q (failed: test_get_service_packs_bad_response due local OCI auth config/default signer; 24 passed)
/tmp/ads-py314-core-venv/bin/python -m pytest tests/unitary/with_extras/model/test_artifact.py -q (blocked: ModuleNotFoundError: No module named 'skl2onnx')

@oracle-contributor-agreement

Copy link
Copy Markdown

Thank you for your pull request and welcome to our community! To contribute, please sign the Oracle Contributor Agreement (OCA).
The following contributors of this PR have not signed the 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.

@oracle-contributor-agreement oracle-contributor-agreement Bot added the OCA Required At least one contributor does not have an approved Oracle Contributor Agreement. label Jul 7, 2026
@cheryloracle cheryloracle changed the title ODSC-88491: CSA implementation ODSC-88491: Fix conda environment resolution for slug versus full OCI path inputs Jul 7, 2026
@oracle-contributor-agreement oracle-contributor-agreement Bot added OCA Verified All contributors have signed the Oracle Contributor Agreement. and removed OCA Required At least one contributor does not have an approved Oracle Contributor Agreement. labels Jul 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

OCA Verified All contributors have signed the Oracle Contributor Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants