McStasMcXtrace / McWeb

Code and documentation for e-neutrons.org platform
3 stars 3 forks source link

Bad gateway 502 periodic error #162

Closed mads-bertelsen closed 6 years ago

mads-bertelsen commented 6 years ago

When testing quizzes during the Tartu NNSP SwedNESS summerschool, the quizes frequently produced the error: "Bad gateway 502" when checking an answer, trying to progress to the next page and so on. Initially the error only occurred when all students were doing the quiz, so I assumed it was related to load on the server. This morning however it occurred in every question of the SANS I quiz during a lecture where I presumably was the only one using the system.

willend commented 6 years ago

Hi @mads-bertelsen and @lindaudby, I am almost 100% sure this is a duplicate of https://github.com/McStasMcXtrace/McWeb/issues/152 - and currently there is no other solution than to reimplement the quizzes - which is on Linda's table.

The issue is that we are running an unsupported question type called qtype_algebra (doesn't run with recent moodle versions) - the log is full of warnings like these:

PHP message: PHP Deprecated:  Methods with the same name as their class will not be constructors in a future version of PHP; qtype_algebra_parser_nullterm has a deprecated constructor in /home/srv/mcweb/moodle/question/type/algebra/parser.php on line 422
PHP message: PHP Deprecated:  Methods with the same name as their class will not be constructors in a future version of PHP; qtype_algebra_parser_number has a deprecated constructor in /home/srv/mcweb/moodle/question/type/algebra/parser.php on line 473
PHP message: PHP Deprecated:  Methods with the same name as their class will not be constructors in a future version of PHP; qtype_algebra_parser_variable has a deprecated constructor in /home/srv/mcweb/moodle/question/type/algebra/parser.php on line 590
PHP message: PHP Deprecated:  Methods with the same name as their class will not be constructors in a future version of PHP; qtype_algebra_parser_power has a deprecated constructor in /home/srv/mcweb/moodle/question/type/algebra/parser.php on line 753
PHP message: PHP Deprecated:  Methods with the same name as their class will not be constructors in a future version of PHP; qtype_algebra_parser_divide has a deprecated constructor in /home/srv/mcweb/moodle/question/type/algebra/parser.php on line 800
PHP message: PHP Deprecated:  Methods with the same name as their class will not be constructors in a future version of PHP; qtype_algebra_parser_multiply has a deprecated constructor in /home/srv/mcweb/moodle/question/type/algebra/parser.php on line 855
willend commented 6 years ago

As attempted for #152 I have tried reinstalling the qtype_algebra class - which seems to have momentarily improved the situation... But I am also convinced this will come back

mads-bertelsen commented 6 years ago

Earlier this only happened when checking answers and similar, in this instance it happened even when going to the next page in the quiz. I thought this would provide more information on the issue, but didn't see the previous topic.

lindaudby commented 6 years ago

OK, if you can patch it like that if the error occurs again during the next week of the Tartu school I will take a look at converting all the qtype_algebra questions into something else. But it is quite a big task.

willend commented 6 years ago

I know it is a big task @lindaudby, but there is no way around it it seems - see below...

I've posted a question about support for moodle 3.3 at the plugin page (https://moodle.org/plugins/qtype_algebra)

safariscreensnapz034

safariscreensnapz035

willend commented 6 years ago

And I can not be on "urgent call" during all of next week to randomly try reinstalling, as we have no proof that this is really "working".

What I could do is to hack in a (hourly?) cronjob that does this reinstall "all the time", but that has a good chance of having other side effects.

lindaudby commented 6 years ago

OK, please try the cronjob hack.

And as I said I will remake the content as soon as possible to not use the algebra question type. But we are talking 10 quiz questions implemented in 4 courses so it is several full days of work and I cannot do it on top of servicing the running course which is already making me work overtime.

willend commented 6 years ago

cron-hack in place... Have no idea if this will help or not, but every 5 minutes past the hour, qtype_algebra is reinstalled - status page for control: https://sim.e-neutrons.org/static/qtype_algebra_status.html

willend commented 6 years ago

@lindaudby I guess we could close this one if there is a related content-issue in place?

willend commented 6 years ago

Oh yes, here it was https://github.com/McStasMcXtrace/McWeb/issues/152