<html dir="ltr">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<style id="owaParaStyle" type="text/css">P {margin-top:0;margin-bottom:0;}</style>
</head>
<body ocsi="0" fpstyle="1">
<div style="direction: ltr;font-family: Tahoma;color: #000000;font-size: 10pt;">Hi,<br>
<br>
We have faced this situation in Solum several times. And in fact this was one of the topics<br>
that we discussed in our last irc meeting.<br>
<br>
We landed on separating the sample check from pep8 gate into a non-voting gate.<br>
One reason to keep the sample check is so that when say a feature in your code fails<br>
due to some upstream changes and for which you don't have coverage in your functional tests then<br>
a non-voting but failing sample check gate can be used as a starting point of the failure investigation.<br>
<br>
More details about the discussion can be found here:<br>
http://eavesdrop.openstack.org/meetings/solum_team_meeting/2014/solum_team_meeting.2014-09-23-16.00.log.txt<br>
<br>
- Devdatta<br>
<br>
<div style="font-family: Times New Roman; color: #000000; font-size: 16px">
<hr tabindex="-1">
<div style="direction: ltr;" id="divRpF479448"><font color="#000000" face="Tahoma" size="2"><b>From:</b> David Shrewsbury [shrewsbury.dave@gmail.com]<br>
<b>Sent:</b> Thursday, September 25, 2014 12:42 PM<br>
<b>To:</b> OpenStack Development Mailing List (not for usage questions)<br>
<b>Subject:</b> Re: [openstack-dev] [Ironic] Get rid of the sample config file<br>
</font><br>
</div>
<div></div>
<div>
<div dir="ltr">Hi!<br>
<div>
<div class="gmail_extra"><br>
<div class="gmail_quote">On Thu, Sep 25, 2014 at 12:23 PM, Lucas Alvares Gomes <span dir="ltr">
<<a href="mailto:lucasagomes@gmail.com" target="_blank">lucasagomes@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex; border-left:1px #ccc solid; padding-left:1ex">
Hi,<br>
<br>
Today we have hit the problem of having an outdated sample<br>
configuration file again[1]. The problem of the sample generation is<br>
that it picks up configuration from other projects/libs<br>
(keystoneclient in that case) and this break the Ironic gate without<br>
us doing anything.<br>
<br>
So, what you guys think about removing the test that compares the<br>
configuration files and makes it no longer gate[2]?<br>
<br>
We already have a tox command to generate the sample configuration<br>
file[3], so folks that needs it can generate it locally.<br>
<br>
Does anyone disagree?<br>
<br clear="all">
</blockquote>
<div><br>
</div>
<div>+1 to this, but I think we should document how to generate the sample config<br>
in our documentation (install guide?).<br>
</div>
</div>
<br>
</div>
<div class="gmail_extra">-Dave<br>
</div>
<div class="gmail_extra">-- <br>
<div dir="ltr">
<div>David Shrewsbury (Shrews)<br>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</body>
</html>