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

Added variable os_immutable_fs to oshardening #669

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

millerthegorilla
Copy link

to compensate for immutable filesystems, in particular ostree systems. Updated oshardening/README.md to reflect this

Signed-off-by: James Miller [email protected]

…able filesystems, in particular ostree systems. Updated oshardening/README.md to reflect this

Signed-off-by: James Miller <[email protected]>
@millerthegorilla
Copy link
Author

woot! I think this might work. I made all the changes you request, hope its ok.

@schurzi
Copy link
Contributor

schurzi commented May 3, 2023

superseeds #666

@millerthegorilla
Copy link
Author

millerthegorilla commented May 3, 2023

I am totally new to molecule, but need to learn it to write tests for my roles. I am guessing that coreos or similar needs to be added to the list of distros and a conditional check in molecule when testing os_immutable_fs... Molecule for the next week or so for me, until I can write tests for my own projects, and then I can see if I can make sense of this ...

@millerthegorilla
Copy link
Author

I just added a check that was missing from my initial commit, which I didn't pick up on until I used the os_immutable_fs branch in my code.

Signed-off-by: James Miller <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants