bemer / containers-on-aws-workshop Goto Github PK

"bellow" should be "below"

It uses bridge mode. I think that for the purposes of this lab, we can use awsvpc mode. This way, users can leverage this networking mode capabilities, benefits and speed

The workshop is supposed to be region agnostic (ex: CF templates are provided for different regions).

But, in some pages, there are lines that supposes the user is in the us-east-1 region. In addition to this, in the page 05-ContinuousDelivery, there are lines that supposes the user is in the us-east-2 region. This may confuse users.

Moreover, if the user copy/paste these lines, things will gonna be wrong, and it will hard to new-come users troubleshoot the errors.

It says: " you can go to the next chapter: 3. Deploying An ECS Cluster."

It should say "you can go to the next chapter: 2. Creating your Docker image."

The redirect of the image is OK.

I think it would be easier to state that the user should run the commands that appear after creating the repo using the console

File: https://github.com/bemer/containers-on-aws-workshop/tree/master/02-CreatingDockerImage

The link to "Setup Environment tutorial" in the following sentence is broken:

In this tutorial, we assume that you completed the Setup Environment tutorial and:

Target is:
https://github.com/bemer/containers-on-aws-workshop/blob/master/01-SetupEnvironment

Should be:

https://github.com/bemer/containers-on-aws-workshop/blob/master/01-EnvironmentSetup/

When creating the task definition for the EC2-type task on Chapter3, it only states:

"In the Task execution IAM role select Create new role and under Task size add 128 in the Task memory (MiB) field" and when clicking add container, you are prompted with an error stating you should declare the CPU Units.

I think you can change that sentence to:
"
"Under Task execution IAM role, select "Create new role"

Then, under Task Size use 128 and CPU units use 256
"

It says "bellow"
It should say "below"

There are no comments after the user reviews that the docker image is now on the repo. I´d recommend to add a note in the end of "02-CreatingDockerImage" that says:

Now we will create an ECS cluster to deploy our Docker image stored on ECR. Go to "03-DeployEcsCluster"

What do you think?

You have Clou9 instead of Cloud9