Skip to content

Commit

Permalink
Use env in service manifests for PORT (GoogleCloudPlatform#212)
Browse files Browse the repository at this point in the history
* fix shipping service ports

* fix product catalog servicde ports

* fix paymentservice ports

* fix currenservice ports

* fix checkoutservice ports

* fix emailservice ports

* fix frontend app ports

* fix recommendationservice ports

* fix indentation in service yaml files
  • Loading branch information
jan25 authored and ahmetb committed May 9, 2019
1 parent 876d4f9 commit 55f5061
Show file tree
Hide file tree
Showing 12 changed files with 33 additions and 9 deletions.
2 changes: 2 additions & 0 deletions kubernetes-manifests/checkoutservice.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,8 @@ spec:
exec:
command: ["/bin/grpc_health_probe", "-addr=:5050"]
env:
- name: PORT
value: "5050"
- name: PRODUCT_CATALOG_SERVICE_ADDR
value: "productcatalogservice:3550"
- name: SHIPPING_SERVICE_ADDR
Expand Down
3 changes: 3 additions & 0 deletions kubernetes-manifests/currencyservice.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,9 @@ spec:
ports:
- name: grpc
containerPort: 7000
env:
- name: PORT
value: "7000"
readinessProbe:
exec:
command: ["/bin/grpc_health_probe", "-addr=:7000"]
Expand Down
3 changes: 3 additions & 0 deletions kubernetes-manifests/emailservice.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,9 @@ spec:
image: emailservice
ports:
- containerPort: 8080
env:
- name: PORT
value: "8080"
readinessProbe:
periodSeconds: 5
exec:
Expand Down
2 changes: 2 additions & 0 deletions kubernetes-manifests/frontend.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,8 @@ spec:
- name: "Cookie"
value: "shop_session-id=x-liveness-probe"
env:
- name: PORT
value: "8080"
- name: PRODUCT_CATALOG_SERVICE_ADDR
value: "productcatalogservice:3550"
- name: CURRENCY_SERVICE_ADDR
Expand Down
3 changes: 3 additions & 0 deletions kubernetes-manifests/paymentservice.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,9 @@ spec:
image: paymentservice
ports:
- containerPort: 50051
env:
- name: PORT
value: "50051"
readinessProbe:
exec:
command: ["/bin/grpc_health_probe", "-addr=:50051"]
Expand Down
3 changes: 3 additions & 0 deletions kubernetes-manifests/productcatalogservice.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,9 @@ spec:
image: productcatalogservice
ports:
- containerPort: 3550
env:
- name: PORT
value: "3550"
readinessProbe:
exec:
command: ["/bin/grpc_health_probe", "-addr=:3550"]
Expand Down
2 changes: 2 additions & 0 deletions kubernetes-manifests/recommendationservice.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,8 @@ spec:
exec:
command: ["/bin/grpc_health_probe", "-addr=:8080"]
env:
- name: PORT
value: "8080"
- name: PRODUCT_CATALOG_SERVICE_ADDR
value: "productcatalogservice:3550"
- name: ENABLE_PROFILER
Expand Down
3 changes: 3 additions & 0 deletions kubernetes-manifests/shippingservice.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,9 @@ spec:
image: shippingservice
ports:
- containerPort: 50051
env:
- name: PORT
value: "50051"
readinessProbe:
periodSeconds: 5
exec:
Expand Down
2 changes: 1 addition & 1 deletion src/currencyservice/server.js
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ const protoLoader = require('@grpc/proto-loader');
const MAIN_PROTO_PATH = path.join(__dirname, './proto/demo.proto');
const HEALTH_PROTO_PATH = path.join(__dirname, './proto/grpc/health/v1/health.proto');

const PORT = 7000;
const PORT = process.env.PORT;

const shopProto = _loadProto(MAIN_PROTO_PATH).hipstershop;
const healthProto = _loadProto(HEALTH_PROTO_PATH).grpc.health.v1;
Expand Down
4 changes: 2 additions & 2 deletions src/paymentservice/server.js
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ const logger = pino({
});

class HipsterShopServer {
constructor (protoRoot, port = HipsterShopServer.DEFAULT_PORT) {
constructor (protoRoot, port = HipsterShopServer.PORT) {
this.port = port;

this.packages = {
Expand Down Expand Up @@ -99,6 +99,6 @@ class HipsterShopServer {
}
}

HipsterShopServer.DEFAULT_PORT = 50051;
HipsterShopServer.PORT = process.env.PORT;

module.exports = HipsterShopServer;
13 changes: 8 additions & 5 deletions src/productcatalogservice/server.go
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ var (
log *logrus.Logger
extraLatency time.Duration

port = flag.Int("port", 3550, "port to listen at")
port = "3550"

reloadCatalog bool
)
Expand Down Expand Up @@ -106,13 +106,16 @@ func main() {
}
}()

log.Infof("starting grpc server at :%d", *port)
run(*port)
if os.Getenv("PORT") != "" {
port = os.Getenv("PORT")
}
log.Infof("starting grpc server at :%s", port)
run(port)
select {}
}

func run(port int) string {
l, err := net.Listen("tcp", fmt.Sprintf(":%d", port))
func run(port string) string {
l, err := net.Listen("tcp", fmt.Sprintf(":%s", port))
if err != nil {
log.Fatal(err)
}
Expand Down
2 changes: 1 addition & 1 deletion src/shippingservice/main.go
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ func main() {
go initProfiling("shippingservice", "1.0.0")

port := defaultPort
if value, ok := os.LookupEnv("APP_PORT"); ok {
if value, ok := os.LookupEnv("PORT"); ok {
port = value
}
port = fmt.Sprintf(":%s", port)
Expand Down

0 comments on commit 55f5061

Please sign in to comment.