erase of Exports File

[ch3i]

@Uncle-Frank said:

Yes, the installer script creates a new empty (overwrites) exports-file. It’s not easy to write fail-proof script code that checks to see if other entries (or old FOG entries) are in that file. So we better create a clean new one. If you have a good idea how to better do this you are more than welcome to give us a hint.

Please make a backup copy of your file before running the installer. @Tom-Elliott Well, maybe we should do that like we do with /tftpboot…

Maybe use comments before/after the fog exports, like that :

/custom/config1

#FOG settings
/images *(ro,sync,no_wdelay,no_subtree_check,insecure_locks,no_root_squash,insecure,fsid=0)
/images/dev *(rw,async,no_wdelay,no_subtree_check,no_root_squash,insecure,fsid=1)
#END FOG settings

/custom/config2

On each update store all other lines not in these comments

[Wayne Workman]

How about having the installer move any existing file to /etc/exports.old ?

[olivierl]

Hi
In your script before add share it can be this

if [ -f $nfsconfig ]
    then
              if  [ $(grep -c "##FOG-SHARE-START##" $nfsconfig) -ne 0 ]
                   then
                            sed -in  '/##FOG-SHARE-START##/,/##FOG-END-SHARE##/{//d;D}' $nfsconfig
              fi
fi

And add this configureNFS()

echo -e "##FOG-SHARE-START##\n$storageLocation *(ro,sync,no_wdelay,no_subtree_check,insecure_locks,no_root_squash,insecure,fsid=0)\n$storageLocation/dev *(rw,async,no_wdelay,no_subtree_check,no_root_squash,insecure,fsid=1)\n##FOG-END-SHARE##" >> "$nfsconfig";

I am not tested for the moment

Olivier

[olivierl]

Hi
I have tested this norming and it’s worrking for my case.

Olivier

[Tom Elliott]

What’s the actual code?

[Wayne Workman]

@Tom-Elliott has added a new option in /opt/fog/.fogsettings

blexports=

1 means to rebuild /etc/exports
0 means do not rebuild it.

https://forums.fogproject.org/topic/5975/dhcpd-conf-file-gets-rebuilt-on-every-install

[ch3i]

@Tom-Elliott In lib/common/functions.sh

configureNFS() {
    echo -e "$storageLocation *(ro,sync,no_wdelay,no_subtree_check,insecure_locks,no_root_squash,insecure,fsid=0)\n$storageLocation/dev *(rw,async,no_wdelay,no_subtree_check,no_root_squash,insecure,fsid=1)" > "$nfsconfig";
    dots "Setting up and starting RPCBind";
    if [ "$systemctl" == "yes" ]; then
        systemctl enable rpcbind.service >/dev/null 2>&1 && \
        systemctl restart rpcbind.service >/dev/null 2>&1 && \
        systemctl status rpcbind.service >/dev/null 2>&1
    elif [ "$osid" -eq 2 ]; then
        true
    else
        chkconfig rpcbind on >/dev/null 2>&1 && \
        $initdpath/rpcbind restart >/dev/null 2>&1 && \
        $initdpath/rpcbind status >/dev/null 2>&1
    fi
    errorStat $?
    dots "Setting up and starting NFS Server..."
    for nfsItem in $nfsservice; do
        if [ "$systemctl" == "yes" ]; then
            systemctl enable $nfsItem >/dev/null 2>&1 && \
            systemctl restart $nfsItem >/dev/null 2>&1 && \
            systemctl status $nfsItem >/dev/null 2>&1
        else
            if [ "$osid" == 2 ]; then
                sysv-rc-conf $nfsItem on >/dev/null 2>&1 && \
                $initdpath/nfs-kernel-server stop >/dev/null 2>&1 && \
                $initdpath/nfs-kernel-server start >/dev/null 2>&1
            else
                chkconfig $nfsItem on >/dev/null 2>&1 && \
                $initdpath/$nfsItem restart >/dev/null 2>&1 && \
                $initdpath/$nfsItem status >/dev/null 2>&1
            fi
        fi
        if [ "$?" -eq 0 ]; then
            break
        fi
    done
    errorStat $?
}

[ch3i]

@Wayne-Workman said:

@Tom-Elliott has added a new option in /opt/fog/.fogsettings

blexports=

1 means to rebuild /etc/exports
0 means do not rebuild it.

https://forums.fogproject.org/topic/5975/dhcpd-conf-file-gets-rebuilt-on-every-install

It’ll be better to have it automatically in the installer like the @olivierl solution.

[Tom Elliott]

@ch3i much appreciated but I meant the code for the sed statement not the function code that needs to be altered.

[Sebastian Roth]

@Tom-Elliott Code snippet was provided by olivierl in this thread (post number 11).

[ch3i]

@Tom-Elliott said:

@ch3i much appreciated but I meant the code for the sed statement not the function code that needs to be altered.

https://forums.fogproject.org/topic/5952/erase-of-exports-file/11

[olivierl]

Hi,

It’'s work but before use it i think i need to verify before if the line "/image and image/dev"exist
To make the transition from a version that does not contain ##Fog–share-start##
to avoid having 2 times sharing appear in the file

[Wayne Workman]

Oh now I see what @olivierl did here… added those tags to the top and bottom.

##FOG-SHARE-START##
##FOG-END-SHARE##

That’s pretty smart.

But I don’t think the new blexports setting should be undone, but I think that this could be added…
In fact something similar is not a bad idea for all of the conf files that fog touches…