[asterisk-commits] runtests.py: Add --keep-full-logs option (testsuite[master])

[SVN commits to the Asterisk project]

Jenkins2 has submitted this change and it was merged. ( https://gerrit.asterisk.org/6284 )

Change subject: runtests.py:  Add --keep-full-logs option
......................................................................

runtests.py:  Add --keep-full-logs option

Keeps the full logs and pcaps around even if the test passes.

Change-Id: Ib33e31c7cecc78f1d9dc495967bc913177a96ffd
---
M runtests.py
1 file changed, 5 insertions(+), 1 deletion(-)

Approvals:
  Richard Mudgett: Looks good to me, but someone else must approve
  Joshua Colp: Looks good to me, approved
  Jenkins2: Approved for Submit



diff --git a/runtests.py b/runtests.py
index d057f09..d684f86 100755
--- a/runtests.py
+++ b/runtests.py
@@ -105,6 +105,7 @@
         self.stdout = ""
         self.timeout = timeout
         self.cleanup = options.cleanup
+        self.keep_full_logs = options.keep_full_logs
         self.skipped_reason = ""
 
         assert self.test_name.startswith('tests/')
@@ -181,7 +182,7 @@
             self._process_valgrind()
             self._process_ref_debug()
 
-            if not self.passed:
+            if not self.passed or self.keep_full_logs:
                 self._archive_logs()
             elif self.cleanup:
                 try:
@@ -862,6 +863,9 @@
                       dest="pcap", default=False,
                       help="Capture packets. Output will be in "
                       " the test's log directory as packet.pcap.")
+    parser.add_option("--keep-full-logs", action="store_true",
+                      dest="keep_full_logs", default=False,
+                      help="Keep full logs even if test passes.")
     (options, args) = parser.parse_args(argv)
 
     # Install a signal handler for USR1/TERM, and use it to bail out of running

-- 
To view, visit https://gerrit.asterisk.org/6284
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ib33e31c7cecc78f1d9dc495967bc913177a96ffd
Gerrit-Change-Number: 6284
Gerrit-PatchSet: 1
Gerrit-Owner: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-commits/attachments/20170823/605767a7/attachment-0001.html>