action #60413

status favicon for job details shows "scheduled" instead of running on firefox

Added by okurz 3 months ago. Updated 3 months ago.

Status:ResolvedStart date:28/11/2019
Priority:NormalDue date:
Assignee:mkittler% Done:

0%

Category:Concrete Bugs
Target version:Done
Difficulty:
Duration:

Description

Observation

Discussed in https://chat.suse.de/group/openqa-dev?msg=LrNzLDwBTLC5Jb8Lq :

the favicon for "job running" is the same as "job scheduled" for me on my local firefox but with chrome/chromium it looks fine. something seems to be messed up with my cache. I deleted the local favicon sqlite database of firefox as well as manually remove the favicon using the sqlite client but also checked on a different computer. It seems that chrome/chromium load the png fine and firefox reads svg bug is delivered the wrong svg by openQA.
what does https://openqa.opensuse.org/asset/9a221844fe/logo-incomplete-16.png show you? for me even in chromium it shows me the color of scheduled, dark blue. should be dark red cross. but https://openqa.opensuse.org/tests/1098346 shows the correct light blue for running (mkittler) logo-incomplete-16.png has the wrong color here, too. (okurz) but if you check the file in the git repo it should be fine. Somehow I am confused by these cached assets.
(mkittler) It is also confusing that it renders a different icon for the different sizes (png vs svg):

      <link rel="icon" href="/asset/f630e03ca4/logo-running-16.png" sizes="16x16" type="image/png">
      <link rel="icon" href="/asset/9a221844fe/logo-scheduled.svg" sizes="any" type="image/svg+xml">

from view-source:https://openqa.opensuse.org/tests/1098346

History

#1 Updated by mkittler 3 months ago

  • Status changed from New to In Progress
  • Assignee set to mkittler
  • Target version set to Current Sprint

I think I've got it. It is just a typo in the asset definitions. I'll create a PR.

#2 Updated by okurz 3 months ago

  • Status changed from In Progress to Resolved
  • Target version changed from Current Sprint to Done

https://github.com/os-autoinst/openQA/pull/2558 , merged. Deployed to o3 and verified working by looking at a "running" job from firefox. Done :) Thx!

Also available in: Atom PDF