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

Move AWS distro tutorials to distributions/aws directory #42

Merged
merged 1 commit into from
Dec 9, 2021

Conversation

surajkota
Copy link
Contributor

@surajkota surajkota commented Dec 9, 2021

Description of your changes:

  • Change aws examples directory location to be in dist/aws to isolate aws related changes
  • Revert generic distro example to original location and revert README changes
  • Update main README to reflect AWS distro and pointer to tutorials
  • Update ingress, alb-controller and envoy-filter location
  • Fixed a few broken links in RDS and one in cognito-automated readme
  • Some feedback from AWS distro tutorials for KF-1.4 #39

Testing

Manual, WIP

  • All links are working including cross readme links
  • Images are loading
  • Cognito readme
  • RDS-S3 reame
  • Cognito-RDS-S3 readme

Checked login, ran a sample pipeline to verify rds and s3 connections and verified a notebook server default options have DLC image

#26

@surajkota surajkota self-assigned this Dec 9, 2021
@surajkota surajkota force-pushed the v1.3-readme-update branch 2 times, most recently from 10d7d54 to 11bf83a Compare December 9, 2021 04:22
@@ -33,7 +33,7 @@ This guide assumes that you have:
```
1. AWS IAM permissions to create roles and attach policies to roles.

1. Clone the `awslabs/kubeflow-manifest` repo.
1. Clone the `awslabs/kubeflow-manifest` repo and checkout release branch.
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: Calling it out as release branch here as of right now it doesn't currently exist? Also how will we handle the different versions i.e (1.3 ... 1.4)? maybe state desired release branch?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

the branch we are working on are release branches, e.g. v1.3-branch but this will be changed when we get closer to release since we will cut a tag

@surajkota surajkota merged commit 8512657 into awslabs:v1.3-branch Dec 9, 2021
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