Issue #6845 has been updated by John Warburton.

I am not quite sure what you mean, as the code in 
lib/puppet/provider/mount/parsed.rb (line 23) does the parsing for Solaris 
correctly

As I said before, I applied the patch in this issue and it all works again, 
however for posterity, here is the result of an unpatched 2.6.7 on an 
/etc/vfstab entry created with 2.6.4 and the patch from issue #4904

    # tail -1 /etc/vfstab
    swap    -       /tmp    tmpfs   -       yes     size=512m
    
And this is what an unpatched 2.6.7 thinks about it:

    # puppet resource mount
    mount { '-':
    ensure  => 'unmounted',
    device  => 'swap',
    dump    => '-',
    fstype  => '/tmp',
    options => 'tmpfs',
    pass    => 'yes       size=512m',
    target  => '/etc/vfstab',
    }
    
When I apply the patch (line 21 lib/puppet/provider/mount/parsed.rb:: case 
Facter.value(:operatingsystem)), it parses correctly:

    # puppet resource mount
    mount { '/tmp':
    ensure      => 'unmounted',
    atboot      => 'yes',
    blockdevice => '-',
    device      => 'swap',
    fstype      => 'tmpfs',
    options     => 'size=512m',
    pass        => '-',
    target      => '/etc/vfstab',
    }
    



----------------------------------------
Bug #6845: mount tmpfs on Solaris fail
https://projects.puppetlabs.com/issues/6845

Author: John Warburton
Status: Ready For Testing
Priority: High
Assignee: 
Category: mount
Target version: 
Affected Puppet version: 2.6.7
Keywords: Solaris mount /tmp tempfs tmpfs
Branch: https://github.com/stschulte/puppet/tree/ticket/2.6.x/6845


The newly released 2.6.7 mount provider fails on Solaris 10 U9, and does not 
modify the /etc/vfstab, but just adds another (wrong) line

mount.pp:
    mount{ "/tmp":
        device  => "swap",
        blockdevice  => "-",
        fstype  => "tmpfs",
        pass    => "-",
        atboot  => "yes",
        options => "size=512m",
        # http://projects.puppetlabs.com/issues/4904
        ensure => $puppetversion ? {
            /0\.25/  => mounted,
            default => present,
        },
    }

/etc/vfstab before:
    swap    -       /tmp    tmpfs   -       yes     -

Apply the change
    # puppet apply --verbose --color true /tmp/mount.pp
    info: Applying configuration version '1301028968'
    notice: /Stage[main]//Mount[/tmp]/ensure: ensure changed 'ghost' to 
'defined'
    info: FileBucket adding {md5}de855ac78119686e92d519e9068c3e70
    info: /Stage[main]//Mount[/tmp]: Scheduling refresh of Mount[/tmp]
    notice: /Stage[main]//Mount[/tmp]: Triggered 'refresh' from 1 events
    info: /Stage[main]//Mount[/tmp]: Scheduling refresh of Mount[/tmp]
    notice: Finished catalog run in 0.22 seconds

/etc/vfstab after:
    swap    -       /tmp    tmpfs   -       yes     -
    swap    /tmp    tmpfs   size=512m       0       -
    
Expected result:
    swap    -       /tmp    tmpfs   -       yes     size=512m




-- 
You have received this notification because you have either subscribed to it, 
or are involved in it.
To change your notification preferences, please click here: 
http://projects.puppetlabs.com/my/account

-- 
You received this message because you are subscribed to the Google Groups 
"Puppet Bugs" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/puppet-bugs?hl=en.

Reply via email to