Added by kraih almost 4 years ago. Updated almost 4 years ago.
Description
Mojolicious 9.0 has just been released and there are quite a few breaking changes. Some also affect openQA, such as the use of reserved stash values as route placeholders, and need to be fixed.
Started working on it in a branch. https://github.com/os-autoinst/openQA/pull/3726
I think we should consider this part of the backlog and effectively a bug fix, because we'll be in trouble if we want to use the latest Mojolicious.
Thank you, @kraih, for being on top of it. Although I would prefer the API wasn't broken all the time, but well, it is what it is.
Agreeing with cdywan, I assume this is not just "for fun" but we would need to work on that anyway at the latest when our tests break when 9.0 reaches openSUSE:Factory, right?
No it's not just "for fun". :) The things i've found in openQA so far are actual bugs, like /:status being used as a route placeholder. And status is special because it allows the response code to be changed ($c->render(text => '...', status => 302)). So we might end up allowing the request URL to pick a specific response code. You get the idea. Mojolicious 9.0 turned a lot of these easy to make mistakes into exceptions.
And yes, once 9.0 reaches factory our tests would fail.
Setting due date based on mean cycle time of SUSE QE Tools
Mojolicious 9.0 is on the way to factory now. https://build.opensuse.org/request/show/872779
The pull request should be ready now. The colored log message feature has been moved upstream and will probably be part of Mojolicious 9.01.
Mojolicious 9.01 with MOJO_LOG_COLOR=1 feature is on its way to factory. https://build.opensuse.org/request/show/873064
Executing /usr/share/openqa/script/openqa-gru fails
Version:
# output of /usr/share/openqa/script/openqa-gru
[2021-02-17 13:07:06.55734] [3763] [warn] Deprecated use of config key '[audit]: blacklist'. Use '[audit]: blocklist' instead
Route pattern "/children/:status" contains a reserved stash value at /usr/share/openqa/script/../lib/OpenQA/WebAPI.pm line 361.
# exit code 30
The PR has been merged.