-
Notifications
You must be signed in to change notification settings - Fork 158
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
tests: add tests for systemd reading files labeled kubernetes_file_t #1116
Conversation
This builds on #1064 and will require a new |
5edc637
to
be1eff6
Compare
0586220
to
44fc622
Compare
@dustymabe @mike-nguyen tagging you to pop this closer to the top of your queues |
echo "$@" >&2 | ||
exit 1 | ||
} | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would add an additional check here to make sure the file at /etc/kubernetes/envfile
has the selinux label we expect. Otherwise this test isn't really testing what we expect it to.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
one recommendation, otherwise LGTM
Adds a test for the ability of `systemd` to read files labeled with `kubernetes_file_t`. See: https://bugzilla.redhat.com/show_bug.cgi?id=1973418
44fc622
to
a2a83a0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Adds a test for the ability of
systemd
to read fileslabeled with
kubernetes_file_t
.See: https://bugzilla.redhat.com/show_bug.cgi?id=1973418