Skip to content
This repository has been archived by the owner on Apr 25, 2024. It is now read-only.

Minor doc updates for app discovery: #471

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Minor doc updates for app discovery: #471

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
e092a7f
Minor doc updates for app discovery:
rddefauw Jun 4, 2018
c70ecd1
Use 'host' rather than 'ingress' in the URL examples.
rddefauw Jun 8, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 8 additions & 6 deletions 03-path-application-development/302-app-discovery/readme.adoc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,9 @@ This chapter shows an example of how different microservices within an applicati

This chapter uses a cluster with 3 master nodes and 5 worker nodes as described here: link:../../01-path-basics/102-your-first-cluster#multi-master-cluster[multi-master cluster].

All configuration files for this chapter are in the `microservices` directory. Make sure you change to that directory before giving any commands in this chapter.
All configuration files for this chapter are in the `app-discovery` directory. Make sure you change to that directory before giving any commands in this chapter. On Cloud9:

cd ~/environment/aws-workshop-for-kubernetes/03-path-application-development/302-app-discovery

=== Application Architecture

Expand Down Expand Up @@ -121,12 +123,12 @@ Wait for ~3 mins for the load balancer to accept request.

Access the application using the following URLs with `curl` or via a browser:

http://<host>
http://<host>?greet=ho
http://<host>?id=1
http://<host>?greet=ho&id=1
http://<ingress>
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this should be left as <host>. We can add clarity around the instruction, such as running the kubectl get svc/webapp-service command first, then stating to use that value as <host>.

http://<ingress>?greet=ho
http://<ingress>?id=1
http://<ingress>?greet=ho&id=1

`<host>` is the value of the ingress load balancer's address:
`<ingress>` is the value of the ingress load balancer's address:

$ kubectl get svc/webapp-service -o jsonpath={.status.loadBalancer.ingress[0].hostname}
a5427e128847211e782280a896fc2bfc-283874069.us-east-1.elb.amazonaws.com
Expand Down