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

As a user of the geopm service I would like an option to disable the use of system dbus #2774

Open
cmcantalupo opened this issue Jan 4, 2023 · 0 comments · May be fixed by #2778
Open

As a user of the geopm service I would like an option to disable the use of system dbus #2774

cmcantalupo opened this issue Jan 4, 2023 · 0 comments · May be fixed by #2778
Labels
feature Description of a new feature as a story

Comments

@cmcantalupo
Copy link
Contributor

cmcantalupo commented Jan 4, 2023 •
edited

Is your feature request related to a problem? Please describe.
When deploying the GEOPM systemd service, the geopmd process may not always have permission to publish the io.github.geopm dbus interface on the system message bus. It may also be the case that the clients may not have permission to to submit requests to the system dbus interface in /run/dbus/system_bus_socket.

Describe the solution you'd like
Provide a command line option to geopmd --session-busthat will create a socket in /run/geopm-service and start a background process running the dbus-daemon running on that session bus. The dbus bus must be configured for multi-user use with the --config option to the daemon. This socket will have world rwx permissions.

Describe alternatives you've considered
Another solution would be to switch to a different message protocol like gRPC. This would easily solve the communication problem, but the credentials provided by dbus would have to be managed differently.

Additional context
Some related work is in this draft PR:
#2771
@cmcantalupo cmcantalupo added the feature Description of a new feature as a story label Jan 4, 2023
cmcantalupo added a commit to cmcantalupo/geopm that referenced this issue Jan 4, 2023
@cmcantalupo
Enable a session bus option for geopmd
98bf934 
- Add a configuration file for dbus-daemon
- Fork a dbus deamon
- Fixes geopm#2774

Signed-off-by: Christopher M. Cantalupo <christopher.m.cantalupo@intel.com>
@cmcantalupo cmcantalupo linked a pull request Jan 4, 2023 that will close this issue
Draft
cmcantalupo added a commit to cmcantalupo/geopm that referenced this issue Jan 4, 2023
@cmcantalupo
Enable a session bus option for geopmd
2e5d56c 
- Add a configuration file for dbus-daemon
- Fork a dbus deamon
- Fixes geopm#2774

Signed-off-by: Christopher M. Cantalupo <christopher.m.cantalupo@intel.com>
cmcantalupo added a commit to cmcantalupo/geopm that referenced this issue Jan 4, 2023
@cmcantalupo
Enable a session bus option for geopmd
2394657 
- Add a configuration file for dbus-daemon
- Fork a dbus deamon
- Fixes geopm#2774

Signed-off-by: Christopher M. Cantalupo <christopher.m.cantalupo@intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature Description of a new feature as a story
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Enable a session bus option for geopmd cmcantalupo/geopm
1 participant
@cmcantalupo