Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sagemaker update space AppType parameter makes request fail #4375

Closed
1 task
akoshel opened this issue Dec 11, 2024 · 2 comments
Closed
1 task

Sagemaker update space AppType parameter makes request fail #4375

akoshel opened this issue Dec 11, 2024 · 2 comments
Assignees
Labels
bug This issue is a confirmed bug. documentation This is a problem with documentation. p2 This is a standard priority issue sagemaker service-api This issue is caused by the service API, not the SDK implementation.

Comments

@akoshel
Copy link

akoshel commented Dec 11, 2024

Describe the bug

The API includes an AppType parameter, but providing any value for AppType causes the request to fail.
https://boto3.amazonaws.com/v1/documentation/api/latest/reference/services/sagemaker/client/update_space.html

Regression Issue

  • Select this option if this issue appears to be a regression.

Expected Behavior

The space should be updated if the same AppType is provided

Current Behavior

botocore.exceptions.ClientError: An error occurred (ValidationException) when calling the UpdateSpace operation: Updating AppType for Space is not supported.

Reproduction Steps

import boto3

sm = boto3.client("sagemaker")

domain_id = ""
space_name= ""

fails

response = sm.update_space(
DomainId=domain_id,
SpaceName=space_name,
SpaceSettings={"AppType": "JupyterLab",}
)

ok

response = sm.update_space(
DomainId=domain_id,
SpaceName=space_id,
)

Possible Solution

No response

Additional Information/Context

No response

SDK version used

1.35.78

Environment details (OS name and version, etc.)

Any

@akoshel akoshel added bug This issue is a confirmed bug. needs-triage This issue or PR still needs to be triaged. labels Dec 11, 2024
@tim-finnigan tim-finnigan self-assigned this Dec 11, 2024
@tim-finnigan tim-finnigan added investigating This issue is being investigated and/or work is in progress to resolve the issue. service-api This issue is caused by the service API, not the SDK implementation. sagemaker p2 This is a standard priority issue labels Dec 11, 2024
@tim-finnigan
Copy link
Contributor

Thanks for reaching out. The ValidationException is being returned from the underlying SageMaker UpdateSpace API. I brought this up with a member of the SageMaker team who confirmed that this is the expected behavior. You cannot update AppType once the space is created. They acknowledged that this could be better clarified in the documentation, and they took an action item to improve the docs. Once the API documentation is updated then the changes will be reflected in the Boto3 command docs as well.

@tim-finnigan tim-finnigan added documentation This is a problem with documentation. and removed investigating This issue is being investigated and/or work is in progress to resolve the issue. needs-triage This issue or PR still needs to be triaged. labels Dec 12, 2024
@akoshel
Copy link
Author

akoshel commented Dec 12, 2024

Hi @tim-finnigan
Thank you for your help.
However, such behavior causes both CloudFormation and Terraform to fail on space update with the error mentioned above.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue is a confirmed bug. documentation This is a problem with documentation. p2 This is a standard priority issue sagemaker service-api This issue is caused by the service API, not the SDK implementation.
Projects
None yet
Development

No branches or pull requests

2 participants