Skip to content

Commit afa49c0

Browse files
iblancasarogercollmviitane
authored
Fix accountingservice container for non-root users (open-telemetry#1696)
* Fix accountingservice container for non-root users Signed-off-by: Israel Blancas <iblancasa@gmail.com> * Add changelog entry Signed-off-by: Israel Blancas <iblancasa@gmail.com> * Fix lint Signed-off-by: Israel Blancas <iblancasa@gmail.com> * Update src/accountingservice/Dockerfile Co-authored-by: Roger Coll <roger.coll@elastic.co> --------- Signed-off-by: Israel Blancas <iblancasa@gmail.com> Co-authored-by: Roger Coll <roger.coll@elastic.co> Co-authored-by: Mikko Viitanen <74129181+mviitane@users.noreply.github.com>
1 parent 8ea2091 commit afa49c0

File tree

2 files changed

+4
-0
lines changed

2 files changed

+4
-0
lines changed

CHANGELOG.md

+3
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,9 @@ the release.
77

88
## Unreleased
99

10+
* [accountingservice] allow running the container with non root user
11+
([#1692](https://github.com/open-telemetry/opentelemetry-demo/pull/1692))
12+
1013
## 1.11.1
1114

1215
* [otel-col] Add docker stats receiver

src/accountingservice/Dockerfile

+1
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@ COPY --from=publish /app/publish .
2626
USER root
2727
RUN mkdir -p "/var/log/opentelemetry/dotnet"
2828
RUN chown app "/var/log/opentelemetry/dotnet"
29+
RUN chown app "/app/instrument.sh"
2930
USER app
3031

3132
ENTRYPOINT ["./instrument.sh", "dotnet", "AccountingService.dll"]

0 commit comments

Comments
 (0)