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

Add notification events for add/remove device definitions #27

Closed
jonner opened this issue Jul 13, 2020 · 6 comments
Closed

Add notification events for add/remove device definitions #27

jonner opened this issue Jul 13, 2020 · 6 comments

Comments

@jonner
Copy link
Member

jonner commented Jul 13, 2020

A management app like libvirt would like to be able to monitor new or removed mediated device definitions. Currently, we have resort to something like periodically polling mdevctl list --defined to detect new or removed devices. It would be nice if we could instead listen for events indicating that a device is defined or undefined.

I envision something similar in concept to udev 'add' or 'remove' events for regular devices. Perhaps DBus could be the mechanism for announcing these events? I'm open to other ideas.

@cohuck
Copy link
Member

cohuck commented Jul 14, 2020

I'm wondering whether we can combine this with the 'callout' interface suggested in #20 -- we need a mechanism to invoke scripts etc. for that as well.

A post-create/post-remove callout should basically cover the add/remove events. If we do not introduce a more formal message via DBus, would the ability to invoke a custom script be already enough to process state changes?

@jonner
Copy link
Member Author

jonner commented Jul 15, 2020 via email

@cohuck
Copy link
Member

cohuck commented Oct 25, 2021

Looking through old issues... I'm wondering whether the callout feature as it is merged does what you need?

@collinwalling
Copy link
Contributor

A notification script should hopefully suffice? :)

@cohuck
Copy link
Member

cohuck commented Oct 25, 2021

I'm mostly asking whether we can close this issue :)

@jonner
Copy link
Member Author

jonner commented Oct 26, 2021

yep, I think we can close it.

@jonner jonner closed this as completed Oct 26, 2021
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

No branches or pull requests

3 participants