Skip to content

Conversation

@evgeni
Copy link
Member

@evgeni evgeni commented Jun 24, 2025

Since pulpcore 3.13 the worker has an own "binary" (Python entrypoint), that is correctly labeled as pulpcore_exec_t (since pulpcore-selinux 2.0.1) and thus the workaround with an wrapper is not necessary anymore.

It's actually harmful, as the wrapper is labeled pulpcore_exec_t, so on execution it auto-transitions into pulpcore_t, but then it wants to execute /usr/bin/pulpcore-worker, which is again pulpcore_exec_t but pulpcore_t is not allowed to transition to pulpcore_exec_t and you get

denied  { execute_no_trans } for comm="pulpcore-worker" path="/usr/bin/pulpcore-worker" scontext=system_u:system_r:pulpcore_t:s0 tcontext=system_u:object_r:pulpcore_exec_t:s0 tclass=file

Fixes: 137128e

Since pulpcore 3.13 the worker has an own "binary" (Python entrypoint),
that is correctly labeled as pulpcore_exec_t (since pulpcore-selinux
2.0.1) and thus the workaround with an wrapper is not necessary anymore.

It's actually harmful, as the wrapper is labeled pulpcore_exec_t, so on
execution it auto-transitions into pulpcore_t, but then it wants to
execute /usr/bin/pulpcore-worker, which is again pulpcore_exec_t but
pulpcore_t is not allowed to transition to pulpcore_exec_t and you get

    denied  { execute_no_trans } for comm="pulpcore-worker" path="/usr/bin/pulpcore-worker" scontext=system_u:system_r:pulpcore_t:s0 tcontext=system_u:object_r:pulpcore_exec_t:s0 tclass=file

Fixes: 137128e
Copy link
Member

@ekohl ekohl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On my phone so harder to check. What creates the libexec wrapper? Should we clean those up too?

@evgeni
Copy link
Member Author

evgeni commented Jun 24, 2025

The packaging, will clean up there later, yes.

@evgeni evgeni added the Bug Something isn't working label Jun 24, 2025
@evgeni evgeni merged commit 5e1d0fd into master Jun 24, 2025
21 checks passed
@evgeni evgeni deleted the direct-worker branch June 24, 2025 08:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants