Skip to content

Conversation

@tobiasehlert
Copy link
Member

This pull request makes a minor adjustment to the file permissions for the application binary in the Dockerfile. The change increases the permissions from 544 to 555, allowing all users to execute the binary.

When in Kubernetes another user is set (via runAsUser), such as 65534, the binary fails to start because other doesn't have the execute flag on the app binary.

@tobiasehlert tobiasehlert requested a review from Copilot September 8, 2025 13:00
@tobiasehlert tobiasehlert self-assigned this Sep 8, 2025
@tobiasehlert tobiasehlert added bug Something isn't working docker Pull requests that update Docker code labels Sep 8, 2025
@sonarqubecloud
Copy link

sonarqubecloud bot commented Sep 8, 2025

@codecov
Copy link

codecov bot commented Sep 8, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.30%. Comparing base (8417aec) to head (dda1cd4).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #499   +/-   ##
=======================================
  Coverage   80.30%   80.30%           
=======================================
  Files          22       22           
  Lines        3697     3697           
=======================================
  Hits         2969     2969           
  Misses        587      587           
  Partials      141      141           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request fixes a Docker permission issue by updating the binary file permissions from 544 to 555 in the Dockerfile. The change ensures the application binary can be executed by any user, preventing startup failures when Kubernetes runs the container with different user IDs (such as 65534) via runAsUser.

  • Updated file permissions for the application binary from 544 (r-xr--r--) to 555 (r-xr-xr-x)
Comments suppressed due to low confidence (2)

Dockerfile:1

  • The Go version 1.25.0 does not exist. The latest stable Go version as of my knowledge cutoff is 1.22.x. Please verify this version exists or use a valid Go version.
# get golang container

Dockerfile:1

  • Alpine Linux version 3.22.1 does not exist. Alpine follows a different versioning scheme with versions like 3.19, 3.18, etc. Please use a valid Alpine version.
# get golang container

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@tobiasehlert tobiasehlert merged commit 1a1305e into main Sep 8, 2025
7 checks passed
@tobiasehlert tobiasehlert deleted the fix(docker)-use-555-for-binary branch September 8, 2025 13:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working docker Pull requests that update Docker code

Development

Successfully merging this pull request may close these issues.

2 participants