1 |
slords |
1.1 |
Add a "repowait" stage right after "add_to_repo", so a package job |
2 |
|
|
can request_copy only once and nothing executes add_to_repo more than |
3 |
|
|
once. |
4 |
|
|
|
5 |
|
|
diff -Nur plague-0.4.4.1-orig/server/PackageJob.py plague-0.4.4.1/server/PackageJob.py |
6 |
|
|
--- plague-0.4.4.1-orig/server/PackageJob.py 2006-03-13 05:10:49.000000000 +0100 |
7 |
|
|
+++ plague-0.4.4.1/server/PackageJob.py 2008-01-18 13:40:29.000000000 +0100 |
8 |
|
|
@@ -87,7 +87,7 @@ |
9 |
|
|
|
10 |
|
|
def is_package_job_stage_valid(stage): |
11 |
|
|
""" Validate a job stage """ |
12 |
|
|
- stages = ['initialize', 'checkout_wait', 'checkout_wait_done', 'checkout', 'make_srpm', 'prep', 'waiting', 'building', 'build_done', 'add_to_repo', 'repodone', 'needsign', 'failed', 'finished'] |
13 |
|
|
+ stages = ['initialize', 'checkout_wait', 'checkout_wait_done', 'checkout', 'make_srpm', 'prep', 'waiting', 'building', 'build_done', 'add_to_repo', 'repowait', 'repodone', 'needsign', 'failed', 'finished'] |
14 |
|
|
if stage in stages: |
15 |
|
|
return True |
16 |
|
|
return False |
17 |
|
|
@@ -615,6 +615,7 @@ |
18 |
|
|
self._set_cur_stage('repodone') |
19 |
|
|
else: |
20 |
|
|
self._set_cur_stage('add_to_repo') |
21 |
|
|
+ self.endtime = time.time() |
22 |
|
|
return False # Don't want to wait |
23 |
|
|
|
24 |
|
|
return True |
25 |
|
|
@@ -687,9 +688,13 @@ |
26 |
|
|
# list from this object directly when the copy operation |
27 |
|
|
# happens |
28 |
|
|
if len(self.repofiles): |
29 |
|
|
+ self.endtime = time.time() |
30 |
|
|
+ self._set_cur_stage('repowait') |
31 |
|
|
self.repo.request_copy(self) |
32 |
|
|
|
33 |
|
|
- self.endtime = time.time() |
34 |
|
|
+ return False |
35 |
|
|
+ |
36 |
|
|
+ def _stage_repowait(self): |
37 |
|
|
return True |
38 |
|
|
|
39 |
|
|
def repo_add_callback(self, success, bad_file=None): |
40 |
|
|
diff -Nur plague-0.4.4.1-orig/www/css/style.css plague-0.4.4.1/www/css/style.css |
41 |
|
|
--- plague-0.4.4.1-orig/www/css/style.css 2005-08-06 18:06:41.000000000 +0200 |
42 |
|
|
+++ plague-0.4.4.1/www/css/style.css 2008-01-18 13:13:20.000000000 +0100 |
43 |
|
|
@@ -201,6 +201,12 @@ |
44 |
|
|
font-size: 12px; |
45 |
|
|
} |
46 |
|
|
|
47 |
|
|
+font.status-repowait { |
48 |
|
|
+ color: #aaaa00; |
49 |
|
|
+ font-weight: bold; |
50 |
|
|
+ font-size: 12px; |
51 |
|
|
+} |
52 |
|
|
+ |
53 |
|
|
font.status-needsign { |
54 |
|
|
color: #00aa00; |
55 |
|
|
font-weight: bold; |
56 |
|
|
diff -Nur plague-0.4.4.1-orig/www/indiv.psp plague-0.4.4.1/www/indiv.psp |
57 |
|
|
--- plague-0.4.4.1-orig/www/indiv.psp 2005-08-31 21:31:00.000000000 +0200 |
58 |
|
|
+++ plague-0.4.4.1/www/indiv.psp 2008-01-18 03:29:07.000000000 +0100 |
59 |
|
|
@@ -59,7 +59,7 @@ |
60 |
|
|
# default cutoff is 1 day |
61 |
|
|
cutoff_time = 86400 |
62 |
|
|
|
63 |
|
|
- status_list = ['ALL', 'waiting', 'building', 'add_to_repo', 'needsign', 'failed', 'success'] |
64 |
|
|
+ status_list = ['ALL', 'waiting', 'building', 'add_to_repo', 'repowait', 'needsign', 'failed', 'success'] |
65 |
|
|
%> |
66 |
|
|
|
67 |
|
|
<center> |
68 |
|
|
diff -Nur plague-0.4.4.1-orig/www/success.psp plague-0.4.4.1/www/success.psp |
69 |
|
|
--- plague-0.4.4.1-orig/www/success.psp 2005-08-31 21:31:00.000000000 +0200 |
70 |
|
|
+++ plague-0.4.4.1/www/success.psp 2008-01-18 14:03:07.000000000 +0100 |
71 |
|
|
@@ -10,7 +10,7 @@ |
72 |
|
|
try: |
73 |
|
|
# show any jobs that have successfully built |
74 |
|
|
args = {} |
75 |
|
|
- args['status'] = ['add_to_repo', 'needsign', 'repodone'] |
76 |
|
|
+ args['status'] = ['add_to_repo', 'needsign', 'repowait', 'repodone'] |
77 |
|
|
args['orderby'] = ['status', 'endtime desc'] |
78 |
|
|
args['maxrows'] = 100 |
79 |
|
|
(e, msg, jobs) = server.list_jobs(args) |