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

transient: Warn or disallow changes outside /usr and /etc #2199

Open
evan-goode opened this issue Jan 21, 2025 · 1 comment
Open

transient: Warn or disallow changes outside /usr and /etc #2199

evan-goode opened this issue Jan 21, 2025 · 1 comment
Labels
Priority:LOW RFE Request for Enhancement (as opposed to a bug) triaged

Comments

@evan-goode
Copy link
Member

@cgwalters said #2195 (comment):

At some point we are going to need to fix the problem that dnf upgrade should probably error out if an attempt is made to change nontrivial things outside of /usr and probably /etc. The key ones here are kernel (needs deeper integration, ref coreos/rpm-ostree#5135 ) and selinux-policy (ref https://bugzilla.redhat.com/show_bug.cgi?id=1290659 ) and shim/grub (ref https://github.com/coreos/bootupd/ )

Ideally we error before we even start an install; if we download filelists this would be pretty easy; without filelists maybe we should have a Provides that flags these special cases? Or we could inject into the base image something that denylists them in the dnf config?

@evan-goode
Copy link
Member Author

I'd prefer something based on filelists rather than doing this on a per-package level. That way, third-party packages will be handled correctly too. Either way, we are going to need to have some list, either of "untouchable paths" or "forbidden packages" that we'll need to keep updated. The list of paths is probably going to change less often than the list of packages.

Implementation-wise it should be easy to have persistence=transient imply loading the filelists.

And there should probably be some way for the user to override this check for testing. (Maybe that way will just be to run ostree admin unlock and not use --transient).

@ppisar ppisar added triaged Priority:LOW RFE Request for Enhancement (as opposed to a bug) labels Jan 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority:LOW RFE Request for Enhancement (as opposed to a bug) triaged
Projects
None yet
Development

No branches or pull requests

2 participants