This branch rocks it hard; thanks a million for this much-needed cleanup.  I&#39;ll review and if all looks well, should be in master by EOD.<div><br><div class="gmail_quote">On Fri, Feb 8, 2013 at 3:11 PM, S.Çağlar Onur <span dir="ltr">&lt;<a href="mailto:caglar@10ur.org" target="_blank">caglar@10ur.org</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">For starters I just pushed a branch called whitespace to upstream (diffstat follows) to make pylint happy. It also converts some files to UNIX EOF as well, please feel free to merge it to master it you think it&#39;s appropriate.<div>


<br></div><div><div>~/Projects/fedoraproject/func(master)] git diff whitespace --stat</div><div> func/minion/AuthedXMLRPCServer.py   |   2 +-</div><div> func/minion/modules/delegation.py   |   2 +-</div><div> func/minion/modules/func_module.py  |   2 +-</div>


<div> func/minion/modules/getfile.py      |   1 +</div><div> func/minion/modules/nagios_check.py |   6 +-</div><div> func/minion/modules/portinfo.py     |  16 +--</div><div> func/minion/modules/users.py        |   2 +-</div>


<div> func/minion/modules/yumcmd.py       |   4 +-</div><div> func/minion/server.py               |   8 +-</div><div> func/module_loader.py               |   4 +-</div><div> func/overlord/client.py             |  26 ++---</div>


<div> func/overlord/delegation_tools.py   |   6 +-</div><div> func/overlord/modules/getfile.py    |  18 ++--</div><div> func/utils.py                       |   6 +-</div><div> func/yaml/__init__.py               |  46 ++++----</div>


<div> func/yaml/dump.py                   | 607 +++++++++++++++++++++++++++++++++++++++++++++++++++----------------------------------------------------</div><div> func/yaml/implicit.py               | 103 +++++++++---------</div>


<div> func/yaml/inline.py                 |  87 +++++++--------</div><div> func/yaml/klass.py                  | 108 +++++++++----------</div><div> func/yaml/load.py                   | 665 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++---------------------------------------------------------</div>


<div> func/yaml/stream.py                 | 398 ++++++++++++++++++++++++++++++++++----------------------------------</div><div> func/yaml/ypath.py                  | 938 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++--------------------------------------------------------------------------------</div>


<div> 22 files changed, 1531 insertions(+), 1524 deletions(-)</div></div></div><div class="gmail_extra"><div><div class="h5"><br><br><div class="gmail_quote">On Fri, Feb 8, 2013 at 3:03 PM, S.Çağlar Onur <span dir="ltr">&lt;<a href="mailto:caglar@10ur.org" target="_blank">caglar@10ur.org</a>&gt;</span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hey Steve,<div><br></div><div>Thanks for quick response :) I&#39;ll give it a try and will let you know if anything comes up.</div>


<div><br></div><div>Cheers,</div></div><div><div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Fri, Feb 8, 2013 at 2:59 PM, Steve Salevan <span dir="ltr">&lt;<a href="mailto:steve@tumblr.com" target="_blank">steve@tumblr.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">



Most of these were changes which were intentionally merged in as part of the ixs-delegation merge, which was outstanding for some time and contained some useful bugfixes and improvements to the delegation codebase, particularly around delegation path determination and parallelization during delegation calls.  Reviewing the diff you supplied, it appears that most of the code in it covered these changes; however, testsingle.py is definitely some old code and came back, like a zombie, with this merge, so I&#39;ve removed it from master.<div>




<br></div><div>At the moment, it looks as if we&#39;re back to a happy state; thanks much for your vigilance, be sure to let me know if you see anything else that looks strange.<div><div><div><div><div><br><div class="gmail_quote">




On Fri, Feb 8, 2013 at 1:41 PM, S.Çağlar Onur <span dir="ltr">&lt;<a href="mailto:caglar@10ur.org" target="_blank">caglar@10ur.org</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">




<div dir="ltr">Hmm it still looks like there are some artifacts in the repo. My local copy was at 4cce9c5b796a0f1f2205deb391b088d7a3e066b6 before the merge and now it shows following diff<div><br></div><div><div>[caglar@kokai:~/Projects/fedoraproject/func(master)] git diff 4cce9c5b796a0f1f2205deb391b088d7a3e066b6 | diffstat</div>






<div> minion/AuthedXMLRPCServer.py  |    2 </div><div> minion/modules/delegation.py  |   47 ++++++++-----</div><div> minion/modules/func_module.py |    2 </div><div> minion/modules/testsingle.py  |  147 ++++++++++++++++++++++++++++++++++++++++++</div>






<div> overlord/client.py            |   39 ++++++-----</div><div> overlord/cmd_modules/call.py  |    2 </div><div> overlord/delegation_tools.py  |   32 +++++----</div><div> overlord/mapper.py            |   10 ++</div><div>






 overlord/sslclient.py         |    2 </div><div> utils.py                      |    2 </div><div> yaml/implicit.py              |    5 +</div><div> 11 files changed, 237 insertions(+), 53 deletions(-)</div></div><div><br>






</div><div>I&#39;m not %100 sure those are the things we want in the master.</div></div><div class="gmail_extra"><div><div><br><br><div class="gmail_quote">On Fri, Feb 8, 2013 at 1:14 PM, Steve Salevan <span dir="ltr">&lt;<a href="mailto:steve@tumblr.com" target="_blank">steve@tumblr.com</a>&gt;</span> wrote:<br>






<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hey man,<div>Thanks hugely for discovering this; I have corrected the merge conflict and pushed a commit up to master to unbreak Func.<br>






<br><div class="gmail_quote"><div><div>On Fri, Feb 8, 2013 at 12:55 PM, S.Çağlar Onur <span dir="ltr">&lt;<a href="mailto:caglar@10ur.org" target="_blank">caglar@10ur.org</a>&gt;</span> wrote:<br>
</div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div><div dir="ltr">Steve,<div><br></div><div>It looks like you accidentally merged an old branch (<a href="http://git.fedorahosted.org/cgit/func.git/commit/" style="font-size:13px;color:rgb(52,101,164);font-family:Cantarell,&#39;Droid Sans&#39;,&#39;DejaVu Sans&#39;,Arial,sans-serif;line-height:19px;white-space:nowrap;background-color:rgb(238,238,238)" target="_blank">xs-delegation</a>) into master [1] and it made things bad as commit contains non python code.</div>









<div><br></div><div>[1] <a href="http://git.fedorahosted.org/cgit/func.git/commit/?id=9a0ae4446ff1b9ad7bccc57ddfb3c765fa68ebbf" target="_blank">http://git.fedorahosted.org/cgit/func.git/commit/?id=9a0ae4446ff1b9ad7bccc57ddfb3c765fa68ebbf</a><br>









</div><div><br clear="all"><div>Cheers,</div><span><font color="#888888">-- <br>S.Çağlar Onur &lt;<a href="mailto:caglar@10ur.org" target="_blank">caglar@10ur.org</a>&gt;
</font></span></div></div>
<br></div></div>_______________________________________________<br>
func mailing list<br>
<a href="mailto:func@lists.fedorahosted.org" target="_blank">func@lists.fedorahosted.org</a><br>
<a href="https://lists.fedorahosted.org/mailman/listinfo/func" target="_blank">https://lists.fedorahosted.org/mailman/listinfo/func</a><br></blockquote></div>--<div>Steve Salevan</div><div><a href="mailto:steve@tumblr.com" target="_blank">steve@tumblr.com</a></div>








</div>
<br>_______________________________________________<br>
func mailing list<br>
<a href="mailto:func@lists.fedorahosted.org" target="_blank">func@lists.fedorahosted.org</a><br>
<a href="https://lists.fedorahosted.org/mailman/listinfo/func" target="_blank">https://lists.fedorahosted.org/mailman/listinfo/func</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br></div></div>S.Çağlar Onur &lt;<a href="mailto:caglar@10ur.org" target="_blank">caglar@10ur.org</a>&gt;
</div>
<br>_______________________________________________<br>
func mailing list<br>
<a href="mailto:func@lists.fedorahosted.org" target="_blank">func@lists.fedorahosted.org</a><br>
<a href="https://lists.fedorahosted.org/mailman/listinfo/func" target="_blank">https://lists.fedorahosted.org/mailman/listinfo/func</a><br>
<br></blockquote></div><br>--<div>Steve Salevan</div><div><a href="mailto:steve@tumblr.com" target="_blank">steve@tumblr.com</a></div>
</div></div></div></div></div></div>
<br>_______________________________________________<br>
func mailing list<br>
<a href="mailto:func@lists.fedorahosted.org" target="_blank">func@lists.fedorahosted.org</a><br>
<a href="https://lists.fedorahosted.org/mailman/listinfo/func" target="_blank">https://lists.fedorahosted.org/mailman/listinfo/func</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br>S.Çağlar Onur &lt;<a href="mailto:caglar@10ur.org" target="_blank">caglar@10ur.org</a>&gt;
</div>
</div></div></blockquote></div><br><br clear="all"><div><br></div></div></div><span class="HOEnZb"><font color="#888888">-- <br>S.Çağlar Onur &lt;<a href="mailto:caglar@10ur.org" target="_blank">caglar@10ur.org</a>&gt;
</font></span></div>
<br>_______________________________________________<br>
func mailing list<br>
<a href="mailto:func@lists.fedorahosted.org">func@lists.fedorahosted.org</a><br>
<a href="https://lists.fedorahosted.org/mailman/listinfo/func" target="_blank">https://lists.fedorahosted.org/mailman/listinfo/func</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br>--<div>Steve Salevan</div><div><a href="mailto:steve@tumblr.com" target="_blank">steve@tumblr.com</a></div>
</div>