Project

General

Profile

Actions

action #41486

closed

Port openQA to Mojolicious 8.0

Added by mkittler over 5 years ago. Updated over 5 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Organisational
Target version:
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 open1 closed)

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

Actions
Actions #1

Updated by coolo over 5 years ago

  • Category set to Organisational
  • Target version set to Ready

This needs staging testing afterwards

Actions #2

Updated by coolo over 5 years ago

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

Updated by mkittler over 5 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 #4

Updated by mkittler over 5 years ago

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

Updated by coolo over 5 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 #6

Updated by kraih over 5 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

Also available in: Atom PDF