Bug #13502: Build failure on RHEL 7.2 - docker moved to separate package - SELinux - Foreman

Actions

Copy link

Bug #13502

closed

Build failure on RHEL 7.2 - docker moved to separate package

Added by Lukas Zapletal almost 9 years ago. Updated over 6 years ago.

Status:

Closed

Priority:

Normal

Assignee:

Lukas Zapletal

Category:

Packaging

Target version:

1.11.1

Difficulty:

Triaged:

Bugzilla link:

Pull request:

https://github.com/theforeman/foreman-selinux/pull/56, https://github.com/theforeman/foreman-packaging/pull/1006

Fixed in Releases:

Found in Releases:

Red Hat JIRA:

Description

I need reverse order of optional_policy/ifndef introduced in #11934 because it fails to build:

Compiling targeted foreman module
foreman.te:413: Warning: miscfiles_read_certs() has been deprecated, please use miscfiles_read_generic_certs() instead.
foreman.te":377:ERROR 'syntax error' at token 'docker_stream_connect' on line 7954:
            docker_stream_connect(passenger_t)
#line 377
/usr/bin/checkmodule:  error(s) encountered while parsing configuration
/usr/bin/checkmodule:  loading policy configuration from tmp/foreman.tmp
make[1]: Leaving directory `/builddir/build/BUILD/foreman-selinux-1.11.0/local-tmp-foreman'
make[1]: *** [tmp/foreman.mod] Error 1
make: *** [foreman.pp] Error 2

Actions

Copy link

Updated by Lukas Zapletal almost 9 years ago

After taking second look, it might not be caused by the optional block, investigating. I guess SELinux broke the API again.

Actions

Copy link

Updated by Lukas Zapletal almost 9 years ago

Subject changed from Build failure on RHEL 7.2 due to optional policy block: to Build failure on RHEL 7.2 - docker moved to separate package

Docker policy was removed from RHEL 7.2 base policy and moved to docker-selinux subpackage:

https://github.com/lzap/centos-selinux-interface/blob/master/centos-7.1-vs-7.2.diff
http://mirror.centos.org/centos/7.2.1511/extras/x86_64/Packages/

The package is is extras which is enabled by default in CentOS 7.

Therefore we only need to add a build RPM dependency. For runtime this is not required as this is in optional block, therefore this will only be loaded when docker is present I believe. If this is not true, then we need to add "rhel-7-server-extras-rpms" to our installation instructions on RHEL 7.

Actions

Copy link