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

add sagemaker resource types #2

Merged
merged 16 commits into from
Dec 28, 2020
Merged

add sagemaker resource types #2

merged 16 commits into from
Dec 28, 2020

Conversation

jerrypeng7773
Copy link
Contributor

Issue #, if available:

Description of changes:

add aws-sagemaker-pipeline resource type.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@jerrypeng7773 jerrypeng7773 changed the title add aws-sagemaker-pipeline resource type add sagemaker resource types Dec 18, 2020
- "sagemaker:ListProjects"
- "sagemaker:ListTags"
- "servicecatalog:DescribeProduct"
- "servicecatalog:DescribeProvisioningArtifact"
Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fxied

@@ -0,0 +1,132 @@
{
Copy link
Contributor

Choose a reason for hiding this comment

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

Minor diff from the latest - Updated repositoryservice-github-backup; Please take the latest from https://code.amazon.com/packages/AWSCloudFormationResourceProvidersSageMaker/logs/repositoryservice-github-backup

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

@@ -0,0 +1,136 @@
{
"typeName": "AWS::SageMaker::Pipeline",
Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fxied

@jerrypeng7773
Copy link
Contributor Author

jerrypeng7773 commented Dec 24, 2020

project, model-pacakge-group, and pipeline needs to sync up the mainline branch and the github-backup-branch.

"permissions": [
"sagemaker:DescribeModelPackageGroup",
"sagemaker:GetModelPackageGroupPolicy",
"sagemaker:PutModelPackageGroupPolicy",
Copy link

Choose a reason for hiding this comment

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

PutModelPackageGroupPolicy may not be required here in read block.

}

/**
* This is used to ensure model package group resource has moved from Pending to Scheduled/Failed state.
Copy link

Choose a reason for hiding this comment

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

Is Scheduled a valid state here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

don't think so, this probably is copied from monitoring schedule.

Copy link

@sreedes sreedes 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, a couple of minor comments.

@jerrypeng7773
Copy link
Contributor Author

I'll squash and merge, and open another PR to address that bad comments

@jerrypeng7773 jerrypeng7773 merged commit 0ce258f into main Dec 28, 2020
@jerrypeng7773 jerrypeng7773 deleted the jerrypeng-dev branch December 28, 2020 17:35
Comment on lines +296 to +298
"type" : "string",
"minimum" : 1,
"maximum" : 2048
Copy link

Choose a reason for hiding this comment

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

aws-cloudformation/cloudformation-cli#414
https://json-schema.org/understanding-json-schema/reference/string.html#length

Suggested change
"type" : "string",
"minimum" : 1,
"maximum" : 2048
"type" : "string",
"minLength" : 1,
"maxLength" : 2048

Comment on lines +290 to +292
"type" : "string",
"minimum" : 1,
"maximum" : 2048
Copy link

Choose a reason for hiding this comment

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

aws-cloudformation/cloudformation-cli#414
https://json-schema.org/understanding-json-schema/reference/string.html#length

Suggested change
"type" : "string",
"minimum" : 1,
"maximum" : 2048
"type" : "string",
"minLength" : 1,
"maxLength" : 2048

Comment on lines +310 to +312
"type" : "string",
"minimum" : 1,
"maximum" : 2048
Copy link

Choose a reason for hiding this comment

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

aws-cloudformation/cloudformation-cli#414
https://json-schema.org/understanding-json-schema/reference/string.html#length

Suggested change
"type" : "string",
"minimum" : 1,
"maximum" : 2048
"type" : "string",
"minLength" : 1,
"maxLength" : 2048

Copy link
Contributor Author

Choose a reason for hiding this comment

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

thanks for catching this up, will fix in a new PR

Comment on lines +378 to +380
"type" : "string",
"minimum" : 1,
"maximum" : 2048
Copy link

Choose a reason for hiding this comment

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

aws-cloudformation/cloudformation-cli#414
https://json-schema.org/understanding-json-schema/reference/string.html#length

Suggested change
"type" : "string",
"minimum" : 1,
"maximum" : 2048
"type" : "string",
"minLength" : 1,
"maxLength" : 2048

Comment on lines +218 to +221
"type" : "array",
"description" : "Monitoring outputs for monitoring jobs. This is where the output of the periodic monitoring jobs is uploaded.",
"minLength" : 1,
"maxLength" : 1,

Choose a reason for hiding this comment

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

Comment on lines +212 to +215
"type" : "array",
"description" : "Monitoring outputs for monitoring jobs. This is where the output of the periodic monitoring jobs is uploaded.",
"minLength" : 1,
"maxLength" : 1,

Choose a reason for hiding this comment

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

Comment on lines +197 to +200
"type" : "array",
"description" : "Monitoring outputs for monitoring jobs. This is where the output of the periodic monitoring jobs is uploaded.",
"minLength" : 1,
"maxLength" : 1,

Choose a reason for hiding this comment

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

Comment on lines +232 to +235
"type" : "array",
"description" : "Monitoring outputs for monitoring jobs. This is where the output of the periodic monitoring jobs is uploaded.",
"minLength" : 1,
"maxLength" : 1,

Choose a reason for hiding this comment

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

Comment on lines +300 to +303
"type" : "array",
"description" : "Monitoring outputs for monitoring jobs. This is where the output of the periodic monitoring jobs is uploaded.",
"minLength" : 1,
"maxLength" : 1,

Choose a reason for hiding this comment

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

},
"list": {
"permissions": [
"sagemaker:ListMonitoringSchedule"
Copy link

Choose a reason for hiding this comment

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

Suggested change
"sagemaker:ListMonitoringSchedule"
"sagemaker:ListMonitoringSchedules"

https://docs.aws.amazon.com/service-authorization/latest/reference/list_amazonsagemaker.html

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