Skip to content

Commit

Permalink
remove -service and _service suffixes (#1914)
Browse files Browse the repository at this point in the history
  • Loading branch information
puckpuck authored Jan 15, 2025
1 parent ea8456f commit b604d28
Show file tree
Hide file tree
Showing 12 changed files with 25 additions and 25 deletions.
12 changes: 6 additions & 6 deletions docker-compose.minimal.yml
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ services:
# Cart service
cart:
image: ${IMAGE_NAME}:${DEMO_VERSION}-cart
container_name: cart-service
container_name: cart
build:
context: ./
dockerfile: ${CART_DOCKERFILE}
Expand Down Expand Up @@ -131,7 +131,7 @@ services:
# Currency service
currency:
image: ${IMAGE_NAME}:${DEMO_VERSION}-currency
container_name: currency-service
container_name: currency
build:
context: ./
dockerfile: ${CURRENCY_DOCKERFILE}
Expand Down Expand Up @@ -159,7 +159,7 @@ services:
# Email service
email:
image: ${IMAGE_NAME}:${DEMO_VERSION}-email
container_name: email-service
container_name: email
build:
context: ./src/email
cache_from:
Expand Down Expand Up @@ -355,7 +355,7 @@ services:
# Payment service
payment:
image: ${IMAGE_NAME}:${DEMO_VERSION}-payment
container_name: payment-service
container_name: payment
build:
context: ./
dockerfile: ${PAYMENT_DOCKERFILE}
Expand Down Expand Up @@ -411,7 +411,7 @@ services:
# Quote service
quote:
image: ${IMAGE_NAME}:${DEMO_VERSION}-quote
container_name: quote-service
container_name: quote
build:
context: ./
dockerfile: ${QUOTE_DOCKERFILE}
Expand Down Expand Up @@ -439,7 +439,7 @@ services:
# Recommendation service
recommendation:
image: ${IMAGE_NAME}:${DEMO_VERSION}-recommendation
container_name: recommendation-service
container_name: recommendation
build:
context: ./
dockerfile: ${RECOMMENDATION_DOCKERFILE}
Expand Down
12 changes: 6 additions & 6 deletions docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,7 @@ services:
# Cart service
cart:
image: ${IMAGE_NAME}:${DEMO_VERSION}-cart
container_name: cart-service
container_name: cart
build:
context: ./
dockerfile: ${CART_DOCKERFILE}
Expand Down Expand Up @@ -169,7 +169,7 @@ services:
# Currency service
currency:
image: ${IMAGE_NAME}:${DEMO_VERSION}-currency
container_name: currency-service
container_name: currency
build:
context: ./
dockerfile: ${CURRENCY_DOCKERFILE}
Expand Down Expand Up @@ -197,7 +197,7 @@ services:
# Email service
email:
image: ${IMAGE_NAME}:${DEMO_VERSION}-email
container_name: email-service
container_name: email
build:
context: ./
dockerfile: ${EMAIL_DOCKERFILE}
Expand Down Expand Up @@ -431,7 +431,7 @@ services:
# Payment service
payment:
image: ${IMAGE_NAME}:${DEMO_VERSION}-payment
container_name: payment-service
container_name: payment
build:
context: ./
dockerfile: ${PAYMENT_DOCKERFILE}
Expand Down Expand Up @@ -493,7 +493,7 @@ services:
# Quote service
quote:
image: ${IMAGE_NAME}:${DEMO_VERSION}-quote
container_name: quote-service
container_name: quote
build:
context: ./
dockerfile: ${QUOTE_DOCKERFILE}
Expand Down Expand Up @@ -521,7 +521,7 @@ services:
# Recommendation service
recommendation:
image: ${IMAGE_NAME}:${DEMO_VERSION}-recommendation
container_name: recommendation-service
container_name: recommendation
build:
context: ./
dockerfile: ${RECOMMENDATION_DOCKERFILE}
Expand Down
2 changes: 1 addition & 1 deletion src/ad/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -32,4 +32,4 @@ ADD --chmod=644 https://github.com/open-telemetry/opentelemetry-java-instrumenta
ENV JAVA_TOOL_OPTIONS=-javaagent:/usr/src/app/opentelemetry-javaagent.jar

EXPOSE ${AD_PORT}
ENTRYPOINT [ "./build/install/opentelemetry-demo-ad-service/bin/Ad" ]
ENTRYPOINT [ "./build/install/opentelemetry-demo-ad/bin/Ad" ]
2 changes: 1 addition & 1 deletion src/ad/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ To run the Ad Service:
```sh
export AD_PORT=8080
export FEATURE_FLAG_GRPC_SERVICE_ADDR=featureflagservice:50053
./build/install/opentelemetry-demo-ad-service/bin/Ad
./build/install/opentelemetry-demo-ad/bin/Ad
```

### Upgrading Gradle
Expand Down
2 changes: 1 addition & 1 deletion src/ad/settings.gradle
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@

rootProject.name = 'opentelemetry-demo-ad-service'
rootProject.name = 'opentelemetry-demo-ad'
2 changes: 1 addition & 1 deletion src/currency/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
cmake_minimum_required(VERSION 3.1)
project(currency-service)
project(currency)

find_package(Protobuf REQUIRED)
find_package(gRPC CONFIG REQUIRED)
Expand Down
6 changes: 3 additions & 3 deletions test/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -25,9 +25,9 @@ make run-tracetesting SERVICES_TO_TEST="service-1 service-2 ..."
docker compose run traceBasedTests "service-1 service-2 ..."
```

For instance, if you need to run the tests for `ad-service` and
`payment-service`, you can run them with:
For instance, if you need to run the tests for `ad` and `payment`, you can run
them with:

```sh
make run-tracetesting SERVICES_TO_TEST="ad-service payment-service"
make run-tracetesting SERVICES_TO_TEST="ad payment"
```
2 changes: 1 addition & 1 deletion test/tracetesting/ad/all.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@

type: TestSuite
spec:
id: ad-service-all
id: ad-all
name: 'Ad Service'
description: Run all Ad Service tests enabled in sequence
steps:
Expand Down
2 changes: 1 addition & 1 deletion test/tracetesting/currency/all.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@

type: TestSuite
spec:
id: currency-service-all
id: currency-all
name: 'Currency Service'
description: Run all Currency Service tests enabled in sequence
steps:
Expand Down
2 changes: 1 addition & 1 deletion test/tracetesting/recommendation/all.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@

type: TestSuite
spec:
id: recommendation-service-all
id: recommendation-all
name: 'Recommendation Service'
description: Run all Recommendation Service tests enabled in sequence
steps:
Expand Down
4 changes: 2 additions & 2 deletions test/tracetesting/run.bash
Original file line number Diff line number Diff line change
Expand Up @@ -54,8 +54,8 @@ spec:
value: $RECOMMENDATION_ADDR
- key: SHIPPING_ADDR
value: $SHIPPING_ADDR
- key: KAFKA_SERVICE_ADDR
value: $KAFKA_SERVICE_ADDR
- key: KAFKA_ADDR
value: $KAFKA_ADDR
EOF
}

Expand Down
2 changes: 1 addition & 1 deletion test/tracetesting/shipping/all.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@

type: TestSuite
spec:
id: shipping-service-all
id: shipping-all
name: 'Shipping Service'
description: Run all Shipping Service tests enabled in sequence
steps:
Expand Down

0 comments on commit b604d28

Please sign in to comment.