Skip to content

Conversation

@pagezyhf pagezyhf requested a review from a team as a code owner October 9, 2025 13:57
@pagezyhf pagezyhf requested a review from nargokul October 9, 2025 13:57
@pagezyhf pagezyhf temporarily deployed to manual-approval October 9, 2025 13:58 — with GitHub Actions Inactive
@mollyheamazon
Copy link
Contributor

Make sure to fix unit test error

@arjkesh
Copy link
Contributor

arjkesh commented Oct 16, 2025

@mollyheamazon could you help us retrigger the PR workflow here? We'd like to get these images available in the pysdk as soon as possible

@arjkesh
Copy link
Contributor

arjkesh commented Oct 17, 2025

It looks like the test below failed

Is this a red herring since this PR is unrelated to XGBoost? If so, can this be merged?

FAILED tests/integ/sagemaker/serve/test_serve_mlflow_xgboost_flavor_happy.py::test_happy_xgboost_sagemaker_endpoint_with_torch_serve

cc @mollyheamazon

@mollyheamazon
Copy link
Contributor

I have retriggered the integ test. We usually make sure the integ tests pass before merging, but there can be exception for merging if there is urgency.

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.

4 participants