switch from sa/ma to authority, fix update_membership_list
[sfa.git] / geni / util / record.py
index 523b8d5..368f910 100644 (file)
@@ -8,35 +8,36 @@ import report
 from types import StringTypes
 from gid import *
 from geni.util.rspec import *
-##
-# The GeniRecord class implements a Geni Record. A GeniRecord is a tuple
-# (Name, GID, Type, Info).
-#
-# Name specifies the HRN of the object
-# GID is the GID of the object
-# Type is user | sa | ma | slice | component
-#
-# Info is comprised of the following sub-fields
-#        pointer = a pointer to the record in the PL database
-#        pl_info = planetlab-specific info (when talking to client)
-#        geni_info = geni-specific info (when talking to client)
-#
-# The pointer is interpreted depending on the type of the record. For example,
-# if the type=="user", then pointer is assumed to be a person_id that indexes
-# into the persons table.
-#
-# A given HRN may have more than one record, provided that the records are
-# of different types. For example, planetlab.us.arizona may have both an SA
-# and a MA record, but cannot have two SA records.
+
 
 class GeniRecord:
+    """ 
+    The GeniRecord class implements a Geni Record. A GeniRecord is a tuple
+    (Name, GID, Type, Info).
+    Name specifies the HRN of the object
+    GID is the GID of the object
+    Type is user | authority | slice | component
+    Info is comprised of the following sub-fields
+           pointer = a pointer to the record in the PL database
+           pl_info = planetlab-specific info (when talking to client)
+           geni_info = geni-specific info (when talking to client)
+    The pointer is interpreted depending on the type of the record. For example,
+    if the type=="user", then pointer is assumed to be a person_id that indexes
+    into the persons table.
+    A given HRN may have more than one record, provided that the records are
+    of different types.
+    """
 
     ##
     # Create a Geni Record
     #
     # @param name if !=None, assign the name of the record
     # @param gid if !=None, assign the gid of the record
-    # @param type one of user | sa | ma | slice | component
+    # @param type one of user | authority | slice | component
     # @param pointer is a pointer to a PLC record
     # @param dict if !=None, then fill in this record from the dictionary
 
@@ -67,6 +68,9 @@ class GeniRecord:
     # @param name is a string containing the HRN
 
     def set_name(self, name):
+        """
+        Set the name of the record
+        """
         self.name = name
         self.dirty = True
 
@@ -76,6 +80,10 @@ class GeniRecord:
     # @param gid is a GID object or the string representation of a GID object
 
     def set_gid(self, gid):
+        """
+        Set the GID of the record
+        """
+
         if isinstance(gid, StringTypes):
             self.gid = gid
         else:
@@ -85,9 +93,12 @@ class GeniRecord:
     ##
     # Set the type of the record
     #
-    # @param type is a string: user | sa | ma | slice | component
+    # @param type is a string: user | authority | slice | component
 
     def set_type(self, type):
+        """
+        Set the type of the record
+        """
         self.type = type
         self.dirty = True
 
@@ -97,6 +108,9 @@ class GeniRecord:
     # @param pointer is an integer containing the ID of a PLC record
 
     def set_pointer(self, pointer):
+        """
+        Set the pointer of the record
+        """
         self.pointer = pointer
         self.dirty = True
 
@@ -106,6 +120,19 @@ class GeniRecord:
     # @param pl_info is a dictionary containing planetlab info
 
     def set_pl_info(self, pl_info):
+        """
+        Set the PLC info of the record
+        """ 
+        if isinstance(pl_info, list):
+            pl_info = pl_info[0]
+        
+        # Convert any boolean strings to real bools
+        for key in pl_info:
+            if isinstance(pl_info[key], StringTypes):
+                if pl_info[key].lower() in ["true"]:
+                    pl_info[key] = True
+                elif pl_info[key].lower() in ["false"]:
+                    pl_info[key] = False   
         self.pl_info = pl_info
         self.dirty = True
 
@@ -115,6 +142,11 @@ class GeniRecord:
     # @param geni_info is a dictionary containing geni info
 
     def set_geni_info(self, geni_info):
+        """
+        Set the geni info the record
+        """
+        if isinstance(geni_info, list):
+            geni_info = geni_info[0]
         self.geni_info = geni_info
         self.dirty = True
 
@@ -122,6 +154,9 @@ class GeniRecord:
     # Return the pl_info of the record, or an empty dictionary if none exists
 
     def get_pl_info(self):
+        """
+        Return the pl_info of the record, or an empty dictionary if none exists
+        """
         if self.pl_info:
             return self.pl_info
         else:
@@ -131,6 +166,9 @@ class GeniRecord:
     # Return the geni_info of the record, or an empty dictionary if none exists
 
     def get_geni_info(self):
+        """
+        Return the geni_info of the record, or an empty dictionary if none exists
+        """
         if self.geni_info:
             return self.geni_info
         else:
@@ -140,12 +178,18 @@ class GeniRecord:
     # Return the name (HRN) of the record
 
     def get_name(self):
+        """
+        Return the name (HRN) of the record
+        """
         return self.name
 
     ##
     # Return the type of the record
 
     def get_type(self):
+        """
+        Return the type of the record
+        """
         return self.type
 
     ##
@@ -154,6 +198,11 @@ class GeniRecord:
     # depends on the type of the record
 
     def get_pointer(self):
+        """
+        Return the pointer of the record. The pointer is an integer that may be
+        used to look up the record in the PLC database. The evaluation of pointer
+        depends on the type of the record
+        """
         return self.pointer
 
     ##
@@ -162,6 +211,9 @@ class GeniRecord:
     # gidObjects in the Cred
 
     def get_gid_object(self):
+        """
+        Return the GID of the record, in the form of a GID object
+        """
         return GID(string=self.gid)
 
     ##
@@ -170,6 +222,11 @@ class GeniRecord:
     # database.
 
     def get_key(self):
+        """
+        Return a key that uniquely identifies this record among all records in
+        Geni. This key is used to uniquely identify the record in the Geni
+        database.
+        """
         return self.name + "#" + self.type
 
     ##
@@ -178,6 +235,11 @@ class GeniRecord:
     # database, but are rather computed values from other entities
 
     def get_field_names(self):
+        """
+        Returns a list of field names in this record. pl_info, geni_info are not
+        included because they are not part of the record that is stored in the
+        database, but are rather computed values from other entities
+        """
         return ["name", "gid", "type", "pointer"]
 
     ##
@@ -186,6 +248,9 @@ class GeniRecord:
     # @param name is the name of field to be returned
 
     def get_field_value_string(self, fieldname):
+        """
+        Given a field name ("name", "gid", ...) return the value of that field.
+        """
         if fieldname == "key":
             val = self.get_key()
         else:
@@ -201,6 +266,9 @@ class GeniRecord:
     # @param fieldnames is a list of field names
 
     def get_field_value_strings(self, fieldnames):
+        """
+        Given a list of field names, return a list of values for those fields.
+        """
         strs = []
         for fieldname in fieldnames:
             strs.append(self.get_field_value_string(fieldname))
@@ -210,6 +278,9 @@ class GeniRecord:
     # Return the record in the form of a dictionary
 
     def as_dict(self):
+        """
+        Return the record in the form of a dictionary
+        """
         dict = {}
         names = self.get_field_names()
         for name in names:
@@ -229,16 +300,20 @@ class GeniRecord:
     # @param dict dictionary to load record fields from
 
     def load_from_dict(self, dict):
+        """
+        Load the record from a dictionary 
+        """
         self.set_name(dict['name'])
         gidstr = dict.get("gid", None)
         if gidstr:
             self.set_gid(dict['gid'])
 
         self.set_type(dict['type'])
-        self.set_pointer(dict['pointer'])
-        if "pl_info" in dict:
+        if "pointer" in dict:
+           self.set_pointer(dict['pointer'])
+        if "pl_info" in dict and dict['pl_info']:
            self.set_pl_info(dict["pl_info"])
-        if "geni_info" in dict:
+        if "geni_info" in dict and dict['geni_info']:
            self.set_geni_info(dict["geni_info"])
 
     ##
@@ -246,7 +321,10 @@ class GeniRecord:
     # the record.
 
     def save_to_string(self):
-
+        """
+        Save the record to a string. The string contains an XML representation of
+        the record.
+        """
         dict = self.as_dict()
         record = RecordSpec()
         record.parseDict(dict)
@@ -259,6 +337,10 @@ class GeniRecord:
     # representation of the record.
 
     def load_from_string(self, str):
+        """
+        Load the record from a string. The string is assumed to contain an XML
+        representation of the record.
+        """
         #dict = xmlrpclib.loads(str)[0][0]
         
         record = RecordSpec()
@@ -273,6 +355,9 @@ class GeniRecord:
     # @param dump_parents if true, then the parents of the GID will be dumped
 
     def dump(self, dump_parents=False):
+        """
+        Walk tree and dump records.
+        """
         print "RECORD", self.name
         print "        hrn:", self.name
         print "       type:", self.type
@@ -293,12 +378,17 @@ class GeniRecord:
         pl_info = getattr(self, "pl_info", {})
         if pl_info:
 
-            for key in pl_info.keys():
+            for key in (s for s in pl_info.keys()\
+            if (s.endswith("_ids") or s.endswith("_id")) == False):
                 print "       ", key, ":", pl_info[key]
 
 
     def getdict(self):
         info = {'hrn': self.name, 'type': self.type, 'gid': self.gid}
-        info.update(getattr(self, "geni_info", {}))
-        info.update(getattr(self, "pl_info", {}))
+        geni_info = getattr(self, "geni_info", {})
+        pl_info = getattr(self, "pl_info", {}) 
+        if geni_info:
+            info.update(geni_info)
+        if pl_info:
+            info.update(pl_info)
         return info