Code Monkey home page Code Monkey logo

Comments (16)

nhooyr avatar nhooyr commented on August 23, 2024

Its only like a 5s delay so not crazy but still.

from sail.

ammario avatar ammario commented on August 23, 2024

If we're indiscriminately chmoding home, isn't it possible we override some intentionally set permissions?

from sail.

nhooyr avatar nhooyr commented on August 23, 2024

We're chowning it which should be fine.

from sail.

ammario avatar ammario commented on August 23, 2024

Is that because chown doesn't propogate a mac mount? Confused

from sail.

nhooyr avatar nhooyr commented on August 23, 2024

Its extremely unlikely any of the mounts contain files owned by a different user.

an alternative approach we could take is to pass in the UID to the build container as a build arg so that the image itself has the right uid from the get go.

from sail.

ammario avatar ammario commented on August 23, 2024

In the rare case there is, there can be some minor corruption. E.g, what if they sshfs a server to some directory in home? I know i've done that before.. This could infinitely traverse my remote server and fuck me up good.

from sail.

nhooyr avatar nhooyr commented on August 23, 2024

Just read the manpage, it turns out it only chowns files already owned by the user. Not sure how this interacts with mounts though. I'll give it a test.

from sail.

ammario avatar ammario commented on August 23, 2024

That doesn't make any sense... If the file is already owned by the user what good does chowning it to the same user do?

from sail.

nhooyr avatar nhooyr commented on August 23, 2024

Files aren't owned by the user, they're owned by the UID.

from sail.

ammario avatar ammario commented on August 23, 2024

isn't the uid 1:1 associated with user

from sail.

nhooyr avatar nhooyr commented on August 23, 2024

Yes, but if you change the UID for a user, you need to chown every file such thats its owned by the new UID.

from sail.

nhooyr avatar nhooyr commented on August 23, 2024

So turns out, the osxfs driver does the right thing by default: https://stackoverflow.com/questions/43097341/docker-on-macosx-does-not-translate-file-ownership-correctly-in-volumes

from sail.

nhooyr avatar nhooyr commented on August 23, 2024

So multi user systems are supported on macOS without any changes. Just linux that will end up being an issue.

from sail.

nhooyr avatar nhooyr commented on August 23, 2024

might be useful: https://github.com/boxboat/fixuid
coder/code-server#640

from sail.

nathanpotter avatar nathanpotter commented on August 23, 2024

Using that fixuid library to get CI working, seems to work pretty well

from sail.

nhooyr avatar nhooyr commented on August 23, 2024

Going to close as it has been a while and I'm not sure if this is relevant anymore.

from sail.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.