Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 1443064

Summary: vdsm.conf isn't being written due to vintage condition
Product: [oVirt] ovirt-host-deploy Reporter: Leon Goldberg <lgoldber>
Component: Plugins.VDSMAssignee: Simone Tiraboschi <stirabos>
Status: CLOSED CURRENTRELEASE QA Contact: Aleksei Slaikovskii <aslaikov>
Severity: high Docs Contact:
Priority: unspecified    
Version: masterCC: bugs, danken, lsvaty, oourfali
Target Milestone: ovirt-4.1.2Keywords: Regression
Target Release: 1.6.4Flags: rule-engine: ovirt-4.1+
rule-engine: blocker+
lsvaty: testing_ack+
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
vdsm.conf wasn't correct rewrote by host-deploy due to bad check of vintage node deployment.
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-05-23 08:16:58 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Integration RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1447670    

Description Leon Goldberg 2017-04-18 12:08:47 UTC
Description of problem:
The function responsible for actually writing vdsm.conf ("_validate") has a vintage condition to it.

How reproducible:
100%

Steps to Reproduce:
1. Add non-vintage host.

Actual results:
vdsm.conf remains commented out.

Expected results:
vdsm.conf is written according to the host's configuration

Comment 1 Dan Kenigsberg 2017-04-19 08:51:57 UTC
Since https://gerrit.ovirt.org/#/c/53531/5/src/plugins/ovirt-host-deploy/vdsm/config.py o-h-d no longer modifies vdsm.conf even if requested to do so in its config.

Comment 2 Red Hat Bugzilla Rules Engine 2017-04-19 12:19:50 UTC
This bug report has Keywords: Regression or TestBlocker.
Since no regressions or test blockers are allowed between releases, it is also being identified as a blocker for this release. Please resolve ASAP.

Comment 3 Aleksei Slaikovskii 2017-05-16 12:00:08 UTC
Verified on ovirt-host-deploy-1.6.4-1.el7ev.noarch