Changeset d3c8759 for fedd/federation/protogeni_access.py

Timestamp:

May 25, 2010 11:15:55 AM (14 years ago)

Author:

Ted Faber <faber@…>

Branches:

axis_example, compt_changes, info-ops, master, version-3.01, version-3.02

Children:

f038da1

Parents:

da2208a

Message:

Wholesale change of IOError to EnvironmentError? for file operations. Lots of
uncaught EnvironmentErrors? were causing spurious error conditions, e.g. on disk
full.

File:

• fedd/federation/protogeni_access.py (modified) (10 diffs)

fedd/federation/protogeni_access.py

@@ -261 +261 @@
                 f = open(self.state_filename, 'w')
                 pickle.dump(self.state, f)
-            except IOError, e:
+            except EnvironmentError, e:
                 self.log.error("Can't write file %s: %s" % \
                         (self.state_filename, e))
@@ -282 +282 @@
                 self.log.debug("[read_state]: Read state from %s" % \
                         self.state_filename)
-            except IOError, e:
+            except EnvironmentError, e:
                 self.log.warning(("[read_state]: No saved state: " +\
                         "Can't open %s: %s") % (self.state_filename, e))
@@ -437 +437 @@
             cf = open(conffilename, "w")
             os.chmod(conffilename, stat.S_IRUSR | stat.S_IWUSR)
-        except IOError, e:
+        except EnvironmentError, e:
             raise service_error(service_error.internal, 
                     "Cannot create user configuration data")
@@ -443 +443 @@
         try:
             dev_null = open("/dev/null", "a")
-        except IOError, e:
+        except EnvironmentError, e:
             self.log.error("export_userconf: can't open /dev/null: %s" % e)
 
@@ -596 +596 @@
                 print >>f, alloc_cert
                 f.close()
-            except IOError, e:
+            except EnvironmentError, e:
                 raise service_error(service_error.internal, 
                         "Can't open %s/%s : %s" % (self.certdir, aid, e))
@@ -827 +827 @@
                         secretkey_base
                 f.close()
-            except IOError, e:
+            except EnvironmentError, e:
                 raise service_error(service_error.internal,
                         "Can't write protal config %s: %s" % (cfn, e))
@@ -841 +841 @@
                     print >>f, "ExperimentID: %s" % mexp
                     f.close()
-                except IOError, e:
+                except EnvironmentError, e:
                     raise service_error(service_error.internal, 
                             "Can't write seer.conf: %s" %e)
@@ -858 +858 @@
                     # print >>f, "ExperimentID: %s/%s" % (mproj, meid)
                     f.close()
-                except IOError, e:
+                except EnvironmentError, e:
                     raise service_error(service_error.internal,
                             "Cannot write client.conf: %s" %s)
@@ -971 +971 @@
             tmpdir = tempfile.mkdtemp(prefix="access-")
             softdir = "%s/software" % tmpdir
-        except IOError:
+        except EnvironmentError:
             raise service_error(service_error.internal, "Cannot create tmp dir")
 
@@ -1044 +1044 @@
                             tf.close()
                             get_url(u, tn, tmpdir, "userconf")
-                        except IOError, e:
+                        except EnvironmentError, e:
                             raise service_error(service.error.internal, 
                                     "Cannot create temp file for " +