Remove unnecessary constructors

[invirt/packages/invirt-database.git] / sipb_xen_database / models.py

diff --git a/sipb_xen_database/models.py b/sipb_xen_database/models.py
index b4380b9..d07950a 100644 (file)
--- a/sipb_xen_database/models.py
+++ b/sipb_xen_database/models.py
@@ -6,11 +6,13 @@ from sqlalchemy.ext.assignmapper import assign_mapper
 __all__ = ['meta',
            'ctx',
            'machine_table',
+           'machine_access_table',
            'nic_table',
            'disk_table',
            'types_table',
            'cdroms_table',
            'Machine',
+           'MachineAccess',
            'NIC',
            'Disk',
            'Type',
@@ -55,37 +57,42 @@ cdroms_table = Table('cdroms', meta,
        Column('cdrom_id', String, primary_key=True, nullable=False),
        Column('description', String, nullable=False))
 
+machine_access_table = Table('machine_access', meta,
+       Column('machine_id', Integer, ForeignKey('machines.machine_id'), nullable=False, index=True),
+       Column('user', String, nullable=False, index=True),
+       PrimaryKeyConstraint('machine_id', 'user'))
 
 class Machine(object):
-    pass
+    def __repr__(self):
+        return "<Machine %s: name='%s' owner='%s'>" % (self.machine_id, self.name, self.owner)
+
+class MachineAccess(object):
+    def __repr__(self):
+        return "<MachineAccess machine='%s' user='%s'>" % (self.machine, self.user)
 
 class NIC(object):
-    def __init__(self, machine_id, mac_addr, ip, hostname):
-        self.machine_id = machine_id
-        self.mac_addr = mac_addr
-        self.ip = ip
-        self.hostname = hostname
+    def __repr__(self):
+        return "<NIC: mac='%s' machine='%s' ip='%s' hostname='%s'>" % (self.mac_addr, self.machine_id, self.ip, self.hostname)
 
 class Disk(object):
-    def __init__(self, machine_id, guest, size):
-        self.machine_id = machine_id
-        self.guest_device_name = guest
-        self.size = size
+    def __repr__(self):
+        return "<Disk: machine=%s device=%s size=%s>" % (self.machine_id, self.guest_device_name, self.size)
 
 class Type(object):
-    pass
+    def __repr__(self):
+        return "<Type %s: %s>" % (self.type_id, self.description)
 
 class CDROM(object):
-    def __init__(self, cdrom_id, description):
-        self.cdrom_id = cdrom_id
-        self.description = description
+    def __repr__(self):
+        return "<CDROM %s: %s>" % (self.cdrom_id, self.description)
 
 assign_mapper(ctx, Machine, machine_table,
               properties={'nics': relation(NIC, backref="machine"),
                           'disks': relation(Disk, backref="machine"),
-                          'type': relation(Type)});
+                          'type': relation(Type),
+                          'users': relation(MachineAccess, backref="machine")});
+assign_mapper(ctx, MachineAccess, machine_access_table)
 assign_mapper(ctx, NIC, nic_table)
 assign_mapper(ctx, Disk, disk_table)
 assign_mapper(ctx, Type, types_table)
 assign_mapper(ctx, CDROM, cdroms_table)
-