[Change Request] Enable rw /mnt/fedora on puppet1

[Dennis Gilmore]

+1 from me

Dennis

Toshio Kuratomi <a.badger at gmail.com> wrote:

>Currently we're mounting /mnt/fedora ro on puppet1.  I think that this
>was a change committed in puppet that affected the /etc/fstab file.
>That didn't come into play until we rebooted puppet1 last night -- the
>reboot caused the new fstab to be used and mount /mnt/fedora ro.
>
>Here's the changeset that caused that:
>
>Date:   Fri Jun 26 22:53:26 2009 +0000
>
>    e mount instead of nfs.
>
>diff --git a/modules/puppet/manifests/init.pp
>b/modules/puppet/manifests/init.pp
>index 21b8d62..0af2273 100644
>--- a/modules/puppet/manifests/init.pp
>+++ b/modules/puppet/manifests/init.pp
>@@ -75,9 +75,12 @@ class puppet::master::mounts {
>         ensure => directory,
>     }
>
>-    nfs { "/mnt/fedora":
>+    mount { "/mnt/fedora":
>         device  => "ntap-fedora1.fedora.phx.redhat.com:/vol/fedora/",
>-        require => File["/mnt/fedora/"],
>+        fstype  => "nfs",
>+        ensure  => "mounted",
>+        options => "defaults,ro,soft,intr",
>+        require => File["/mnt/fedora"],
>     }
> }
>
>
>I'd like to make the following change to this:
>
>-        options => "defaults,ro,soft,intr",
>+        options => "defaults,rw,soft,intr",
>
>Can I get two +1's for my change?
>
>-Toshio
>
>_______________________________________________
>Fedora-infrastructure-list mailing list
>Fedora-infrastructure-list at redhat.com
>https://www.redhat.com/mailman/listinfo/fedora-infrastructure-list

--
Sent from my Android phone with K-9. Please excuse my brevity.