Skip to content

feat(apigateway): L2 construct for Sagemaker Integration#25459

Merged
mergify[bot] merged 14 commits intomainfrom
apigateway-sagemaker-integration
Aug 25, 2023
Merged

feat(apigateway): L2 construct for Sagemaker Integration#25459
mergify[bot] merged 14 commits intomainfrom
apigateway-sagemaker-integration

Conversation

@cgarvis
Copy link
Copy Markdown
Contributor

@cgarvis cgarvis commented May 5, 2023

Sagemaker integration for ApiGateway. Pulled the IEndpoint from the alpha module.

Stabilizes sagemaker.IEndpoint from aws-sagemaker-alpha so that it can be used in the integration.


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

@gitpod-io
Copy link
Copy Markdown

gitpod-io bot commented May 5, 2023

@aws-cdk-automation aws-cdk-automation requested a review from a team May 5, 2023 13:32
@github-actions github-actions bot added the p2 label May 5, 2023
@mergify mergify bot added the contribution/core This is a PR that came from AWS. label May 5, 2023
Copy link
Copy Markdown
Collaborator

@aws-cdk-automation aws-cdk-automation left a comment

Choose a reason for hiding this comment

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

The pull request linter has failed. See the aws-cdk-automation comment below for failure reasons. If you believe this pull request should receive an exemption, please comment and provide a justification.

A comment requesting an exemption should contain the text Exemption Request. Additionally, if clarification is needed add Clarification Request to a comment.

@cgarvis cgarvis changed the title feat(apigateway): L2 construct for Sagemaker Integration feat(apigateway): add L2 construct for Sagemaker Integration May 5, 2023
@kaizencc kaizencc changed the title feat(apigateway): add L2 construct for Sagemaker Integration feat(apigateway): L2 construct for Sagemaker Integration Aug 24, 2023
@kaizencc kaizencc marked this pull request as ready for review August 24, 2023 20:30
@kaizencc kaizencc added the pr/do-not-merge This PR should not be merged at this time. label Aug 24, 2023
@kaizencc kaizencc added pr-linter/exempt-integ-test The PR linter will not require integ test changes and removed pr/do-not-merge This PR should not be merged at this time. labels Aug 25, 2023
@aws-cdk-automation aws-cdk-automation dismissed their stale review August 25, 2023 17:23

✅ Updated pull request passes all PRLinter validations. Dismissing previous PRLinter review.

@aws-cdk-automation
Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: AutoBuildv2Project1C6BFA3F-wQm2hXv2jqQv
  • Commit ID: 4530d51
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Aug 25, 2023

Thank you for contributing! Your pull request will be updated from main and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

@mergify mergify bot merged commit 53d61bb into main Aug 25, 2023
@mergify mergify bot deleted the apigateway-sagemaker-integration branch August 25, 2023 18:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contribution/core This is a PR that came from AWS. p2 pr-linter/exempt-integ-test The PR linter will not require integ test changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants