It was suggested in #22534 that warning was too high a level for potential
parse errors from the package query output.  And I think that stuck in my
head.  When I was reviewing, I realized the original level (prior to my
description work) was also warning.  Really it boils down to whether we
would want actual failures to parse dpkg-query/rpm -q data to show up with
or without --debug, I think.


On Tue, Sep 17, 2013 at 4:41 PM, Patrick Carlisle
<notifications@github.com>wrote:

> In lib/puppet/provider/package/dpkg.rb:
>
> > @@ -59,7 +60,7 @@ def self.parse_multi_line(pipe)
> >
> >      line = pipe.gets
> >      unless hash = parse_line(line)
> > -      Puppet.warning "Failed to match dpkg-query line #{line.inspect}"
> > +      Puppet.info "Failed to match dpkg-query line #{line.inspect}" if !self::DPKG_PACKAGE_NOT_FOUND_REGEX.match(line)
>
> Why the change in log level?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/puppetlabs/puppet/pull/1909/files#r6421581>
> .
>



-- 
Josh Partlow
jpartlow@puppetlabs.com
Developer, Puppet Labs

Join us at PuppetConf 2014, September 23-24 in San Francisco