Skip to content

Commit ce335ca

Browse files
committed
rename adservice to ad
1 parent 0edb6b5 commit ce335ca

File tree

4 files changed

+17
-15
lines changed

4 files changed

+17
-15
lines changed

src/ad/Dockerfile

+2
Original file line numberDiff line numberDiff line change
@@ -9,11 +9,13 @@ WORKDIR /usr/src/app/
99
COPY ./src/ad/gradlew* ./src/ad/settings.gradle* ./src/ad/build.gradle ./
1010
COPY ./src/ad/gradle ./gradle
1111

12+
RUN chmod +x ./gradlew
1213
RUN ./gradlew
1314
RUN ./gradlew downloadRepos
1415

1516
COPY ./src/ad/ ./
1617
COPY ./pb/ ./proto
18+
RUN chmod +x ./gradlew
1719
RUN ./gradlew installDist -PprotoSourceDir=./proto
1820

1921
# -----------------------------------------------------------------------------

src/ad/src/main/java/oteldemo/AdService.java

+10-10
Original file line numberDiff line numberDiff line change
@@ -59,8 +59,8 @@ public final class AdService {
5959
private HealthStatusManager healthMgr;
6060

6161
private static final AdService service = new AdService();
62-
private static final Tracer tracer = GlobalOpenTelemetry.getTracer("adservice");
63-
private static final Meter meter = GlobalOpenTelemetry.getMeter("adservice");
62+
private static final Tracer tracer = GlobalOpenTelemetry.getTracer("ad");
63+
private static final Meter meter = GlobalOpenTelemetry.getMeter("ad");
6464

6565
private static final LongCounter adRequestsCounter =
6666
meter
@@ -80,7 +80,7 @@ private void start() throws IOException {
8080
.orElseThrow(
8181
() ->
8282
new IllegalStateException(
83-
"environment vars: AD_SERVICE_PORT must not be null")));
83+
"environment vars: AD_PORT must not be null")));
8484
healthMgr = new HealthStatusManager();
8585

8686
// Create a flagd instance with OpenTelemetry
@@ -132,9 +132,9 @@ private enum AdResponseType {
132132

133133
private static class AdServiceImpl extends oteldemo.AdServiceGrpc.AdServiceImplBase {
134134

135-
private static final String ADSERVICE_FAILURE = "adServiceFailure";
136-
private static final String ADSERVICE_MANUAL_GC_FEATURE_FLAG = "adServiceManualGc";
137-
private static final String ADSERVICE_HIGH_CPU_FEATURE_FLAG = "adServiceHighCpu";
135+
private static final String AD_FAILURE = "adFailure";
136+
private static final String AD_MANUAL_GC_FEATURE_FLAG = "adManualGc";
137+
private static final String AD_HIGH_CPU_FEATURE_FLAG = "adHighCpu";
138138
private static final Client ffClient = OpenFeatureAPI.getInstance().getClient();
139139

140140
private AdServiceImpl() {}
@@ -169,7 +169,7 @@ public void getAds(AdRequest req, StreamObserver<AdResponse> responseObserver) {
169169
}
170170

171171
CPULoad cpuload = CPULoad.getInstance();
172-
cpuload.execute(ffClient.getBooleanValue(ADSERVICE_HIGH_CPU_FEATURE_FLAG, false, evaluationContext));
172+
cpuload.execute(ffClient.getBooleanValue(AD_HIGH_CPU_FEATURE_FLAG, false, evaluationContext));
173173

174174
span.setAttribute("app.ads.contextKeys", req.getContextKeysList().toString());
175175
span.setAttribute("app.ads.contextKeys.count", req.getContextKeysCount());
@@ -202,12 +202,12 @@ public void getAds(AdRequest req, StreamObserver<AdResponse> responseObserver) {
202202
adRequestTypeKey, adRequestType.name(), adResponseTypeKey, adResponseType.name()));
203203

204204
// Throw 1/10 of the time to simulate a failure when the feature flag is enabled
205-
if (ffClient.getBooleanValue(ADSERVICE_FAILURE, false, evaluationContext) && random.nextInt(10) == 0) {
205+
if (ffClient.getBooleanValue(AD_FAILURE, false, evaluationContext) && random.nextInt(10) == 0) {
206206
throw new StatusRuntimeException(Status.UNAVAILABLE);
207207
}
208208

209-
if (ffClient.getBooleanValue(ADSERVICE_MANUAL_GC_FEATURE_FLAG, false, evaluationContext)) {
210-
logger.warn("Feature Flag " + ADSERVICE_MANUAL_GC_FEATURE_FLAG + " enabled, performing a manual gc now");
209+
if (ffClient.getBooleanValue(AD_MANUAL_GC_FEATURE_FLAG, false, evaluationContext)) {
210+
logger.warn("Feature Flag " + AD_MANUAL_GC_FEATURE_FLAG + " enabled, performing a manual gc now");
211211
GarbageCollectionTrigger gct = new GarbageCollectionTrigger();
212212
gct.doExecute();
213213
}

src/flagd/demo.flagd.json

+3-3
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@
1919
},
2020
"defaultVariant": "off"
2121
},
22-
"adServiceManualGc": {
22+
"adManualGc": {
2323
"description": "Triggers full manual garbage collections in the ad service",
2424
"state": "ENABLED",
2525
"variants": {
@@ -28,7 +28,7 @@
2828
},
2929
"defaultVariant": "off"
3030
},
31-
"adServiceHighCpu": {
31+
"adHighCpu": {
3232
"description": "Triggers high cpu load in the ad service",
3333
"state": "ENABLED",
3434
"variants": {
@@ -37,7 +37,7 @@
3737
},
3838
"defaultVariant": "off"
3939
},
40-
"adServiceFailure": {
40+
"adFailure": {
4141
"description": "Fail ad service",
4242
"state": "ENABLED",
4343
"variants": {

src/frontend/gateways/rpc/Ad.gateway.ts

+2-2
Original file line numberDiff line numberDiff line change
@@ -4,9 +4,9 @@
44
import { ChannelCredentials } from '@grpc/grpc-js';
55
import { AdResponse, AdServiceClient } from '../../protos/demo';
66

7-
const { AD_SERVICE_ADDR = '' } = process.env;
7+
const { AD_ADDR = '' } = process.env;
88

9-
const client = new AdServiceClient(AD_SERVICE_ADDR, ChannelCredentials.createInsecure());
9+
const client = new AdServiceClient(AD_ADDR, ChannelCredentials.createInsecure());
1010

1111
const AdGateway = () => ({
1212
listAds(contextKeys: string[]) {

0 commit comments

Comments
 (0)