X-Git-Url: http://git.onelab.eu/?a=blobdiff_plain;f=PLC%2FNodes.py;h=78bbfdc9958e4830b2924aad2b64a74c628ba93f;hb=df89953c9a34cd167b36eaae8a03f2ae1efb03b4;hp=6356c39708b39463332df4200dab5db3f093cc55;hpb=f6f0164afcbada9498622842964024f6fb3e5dfa;p=plcapi.git diff --git a/PLC/Nodes.py b/PLC/Nodes.py index 6356c39..78bbfdc 100644 --- a/PLC/Nodes.py +++ b/PLC/Nodes.py @@ -4,7 +4,7 @@ # Mark Huang # Copyright (C) 2006 The Trustees of Princeton University # -# $Id: Nodes.py,v 1.13 2006/10/24 20:02:22 mlhuang Exp $ +# $Id: Nodes.py,v 1.28 2006/12/05 16:45:03 thierry Exp $ # from types import StringTypes @@ -12,6 +12,7 @@ import re from PLC.Faults import * from PLC.Parameter import Parameter +from PLC.Filter import Filter from PLC.Debug import profile from PLC.Table import Row, Table from PLC.NodeNetworks import NodeNetwork, NodeNetworks @@ -37,45 +38,65 @@ class Node(Row): table_name = 'nodes' primary_key = 'node_id' + join_tables = ['nodegroup_node', 'conf_file_node', 'nodenetworks', 'pcu_node', 'slice_node', 'slice_attribute', 'node_session'] fields = { 'node_id': Parameter(int, "Node identifier"), 'hostname': Parameter(str, "Fully qualified hostname", max = 255), 'site_id': Parameter(int, "Site at which this node is located"), 'boot_state': Parameter(str, "Boot state", max = 20), - 'model': Parameter(str, "Make and model of the actual machine", max = 255), + 'model': Parameter(str, "Make and model of the actual machine", max = 255, nullok = True), 'boot_nonce': Parameter(str, "(Admin only) Random value generated by the node at last boot", max = 128), 'version': Parameter(str, "Apparent Boot CD version", max = 64), 'ssh_rsa_key': Parameter(str, "Last known SSH host key", max = 1024), 'date_created': Parameter(int, "Date and time when node entry was created", ro = True), 'last_updated': Parameter(int, "Date and time when node entry was created", ro = True), 'key': Parameter(str, "(Admin only) Node key", max = 256), - 'session': Parameter(str, "(Admin only) Node session value", max = 256), - 'nodenetwork_ids': Parameter([int], "List of network interfaces that this node has", ro = True), - 'nodegroup_ids': Parameter([int], "List of node groups that this node is in", ro = True), - 'conf_file_ids': Parameter([int], "List of configuration files specific to this node", ro = True), - # 'root_person_ids': Parameter([int], "(Admin only) List of people who have root access to this node", ro = True), - 'slice_ids': Parameter([int], "List of slices on this node", ro = True), - 'pcu_ids': Parameter([int], "List of PCUs that control this node", ro = True), - 'ports': Parameter([int], "List of PCU ports that this node is connected to", ro = True), + 'session': Parameter(str, "(Admin only) Node session value", max = 256, ro = True), + 'nodenetwork_ids': Parameter([int], "List of network interfaces that this node has"), + 'nodegroup_ids': Parameter([int], "List of node groups that this node is in"), + 'conf_file_ids': Parameter([int], "List of configuration files specific to this node"), + # 'root_person_ids': Parameter([int], "(Admin only) List of people who have root access to this node"), + 'slice_ids': Parameter([int], "List of slices on this node"), + 'pcu_ids': Parameter([int], "List of PCUs that control this node"), + 'ports': Parameter([int], "List of PCU ports that this node is connected to"), + 'peer_id': Parameter(int, "Peer at which this node is managed", nullok = True), } + # for Cache + class_key = 'hostname' + foreign_fields = ['boot_state','model','version'] + # forget about these ones, they are read-only anyway + # handling them causes Cache to re-sync all over again + # 'date_created','last_updated' + foreign_xrefs = [ + # in this case, we dont need the 'table' but Cache will look it up, so... + {'field' : 'site_id' , 'class' : 'Site' , 'table' : 'unused-on-direct-refs' } , + ] + def validate_hostname(self, hostname): if not valid_hostname(hostname): raise PLCInvalidArgument, "Invalid hostname" conflicts = Nodes(self.api, [hostname]) - for node_id, node in conflicts.iteritems(): - if 'node_id' not in self or self['node_id'] != node_id: + for node in conflicts: + if 'node_id' not in self or self['node_id'] != node['node_id']: raise PLCInvalidArgument, "Hostname already in use" return hostname def validate_boot_state(self, boot_state): - if boot_state not in BootStates(self.api): + boot_states = [row['boot_state'] for row in BootStates(self.api)] + if boot_state not in boot_states: raise PLCInvalidArgument, "Invalid boot state" return boot_state + # timestamps + def validate_date_created (self, timestamp): + return self.validate_timestamp (timestamp) + def validate_last_updated (self, timestamp): + return self.validate_timestamp (timestamp) + def delete(self, commit = True): """ Delete existing node. @@ -83,54 +104,38 @@ class Node(Row): assert 'node_id' in self - # Delete all nodenetworks - nodenetworks = NodeNetworks(self.api, self['nodenetwork_ids']) - for nodenetwork in nodenetworks.values(): - nodenetwork.delete(commit = False) - # Clean up miscellaneous join tables - for table in ['nodegroup_node', 'slice_node', 'slice_attribute']: - self.api.db.do("DELETE FROM %s" \ - " WHERE node_id = %d" % \ + for table in self.join_tables: + self.api.db.do("DELETE FROM %s WHERE node_id = %d" % \ (table, self['node_id'])) # Mark as deleted self['deleted'] = True self.sync(commit) + class Nodes(Table): """ Representation of row(s) from the nodes table in the database. """ - def __init__(self, api, node_id_or_hostname_list = None): - self.api = api + def __init__(self, api, node_filter = None, columns = None): + Table.__init__(self, api, Node, columns) sql = "SELECT %s FROM view_nodes WHERE deleted IS False" % \ - ", ".join(Node.fields) - - if node_id_or_hostname_list: - # Separate the list into integers and strings - node_ids = filter(lambda node_id: isinstance(node_id, (int, long)), - node_id_or_hostname_list) - hostnames = filter(lambda hostname: isinstance(hostname, StringTypes), - node_id_or_hostname_list) - sql += " AND (False" - if node_ids: - sql += " OR node_id IN (%s)" % ", ".join(map(str, node_ids)) - if hostnames: - sql += " OR hostname IN (%s)" % ", ".join(api.db.quote(hostnames)).lower() - sql += ")" - - rows = self.api.db.selectall(sql) - - for row in rows: - self[row['node_id']] = node = Node(api, row) - for aggregate in ['nodenetwork_ids', 'nodegroup_ids', - 'conf_file_ids', 'root_person_ids', 'slice_ids', - 'pcu_ids']: - if not node.has_key(aggregate) or node[aggregate] is None: - node[aggregate] = [] - else: - node[aggregate] = map(int, node[aggregate].split(',')) + ", ".join(self.columns) + + if node_filter is not None: + if isinstance(node_filter, (list, tuple, set)): + # Separate the list into integers and strings + ints = filter(lambda x: isinstance(x, (int, long)), node_filter) + strs = filter(lambda x: isinstance(x, StringTypes), node_filter) + node_filter = Filter(Node.fields, {'node_id': ints, 'hostname': strs}) + sql += " AND (%s)" % node_filter.sql(api, "OR") + elif isinstance(node_filter, dict): + node_filter = Filter(Node.fields, node_filter) + sql += " AND (%s)" % node_filter.sql(api, "AND") + + self.selectall(sql) +