Project

General

Profile

Actions
Copy link

action #41486

closed

Port openQA to Mojolicious 8.0

Added by mkittler about 6 years ago. Updated about 6 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
mkittler
Category:
Organisational
Target version:
Done
Start date:
2018-09-24
Due date:
% Done:

0%

Estimated time:

Description

Almost all of our tests still pass after updating to Mojolicious 8.0 - so likely only a few adjustments are required. In particular, the tests 24-worker.t and 25-serverstartup.t fail.

Related issues 1 (0 open — 1 closed)

Has duplicate openQA Project (public) - action #41801: 25-serverstartup.t test fails with recent MojoliciousRejected2018-09-29

Actions
Actions
Copy link
 #1

Updated by coolo about 6 years ago

  • Category set to Organisational
  • Target version set to Ready

This needs staging testing afterwards

Actions
Copy link
 #2

Updated by coolo about 6 years ago

  • Has duplicate action #41801: 25-serverstartup.t test fails with recent Mojolicious added
Actions
Copy link
 #3

Updated by mkittler about 6 years ago

It seems that setting the log level via environment variable MOJO_LOG_LEVEL doesn't work anymore.

Eg. this would let the tests pass: https://github.com/os-autoinst/openQA/compare/master...Martchus:port_to_mojo_8?expand=1

Actions
Copy link
 #4

Updated by mkittler about 6 years ago

  • Status changed from New to In Progress
  • Assignee set to mkittler
Actions
Copy link
 #5

Updated by coolo about 6 years ago

  • Status changed from In Progress to Resolved
  • Target version changed from Ready to Done

Let's stick to 7.94 in production until Sebastian finds out

Actions
Copy link
 #6

Updated by kraih about 6 years ago

Easier testing was actually one of the motivations for the change in Mojolicious. I don't think there's a need to use the MOJO_LOG_LEVEL environment variable at all anymore in openQA. Calls to $app->log->level('...') now always have a higher precedence.

Actions
Copy link

Also available in: Atom PDF