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

INSTALL: simplify documentation for subnet selection #422

Merged
merged 1 commit into from
Oct 16, 2019
Merged

Conversation

tjkirch
Copy link
Contributor

@tjkirch tjkirch commented Oct 16, 2019

Uses more advanced aws jmespath/jq queries to bring together information with
less work from the user, and with clearer output.

The subnet commands were in the middle of the cluster settings, too, which are
now moved above the subnet commands, so the user doesn't have to keep track of
those two sections mentally.


Fixes #384, and then some.

Don't forget to click the "display the rich diff" button at the top right of the diff.

@tjkirch tjkirch requested a review from iliana October 16, 2019 21:37
INSTALL.md Outdated Show resolved Hide resolved
@tjkirch
Copy link
Contributor Author

tjkirch commented Oct 16, 2019

This push removes the subnet info from the first command, where it's now irrelevant; oops.

INSTALL.md Show resolved Hide resolved
Uses more advanced aws jmespath/jq queries to bring together information with
less work from the user, and with clearer output.

The subnet commands were in the middle of the cluster settings, too, which are
now moved above the subnet commands, so the user doesn't have to keep track of
those two sections mentally.
@tjkirch
Copy link
Contributor Author

tjkirch commented Oct 16, 2019

This push adds a note about availability zone selection.

@tjkirch tjkirch merged commit 285e64c into develop Oct 16, 2019
@tjkirch tjkirch deleted the subnet-doc branch October 16, 2019 23:24
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.

INSTALL.md: It's not clear how to tell which subnets are public
3 participants