Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update SELinux policy to support linkerd-cni #1162

Merged
merged 1 commit into from
Jan 8, 2024
Merged

Update SELinux policy to support linkerd-cni #1162

merged 1 commit into from
Jan 8, 2024

Conversation

lachlan-smith
Copy link
Contributor

Found some issues with the SELinux policy when trying to chain the Linkerd CNI plugin with Cilium. Determined additional required policy statements by running grep "denied" /var/log/audit/audit.log | audit2allow -M linkerd until no more denied errors logged. Patched in the additional statements to existing SELinux module.

For reference, generated module was:

require {
        type etc_t;
        type container_t;
        class file { create setattr unlink write };
        class dir watch;
}

#============= container_t ==============

#!!!! This avc is allowed in the current policy
allow container_t etc_t:dir watch;

#!!!! This avc is allowed in the current policy
allow container_t etc_t:file { create setattr unlink write };```

@Silvest89
Copy link
Contributor

@mysticaltech

Copy link
Collaborator

@mysticaltech mysticaltech left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @lachlan-smith

@mysticaltech mysticaltech merged commit 00227b5 into kube-hetzner:master Jan 8, 2024
3 checks passed
@mysticaltech
Copy link
Collaborator

@lachlan-smith @Silvest89 Fix released in v2.11.7. Enjoy and don't hesitate to shoot more PRs please, this help is truly appreciate it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants