You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
feature: AutoGluon image URIs update for v2 - #5891
All four exercise @step-decorator pipeline serialization and execution against a per-run dummy remote-function container, failing inside create_and_execute_pipeline. This PR changes exactly one file — src/sagemaker/image_uri_config/autogluon.json, adding the 1.4.0 and 1.5.0 version keys and their aliases — so there is no code path from it to step-decorator serialization.
The full integ suite runs here only because the CodeBuild filter matches src/sagemaker/image_uri_config/*.json, so any image-config PR picks up whatever is currently flaky in the suite. These tests run under -n 120 with --reruns 1 --reruns-delay 15, and the summary shows 14 reruns occurred.
codestyle-doc-tests — pre-existing lint drift on master-v2
13 files are flagged with F401 ... imported but unused and one E501 (121 > 120), in files such as src/sagemaker/amazon/amazon_estimator.py and src/sagemaker/apiutils/_base_types.py. The file touched by this PR does not appear in that output. The same job fails on other current master-v2 PRs (e.g. #6396) with unrelated diffs.
What does pass:unit-tests on all four interpreters (py39, py310, py311, py312), codecov/patch at 97.41%, collab-check, and wait-for-approval. The earlier py39-only unit-test failure on the previous head is gone. tests/unit/sagemaker/image_uris/test_autogluon.py is data-driven — it iterates whatever versions the JSON contains — so the two new versions are covered without a test change.
A rerun of integ-tests should be enough to confirm the four workflow failures are environmental.
Edited to correct the file list: an earlier version of this comment also referenced a KNOWN_MISSING_COMBINATIONS entry in tests/unit/sagemaker/image_uris/test_djl.py. That belongs to a different PR; this one touches only autogluon.json.
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
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.
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.