Skip to content
This repository has been archived by the owner on Sep 18, 2020. It is now read-only.

Enable SELinux on arm64 #2790

Closed
wants to merge 12 commits into from
Closed

Enable SELinux on arm64 #2790

wants to merge 12 commits into from

Conversation

dm0-
Copy link
Contributor

@dm0- dm0- commented Sep 25, 2017

I'm creating these PRs to build test images. Just about all of these packages could use updating, but this just attempts to get them working as is first.

@dm0-
Copy link
Contributor Author

dm0- commented Sep 25, 2017

The current state of this is that everything builds, the userland tools work, and all kola tests pass on arm64. The policy itself may not be built or installed correctly, so it's not restricting everything it should. The journal says SELinux: the above unknown classes and permissions will be allowed. So, this should be investigated before merging, but the PRs can be used as is for building test images with clean kola runs.

@euank
Copy link
Contributor

euank commented Sep 25, 2017

@dm0- we have:

SELinux:  Class cap_userns not defined in policy.
SELinux:  Class cap2_userns not defined in policy.
SELinux: the above unknown classes and permissions will be allowed

on amd64.

Do we have a different set on arm?

We should probably fix both of them and have a kola test that would fail on that message tbh.

@dm0-
Copy link
Contributor Author

dm0- commented Sep 25, 2017

Yes, it's a much bigger set.

@euank
Copy link
Contributor

euank commented Nov 16, 2017

@ajeddeloh was looking at possibly updating our amd64 selinux policy. @ajeddeloh, want to review this as well?

@ajeddeloh
Copy link
Contributor

Sure, For the commits changing configuration flags (e.g. f722cbc) can you add a commit body about why that change was needed? It would be helpful for when I go through and update these pacakges in the future as well.

@glevand
Copy link
Contributor

glevand commented Nov 17, 2017

@ajeddeloh @euank I've already done some work to update the selinux related packages. Let's try to minimize duplicated work. See:
https://github.com/glevand/coreos--coreos-overlay/commits/master
https://github.com/glevand/coreos--portage-stable/commits/master

@dm0-
Copy link
Contributor Author

dm0- commented Apr 20, 2018

Replaced by #3100 and #3155

@dm0- dm0- closed this Apr 20, 2018
@dm0- dm0- deleted the arm64-selinux branch April 20, 2018 22:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants