Skip to content
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

Windows (WCOW) bug: volume mounts seem not to work correctly when running the 5 phases #2147

Open
natalieparellano opened this issue May 2, 2024 · 0 comments
Labels
status/ready Issue ready to be worked on. type/bug Issue that reports an unexpected behaviour.

Comments

@natalieparellano
Copy link
Member

Summary

See this branch #2146 and these failing tests:

bpOutputAsserts.ReportsReadingFileContents("Detect", testFilePath, "some-content")

fails here with output

        + 	[detector] Starting detector...
        + 	[detector] Parsing inputs...
        + 	[detector] Ensuring privileges...
        + 	[detector] Executing command...
        + 	[detector] Timer: Detector started at 2024-05-02T18:18:51Z
        + 	[detector] Checking for match against descriptor: {windows   []}
        + 	[detector] ======== Output: volume/bp@volume-bp-version ========
        + 	[detector] --- Detect: Volume Buildpack
        + 	[detector] Access is denied.
        + 	[detector] Detect: Reading file 'c:\platform\volume-mount-target\some-file': 
        + 	[detector] --- Done

bpOutputAsserts.ReportsReadingFileContents("Detect", testDetectFilePath, "some-content")

fails here with output

       [detector] Starting detector...
        [detector] Parsing inputs...
        [detector] Ensuring privileges...
        [detector] Executing command...
        [detector] Timer: Detector started at 2024-05-02T18:19:24Z
        [detector] Checking for match against descriptor: {windows   []}
        [detector] ======== Output: rw-volume/bp@rw-volume-bp-version ========
        [detector] --- Detect: Read/Write Volume Buildpack
        [detector] Access is denied.
        [detector] Detect: Writing file 'c:\volume-mount-target\detect-file': failed
        [detector] The system cannot find the file specified.
        [detector] Detect: Reading file 'c:\volume-mount-target\detect-file': 
        [detector] --- Done

I wonder if it is similar to the issue fixed by #2129 in that with the 5 phases, the volume is getting mounted first to the analyze container (which is started as root), then to the detect container (not root). Though, the creator does drop to a non-root user by the time detect starts.


Environment

Windows Containers on Windows

@natalieparellano natalieparellano added type/bug Issue that reports an unexpected behaviour. status/triage Issue or PR that requires contributor attention. labels May 2, 2024
@jjbustamante jjbustamante added status/ready Issue ready to be worked on. and removed status/triage Issue or PR that requires contributor attention. labels May 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status/ready Issue ready to be worked on. type/bug Issue that reports an unexpected behaviour.
Projects
None yet
Development

No branches or pull requests

2 participants