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

As a User, I want to have the ability to ensure that builder is able to always build despite the choose container solution. #86

Open
Akendo opened this issue May 21, 2024 · 4 comments

Comments

@Akendo
Copy link
Contributor

Akendo commented May 21, 2024

As a first-time user, I had difficulty building the project. My first impression was that I did something wrong. Therefore, I switched from docker to podman which lead to a set of unexpected problems since docker creating artifacts with root permission where podman doesn't.

To solve this, we should keep track of the used container solutions and make sure that the permissions are correct at given time.

@fwilhe
Copy link
Member

fwilhe commented May 21, 2024

Thanks for opening that issue. I think the problem needs solving, I'm not sure if I agree with the proposed solution.

I feel like keeping track of how the builder was invoked before will add hard to manage complexity.

Better documentation and/or error messages for common scenarios might also help a lot here, like testing for ownership of certain files.

@Akendo
Copy link
Contributor Author

Akendo commented May 22, 2024

@fwilhe purposed some food for thoughts: He's uncertain about the complexity we might introduce by keeping track of the state. @nkraetzschmar might add some ideas as well.

@fwilhe
Copy link
Member

fwilhe commented May 22, 2024

Question, not sure how it works exactly: People might use docker with very different setups, i.e. they might always prefix it with sudo to avoid having to add their own account to the docker group. Does that influence the behaviour you described? The debian wiki says sudo is preferred, I always used to add my user to the group, I'm not sure if both setups produce the issue you described.

My suggestion would be to look for good assertions on 'if this file exists it needs to be writable' and use this to bail out with a better error message. The user might sill choose how to resolve, i.e. build with docker again instead of podman, or delete the .build directory, or selectively chown files.

@fwilhe fwilhe closed this as completed May 22, 2024
@fwilhe
Copy link
Member

fwilhe commented May 22, 2024

wtf did not want to close the issue

@fwilhe fwilhe reopened this May 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants