Initialized to use tito and added spec file #3
Conversation
|
Also tested on a 2.1 and install with this and it seemed to install with no errors. |
| Requires(post): /usr/sbin/semodule, /sbin/restorecon, /usr/sbin/setsebool, /usr/sbin/selinuxenabled, /usr/sbin/semanage | ||
| Requires(post): policycoreutils-python | ||
| Requires(post): selinux-policy-targeted | ||
| Requires(postun): /usr/sbin/semodule, /sbin/restorecon |
There was a problem hiding this comment.
We actually need Require: foreman-selinux with version specifier (current version + 1 - the removal was not yet merged).
If we have no time to wait, then without version for now.
There was a problem hiding this comment.
Doesn't this work just fine with the current foreman-selinux ? I'm not sure i get why the version specifier for 1.8 (i presume)
There was a problem hiding this comment.
We need it for elasticsearch port definition which will prevent foreman policy from loading (katello is not the problematic one in this case). See the referenced PR. We will likely bump the release number to 1.8.0-1 to get there.
|
LGTM, add the require please |
|
Let's wait after FOSDEM, we need to bump version of the foreman package to get it working. Katello package can be only installed AFTER foreman-selinux was properly upgraded (and the port was undefined). |
|
@lzap good with merging? |
|
I haven't tested yet. |
Initialized to use tito and added spec file
No description provided.