Skip to content

Conversation

@aledsage
Copy link
Contributor

@aledsage aledsage commented Jun 4, 2018

Copy link
Member

@tbouron tbouron left a comment

Choose a reason for hiding this comment

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

LGTM @aledsage, I have just one comment, see below

For example, [security groups for EC2-classic]
(http://docs.aws.amazon.com/AWSEC2/latest/UserGuide/using-network-security.html#ec2-classic-security-groups)
For example,
[security groups for EC2-classic](http://docs.aws.amazon.com/AWSEC2/latest/UserGuide/using-network-security.html)
Copy link
Member

Choose a reason for hiding this comment

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

Why did you removed the anchor here? It is still valid.
Having said that, the section doesn't seem to have specificities to firewalls. So the title probably need to change to security groups for EC2, or be dropped all together

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good spot. I removed the anchor because it's not just ec2-classic that is used in Brooklyn (one can point it at VPC security groups). Therefore I've changed the link text to 'security groups for AWS EC2'.

Copy link
Member

@tbouron tbouron left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @aledsage. Merging

@asfgit asfgit merged commit 4219028 into apache:master Jun 5, 2018
asfgit pushed a commit that referenced this pull request Jun 5, 2018
@aledsage aledsage deleted the BROOKLYN-588 branch June 20, 2018 12:34
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