Skip to content

Commit

Permalink
Fix env variable
Browse files Browse the repository at this point in the history
  • Loading branch information
ladatz committed Jan 25, 2024
1 parent 2abc15e commit 78b5b41
Show file tree
Hide file tree
Showing 4 changed files with 10 additions and 10 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/e2e-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ concurrency:

env:
CARGO_TERM_COLOR: always
CHARIOTT_REGISTRY_TTL_SECS: 7
INTENT_BROKERING_REGISTRY_TTL_SECS: 7

jobs:
e2e:
Expand Down Expand Up @@ -67,7 +67,7 @@ jobs:
run: |
docker run -d --name intent_brokering --network host \
-e RUST_LOG=debug \
-e CHARIOTT_REGISTRY_TTL_SECS \
-e INTENT_BROKERING_REGISTRY_TTL_SECS \
intent_brokering:1
- name: Start KV-App
Expand Down
2 changes: 1 addition & 1 deletion Dockerfile.intent_brokering
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ FROM docker.io/library/rust:${RUST_VERSION} AS builder
# Dockerfile for building Eclipse Chariott Intent Brokering runtime container
#
# This Dockerfile utilizes a two step build process. It builds Intent Brokering
# service with statically linked dependencies (using musl) for a x86_64
# service with statically linked dependencies (using musl) for a x86_64
# architecture.

# Chariott user id
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,9 +5,9 @@
ARG RUST_VERSION=1.70
FROM docker.io/library/rust:${RUST_VERSION} AS builder

# Dockerfile for building the "latency test" container
# Dockerfile for building the "load test" container
#
# This Dockerfile utilizes a two step build process. It builds "latency test"
# This Dockerfile utilizes a two step build process. It builds "load test"
# with statically linked dependencies (using musl vs. glibc to accomplish this)
# for a specific architecture such that we can utilize a scratch container
# without further dependencies for our final container, minimizing container
Expand All @@ -30,7 +30,7 @@ RUN rustup target add x86_64-unknown-linux-musl

RUN cargo build --release --target=x86_64-unknown-linux-musl --package lt-provider-app

# unprivileged identity to run "latency test" as
# unprivileged identity to run "load test" as
RUN adduser \
--disabled-password \
--gecos "" \
Expand All @@ -46,7 +46,7 @@ RUN adduser \
FROM alpine:latest
ARG APP_NAME
ARG USER_NAME
# Import "latency test" user and group from builder.
# Import "load test" user and group from builder.
COPY --from=builder /etc/passwd /etc/passwd
COPY --from=builder /etc/group /etc/group

Expand All @@ -55,6 +55,6 @@ WORKDIR /sdv
# Copy our build
COPY --from=builder /sdv/target/x86_64-unknown-linux-musl/release/lt-provider-app /sdv/lt-provider-app

# Use the unprivileged "latency test" user during execution.
# Use the unprivileged "load test" user during execution.
USER ${USER_NAME}:${USER_NAME}
CMD ["sh", "-c", "./lt-provider-app"]
4 changes: 2 additions & 2 deletions intent_brokering/tests/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,9 +11,9 @@ Assuming the current working directory is the project root, run:

```sh
cargo build --release -p intent_brokering -p kv-app
CHARIOTT_REGISTRY_TTL_SECS=7 ./target/release/chariott &
INTENT_BROKERING_REGISTRY_TTL_SECS=7 ./target/release/chariott &
./target/release/kv-app &
CHARIOTT_REGISTRY_TTL_SECS=7 cargo test --test '*e2e'
INTENT_BROKERING_REGISTRY_TTL_SECS=7 cargo test --test '*e2e'
```

If you want to display debug logs, make sure to set the log level via the `RUST_LOG` environment variable.
Expand Down

1 comment on commit 78b5b41

@github-actions
Copy link
Contributor

Choose a reason for hiding this comment

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

ESS Benchmark

Benchmark suite Current: 78b5b41 Previous: 5886244 Ratio
ess/1-subscribers/1000-events 185127 ns/iter (± 4939) 188873 ns/iter (± 6213) 0.98
ess/10-subscribers/1000-events 1286478 ns/iter (± 23806) 1368890 ns/iter (± 32371) 0.94
ess/100-subscribers/1000-events 12535966 ns/iter (± 496228) 13413472 ns/iter (± 574355) 0.93
ess/1-subscribers/10000-events 1711208 ns/iter (± 9758) 1712373 ns/iter (± 59392) 1.00
ess/10-subscribers/10000-events 12770979 ns/iter (± 60031) 13528032 ns/iter (± 51857) 0.94
ess/100-subscribers/10000-events 125917122 ns/iter (± 2577305) 131653141 ns/iter (± 461857) 0.96

This comment was automatically generated by workflow using github-action-benchmark.

Please sign in to comment.