Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

selinux: expand the selinuxpolicy status subresource to include error messages #223

Closed
jhrozek opened this issue Jan 22, 2021 · 3 comments · Fixed by #552
Closed

selinux: expand the selinuxpolicy status subresource to include error messages #223

jhrozek opened this issue Jan 22, 2021 · 3 comments · Fixed by #552
Labels
kind/feature Categorizes issue or PR as related to a new feature. lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale.

Comments

@jhrozek
Copy link
Contributor

jhrozek commented Jan 22, 2021

What would you like to be added:

  • expand the status subresource of the selinuxpolicy object with an error message
  • populate it on errors
  • add an e2e test that tries loading a broken policy

Why is this needed:

the selinux controller already reads errors from selinuxd, but only displays them in the logs. It would be more user-friendly if the error was displayed in the status subresource.
@jhrozek jhrozek added the kind/feature Categorizes issue or PR as related to a new feature. label Jan 22, 2021
@jhrozek jhrozek mentioned this issue Apr 21, 2021
Closed
@fejta-bot
Copy link

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Send feedback to sig-contributor-experience at kubernetes/community.
/lifecycle stale

@k8s-ci-robot k8s-ci-robot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Apr 22, 2021
@jhrozek
Copy link
Contributor Author

jhrozek commented Apr 22, 2021

/remove-lifecycle stale
I'll get to this soon

@k8s-ci-robot k8s-ci-robot removed the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Apr 22, 2021
@fejta-bot
Copy link

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Send feedback to sig-contributor-experience at kubernetes/community.
/lifecycle stale

@k8s-ci-robot k8s-ci-robot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Jul 21, 2021
@jhrozek jhrozek mentioned this issue Jul 27, 2021
Merged
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
kind/feature Categorizes issue or PR as related to a new feature. lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale.
Projects
None yet
Milestone
No milestone
3 participants
@jhrozek @k8s-ci-robot @fejta-bot