Bug 1274063
Summary: | vdsm fails to start due to incorrect permissions on /tmp/ovirt.log | ||||||
---|---|---|---|---|---|---|---|
Product: | [oVirt] vdsm | Reporter: | Ivan Bulatovic <combuster> | ||||
Component: | General | Assignee: | Dan Kenigsberg <danken> | ||||
Status: | CLOSED NEXTRELEASE | QA Contact: | Meni Yakove <myakove> | ||||
Severity: | urgent | Docs Contact: | |||||
Priority: | urgent | ||||||
Version: | --- | CC: | bugs, danken, dfediuck, didi, fdeutsch, lveyde, rmartins, sbonazzo, stirabos | ||||
Target Milestone: | ovirt-3.6.1 | Flags: | danken:
ovirt-3.6.z?
danken: ovirt-4.0.0? rule-engine: planning_ack? fdeutsch: devel_ack+ rule-engine: testing_ack? |
||||
Target Release: | 4.17.11 | ||||||
Hardware: | x86_64 | ||||||
OS: | Linux | ||||||
Whiteboard: | network | ||||||
Fixed In Version: | Doc Type: | Bug Fix | |||||
Doc Text: | Story Points: | --- | |||||
Clone Of: | Environment: | ||||||
Last Closed: | 2015-11-18 10:49:30 UTC | Type: | Bug | ||||
Regression: | --- | Mount Type: | --- | ||||
Documentation: | --- | CRM: | |||||
Verified Versions: | Category: | --- | |||||
oVirt Team: | Network | RHEL 7.3 requirements from Atomic Host: | |||||
Cloudforms Team: | --- | Target Upstream Version: | |||||
Embargoed: | |||||||
Attachments: |
|
Description
Ivan Bulatovic
2015-10-21 19:42:53 UTC
Apparently, the mere from ovirtnode import ovirtfunctions logs onto /tmp/ovirt.log. If the first import happened to be in root context, we'd get this error. This bug is not marked for z-stream, yet the milestone is for a z-stream version, therefore the milestone has been reset. Please set the correct milestone or add the z-stream flag. Created attachment 1085643 [details]
chown the /tmp/ovirt.log
When I remove ovirt.log from the tmp directory and reboot, vdsm starts successfully. Supervdsmd creates the temporary log file with root privileges, and when vdsm starts with user privileges, it changes ownership of /tmp/ovirt.log to vdsm:kvm.
This is an relict from old ovirt-node times. The logging is setup when ovirtnode.ovirtfunctions is getting imported, which is not good. Normally the import happens because of persistence, in that case ovirt.node.utils.fs.Config can be used these days, which should not suffer the import problems. Target release should be placed once a package build is known to fix a issue. Since this bug is not modified, the target version has been reset. Please use target milestone to plan a fix for a oVirt release. (In reply to Fabian Deutsch from comment #6) > > Normally the import happens because of persistence, in that case > ovirt.node.utils.fs.Config can be used these days, which should not suffer > the import problems. Yes, forcing import from ovirt.node.utils.fs (commenting out importing from ovirtnode) in vdsm/utils.py does not prevent mom-vdsm and vdsm units from starting anymore. Thanks Fabian, I caught this by accident, thought I should report it (even though I don't fully understand implications of it all, yet). Ivan, cool, nice to hear that it worked. Moving this over to vdsm where the patch resides. |