Skip to content

Conversation

SiddharthaBose30
Copy link

  1. In order to delete Domains and UserProfiles, the associated Spaces needs to be deleted. (sagemaker-spaces.go takes care of it).
  2. The Spaces can have associated Apps which needs to be deleted first. (sagemaker-spaces.go takes care of it).
  3. The Algorithms needs to be deleted first. (sagemaker-algorithms.go takes care of it).
  4. After deleting the Algorithms, we can delete the Training Jobs. (sagemaker-trainingjobs.go takes care of it).

Copy link

@bjfish25 bjfish25 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, great work.

Could you use the new aws-sdk-go-v2 for the new resources as all aws-sdk-go will have to be replaced eventually

@SiddharthaBose30
Copy link
Author

Looks good, great work.

Could you use the new aws-sdk-go-v2 for the new resources as all aws-sdk-go will have to be replaced eventually

Updated. 👍

Copy link

@dilip0515 dilip0515 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@bjfish25 bjfish25 merged commit 5af4289 into Optum:main Oct 29, 2024
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