/etc/profile.d/libvirt-uri.sh (LIBVIRT_DEFAULT_URI setting) overrides configured default

[Daniel Richard G.]

This concerns libvirt-clients 8.0.0-1ubuntu7.5 in Ubuntu 22.04/jammy.

The /etc/profile.d/libvirt-uri.sh profile script sets LIBVIRT_DEFAULT_URI to a value of "qemu:///system" or "xen:///". This setting occurs irrespective of any previous value (so this cannot be overridden by an earlier profile script or environment setting) or setting for uri_default in /etc/libvirt/libvirt.conf .

I wanted the default connection URI to be "qemu:///session", and was wondering why virsh(1) insisted on connecting to the system session despite my having set uri_default in libvirt.conf.

It seems to me that the profile script may no longer be needed in light of the uri_default setting, and should thus be removed. (Notably, the Debian libvirt-clients package does not contain this script.) If there is still a need for it, then it should at least be made more admin-friendly.

[Bryce Harrington (bryce)]

From the debian changelog, it appears this script was added to perform an automated switch when xen is in use. It was added prior to 2015, and does not appear to have been substantially modified since then. So, I suspect a question may be if this xen use case is still important. If not, then your suggested course of action could make sense.

OTOH, if the automatic behavior with xen is important, then it sounds like what is needed would be either a way to ensure user settings in /etc/libvirt/libvirt.conf take precedence, or to provide administrators with a straightforward way to disable the automated behavior.

[Daniel Richard G. (skunk)]

Wouldn't it make sense to have a libvirt-clients package script set the appropriate value (Xen or no Xen) in libvirt.conf, perhaps with an associated debconf question? The current approach doesn't seem consistent with normal Debian/Ubuntu practice.

[Bryce Harrington (bryce)]

Hi Daniel, thanks for the quick reply. I am not personally familiar enough with libvirt to say, but offhand I do agree with you that the current approach feels like it could be done more cleanly. I've tagged the bug as 'server-triage-discuss' which adds it to a list of review items for the server team's Wednesday meeting.

[Christian Ehrhardt  (paelzer)]

I think we need to keep qemu:system as the default.
But you are right
- we do no more need xen nowadays
- we want users to be able to override it in non-weird ways

We'll have a look at this when merging a recent libvirt for 23.10

[Lena Voytek (lvoytek)]

I'll add this as part of the libvirt merge

[Lena Voytek (lvoytek)]

Fixed in mantic merge, I'll start the SRUs soon

[Lena Voytek (lvoytek)]

Going to update the fix with a full removal of the script while making sure qemu:///system remains as the default uri_default, setting back to in progress and adding mantic

[Brian Murray (brian-murray)]

Ubuntu 23.04 (Lunar Lobster) has reached end of life, so this bug will not be fixed for that specific release.

[Brian Murray (brian-murray)]

Ubuntu 23.10 (Mantic Minotaur) has reached end of life, so this bug will not be fixed for that specific release.