[backend] only do packstatus queries if the packstatus was in the cache

We do not want to use them if the bininfo cache has timed out, as we want to do a refetch every hour.
openSUSE · Mar 15, 2023 · 1a7395c · 1a7395c
1 parent 5034401
commit 1a7395c
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/src/backend/BSSched/Remote.pm b/src/backend/BSSched/Remote.pm
@@ -581,7 +581,7 @@ sub cleanup_remotepackstatus {
 
 ###########################################################################
 ###
-### remote BuildResult (aka gbininfo) support
+### gbininfo cache handling
 ###
 
 sub read_gbininfo_remote_cache {
@@ -611,6 +611,11 @@ sub update_gbininfo_remote_cache {
   $remotegbininfos->{$prpa}->{'cookie'} = $cookie if defined $cookie;
 }
 
+###########################################################################
+###
+### remote BuildResult (aka gbininfo) support
+###
+
 sub read_gbininfo_remote {
   my ($ctx, $prpa, $remoteproj, $packstatus) = @_;
 
@@ -641,7 +646,7 @@ sub read_gbininfo_remote {
 
   # need to fetch new data, first try the packstatus if we have a bininfo cookie
   my ($projid, $repoid, $arch) = split('/', $prpa, 3);
-  if ($remoteproj->{'partition'} && $remotegbininfos->{$prpa} && $remotegbininfos->{$prpa}->{'cookie'}) {
+  if ($remoteproj->{'partition'} && $remotegbininfos->{$prpa} && $remotegbininfos->{$prpa}->{'cookie'} && ($packstatus && !$rpackstatus)) {
     print "    fetching remote packstatus for $prpa\n";
     my $param = {
       'uri' => "$remoteproj->{'remoteurl'}/build/$remoteproj->{'remoteproject'}/$repoid/$arch",