Puppet users - Jan 2007 - yumrepo err

I am just starting with puppet to replace a ''homegrown'' config
distribution system and it is looking very promising, i will be playing 
with the templating next.

I have run into an error when attempting to disable the shipped repo''s 
with CentOS 4.4.  See the relevant info below.

This is obviously a problem when repo definitions include multi-line 
baseurl''s.  Am i missing something or is this a bug to be logged?  In 
addition to this i assume then that it is not possible to set multiple 
''baseurl'' when defining a new yumrepo with puppet?

PS. Defining the "Package { provider => yum }" as a default for
CentOS
doesn''t appear to have any affect, setting it to
''rpm'' doesn''t change
the clients behavior, it still uses yum.

Any help would be appreciated.

Your truly,

Ben

-------------------------------

Here''s the puppet class:

class centos {
        Package { provider => yum }

        yumrepo {
                [ base, updates, addons, extras, centosplus, contrib, 
c4-media ]:
                        enabled => absent
                }
}


Here''s the offending repo file: /etc/yum.repos.d/CentOS-Media.repo
(This
is a standard repo shipped with 4.4)

[c4-media]
name=CentOS-$releasever - Media
baseurl=file:///media/cdrom/
        file:///media/cdrecorder/
gpgcheck=1
enabled=0
gpgkey=file:///usr/share/doc/centos-release-4/RPM-GPG-KEY-centos4

Here''s the puppet error:

err: //default/centos/yumrepo=c4-media: Failed to retrieve current 
state: /etc/yum.repos.d/CentOS-Media.repo:17: Can''t parse ''
file:///media/cdrecorder/''

On Thu, 2007-01-04 at 11:30 +1100, Abnormaliti wrote:
> Here''s the offending repo file: /etc/yum.repos.d/CentOS-Media.repo
(This
> is a standard repo shipped with 4.4)
> 
> [c4-media]
> name=CentOS-$releasever - Media
> baseurl=file:///media/cdrom/
>         file:///media/cdrecorder/
Ugh .. yes. The parser for yum config files currently doesn''t support
continuation lines like that; does CentOS ship with such yum config
files ?

In any event, can you open a bug for this so I don''t forget about it ?

David

On Wed, 2007-01-03 at 16:46 -0800, David Lutterkort
wrote:
> On Thu, 2007-01-04 at 11:30 +1100, Abnormaliti wrote:
> > Here''s the offending repo file:
/etc/yum.repos.d/CentOS-Media.repo (This
> > is a standard repo shipped with 4.4)
> > 
> > [c4-media]
> > name=CentOS-$releasever - Media
> > baseurl=file:///media/cdrom/
> >         file:///media/cdrecorder/
> 
> Ugh .. yes. The parser for yum config files currently doesn''t
support
> continuation lines like that; does CentOS ship with such yum config
> files ?
> 
> In any event, can you open a bug for this so I don''t forget about
it ?
Actually, don''t worry - I committed a fix in r2032; will be included in
the next release.

David

David Lutterkort wrote:
> On Wed, 2007-01-03 at 16:46 -0800, David Lutterkort wrote:
>   
>> On Thu, 2007-01-04 at 11:30 +1100, Abnormaliti wrote:
>>     
>>> Here''s the offending repo file:
/etc/yum.repos.d/CentOS-Media.repo (This
>>> is a standard repo shipped with 4.4)
>>>
>>> [c4-media]
>>> name=CentOS-$releasever - Media
>>> baseurl=file:///media/cdrom/
>>>         file:///media/cdrecorder/
>>>       
>> Ugh .. yes. The parser for yum config files currently doesn''t
support
>> continuation lines like that; does CentOS ship with such yum config
>> files ?
>>
>> In any event, can you open a bug for this so I don''t forget
about it ?
>>     
>
> Actually, don''t worry - I committed a fix in r2032; will be
included in
> the next release.
>
> David
>
>
> _______________________________________________
> Puppet-users mailing list
> Puppet-users@madstop.com
> https://mail.madstop.com/mailman/listinfo/puppet-users
>
>
>   
David,

Thankyou for your prompt response.

Can I just download ''inifile.rb 
<http://reductivelabs.com/cgi-bin/puppet.cgi/browser/trunk/lib/puppet/inifile.rb?rev=2032>''
from Trac and it will work with 0.20.1?  I can then create a patch and 
rebuild my RPM.

Ticket has already been created,  I will go and close it now.

On Thu, 2007-01-04 at 13:43 +1100, Abnormaliti wrote:
> Can I just download ''inifile.rb 
>
<http://reductivelabs.com/cgi-bin/puppet.cgi/browser/trunk/lib/puppet/inifile.rb?rev=2032>''
> from Trac and it will work with 0.20.1?  I can then create a patch and 
> rebuild my RPM.
I haven''t tried that, but I think that should work without a problem.

David