Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-51111

Navigating to downstream job via Blue Ocean Triggered Builds does not show tests in Tests tab

    XMLWordPrintable

Details

    • Bug
    • Status: Resolved (View Workflow)
    • Minor
    • Resolution: Fixed
    • blueocean-plugin
    • None
    • Jenkins 2.117
      Blue Ocean 1.5
    • 1.14.0

    Description

      Steps to reproduce using two jobs:

      • First job: triggers the second job (wait = true)
      • Second job: record unit tests
      • Open Blue Ocean from the first job, click Triggered Builds to access the second job
      • The Tests tab will say no results are recorded
      • Exit Blue Ocean, and access it again from the second job
      • The Tests tab will show results

      I noticed this happening both while the job was running and complete.

      Attachments

        1. 01_Job1.png
          01_Job1.png
          74 kB
        2. 02_Job2.png
          02_Job2.png
          31 kB
        3. 03_Job2_Tests_Via_Job1.png
          03_Job2_Tests_Via_Job1.png
          32 kB
        4. 04_Job2_Direct.png
          04_Job2_Direct.png
          23 kB
        5. 05_Job2_Tests.png
          05_Job2_Tests.png
          26 kB
        6. 06_Job2_Tests.png
          06_Job2_Tests.png
          33 kB
        7. screenshot-bug.png
          screenshot-bug.png
          28 kB
        8. screenshot-okay.png
          screenshot-okay.png
          42 kB

        Issue Links

          Activity

            stuartrowe Stuart Rowe added a comment -

            I don't believe my change would have fixed this issue. It sounds like a similar bug though (retrieving and caching the run summary for the last run of a project leads to false cache hit when retrieving the full details for the same run). This code snippet looks suspicious:

            https://github.com/jenkinsci/blueocean-plugin/blob/3f049df3d253727d021e95ee53a4d1be228bf8cf/blueocean-rest-impl/src/main/java/io/jenkins/blueocean/service/embedded/rest/AbstractPipelineImpl.java#L106

            halkeye could you take a look / comment on this?

            stuartrowe Stuart Rowe added a comment - I don't believe my change would have fixed this issue. It sounds like a similar bug though (retrieving and caching the run summary for the last run of a project leads to false cache hit when retrieving the full details for the same run). This code snippet looks suspicious: https://github.com/jenkinsci/blueocean-plugin/blob/3f049df3d253727d021e95ee53a4d1be228bf8cf/blueocean-rest-impl/src/main/java/io/jenkins/blueocean/service/embedded/rest/AbstractPipelineImpl.java#L106 halkeye could you take a look / comment on this?
            halkeye Gavin Mogan added a comment -

            stuartrowe I'm confused by your comment. Sounds like you suspect its not fixed, but reinholdfuereder says it is actually fixed for them?

            I agree, the null looks suspicious, but the PR it was merged in with (https://github.com/jenkinsci/blueocean-plugin/commit/1e05a5cf487cdb85112cbbb61d6ebf1f9cb889b4) says it was done for performance issues.

            I think a longer term better fix is making the tests page pull the data it needs if its not available, but i'm way less familiar with the actual frontend code here.

             

            halkeye Gavin Mogan added a comment - stuartrowe I'm confused by your comment. Sounds like you suspect its not fixed, but reinholdfuereder says it is actually fixed for them? I agree, the null looks suspicious, but the PR it was merged in with ( https://github.com/jenkinsci/blueocean-plugin/commit/1e05a5cf487cdb85112cbbb61d6ebf1f9cb889b4)  says it was done for performance issues. I think a longer term better fix is making the tests page pull the data it needs if its not available, but i'm way less familiar with the actual frontend code here.  
            stuartrowe Stuart Rowe added a comment - - edited

            I'm just saying that if it is fixed, it's not due to my PR. I looped you in in case there were any other recent BO changes that came to mind. I don't have jobs that produce xUnit style output so I haven't noticed this issue and can't confirm a fix.

            stuartrowe Stuart Rowe added a comment - - edited I'm just saying that if it is fixed, it's not due to my PR. I looped you in in case there were any other recent BO changes that came to mind. I don't have jobs that produce xUnit style output so I haven't noticed this issue and can't confirm a fix.
            halkeye Gavin Mogan added a comment -

            Okay cool, I agree, i don't think its related to your change, but its a complicated system so who knows.

            I'm leaving this as resolved! 

            halkeye Gavin Mogan added a comment - Okay cool, I agree, i don't think its related to your change, but its a complicated system so who knows. I'm leaving this as resolved! 

            stuartrowe and halkeye: Sorry for the confusion. I am really extremely pretty sure that the problem I described in https://issues.jenkins-ci.org/browse/JENKINS-51111?focusedCommentId=343321&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-343321 (with rather straightforward reproducer steps) works now as expected (for "changes" AND for "tests") – and I think that was really the same problem as this issue's original description.

            However, I could also imagine that stuartrowe is perfectly right about that he only fixed the "changes" now in the course of JENKINS-55497, and "tests" was already fixed some time ago...

            reinholdfuereder Reinhold Füreder added a comment - stuartrowe and halkeye : Sorry for the confusion. I am really extremely pretty sure that the problem I described in https://issues.jenkins-ci.org/browse/JENKINS-51111?focusedCommentId=343321&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-343321 (with rather straightforward reproducer steps) works now as expected (for "changes" AND for "tests") – and I think that was really the same problem as this issue's original description. However, I could also imagine that stuartrowe is perfectly right about that he only fixed the "changes" now in the course of JENKINS-55497 , and "tests" was already fixed some time ago...

            People

              sophistifunk Josh McDonald
              woolzer Roger Wooley
              Votes:
              2 Vote for this issue
              Watchers:
              9 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: