Ratings don't appear to be updating

[robbieaverill]

Example: phptek/verifiable

It's scheduled in jobs, but running the job reports a 404 in Codecov (that's fine) but also zero points overall when the job finishes.

We need to have another look at this. Note there's some other issues e.g. https://github.com/silverstripe/moduleratings/issues/11 and an open pull request to help with diagnosing this kind of problem at #209

[phptek]

Well at least the module score is automated now - I thought I still needed to send an email to community@silverstripe.org - indeed, I did that a couple of weeks ago because that's what the docs on addons.silverstripe.org still say.

[phptek]

Ah, not the same thing as "Module Score" after all. So how does one go about getting one of these for phptek/verifiable?)

[robbieaverill]

Has it still not updated!? Well, the PR to add logging for diagnostics was finally merged a couple of days ago so now at least we can work out why the jobs are failing. It should all be automatic

[phptek]

Nope. Still says "N/A" :-(

[robbieaverill]

@phptek still getting around to looking at fixing this. I've deployed the latest master branch to UAT which includes config for the internal ratings system to send logs to Graylog so we can try and see if there's some rate limiting being hit, or something like that. We'll hopefully be looking at this in the next couple of weeks.

[robbieaverill]

Logging is tricky to monitor (see #216). I didn't see anything obvious about the Travis API though. It kind of looks like some jobs aren't being run at all... will keep investigating passively

[robbieaverill]

🤦‍♂️ 🤦‍♂️ 🤦‍♂️

        // Provide the checksuite with a logger in case API calls fail
        $logger = new \Monolog\Logger('module_ratings_logs', [
-             new SyslogHandler('SilverStripe_log'),
+             $handler = new SyslogHandler('SilverStripe_log'),
        ]);
        $formatter = new LineFormatter("%level_name%: %message% %context% %extra%");
-         $logger->setFormatter($formatter);
+         $handler->setFormatter($formatter);

[robbieaverill]

Fixes are at https://github.com/silverstripe/addons.silverstripe.org/pull/217