Bug 1094936
Summary: | vdsm: script and/or trigger should not directly enable systemd units | ||
---|---|---|---|
Product: | [oVirt] vdsm | Reporter: | Andy Lutomirski <luto> |
Component: | General | Assignee: | Yaniv Bronhaim <ybronhei> |
Status: | CLOSED CURRENTRELEASE | QA Contact: | Petr Kubica <pkubica> |
Severity: | high | Docs Contact: | |
Priority: | unspecified | ||
Version: | --- | CC: | bazulay, bugs, danken, dougsland, ecohen, fsimonce, gklein, lsurette, mgoldboi, oourfali, rbalakri, s.kieske, ybronhei, yeylon |
Target Milestone: | ovirt-3.6.0-rc | Flags: | rule-engine:
ovirt-3.6.0+
ylavi: planning_ack+ rule-engine: devel_ack+ rule-engine: testing_ack+ |
Target Release: | 4.17.8 | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Whiteboard: | infra | ||
Fixed In Version: | vdsm-4.17.0-1198.git6ede99a | Doc Type: | Bug Fix |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2016-01-13 14:40:30 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | Infra | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 1090684 |
Description
Andy Lutomirski
2014-05-06 17:23:51 UTC
Vdsm does it during installation, see vdsm.spec in %post Mooli, please check the macroized scriptlet that we can use instead As I understand from reading: https://fedoraproject.org/wiki/Packaging:ScriptletSnippets#Systemd vdsmd needs FESCo approval for starting by default, since it is persistent and listens to a network socket. We do not need the same as far as I understand for spervdsmd/vdsm-reg: https://fedoraproject.org/wiki/Starting_services_by_default If there is agreement we need to file a bug against systemd to be included in http://pkgs.fedoraproject.org/cgit/systemd.git/plain/90-default.preset and explain why vdsm needs to be enabled by default: (notice libvirtd.service is present there under "Network facing" section) much like https://bugzilla.redhat.com/show_bug.cgi?id=885406 This is AFAIU regardless of starting via Macroized scriptlets/Manual scriptlets for which I will submit a patch soon. Doing some more reading: http://www.freedesktop.org/software/systemd/man/systemd.preset.html This explains well why this is needed: http://freedesktop.org/wiki/Software/systemd/Preset/ (To allow enablement policy per distribution/site and not per package.) Two important relevant commits should be considered: spec: use the fedora systemd macros http://gerrit.ovirt.org/#/c/12285/ spec: fix that vdsmd.service is not enabled after installation http://gerrit.ovirt.org/#/c/16754/ This one is relevant as well. spec: restart systemd-modules-load.service on post installation http://gerrit.ovirt.org/#/c/16610/ Oved, complying with this Fedora guideline makes sense, and should be simply done in 3.6. We should drop all `/bin/systemctl enable` from the spec, and move them to a vdsm-tool configurator. Please reconsider doing this in 3.6. Bug tickets that are moved to testing must have target release set to make sure tester knows what to test. Please set the correct target release before moving to ON_QA. Verified in vdsm-4.17.15-0.el7ev.noarch oVirt 3.6.0 has been released, closing current release |