<html><body><p><font size="2">Thanks for sharing this info , it make sense to leave a -2 here, I will keep modifying follow on patches and get more reviews. thanks </font><br><br><font size="2">Best Regards! <br><br>Kevin (Chen) Ji ¼Í ³¿<br><br>Engineer, zVM Development, CSTL<br>Notes: Chen CH Ji/China/IBM@IBMCN   Internet: jichenjc@cn.ibm.com<br>Phone: +86-10-82451493<br>Address: 3/F Ring Building, ZhongGuanCun Software Park, Haidian District, Beijing 100193, PRC </font><br><br><img width="16" height="16" src="cid:1__=8FBB0812DFB3E61A8f9e8a93df938690918c8FB@" border="0" alt="Inactive hide details for Dan Smith ---04/30/2018 10:55:51 PM---> According to requirements and comments, now we opened the CI "><font size="2" color="#424282">Dan Smith ---04/30/2018 10:55:51 PM---> According to requirements and comments, now we opened the CI runs with > run_validation = True And</font><br><br><font size="2" color="#5F5F5F">From:        </font><font size="2">Dan Smith <dms@danplanet.com></font><br><font size="2" color="#5F5F5F">To:        </font><font size="2">"Chen CH Ji" <jichenjc@cn.ibm.com></font><br><font size="2" color="#5F5F5F">Cc:        </font><font size="2">"OpenStack Development Mailing List \(not for usage questions\)" <openstack-dev@lists.openstack.org></font><br><font size="2" color="#5F5F5F">Date:        </font><font size="2">04/30/2018 10:55 PM</font><br><font size="2" color="#5F5F5F">Subject:        </font><font size="2">Re: [openstack-dev] [Nova] z/VM introducing a new config driveformat</font><br><hr width="100%" size="2" align="left" noshade style="color:#8091A5; "><br><br><br><tt><font size="2">> According to requirements and comments, now we opened the CI runs with<br>> run_validation = True And according to [1] below, for example, [2]<br>> need the ssh validation passed the test<br>><br>> And there are a couple of comments need some enhancement on the logs<br>> of CI such as format and legacy incorrect links of logs etc the newest<br>> logs sample can be found [3] (take n-cpu as example and those logs are<br>> with _white.html)<br>><br>> Also, the blueprint [4] requested by previous discussion post here<br>> again for reference<br>><br>> please let us know whether the procedure -2 can be removed in order to<br>> proceed . thanks for your help<br><br>The CI log format issues look fixed to me and validation is turned on<br>for the stuff supported, which is what was keeping it out of the<br>runway.<br><br>I still plan to leave the -2 on there until the next few patches have<br>agreement, just so we don't land an empty shell driver before we are<br>sure we're going to land spawn/destroy, etc. That's pretty normal<br>procedure and I'll be around to remove it when appropriate.<br><br>--Dan<br><br></font></tt><br><br><BR>
</body></html>