Skip to content

Compose publish doesn't apply ownership for ContainerFileSystemCallbackAnnotation with existing files #9410

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
1 task done
danegsta opened this issue May 20, 2025 · 0 comments
Milestone

Comments

@danegsta
Copy link
Member

Is there an existing issue for this?

  • I have searched the existing issues

Describe the bug

In #8908, we updated the Docker Compose publish path to use the existing configs feature to provide an analog to the runtime behavior for ContainerFileSystemCallbackAnnotation. However, while compose allows us to set ownership and permission settings for all configs references in a service, it only honors those settings when providing the file content inline (as internally it uses docker cp like we do at runtime). With existing file references, compose uses volume mounts to inject the files and doesn't apply permissions (the file is added to the container, but with default ownership/mode settings).

One potential workaround would be to convert all text based ContainerFileSystemCallbackAnnotation to inline by reading the file contents, but this could cause the compose file to grow very large and wouldn't be a viable option for binary files.

Expected Behavior

No response

Steps To Reproduce

No response

Exceptions (if any)

No response

.NET Version info

No response

Anything else?

No response

@github-actions github-actions bot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label May 20, 2025
@danegsta danegsta added this to the 9.4 milestone May 20, 2025
@danmoseley danmoseley added area-deployment and removed needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners labels May 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants