[plague] Wait max. 60 s for Mock state.log to appear (prev. 15 s).
Michael Schwendt
mschwendt at fedoraproject.org
Mon Mar 2 00:52:19 UTC 2015
commit b8185815e5775d3411bdec2989921f7cbbd346b6
Author: Michael Schwendt <mschwendt at fedoraproject.org>
Date: Mon Mar 2 01:52:15 2015 +0100
Wait max. 60 s for Mock state.log to appear (prev. 15 s).
- Ignore SSL.Error exceptions in recv().
plague-0.4.5.8-fedora-mock-state.patch | 12 ++++++++++--
plague-0.4.5.8-wakeup-serve_forever.patch | 16 ++++++++++++++--
plague.spec | 6 +++++-
3 files changed, 29 insertions(+), 5 deletions(-)
---
diff --git a/plague-0.4.5.8-fedora-mock-state.patch b/plague-0.4.5.8-fedora-mock-state.patch
index 8808385..e44ffbb 100644
--- a/plague-0.4.5.8-fedora-mock-state.patch
+++ b/plague-0.4.5.8-fedora-mock-state.patch
@@ -1,6 +1,6 @@
diff -Nur plague-0.4.5.8-orig/builder/builder.py plague-0.4.5.8/builder/builder.py
--- plague-0.4.5.8-orig/builder/builder.py 2010-05-05 12:45:20.000000000 +0200
-+++ plague-0.4.5.8/builder/builder.py 2015-02-28 11:40:40.226331650 +0100
++++ plague-0.4.5.8/builder/builder.py 2015-03-02 01:46:22.511456796 +0100
@@ -253,15 +253,6 @@
self._childpid = ExecUtils.exec_with_redirect(cmd, args, None, None, None)
self._status = 'cleanup'
@@ -43,7 +43,15 @@ diff -Nur plague-0.4.5.8-orig/builder/builder.py plague-0.4.5.8/builder/builder.
def _read_mock_config(self):
mockconfigfile = os.path.join(self._result_dir, 'mockconfig.log')
-@@ -365,9 +357,7 @@
+@@ -359,15 +351,13 @@
+ # We need to make sure that mock has dumped the status file withing a certain
+ # amount of time, otherwise we can't tell what it's doing
+ if not os.path.exists(self._state_file):
+- # something is wrong if mock takes more than 15s to write the status file
+- if time.time() > self._mockstarttime + 15:
++ # something is wrong if mock takes more than 60s to write the status file
++ if time.time() > self._mockstarttime + 60:
+ self._mockstarttime = 0
self._log("ERROR: Timed out waiting for the mock status file! %s\n" % self._state_file)
self.die()
else:
diff --git a/plague-0.4.5.8-wakeup-serve_forever.patch b/plague-0.4.5.8-wakeup-serve_forever.patch
index bce7e96..ba4fd3e 100644
--- a/plague-0.4.5.8-wakeup-serve_forever.patch
+++ b/plague-0.4.5.8-wakeup-serve_forever.patch
@@ -1,6 +1,6 @@
diff -Nur plague-0.4.5.8-orig/builder/builder.py plague-0.4.5.8/builder/builder.py
--- plague-0.4.5.8-orig/builder/builder.py 2010-05-05 12:45:20.000000000 +0200
-+++ plague-0.4.5.8/builder/builder.py 2013-04-22 15:35:24.683026931 +0200
++++ plague-0.4.5.8/builder/builder.py 2015-03-02 01:44:47.343591868 +0100
@@ -799,6 +799,14 @@
log("Received SIGTERM, quitting...\n")
bserver.stop()
@@ -16,9 +16,21 @@ diff -Nur plague-0.4.5.8-orig/builder/builder.py plague-0.4.5.8/builder/builder.
def main():
global bserver
+diff -Nur plague-0.4.5.8-orig/common/SSLConnection.py plague-0.4.5.8/common/SSLConnection.py
+--- plague-0.4.5.8-orig/common/SSLConnection.py 2010-08-10 17:19:22.000000000 +0200
++++ plague-0.4.5.8/common/SSLConnection.py 2015-03-02 01:00:55.000000000 +0100
+@@ -149,6 +149,8 @@
+ except SSL.SysCallError, (e, err):
+ time.sleep(0.2)
+ return None
++ except Error:
++ return None
+ return None
+
+ class PlgFileObject(socket._fileobject):
diff -Nur plague-0.4.5.8-orig/server/main.py plague-0.4.5.8/server/main.py
--- plague-0.4.5.8-orig/server/main.py 2008-01-31 14:28:05.000000000 +0100
-+++ plague-0.4.5.8/server/main.py 2013-04-22 15:36:00.412107870 +0200
++++ plague-0.4.5.8/server/main.py 2015-03-02 01:44:47.365590682 +0100
@@ -68,6 +68,14 @@
print "Received SIGTERM, quitting..."
bm_server.stop()
diff --git a/plague.spec b/plague.spec
index e6b7653..d1a2f02 100644
--- a/plague.spec
+++ b/plague.spec
@@ -6,7 +6,7 @@ BuildArch: noarch
Summary: Distributed build system for RPMs
Name: plague
Version: 0.4.5.8
-Release: 22%{?dist}
+Release: 23%{?dist}
License: GPLv2+
Group: Development/Tools
#Source: http://fedoraproject.org/projects/plague/releases/%{name}-%{version}.tar.bz2
@@ -185,6 +185,10 @@ mkdir -p $RPM_BUILD_ROOT/var/lib/plague/builder
%changelog
+* Mon Mar 2 2015 Michael Schwendt <mschwendt at fedoraproject.org> - 0.4.5.8-23
+- Wait max. 60 s for Mock state.log to appear (prev. 15 s).
+- Ignore SSL.Error exceptions in recv().
+
* Sat Feb 28 2015 Michael Schwendt <mschwendt at fedoraproject.org> - 0.4.5.8-22
- Builder upgrade revealed that Mock in Fedora has changed and creates a
different state.log file than what has been compatible with Plague Builder.
More information about the scm-commits
mailing list