<div>Thanks @doug and that saves me days!</div><div><includetail><div> </div><div>It's reasonable and I think we should make the private patch for our environment. Thanks again.</div><div><br></div><div>-tobe from UnitedStack</div><div> </div><div style="font:Verdana normal 14px;color:#000;"><div style="FONT-SIZE: 12px;FONT-FAMILY: Arial Narrow;padding:2px 0 2px 0;">------------------ Original ------------------</div><div style="FONT-SIZE: 12px;background:#efefef;padding:8px;"><div id="menu_sender"><b>From: </b> "Doug Shelley"<doug@tesora.com>;</div><div><b>Date: </b> Mon, Jul 6, 2015 07:29 PM</div><div><b>To: </b> "openstack-dev@lists."<openstack-dev@lists.openstack.org>; <wbr></div><div></div><div><b>Subject: </b> Re: [openstack-dev] [trove]The patch from 191859 breaks thecompatibility with python 2.6</div></div><div> </div>




<div>
<div>
<div>Hi,</div>
</div>
</div>
<div><br>
</div>
<div>Python 2.6 support was dropped in Kilo (for all of OpenStack). The patchset you mention was just merged last week and will be in Liberty. Trove is no longer being tested with Python 2.6 so it is likely things like this will continue to occur going forward. </div>
<div><br>
</div>
<div>Regards,</div>
<div>Doug</div>
<div><br>
</div>
<span id="OLK_SRC_BODY_SECTION">
<div style="font-family:Calibri; font-size:11pt; text-align:left; color:black; BORDER-BOTTOM: medium none; BORDER-LEFT: medium none; PADDING-BOTTOM: 0in; PADDING-LEFT: 0in; PADDING-RIGHT: 0in; BORDER-TOP: #b5c4df 1pt solid; BORDER-RIGHT: medium none; PADDING-TOP: 3pt">
<span style="font-weight:bold">From: </span>陈迪豪 <<a href="mailto:chendihao@unitedstack.com" onclick="window.open('https://mail.google.com/mail/?view=cm&tf=1&to=chendihao@unitedstack.com&cc=&bcc=&su=&body=','_blank');return false;">chendihao@unitedstack.com</a>><br>
<span style="font-weight:bold">Reply-To: </span>OpenStack List <<a href="mailto:openstack-dev@lists.openstack.org" onclick="window.open('https://mail.google.com/mail/?view=cm&tf=1&to=openstack-dev@lists.openstack.org&cc=&bcc=&su=&body=','_blank');return false;">openstack-dev@lists.openstack.org</a>><br>
<span style="font-weight:bold">Date: </span>Monday, July 6, 2015 at 6:16 AM<br>
<span style="font-weight:bold">To: </span>OpenStack List <<a href="mailto:openstack-dev@lists.openstack.org" onclick="window.open('https://mail.google.com/mail/?view=cm&tf=1&to=openstack-dev@lists.openstack.org&cc=&bcc=&su=&body=','_blank');return false;">openstack-dev@lists.openstack.org</a>><br>
<span style="font-weight:bold">Subject: </span>[openstack-dev] [trove]The patch from 191859 breaks the compatibility with python 2.6<br>
</div>
<div><br>
</div>
<div>
<div>
<div>Hi all,</div>
<div><br>
</div>
<div>We're deploying trove with python 2.6 in production. And the latest code from <a href="https://review.openstack.org/#/c/191859">https://review.openstack.org/#/c/191859</a> has broken the compatible with python 2.6.</div>
<div><br>
</div>
<div>The actual code which causes it is in trove/guestagent/common/operating_system.py and looks like thest. Python 2.6 has syntax error for this "list" expression.</div>
<div><br>
</div>
<div>
<div>def list_files_in_directory(root_dir, recursive=False, pattern=None):</div>
<div>    return {os.path.abspath(os.path.join(root, name))</div>
<div>            for (root, _, files) in os.walk(root_dir, topdown=True)</div>
<div>            if recursive or (root == root_dir)</div>
<div>            for name in files</div>
<div>            if not pattern or re.match(pattern, name)}</div>
</div>
<div><br>
</div>
<div>It would be great for anyone to fix it for both python 2.6 and 2.7, right?</div>
<div><br>
</div>
<div>- tobe</div>
<div><tincludetail></tincludetail></div>
</div>
</div>
</span>



</div><!--<![endif]--></includetail></div>