Michal Skrivanek has posted comments on this change.

Change subject: Implement default configuration values
......................................................................


Patch Set 10:

(1 comment)

out of curiosity - why splitting the conf?

http://gerrit.ovirt.org/#/c/28045/10/configurations/ovirt-guest-agent.ini
File configurations/ovirt-guest-agent.ini:

Line 1: #
Line 2: # rhev-agent.ini - RHEV-agent/Windows configuration file
since you're splitting and introducing a new file you may as well fix the 
header….
Line 3: #
Line 4: 
Line 5: [general]
Line 6: 


-- 
To view, visit http://gerrit.ovirt.org/28045
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I5ce9d457e826448aad205ce4abda5dc4b0210804
Gerrit-PatchSet: 10
Gerrit-Project: ovirt-guest-agent
Gerrit-Branch: master
Gerrit-Owner: Vinzenz Feenstra <[email protected]>
Gerrit-Reviewer: Francesco Romani <[email protected]>
Gerrit-Reviewer: Michal Skrivanek <[email protected]>
Gerrit-Reviewer: Vinzenz Feenstra <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to