<div dir="ltr">Please make sure all the changes are noted in v0.26 release notes.<div>Y.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Sep 25, 2016 at 9:02 AM, Nadav Goldin <span dir="ltr">&lt;<a href="mailto:ngoldin@redhat.com" target="_blank">ngoldin@redhat.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi all,<br>
As a side-effect of my proposed PR[1], 3 backward-incompatible<br>
changes will be added:<br>
<br>
variables names in lago.conf<br>
------------------------------<wbr>------<br>
1. &#39;default_ssh_user&#39;, changed to: &#39;ssh_user&#39;, value: no change.<br>
2. &#39;default_ssh_password, changed to: &#39;ssh_password&#39;, value: no change.<br>
3. &#39;template_default_repo&#39;, changed to: &#39;template_repo_name&#39;(same as<br>
the existing CLI), value: was never set.<br>
The above 3 were hard-coded in config.py&#39;s defaults dictionary and<br>
never exposed externally. They had no documents, but, if a user knew<br>
about them he could have added them manually to lago.conf<br>
<br>
The reason for changing is that the PR unifies the configuration files<br>
and CLI parameters. So keeping the &#39;default&#39; prefix is misleading and<br>
will lead to awkward results, such as:<br>
lago --default-ssh-user root --default-ssh-password ....<br>
instead of:<br>
lago --ssh-user root --ssh-password ...<br>
<br>
<br>
&#39;template_*&#39; section<br>
-------------------------<br>
&#39;template_*&#39; parameters which are used by lago init command, will be<br>
move to [init] section in lago.conf.<br>
<br>
<br>
lago.conf location<br>
-----------------------<br>
1. Drop /etc/lago.d/lago.conf in favour of /etc/lago/lago.conf<br>
2. The installed file from the RPM will be all commented out(i.e.<br>
installed just to use as a reference).<br>
3. /etc/lago/lago.conf will be the least important in the<br>
hierarchy(you can read more details about the hierarchy in the PR)<br>
I&#39;ll note that Lago will be fully functional without any configuration<br>
file at all.<br>
<br>
These changes will possibly affect only already-customized<br>
lago.conf files. But, changing to the new configuration format will be<br>
easy, using the &#39;generate-config&#39; command:<br>
mkdir -p ~/.config/lago &amp;&amp; lago generate-config &gt; ~/.config/lago/lago.conf<br>
<br>
<br>
The changes are needed as the the PR mainly deals with standardizing<br>
the configuration loading module, so it is a good time for clean-up,<br>
and dropping misleading variable names(which were probably good at the<br>
time written). Second, I&#39;m not sure how widely these parameters were<br>
used, as some of them didn&#39;t have any docs. For instance I couldn&#39;t<br>
find any evidence of usage for &#39;template_default_repo&#39;(the CLI<br>
parameter is the same as it was, &#39;template_repo_name&#39;).<br>
<br>
<br>
<br>
Some possible alternatives:<br>
1. Issue a warning about variables name change, and use both, then in<br>
2-3 versions from now, completely remove them.<br>
2. lago.conf location: Not a major issue, as far as I know, &#39;.d&#39;<br>
suffix was used historically for distinguishing a &#39;directory&#39; from a<br>
single configuration file, so having &#39;/etc/lago/&#39; is better(and<br>
keeping lago.d if someday we will need to load other user-defined<br>
configurations). But this can be easily changed.<br>
<br>
<br>
If there is no request for the alternatives or objection, I&#39;ll try to<br>
get it merged for the next version and add a short summary of this<br>
message to the release email.<br>
<br>
<br>
Thanks,<br>
<br>
Nadav.<br>
<br>
[1] <a href="https://github.com/lago-project/lago/pull/324" rel="noreferrer" target="_blank">https://github.com/lago-<wbr>project/lago/pull/324</a><br>
______________________________<wbr>_________________<br>
lago-devel mailing list<br>
<a href="mailto:lago-devel@ovirt.org">lago-devel@ovirt.org</a><br>
<a href="http://lists.ovirt.org/mailman/listinfo/lago-devel" rel="noreferrer" target="_blank">http://lists.ovirt.org/<wbr>mailman/listinfo/lago-devel</a><br>
</blockquote></div><br></div>