Skip to content

chore(azure-ai-ml): remove stale live_test_only markers, re-enable e2e tests in playback CI#48230

Draft
Chakradhar886 with Copilot wants to merge 1 commit into
mainfrom
copilot/re-record-enable-azure-ai-ml-e2e-tests
Draft

chore(azure-ai-ml): remove stale live_test_only markers, re-enable e2e tests in playback CI#48230
Chakradhar886 with Copilot wants to merge 1 commit into
mainfrom
copilot/re-record-enable-azure-ai-ml-e2e-tests

Conversation

Copilot AI commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

CLI v2.44.0 introduced a serialization regression (SdkJSONEncoder couldn't serialize the legacy msrest Datastore model) that went undetected in CI because the affected datastore e2e tests were already gated behind @pytest.mark.live_test_only. The hotfix shipped in v2.44.1; the markers are now stale.

Changes

  • Removed 11 stale @pytest.mark.live_test_only("Needs re-recording…") markers across 5 test files so these tests participate in playback CI again:
    • tests/datastore/e2etests/test_datastore.py — 4 tests (test_blob_store, test_blob_store_credential_less, test_file_store, test_credential_less_adls_gen2_store)
    • tests/connection/e2etests/test_connections.py — 3 tests
    • tests/environment/e2etests/test_environment.py — 2 tests
    • tests/component/e2etests/test_component.py — 1 test
    • tests/internal/e2etests/test_pipeline_job.py — 1 test

Required follow-up

New cassette recordings must be pushed to azure-sdk-assets (run with AZURE_TEST_RUN_LIVE=true and update assets.json tag) before playback CI passes for these tests.

Co-authored-by: Chakradhar886 <259224138+Chakradhar886@users.noreply.github.com>
Copilot AI changed the title chore(azure-ai-ml): remove stale live_test_only markers from e2e tests chore(azure-ai-ml): remove stale live_test_only markers, re-enable e2e tests in playback CI Jul 23, 2026
Copilot AI requested a review from Chakradhar886 July 23, 2026 12:33
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
9 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants