Skip to content

enable_network_isolation documented but not implemented in remote jobs. #4425

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

Closed
ArtemioPadilla opened this issue Feb 13, 2024 · 1 comment · Fixed by #4465
Closed

enable_network_isolation documented but not implemented in remote jobs. #4425

ArtemioPadilla opened this issue Feb 13, 2024 · 1 comment · Fixed by #4465
Labels
remote-function Relates to Pathways feature type: bug

Comments

@ArtemioPadilla
Copy link

Describe the bug
A clear and concise description of what the bug is.
The parameter enable_network_isolation is documented in the RemoteExector, yet it is not a usable parameter for either the decorator remote or the class.

To reproduce

Just try to use enable_network_isolation set as True and run anything.

Expected behavior
SageMaker job ran with enable_network_isolation configured as True.

Screenshots or logs
---> 10 @Remote(instance_type="ml.c5.xlarge",
11 sagemaker_session=session,
12 job_name_prefix="test-job",
23
24 enable_network_isolation=True
25 )
26 def matrix_multiply(a, b):
27 return np.matmul(a, b)
29 a = np.array([[1, 0],
30 [0, 1]])

TypeError: remote() got an unexpected keyword argument 'enable_network_isolation'

System information
SageMaker Jupyter Lab from New SageMaker Studio- SageMaker Python SDK version: 2.198.0

  • Custom Docker image (Y/N): N
@rohangujarathi
Copy link
Member

Thanks for reaching out @ArtemioPadilla. We do not support enable_network_isolation with remote_decorator as of now. I will correct the doc.

@knikure knikure linked a pull request Mar 1, 2024 that will close this issue
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
remote-function Relates to Pathways feature type: bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants