Skip to content

Conversation

brunoklein99
Copy link

Adds the template for a multi-branch training pipeline

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

@brunoklein99 brunoklein99 force-pushed the multi-branch-mlops-train branch from e957546 to 48f1df8 Compare August 29, 2021 16:51
@durgasury
Copy link
Contributor

@brunoklein99 could you make a couple of changes?

  1. Bucket name gets too long for the SageMaker project if the project name >18 chars. Please specify that or reduce the chars in default template name
  2. Launch constraint role doesn't have access to tag IAM roles, please add the policy

@durgasury
Copy link
Contributor

approved PR

@durgasury durgasury closed this Sep 9, 2021
@durgasury durgasury reopened this Sep 9, 2021
@durgasury durgasury merged commit 4f74b86 into aws-samples:main Sep 9, 2021
georschi pushed a commit that referenced this pull request Jun 1, 2022
* update README for infra repository

* update READMEs and the diagrams

* remove some constant values

* removing loaded account ids from config

* add new template option for dynamic accounts through cfn params

- add more to README
- rename stagin to preprod
- remove space from folder name
- remove account specific details

* add comments to constants.py

* update script to handle updating constants.py

* adding minor detail

Co-authored-by: Fatema Alkhanaizi <alkhanai@amazon.co.uk>
georschi added a commit that referenced this pull request Jun 1, 2022
Feature/dynamic accounts setup (#2)
acere pushed a commit to apac-ml-tfc/sagemaker-custom-project-templates that referenced this pull request Jun 13, 2024
…mlops-stacks

This PR enables multiple set of accounts to be deployed and moves project specific SSM parameters to be created together with project
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.

3 participants