chore(mw-jobs): run polling job less often (#611)

* chore(mw-jobs): run polling job less often

* chore(mw-jobs): temporarily return early from job

Author: m90

CHANGELOG.md

@@ -1,5 +1,9 @@
 # api
 
+## 8x.12.3 - 19 June 2023
+- Temporary fix: make polling job a no-op so the queue can empty
+- Run polling job less often
+
 ## 8x.12.2 - 13 June 2023
 - Make sure job for polling wikis does queue properly
 

app/Console/Kernel.php

@@ -45,7 +45,7 @@ protected function schedule(Schedule $schedule)
         // Schedule site stat updates for each wiki and platform-summary
         $schedule->command('schedule:stats')->daily();
 
-        $schedule->job(new PollForMediaWikiJobsJob)->everyMinute();
+        $schedule->job(new PollForMediaWikiJobsJob)->everyFifteenMinutes();
     }
 
     /**

app/Jobs/PollForMediaWikiJobsJob.php

@@ -13,12 +13,15 @@ class PollForMediaWikiJobsJob extends Job implements ShouldQueue, ShouldBeUnique
     public $timeout = 3600;
     public function handle (): void
     {
-        $allWikiDomains = Wiki::all()->pluck('domain');
-        foreach ($allWikiDomains as $wikiDomain) {
-            if ($this->hasPendingJobs($wikiDomain)) {
-                $this->enqueueWiki($wikiDomain);
+        return;
+        /*
+         $allWikiDomains = Wiki::all()->pluck('domain');
+         foreach ($allWikiDomains as $wikiDomain) {
+             if ($this->hasPendingJobs($wikiDomain)) {
+                 $this->enqueueWiki($wikiDomain);
+                }
             }
-        }
+        */
     }
 
     private function hasPendingJobs (string $wikiDomain): bool

tests/Jobs/PollForMediaWikiJobsJobTest.php

@@ -27,6 +27,7 @@ public function setUp(): void
 
     public function testNoJobs()
     {
+        $this->markTestSkipped();
         Http::fake([
             getenv('PLATFORM_MW_BACKEND_HOST').'/w/api.php?action=query&meta=siteinfo&siprop=statistics&format=json' => Http::response([
                 'query' => [
@@ -50,6 +51,7 @@ public function testNoJobs()
 
     public function testWithJobs()
     {
+        $this->markTestSkipped();
         Http::fake([
             getenv('PLATFORM_MW_BACKEND_HOST').'/w/api.php?action=query&meta=siteinfo&siprop=statistics&format=json' => Http::response([
                 'query' => [
@@ -74,6 +76,7 @@ public function testWithJobs()
 
     public function testWithFailure()
     {
+        $this->markTestSkipped();
         Http::fake([
             getenv('PLATFORM_MW_BACKEND_HOST').'/w/api.php?action=query&meta=siteinfo&siprop=statistics&format=json' => Http::response([
                 'error' => 'Something went wrong'