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

tuned-ppd: Support the new UPower PPD namespace #684

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

Conversation

zacikpa
Copy link
Contributor

@zacikpa zacikpa commented Sep 3, 2024

Since version 0.20, power-profiles-daemon exposes
its DBus interface under "org.freedesktop.UPower.PowerProfiles" alongside the previous name. To be compatible with PPD, this commit implements the same change into tuned-ppd.

Resolves: #683

Since version 0.20, power-profiles-daemon exposes
its DBus interface under "org.freedesktop.UPower.PowerProfiles"
alongside the previous name. To be compatible with PPD, this commit
implements the same change into tuned-ppd.

Resolves: redhat-performance#683
@superm1
Copy link
Contributor

superm1 commented Sep 9, 2024

Please be careful with this, the PPD namespace has other changes that are in flight.
See these:

@zacikpa
Copy link
Contributor Author

zacikpa commented Sep 11, 2024

Thanks for the heads up, @superm1. I just had a look at the proposed power-profiles-daemon changes, but I don't see anything that should be an immediate problem for us.

IIUC, you're adding some new functionality to the existing PPD namespace. As of now, tuned-ppd does not aim to cover the entire functionality - we're mostly interested in what's actively used by power management daemons such as powerdevil or gnome-control-center. The new functionality may or may not be added in the future.

@superm1
Copy link
Contributor

superm1 commented Sep 11, 2024

Thanks for the heads up, @superm1. I just had a look at the proposed power-profiles-daemon changes, but I don't see anything that should be an immediate problem for us.

IIUC, you're adding some new functionality to the existing PPD namespace. As of now, tuned-ppd does not aim to cover the entire functionality - we're mostly interested in what's actively used by power management daemons such as powerdevil or gnome-control-center. The new functionality may or may not be added in the future.

Yeah I mostly worry about both software suites using the same name and interface but then downstream software making assumptions on that.

Just want to avoid stepping on egg shells in the API design where possible.

Maybe it's best to advertise a compatibility level as well? I recall PPD has a version property. Maybe you can use "0.23" to indicate you match the API from 0.23 release?

@zacikpa
Copy link
Contributor Author

zacikpa commented Sep 18, 2024

Maybe it's best to advertise a compatibility level as well? I recall PPD has a version property. Maybe you can use "0.23" to indicate you match the API from 0.23 release?

Thanks, that's a nice suggestion, I added it into another open PR.

@Conan-Kudo
Copy link

Can we please get this landed? Final Freeze is on Tuesday, and this needs to be in before then as things are listening on the UPower namespace exclusively already.

@yarda
Copy link
Contributor

yarda commented Oct 14, 2024

Can we please get this landed? Final Freeze is on Tuesday, and this needs to be in before then as things are listening on the UPower namespace exclusively already.

Thanks for the info, @zacikpa will backport it to Fedora today to land in Fedora on time.

@Conan-Kudo
Copy link

Thanks!

@yarda
Copy link
Contributor

yarda commented Oct 15, 2024

https://bodhi.fedoraproject.org/updates/FEDORA-2024-ceee3c1021 it seems stuck in testing.

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.

Advertise p-p-d compatibility D-Bus interface under UPower namespace
4 participants