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

Enable SELinux on arm64 - #2790

Closed
dm0- wants to merge 12 commits into
coreos:masterfrom
dm0-:arm64-selinux
Closed

Enable SELinux on arm64#2790
dm0- wants to merge 12 commits into
coreos:masterfrom
dm0-:arm64-selinux

Conversation

@dm0-

@dm0- dm0- commented Sep 25, 2017

Copy link
Copy Markdown
Contributor

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-

dm0- commented Sep 25, 2017

Copy link
Copy Markdown
Contributor Author

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

euank commented Sep 25, 2017

Copy link
Copy Markdown
Contributor

@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-

dm0- commented Sep 25, 2017

Copy link
Copy Markdown
Contributor Author

Yes, it's a much bigger set.

@euank

euank commented Nov 16, 2017

Copy link
Copy Markdown
Contributor

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

@ajeddeloh

Copy link
Copy Markdown
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

glevand commented Nov 17, 2017

Copy link
Copy Markdown
Contributor

@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-

dm0- commented Apr 20, 2018

Copy link
Copy Markdown
Contributor Author

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