From Barak Korren <bkorren(a)redhat.com>:
Barak Korren has submitted this change and it was merged.
Change subject: Fix mock_genconfig nested dict settings
......................................................................
Fix mock_genconfig nested dict settings
The latset "fix" to mock_genconfig caused if to destroy the default
values in mock's nested configuration structures, when the user attempts
to customize some values in those structures via the root configuration
file. This made the generated configuration crash mock.
This fix ensures the generated configuration preserves the default
configuration values in nested structures when those are not chaneged
directly.
Change-Id: I7459ae9acf558283502c1b06b3f535ac7093f437
Jira-Ticket:
https://ovirt-jira.atlassian.net/browse/OVIRT-939
Signed-off-by: Barak Korren <bkorren(a)redhat.com>
---
M mock_configs/mock_genconfig
1 file changed, 13 insertions(+), 0 deletions(-)
Approvals:
Eyal Edri: Looks good to me
Barak Korren: Verified; Looks good to me, approved; Ready for merge
Jenkins CI: Passed CI tests
--
To view, visit
https://gerrit.ovirt.org/68503
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I7459ae9acf558283502c1b06b3f535ac7093f437
Gerrit-PatchSet: 2
Gerrit-Project: jenkins
Gerrit-Branch: master
Gerrit-Owner: Barak Korren <bkorren(a)redhat.com>
Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com>
Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com>
Gerrit-Reviewer: David Caro <david(a)dcaro.es>
Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com>
Gerrit-Reviewer: Gil Shinar <gshinar(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nadav Goldin <ngoldin(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Shlomo Ben David <sbendavi(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>