X-Git-Url: http://git.onelab.eu/?a=blobdiff_plain;f=NodeUpdate.py;h=8d07729f786d4e6a3f0c0b60a9ee846da361c279;hb=09f1706948e213a4b53ce20c69db8e70d3c79214;hp=35530358ab65a4505df2e8b0bcb662f101b43e90;hpb=47049f839a9ed6f3ce8b34f4730e230cde498403;p=nodeupdate.git diff --git a/NodeUpdate.py b/NodeUpdate.py old mode 100644 new mode 100755 index 3553035..8d07729 --- a/NodeUpdate.py +++ b/NodeUpdate.py @@ -23,7 +23,6 @@ RPM_PATH = "/bin/rpm" RPM_GPG_PATH = "/etc/pki/rpm-gpg" - # location of file containing http/https proxy info, if needed PROXY_FILE = '/etc/planetlab/http_proxy' @@ -35,9 +34,11 @@ REBOOT_FLAG = '/etc/planetlab/update-reboot' # location of directory containing boot server ssl certs SSL_CERT_DIR='/mnt/cdrom/bootme/cacert/' -# file containing list of extra groups to attempt to update, -# if necessary. -EXTRA_GROUPS_FILE= '/etc/planetlab/extra-node-groups' +# file containing the list of extensions this node has, each +# correspond to a package group in yum repository. +# this is expected to be updated from the 'extensions tag' +# through the 'extensions.php' nodeconfig script +EXTENSIONS_FILE='/etc/planetlab/extensions' # file containing a list of rpms that we should attempt to delete # before updating everything else. This list is not removed with @@ -45,6 +46,23 @@ EXTRA_GROUPS_FILE= '/etc/planetlab/extra-node-groups' # that were not intended to be deleted. DELETE_RPM_LIST_FILE= '/etc/planetlab/delete-rpm-list' +# ok, so the logic should be simple, just yum update the world +# however there are many cases in the real life where this +# just does not work, because of a glitch somewhere +# so, we force the update of crucial pkgs independently, as +# the whole group is sometimes too much to swallow +# this one is builtin +CRUCIAL_PACKAGES_BUILTIN=[ 'NodeUpdate' , 'nodemanager-lib', 'nodemanager-lxc', 'nodemanager-vs' ] +# and operations can also try to push a list through a conf_file +# should use the second one for consistency, try the first one as well for legacy +CRUCIAL_PACKAGES_OPTIONAL_PATHS=[ + # this is a legacy name, please avoid using this one + '/etc/planetlab/NodeUpdate.packages' + # file to use with a hand-written conf_file + '/etc/planetlab/crucial-rpm-list' + # this one could some day be maintained by a predefined config_file + '/etc/planetlab/sliceimage-rpm-list' +] # print out a message only if we are displaying output def Message(message): @@ -152,29 +170,50 @@ class NodeUpdate: yum_options += " --verbose" else: Message( "Unsupported, not using --verbose option" ) - + + # a configurable list of packages to try and update independently + # cautious.. + try: + crucial_packages = [] + for package in CRUCIAL_PACKAGES_BUILTIN: crucial_packages.append(package) + for path in CRUCIAL_PACKAGES_OPTIONAL_PATHS: + try: crucial_packages += file(CRUCIAL_PACKAGES_OPTIONAL_PATH1).read().split() + except: pass + for package in crucial_packages: + # if package is not yet installed, like e.g. slice images, + # need to yum install, not yum update + if os.system("rpm -q %s > /dev/null"%package)==0: + Message( "\nUpdating crucial package %s" % package) + os.system( "%s %s -y update %s" %(YUM_PATH, yum_options, package)) + else: + Message( "\Installing crucial package %s" % package) + os.system( "%s %s -y install %s" %(YUM_PATH, yum_options, package)) + except: + pass + Message( "\nUpdating PlanetLab group" ) - os.system( "%s %s %s -y groupupdate \"PlanetLab\"" % + os.system( "%s %s %s -y groupinstall \"PlanetLab\"" % (YUM_PATH, yum_options, sslcertdir) ) Message( "\nUpdating rest of system" ) - os.system( "%s %s %s -y update" % - (YUM_PATH, yum_options, sslcertdir) ) - - Message( "\nChecking for extra groups to update" ) - if os.access(EXTRA_GROUPS_FILE, os.R_OK) and \ - os.path.isfile(EXTRA_GROUPS_FILE): - extra_groups_contents= file(EXTRA_GROUPS_FILE).read() - extra_groups_contents= string.strip(extra_groups_contents) - if extra_groups_contents == "": + os.system( "%s %s %s -y update" % (YUM_PATH, yum_options, sslcertdir) ) + + Message( "\nChecking for extra groups (extensions) to update" ) + if os.access(EXTENSIONS_FILE, os.R_OK) and \ + os.path.isfile(EXTENSIONS_FILE): + extensions_contents= file(EXTENSIONS_FILE).read() + extensions_contents= string.strip(extensions_contents) + if extensions_contents == "": Message( "No extra groups found in file." ) else: - for group in string.split(extra_groups_contents,"\n"): + extensions_contents.strip() + for extension in extensions_contents.split(): + group = "extension%s" % extension Message( "\nUpdating %s group" % group ) - os.system( "%s %s %s -y groupupdate \"%s\"" % + os.system( "%s %s %s -y groupinstall \"%s\"" % (YUM_PATH, yum_options, sslcertdir, group) ) else: - Message( "No extra groups file found" ) + Message( "No extensions file found" ) if os.access(REBOOT_FLAG, os.R_OK) and os.path.isfile(REBOOT_FLAG) and self.doReboot: Message( "\nAt least one update requested the system be rebooted" ) @@ -200,24 +239,30 @@ class NodeUpdate: Message( "\nLooking for RPMs to be deleted." ) if os.access(DELETE_RPM_LIST_FILE, os.R_OK) and \ os.path.isfile(DELETE_RPM_LIST_FILE): - rpm_list_contents= file(DELETE_RPM_LIST_FILE).read() - rpm_list_contents= string.strip(rpm_list_contents) + rpm_list_contents= file(DELETE_RPM_LIST_FILE).read().strip() if rpm_list_contents == "": Message( "No RPMs listed in file to delete." ) return - rpm_list= string.join(string.split(rpm_list_contents)) + rpm_list= string.split(rpm_list_contents) - Message( "Deleting these RPMs:" ) - Message( rpm_list_contents ) - - rc= os.system( "%s -ev %s" % (RPM_PATH, rpm_list) ) - - if rc != 0: - Error( "Unable to delete RPMs, continuing. rc=%d" % rc ) - else: - Message( "RPMs deleted successfully." ) + Message( "Deleting RPMs from %s: %s" %(DELETE_RPM_LIST_FILE," ".join(rpm_list))) + + # invoke them separately as otherwise one faulty (e.g. already uninstalled) + # would prevent the other ones from uninstalling + for rpm in rpm_list: + # is it installed + is_installed = os.system ("%s -q %s"%(RPM_PATH,rpm))==0 + if not is_installed: + Message ("Ignoring rpm %s marked to delete, already uninstalled"%rpm) + continue + uninstalled = os.system( "%s -ev %s" % (RPM_PATH, rpm) )==0 + if uninstalled: + Message ("Successfully removed RPM %s"%rpm) + continue + else: + Error( "Unable to delete RPM %s, continuing. rc=%d" % (rpm,uninstalled)) else: Message( "No RPMs list file found." )