All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jason Wang <jasowang@redhat.com>
To: lmr@redhat.com, autotest@test.kernel.org
Cc: kvm@vger.kernel.org
Subject: [PATCH v2 10/10] KVM test: Add a helper to search the panic in the log
Date: Tue, 11 May 2010 17:04:31 +0800	[thread overview]
Message-ID: <20100511090431.19914.24287.stgit@localhost.localdomain> (raw)
In-Reply-To: <20100511083338.19914.7719.stgit@localhost.localdomain>

This checker serves as the post_command to find the panic information
in the file which contains the content of guest serial console.

Signed-off-by: Jason Wang <jasowang@redhat.com>
---
 client/tests/kvm/scripts/check_serial.py |   41 ++++++++++++++++++++++++++++++
 client/tests/kvm/tests_base.cfg.sample   |    7 ++++-
 2 files changed, 46 insertions(+), 2 deletions(-)
 create mode 100644 client/tests/kvm/scripts/check_serial.py

diff --git a/client/tests/kvm/scripts/check_serial.py b/client/tests/kvm/scripts/check_serial.py
new file mode 100644
index 0000000..969bbe3
--- /dev/null
+++ b/client/tests/kvm/scripts/check_serial.py
@@ -0,0 +1,41 @@
+import os, sys, glob, re
+
+
+class SerialCheckerError(Exception):
+    """
+    Simple wrapper for the builtin Exception class.
+    """
+    pass
+
+
+class SerialChecker(object):
+    """
+    Serach the panic or other keywords in the guest serial.
+    """
+    def __init__(self):
+        """
+        Gets params from environment variables and sets class attributes.
+        """
+        client_dir =  os.environ['AUTODIR']
+        self.pattern = os.environ['KVM_TEST_search_pattern']
+        self.shortname = os.environ['KVM_TEST_shortname']
+        self.debugdir = os.path.join(client_dir, "results/default/kvm.%s/debug" \
+                                     % self.shortname)
+        self.serial_files = glob.glob(os.path.join(self.debugdir, 'serial*'))
+
+
+    def check(self):
+        """
+        Check whether the pattern were found in the serial files
+        """
+        fail = [ f for f in self.serial_files if
+                 re.findall(self.pattern, file(f).read(), re.I) ]
+        if fail:
+            print "%s is found in %s" % (self.pattern, fail)
+            raise SerialCheckerError("Error found during the check, Please" \
+                                     " check the log")
+
+
+if __name__ == "__main__":
+    checker = SerialChecker()
+    checker.check()
diff --git a/client/tests/kvm/tests_base.cfg.sample b/client/tests/kvm/tests_base.cfg.sample
index 3c0933e..3ac8f0d 100644
--- a/client/tests/kvm/tests_base.cfg.sample
+++ b/client/tests/kvm/tests_base.cfg.sample
@@ -52,6 +52,10 @@ address_index = 0
 # Misc
 profilers = kvm_stat
 
+# pattern serach in guest serial console
+serach_pattern = panic
+post_command = "python scripts/check_serial.py"
+post_command_noncritical = no
 
 # Tests
 variants:
@@ -1314,10 +1318,9 @@ virtio|virtio_blk|e1000|balloon_check:
 variants:
     - @qcow2:
         image_format = qcow2
-        post_command = " python scripts/check_image.py;"
+        post_command = " python scripts/check_image.py; python scripts/check_serial.py"
         remove_image = no
         post_command_timeout = 600
-        post_command_noncritical = yes
     - vmdk:
         only Fedora Ubuntu Windows
         only smp2


  parent reply	other threads:[~2010-05-11  8:59 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-05-11  9:03 [PATCH v2 00/10] Redirct and make use of the guest serial console Jason Wang
2010-05-11  9:03 ` [PATCH v2 01/10] KVM test: Introduce prompt assist Jason Wang
2010-05-12 22:00   ` Lucas Meneghel Rodrigues
2010-05-11  9:03 ` [PATCH v2 02/10] KVM test: Send the username in remote_login() Jason Wang
2010-05-11  9:03 ` [PATCH v2 03/10] KVM test: Make the login re suitable for serial console Jason Wang
2010-05-11  9:03 ` [PATCH v2 04/10] KVM test: Redirect the serial to the unix domain socket Jason Wang
2010-05-11  9:03 ` [PATCH v2 05/10] KVM test: Log the content from guest serial console Jason Wang
2010-05-11  9:03 ` [PATCH v2 06/10] KVM test: Return none when met unknown type in kvm_vm.remote_login() Jason Wang
2010-05-11  9:04 ` [PATCH v2 07/10] KVM test: Introduce local_login() Jason Wang
2010-05-11  9:04 ` [PATCH v2 08/10] KVM test: Enable the serial console for all linux guests Jason Wang
2010-05-11  9:04 ` [PATCH v2 09/10] KVM test: Enable the serial console during unattended installation Jason Wang
2010-05-11  9:04 ` Jason Wang [this message]
2010-05-12  9:44   ` [PATCH v2 10/10] KVM test: Add a helper to search the panic in the log Michael Goldish
2010-05-17  7:28     ` Jason Wang
2010-05-17 21:23       ` Lucas Meneghel Rodrigues
2010-05-25 14:50 ` [PATCH v2 00/10] Redirct and make use of the guest serial console Lucas Meneghel Rodrigues
2010-05-28  8:19   ` Jason Wang

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20100511090431.19914.24287.stgit@localhost.localdomain \
    --to=jasowang@redhat.com \
    --cc=autotest@test.kernel.org \
    --cc=kvm@vger.kernel.org \
    --cc=lmr@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.