Fix accountingservice container for non-root users (#1696)

* Fix accountingservice container for non-root users Signed-off-by: Israel Blancas <[email protected]> * Add changelog entry Signed-off-by: Israel Blancas <[email protected]> * Fix lint Signed-off-by: Israel Blancas <[email protected]> * Update src/accountingservice/Dockerfile Co-authored-by: Roger Coll <[email protected]> --------- Signed-off-by: Israel Blancas <[email protected]> Co-authored-by: Roger Coll <[email protected]> Co-authored-by: Mikko Viitanen <[email protected]>
open-telemetry · Aug 13, 2024 · afa49c0 · afa49c0
1 parent 8ea2091
commit afa49c0
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -7,6 +7,9 @@ the release.
 
 ## Unreleased
 
+* [accountingservice] allow running the container with non root user
+  ([#1692](https://github.com/open-telemetry/opentelemetry-demo/pull/1692))
+
 ## 1.11.1
 
 * [otel-col] Add docker stats receiver

diff --git a/src/accountingservice/Dockerfile b/src/accountingservice/Dockerfile
@@ -26,6 +26,7 @@ COPY --from=publish /app/publish .
 USER root
 RUN mkdir -p "/var/log/opentelemetry/dotnet"
 RUN chown app "/var/log/opentelemetry/dotnet"
+RUN chown app "/app/instrument.sh"
 USER app
 
 ENTRYPOINT ["./instrument.sh", "dotnet", "AccountingService.dll"]