OBS: Fix status detection even more

Require build state = published before considering such a build as being finished. States with other states (scheduling) and stale statuses do happen. We need to filter those out.
Raku · Nov 1, 2022 · 5f64fa2 · 5f64fa2
1 parent 1ab6149
commit 5f64fa2
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/lib/OBS.rakumod b/lib/OBS.rakumod
@@ -163,11 +163,11 @@ method process-worklist() is serial-dedup {
 
             my $test-name = $build.arch ~ "-" ~ $build.repository;
 
-            my $status = $build.state eq "building"                         ?? DB::IN_PROGRESS !!
-                         $build.status.values.grep( * eq "failed")          ?? DB::FAILURE !!
-                         $build.status.values.grep( * eq "building")        ?? DB::IN_PROGRESS !!
-                         ([&&] $build.status.values.map( * eq "succeeded")) ?? DB::SUCCESS !!
-                                                                               DB::UNKNOWN;
+            my $status = $build.state eq "building"                                                        ?? DB::IN_PROGRESS !!
+                                                        $build.status.values.grep( * eq "building")        ?? DB::IN_PROGRESS !!
+                         $build.state eq "published" && $build.status.values.grep( * eq "failed")          ?? DB::FAILURE !!
+                         $build.state eq "published" && ([&&] $build.status.values.map( * eq "succeeded")) ?? DB::SUCCESS !!
+                                                                                                              DB::UNKNOWN;
 
             my $test-is-new = False;
             my $test =