Add sagemaker-mlops, sagemaker-serve and sagemaker-train, sagemaker-schema-inference-artifacts by mollyheamazon · Pull Request #32176 · conda-forge/staged-recipes

mollyheamazon · 2026-02-11T19:30:10Z

Checklist

conda-forge-admin · 2026-02-11T19:31:54Z

Hi! This is the friendly automated conda-forge-linting service.

I wanted to let you know that I linted all conda-recipes in your PR (recipes/sagemaker-train/recipe.yaml, recipes/sagemaker-serve/recipe.yaml, recipes/sagemaker-mlops/recipe.yaml) and found some lint.

Here's what I've got...

For recipes/sagemaker-train/recipe.yaml:

❌ The homepage item is expected in the about section.
❌ There are too few lines. There should be one empty line at the end of the file.

For recipes/sagemaker-train/recipe.yaml:

ℹ️ noarch: python recipes should usually follow the syntax in our documentation for specifying the Python version.
- For the host section of the recipe, you should usually use the pin python ${{ python_min }}.* for the python entry.
- For the run section of the recipe, you should usually use the pin python >=${{ python_min }} for the python entry.
- For the tests[].python.python_version or tests[].requirements.run section of the recipe, you should usually use the pin python_version: ${{ python_min }}.* or python ${{ python_min }}.* for the python_version or python entry.
- If the package requires a newer Python version than the currently supported minimum version on conda-forge, you can override the python_min variable by adding a Jinja2 set statement at the top of your recipe (or using an equivalent context variable for v1 recipes).

For recipes/sagemaker-serve/recipe.yaml:

❌ The homepage item is expected in the about section.
❌ There are too few lines. There should be one empty line at the end of the file.

For recipes/sagemaker-serve/recipe.yaml:

ℹ️ noarch: python recipes should usually follow the syntax in our documentation for specifying the Python version.
- For the host section of the recipe, you should usually use the pin python ${{ python_min }}.* for the python entry.
- For the run section of the recipe, you should usually use the pin python >=${{ python_min }} for the python entry.
- For the tests[].python.python_version or tests[].requirements.run section of the recipe, you should usually use the pin python_version: ${{ python_min }}.* or python ${{ python_min }}.* for the python_version or python entry.
- If the package requires a newer Python version than the currently supported minimum version on conda-forge, you can override the python_min variable by adding a Jinja2 set statement at the top of your recipe (or using an equivalent context variable for v1 recipes).

For recipes/sagemaker-mlops/recipe.yaml:

❌ The homepage item is expected in the about section.
❌ There are too few lines. There should be one empty line at the end of the file.

For recipes/sagemaker-mlops/recipe.yaml:

ℹ️ noarch: python recipes should usually follow the syntax in our documentation for specifying the Python version.
- For the host section of the recipe, you should usually use the pin python ${{ python_min }}.* for the python entry.
- For the run section of the recipe, you should usually use the pin python >=${{ python_min }} for the python entry.
- For the tests[].python.python_version or tests[].requirements.run section of the recipe, you should usually use the pin python_version: ${{ python_min }}.* or python ${{ python_min }}.* for the python_version or python entry.
- If the package requires a newer Python version than the currently supported minimum version on conda-forge, you can override the python_min variable by adding a Jinja2 set statement at the top of your recipe (or using an equivalent context variable for v1 recipes).

_{This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/21919934418. Examine the logs at this URL for more detail.}

github-actions · 2026-02-11T19:35:13Z

Hi! This is the staged-recipes linter and your PR looks excellent! 🚀

conda-forge-admin · 2026-02-11T20:02:47Z

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipes/sagemaker-serve/recipe.yaml, recipes/sagemaker-train/recipe.yaml, recipes/sagemaker-mlops/recipe.yaml) and found it was in an excellent condition.

github-actions · 2026-02-11T20:05:52Z

Hi! This is the staged-recipes linter and I found some lint.

File-specific lints and/or hints:

recipes/sagemaker-schema-inference-artifacts/recipe.yaml:
- lints:
  - The following maintainers have not yet confirmed that they are willing to be listed here: mufaddal-rohawala. Please ask them to comment on this PR if they are.
recipes/sagemaker-mlops/recipe.yaml:
- lints:
  - The following maintainers have not yet confirmed that they are willing to be listed here: mufaddal-rohawala. Please ask them to comment on this PR if they are.
recipes/sagemaker-train/recipe.yaml:
- lints:
  - The following maintainers have not yet confirmed that they are willing to be listed here: mufaddal-rohawala. Please ask them to comment on this PR if they are.
recipes/sagemaker-serve/recipe.yaml:
- lints:
  - The following maintainers have not yet confirmed that they are willing to be listed here: mufaddal-rohawala. Please ask them to comment on this PR if they are.

conda-forge-admin · 2026-02-11T20:27:23Z

Hi! This is the friendly automated conda-forge-linting service.

I wanted to let you know that I linted all conda-recipes in your PR (recipes/sagemaker-serve/recipe.yaml, recipes/sagemaker-train/recipe.yaml, recipes/sagemaker-mlops/recipe.yaml) and found some lint.

Here's what I've got...

For recipes/sagemaker-serve/recipe.yaml:

❌ license_file entry is missing, but is required.

For recipes/sagemaker-mlops/recipe.yaml:

❌ license_file entry is missing, but is required.

_{This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/21921779267. Examine the logs at this URL for more detail.}

…to serve and mlops packages

conda-forge-admin · 2026-02-11T22:51:07Z

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipes/sagemaker-mlops/recipe.yaml, recipes/sagemaker-serve/recipe.yaml, recipes/sagemaker-schema-inference-artifacts/recipe.yaml, recipes/sagemaker-train/recipe.yaml) and found it was in an excellent condition.

conda-forge-admin · 2026-02-11T23:39:27Z

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipes/sagemaker-serve/recipe.yaml, recipes/sagemaker-mlops/recipe.yaml, recipes/sagemaker-schema-inference-artifacts/recipe.yaml, recipes/sagemaker-train/recipe.yaml) and found it was in an excellent condition.

I do have some suggestions for making it better though...

For recipes/sagemaker-schema-inference-artifacts/recipe.yaml:

ℹ️ Detected pure Python wheel(s) in source: https://pypi.org/packages/py3/s/sagemaker-schema-inference-artifacts/sagemaker_schema_inference_artifacts-${{ version }}-py3-none-any.whl. This is generally ok for pure Python wheels and noarch=python packages but it's preferred to use a source distribution (sdist) if possible.

_{This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/21968620937. Examine the logs at this URL for more detail.}

nargokul · 2026-02-12T18:56:50Z

Changes look good to me

MridulS · 2026-02-12T21:14:59Z

recipes/sagemaker-mlops/LICENSE

@@ -0,0 +1,201 @@
+                                 Apache License


There is no need to add LICENSE text files explicitly if they are already linked from the recipe, and it's already there https://github.com/aws/sagemaker-python-sdk/blob/master/LICENSE.

Same comment for other LICENSE files in this PR.

The LICENSE file is needed in the recipe directory because:

The PyPI source packages for sagemaker-mlops and sagemaker-serve don't include a LICENSE file

Conda-forge's linter requires license_file to be present and will fail without it

The GitHub link points to the parent repo, but these are separate PyPI distributions

Do you have suggestion for a better approach to this?

Ah okie, didn't look inside the sdist!

I would argue to make the packages PEP 639 complaint anyway and add the license files to them.

But I would wait for someone from conda-forge-core to comment here (I can't merge this PR). Once the CI is green they can be tagged via @conda-forge/help-python ready for review! as a comment.

I would argue to make the packages PEP 639 complaint anyway and add the license files to them.

I agree on this too. Will include them in our next pypi release.
Thanks Mridul!

MridulS · 2026-02-12T21:31:04Z

recipes/sagemaker-schema-inference-artifacts/recipe.yaml

+
+about:
+  summary: Open source library for Hugging Face Task Sample Inputs and Outputs
+  homepage: https://pypi.org/project/sagemaker-schema-inference-artifacts/


just FYI the homepage link from pypi leads to https://github.com/aws/sagemaker-schema-inference-artifacts which isn't public.

Our team doesn't own sagemaker-schema-inference-artifacts, it's maintained by the SageMaker Inference team. However, we need to create this conda-forge package because our package (sagemaker-serve) depends on it. The manager is aware of this PR.

MridulS · 2026-02-12T21:33:07Z

recipes/sagemaker-schema-inference-artifacts/recipe.yaml

+build:
+  number: 0
+  noarch: python
+  script: ${{ PYTHON }} -m pip install "sagemaker_schema_inference_artifacts-${{ version }}-py3-none-any.whl" -vv --no-deps


It's recommended to build from a sdist instead of a wheel as pointed out in #32176 (comment)

sagemaker_schema_inference_artifacts pypi only seems to have wheel in file downloads.

Can you use the GitHub source instead?

The github repo is private, so I think this is not possible? We don't own this package but need it as a dependency for sagemaker-serve.

Isn't it this repo: https://github.com/aws/sagemaker-python-sdk/tree/v3.4.1 ? I can access it just fine. It sure makes it hard to figure out what version what is b/c every is bundled under the same version there!

sagemaker-schema-inference-artifacts is from https://github.com/aws/sagemaker-schema-inference-artifacts, which is a private repo from a partner team

@ocefpaf would be great if you could help merge this PR, or provide any other feedback for improvement. Thanks!

mufaddal-rohawala · 2026-02-12T23:39:40Z

I confirm to be maintainer of these pacakges

github-actions · 2026-02-12T23:45:00Z

Hi! This is the staged-recipes linter and your PR looks excellent! 🚀

mollyheamazon · 2026-02-12T23:46:53Z

@conda-forge-admin, please rerender

mollyheamazon · 2026-02-12T23:52:02Z

@conda-forge/help-python ready for review!

Add sagemaker-mlops, sagemaker-serve and sagemaker-train

d3e4b5d

lint update

29f0900

mollyheamazon added 2 commits February 11, 2026 12:25

Remove maintainer, update license

b9e0495

Merge branch 'main' into add-sagemaker-v3-recipe

2c7e1eb

mollyheamazon added 2 commits February 11, 2026 14:49

Merge branch 'conda-forge:main' into add-sagemaker-v3-recipe

fc33677

Update python_min to 3.10 for sagemaker-core dependency, add license …

fb2b1a1

…to serve and mlops packages

mollyheamazon added 3 commits February 11, 2026 14:57

add sagemaker-mlflow dependency

f817cb9

Remove python_min, add sagemaker-schema-inference-artifacts

5f239d6

Update source for sagemaker-schema-inference-artifacts

e1c4bf5

mollyheamazon added 4 commits February 11, 2026 15:46

Update sagemaker-schema-inference-artifacts install path

ae6b649

Add json to run dependency of sagemaker-schema-inference-artifacts

ce9446c

Update sagemaker-serve run dependency to add protobuf and pytest

59c0a55

Update pip install path for sagemaker_schema_inference_artifacts

657a4cc

mollyheamazon changed the title ~~Add sagemaker-mlops, sagemaker-serve and sagemaker-train~~ Add sagemaker-mlops, sagemaker-serve and sagemaker-train, sagemaker-schema-inference-artifacts Feb 12, 2026

MridulS reviewed Feb 12, 2026

View reviewed changes

Merge branch 'main' into add-sagemaker-v3-recipe

c58513a

github-actions bot added python review-requested labels Feb 12, 2026

Uh oh!

Conversation

mollyheamazon commented Feb 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

conda-forge-admin commented Feb 11, 2026

Uh oh!

github-actions bot commented Feb 11, 2026

Uh oh!

conda-forge-admin commented Feb 11, 2026

Uh oh!

github-actions bot commented Feb 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

conda-forge-admin commented Feb 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

conda-forge-admin commented Feb 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

conda-forge-admin commented Feb 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

nargokul commented Feb 12, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

mufaddal-rohawala commented Feb 12, 2026

Uh oh!

github-actions bot commented Feb 12, 2026

Uh oh!

mollyheamazon commented Feb 12, 2026

Uh oh!

mollyheamazon commented Feb 12, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

mollyheamazon commented Feb 11, 2026 •

edited

Loading

github-actions bot commented Feb 11, 2026 •

edited

Loading

conda-forge-admin commented Feb 11, 2026 •

edited

Loading

conda-forge-admin commented Feb 11, 2026 •

edited

Loading

conda-forge-admin commented Feb 11, 2026 •

edited

Loading