<div dir="ltr">On Tue, Nov 12, 2013 at 1:08 AM, Petr Bena <span dir="ltr"><<a href="mailto:benapetr@gmail.com" target="_blank">benapetr@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Tue, Nov 12, 2013 at 9:48 AM, Ryan Lane <<a href="mailto:rlane@wikimedia.org">rlane@wikimedia.org</a>> wrote:<br>


> On Tue, Nov 12, 2013 at 12:42 AM, Petr Bena <<a href="mailto:benapetr@gmail.com">benapetr@gmail.com</a>> wrote:<br>
>><br>
>> Not really, this give me a list of packages that are installed by all<br>
>> classes recursively, but what if someone decide to remove some package<br>
>> from some class that the tool labs classes are inherited from?<br>
>><br>
>> I think that only logical way of maintaining such a list would be to<br>
>> make it by hand and ensure that all packages on such a list are<br>
>> somewhere included (in some class).<br>
>><br>
><br>
> I'm sorry, but when you suggest doing anything by hand, ever, you're<br>
> suggesting the wrong thing. If a package gets removed and it was needed,<br>
> then it should have been listed in the puppet config. The goal of<br>
> configuration management is to never, ever do anything manually that affects<br>
> the state of the system.<br>
><br>
<br>
</div>But what if someone removes something from puppet? Not directly from a<br>
class of tool labs but some other class that tool labs inherits. How<br>
are people supposed to know that this package was actually never<br>
"supported" by tool labs, but was installed as dependency of different<br>
class, which maintainer of decided it's no longer needed.<br>
<br></blockquote><div><br></div><div>In general this would only happen if the package was removed from the base class and we don't have a lot of packages listed in there anyway.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


It wouldn't be the first time when list of default packages that are<br>
installed on every instance by default was shortened. I remember<br>
original instances on labs, time ago, had much more packages than<br>
newly created instances.<br>
<div class="im"><br></div></blockquote><div><br></div><div>This is likely because the new images are missing packages that are installed in ubuntu-server default. You have to be specific when making the image, so this is a bug. The base packages for the images are also in puppet, btw, so if we're missing some packages in the image, we should add them.</div>

<div><br></div><div>- Ryan</div></div></div></div>