Changeset 26292 for trunk/Nebulous-Server/bin/nebdiskd

Timestamp:

Nov 30, 2009, 1:54:49 PM (17 years ago)

Author:

watersc1

Message:

Nebulous: Implemented a number of bug fixes and modifications to

ensure no loss of data.

• Each volume can now be associated into a cabinet, defining their physical arrangement.
• File creation: new files without a specified volume are created on a random volume from a subset of the N least full.
• Replication: file replication uses the same volume selection method, with the constraints that a copy not be automatically placed on the same volume as another, and that the first copy not be housed within the same volume.
• Cull: instead of removing the first available instance, first attempt to remove an instance on the same volume as another, then attempt to remove an instance in the same cabinet as another, and then remove the first only if those fail.
• Corrected bug that prevented the deletion of an object if it had more than one unavailable instance.
• Fixed nebdiskd to synchronize between volume->mountedvol tables and to properly remove volumes that were no longer accessible.

File:

• trunk/Nebulous-Server/bin/nebdiskd (modified) (4 diffs)

trunk/Nebulous-Server/bin/nebdiskd

@@ -177 +177 @@
 
     eval {
-        my $r_query = $dbh->prepare_cached("REPLACE INTO mountedvol SELECT *, ?, ? FROM volume WHERE mountpoint = ?");
+        my $r_query = $dbh->prepare_cached("REPLACE INTO mountedvol SELECT vol_id,name,host,path,allocate,available,xattr,mountpoint, ?, ? FROM volume WHERE mountpoint = ?");
+#       my $d_query = $dbh->prepare_cached("UPDATE mountedvol SET allocate = ?, available = ? WHERE mountpoint = ?");
         my $d_query = $dbh->prepare_cached("DELETE FROM mountedvol WHERE mountpoint = ?");
 
@@ -185 +186 @@
             # there may be multiple vol_ids per mountpoint but we only need to
             # check each mointpont once
-            my $query = $dbh->prepare_cached("SELECT DISTINCT mountpoint FROM volume WHERE available = 1");
+            my $query = $dbh->prepare_cached("SELECT DISTINCT mountpoint FROM volume");
             $query->execute;
             while (my $row = $query->fetchrow_hashref) {
@@ -200 +201 @@
             # automounter and it fails to mount
             my $tries = 0;
+            my $valid_mountpoint = 1;
             TEST: eval {
                 $tries++;
                 unless (is_mountpoint($mountpoint)) {
-                    $log->warn("$mnt is not a valid mountpoint");
+                    $log->warn("$mountpoint is not a valid mountpoint ($tries)");
+                    $valid_mountpoint = 0;
                 }
             };
-            if ($@) {
+            if (!$valid_mountpoint) {
                 # try is_mountpoint() again if $retry > 1
                 if ($tries < $retry) {
@@ -212 +215 @@
                     goto TEST; 
                 }
-                $log->warn($@);
+                $log->warn("Removing $mountpoint from the mountedvol table ($tries > $retry)");
+ #               $log->warn($@);
+
                 $d_query->execute($mountpoint);
                 next;