Changeset 2261 for trunk

Timestamp:

Mar 18, 2009, 6:52:03 PM (16 years ago)

Author:

quentin

Message:

Make invirt-update-vnc-cert more robust to failure

File:

• trunk/packages/invirt-vnc-client/debian/invirt-update-vnc-cert (modified) (2 diffs)

trunk/packages/invirt-vnc-client/debian/invirt-update-vnc-cert

@@ -2 +2 @@
 
 from invirt.config import structs as config
-from subprocess import Popen, call, PIPE
+from subprocess import Popen, check_call, PIPE, CalledProcessError
 import tempfile
 import os
@@ -8 +8 @@
 import shutil
 
+def check_wait(popen):
+    retcode = popen.wait()
+    if retcode != 0:
+        raise CalledProcessError(retcode, popen.pid)
+    return retcode
+
 def main():
-    call(['kinit', '-k', 'daemon/%s' % config.web.hostname])
-    
-    shutil.copy('/usr/share/invirt-vnc-client/VncViewer.src.jar',
-                '/usr/share/invirt-vnc-client/VncViewer.jar')
+    check_call(['kinit', '-k', 'daemon/%s' % config.web.hostname])
     
     temp_dir = tempfile.mkdtemp()
+
+    jarfile = os.path.join(temp_dir, 'VncViewer.jar')
+
+    shutil.copy('/usr/share/invirt-vnc-client/VncViewer.src.jar',
+                jarfile)
+    
     keystore = os.path.join(temp_dir, 'trust.store')
     for host in config.hosts:
         cert = Popen(['remctl', config.remote.hostname, 'web', 'vnccert', host.hostname],
                      stdout=PIPE)
-        cert.wait()
-        call(['keytool', '-import', '-noprompt', '-alias', host.hostname,
-              '-keystore', keystore, '-storepass', 'foobar'],
-             stdin=cert.stdout)
+        check_wait(cert)
+        check_call(['keytool', '-import', '-noprompt', '-alias', host.hostname,
+                    '-keystore', keystore, '-storepass', 'foobar'],
+                   stdin=cert.stdout)
     
-    call(['jar', 'uf', '/usr/share/invirt-vnc-client/VncViewer.jar',
-          '-C', temp_dir, 'trust.store'])
+    check_call(['jar', 'uf', jarfile,
+                '-C', temp_dir, 'trust.store'])
+
+    shutil.move(jarfile, '/usr/share/invirt-vnc-client/VncViewer.jar')
     
     shutil.rmtree(temp_dir)