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.

Sign up for GitHub

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

Jump to bottom

v23/security: add a notification mechanism for blessing root changes #236

Open
cosnicolaou opened this issue Nov 5, 2021 · 0 comments
Open

v23/security: add a notification mechanism for blessing root changes #236

cosnicolaou opened this issue Nov 5, 2021 · 0 comments

Comments

@cosnicolaou
Copy link
Contributor

There is currently no way for a process to be notified (other than polling) of changes to its blessing roots, whilst there is such a mechanism for blessings themselves. See the TestDaemonMode test in x/ref/lib/security which currently polls for changes.
@cosnicolaou cosnicolaou mentioned this issue Nov 5, 2021
Merged
cosnicolaou added a commit that referenced this issue Nov 6, 2021
@cosnicolaou
x/ref/lib/security,x/runtime: fix flaky tests, including TestDaemonMo…
6fa340f 
…de and overly tight timeouts in RPC tests. (#235)

This PR fixes a flaky test by waiting for the expected changes to take place. Issue #236 calls for a better mechanism for doing so.

Tight timeouts are extended in the rpc tests.

Also, add the O_CREATE flag when creating read-only locks to cleanup unnecessary complexity in the code that uses the internal lockedfile package.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@cosnicolaou