Bug 1527052

Summary: no assigned port to container_port_t label
Product: Red Hat Enterprise Linux 7 Reporter: Lukas Pramuk <lpramuk>
Component: dockerAssignee: Daniel Walsh <dwalsh>
Status: CLOSED WONTFIX QA Contact: atomic-bugs <atomic-bugs>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 7.4CC: amurdaca, dwalsh, lsm5, lzap, tsweeney
Target Milestone: rcKeywords: Extras
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2020-06-09 21:34:07 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Lukas Pramuk 2017-12-18 12:43:24 UTC
Description of problem:
There is no assigned port number to the container_port_t label.
Before container_port_t has at least 2375/tcp port assigned.
Is this a regression or intentional change and port assigment is now left to be done by sysadmin? documented?

Version-Release number of selected component (if applicable):
container-selinux-2.33-1.git86f33cd.el7.noarch

How reproducible:
deterministic

Steps to Reproduce:
1. Install container-selinux 
# yum install container-selinux
...
# semodule -l | grep -e container -e docker
container	2.33.0

2. List defined port labels 
# semanage port -l |grep -e container -e docker
<empty>

Actual results:
no port defined for container_port_t label

Expected results:
2375/tcp (and 2376/tcp?) defined for container_port_t label


Additional info:
# seinfo -xtcontainer_port_t
   container_port_t
      port_type
   Aliases
      docker_port_t

>>> defined_port_type is missing

Comment 2 Daniel Walsh 2017-12-18 15:34:04 UTC
Franticek add this 

semanage port -a -t container_port_t -P tcp 2375
TO docker post install

Comment 3 Lukas Zapletal 2018-01-08 10:35:27 UTC
Just FYI this port is already assigned to foreman_container_port_t on Satellite 6 installations, therefore your command will fail on these deployments. I recommend to grep port list before adding the port.

I wish semanage allowed "add-when-missing" or "ignore-when-present" options, grepping is hard as ports can be specified also as ranges.

Comment 4 Lukas Zapletal 2018-01-08 10:45:12 UTC
Here is how we do it in Satellite 6:

https://github.com/theforeman/foreman-selinux/blob/develop/foreman-selinux-enable#L23-L27

Comment 6 Tom Sweeney 2020-06-09 21:34:07 UTC
We have no plans to ship another version of Docker at this time. RHEL7 is in final support stages where only security fixes will get released.  Customers should move to use Podman which is available starting in RHEL 7.6.