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

Ability to subscribe to PropertiesChanged signal on systemd's dbus interface thru system-observe interface #14890

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

prash813
Copy link

The following changes are meant to include access to 'PropertiesChanged' signal on systemd's dbus interface.

This change enables snap applications with the 'system-observe' plug to subscribe to the 'PropertiesChanged' signal on systemd's DBus interface.

  • template.go-: '/usr/bin/dbus-monitor' has been included in the list, as this executable allows monitoring signals on dbus.
  • system_observe.go-: Adds the necessary DBus API string to the 'systemObserveConnectedPlugAppArmor' constant for the 'PropertiesChanged' signal.
  • system_observe_test.go-: Adds corresponding test case for the new functionality.

…ed' signal on systemd's dbus interface.

This change enables snap applications with the 'system-observe' plug to subscribe
to the 'PropertiesChanged' signal on systemd's DBus interface.
- **template.go**-: '/usr/bin/dbus-monitor' has been included in the list, as this executable allows monitoring signals on dbus.
- **system_observe.go**-: Adds the necessary DBus API string to the   'systemObserveConnectedPlugAppArmor' constant for the 'PropertiesChanged' signal.
- **system_observe_test.go**-: Adds corresponding test case for the new functionality.
Copy link

Hey! prash813 has not signed the Canonical CLA which is required to get this contribution merged on this project.

Please head over to https://ubuntu.com/legal/contributors to read more about 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.

1 participant