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

REGRESSION detail view log links for steps only are broken

    Details

    • Similar Issues:
    • Sprint:
      Blue Ocean 1.0-rc3

      Description

      Go to https://ci.blueocean.io/blue/organizations/jenkins/blueocean/detail/master/797/pipeline/

       

      Press either log button both lead to 404 urls https://ci.blueocean.io/blue/rest/organizations/jenkins/pipelines/blueocean/branches/master/runs/797/nodes/undefined/log/?start=0

       

      See the 'undefined' in the path that is because

      const logUrl = shouldShowCV ? logGeneral.url : `${nodeKey}${this.mergedConfig.node}/log/`;
      

      only works if there is a node.

        Attachments

          Activity

          tscherler Thorsten Scherler created issue -
          jamesdumay James Dumay made changes -
          Field Original Value New Value
          Sprint Blue Ocean 1.0-rc2 [ 261 ]
          jamesdumay James Dumay made changes -
          Rank Ranked higher
          jamesdumay James Dumay made changes -
          Priority Minor [ 4 ] Critical [ 2 ]
          jamesdumay James Dumay made changes -
          Assignee Ivan Meredith [ imeredith ]
          michaelneale Michael Neale made changes -
          Description Go to [https://ci.blueocean.io/blue/organizations/jenkins/blueocean/detail/master/797/pipeline/]

           

          Press either log button both lead to 404 urls https://ci.blueocean.io/blue/rest/organizations/jenkins/pipelines/blueocean/branches/master/runs/797/nodes/undefined/log/?start=0

           

          See the 'undefined' in the path that is because
          {code:java}
          const logUrl = shouldShowCV ? logGeneral.url : `${nodeKey}${this.mergedConfig.node}/log/`;
          {code}
          only works if there is a node.
          Go to [https://ci.blueocean.io/blue/organizations/jenkins/blueocean/detail/master/797/pipeline/]

           

          Press either log button both lead to 404 urls [https://ci.blueocean.io/blue/rest/organizations/jenkins/pipelines/blueocean/branches/master/runs/797/nodes/undefined/log/?start=0]

           

          See the 'undefined' in the path that is because
          {code:java}
          const logUrl = shouldShowCV ? logGeneral.url : `${nodeKey}${this.mergedConfig.node}/log/`;
          {code}
          only works if there is a node.

           

          Expected behavior: 

          should default to showing full log in these cases (ie no steps). 

           
          jamesdumay James Dumay made changes -
          Epic Link JENKINS-35750 [ 171713 ]
          jamesdumay James Dumay made changes -
          Description Go to [https://ci.blueocean.io/blue/organizations/jenkins/blueocean/detail/master/797/pipeline/]

           

          Press either log button both lead to 404 urls [https://ci.blueocean.io/blue/rest/organizations/jenkins/pipelines/blueocean/branches/master/runs/797/nodes/undefined/log/?start=0]

           

          See the 'undefined' in the path that is because
          {code:java}
          const logUrl = shouldShowCV ? logGeneral.url : `${nodeKey}${this.mergedConfig.node}/log/`;
          {code}
          only works if there is a node.

           

          Expected behavior: 

          should default to showing full log in these cases (ie no steps). 

           
          Go to [https://ci.blueocean.io/blue/organizations/jenkins/blueocean/detail/master/797/pipeline/]

           

          Press either log button both lead to 404 urls https://ci.blueocean.io/blue/rest/organizations/jenkins/pipelines/blueocean/branches/master/runs/797/nodes/undefined/log/?start=0

           

          See the 'undefined' in the path that is because
          {code:java}
          const logUrl = shouldShowCV ? logGeneral.url : `${nodeKey}${this.mergedConfig.node}/log/`;
          {code}
          only works if there is a node.
          michaelneale Michael Neale made changes -
          Rank Ranked higher
          michaelneale Michael Neale made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          jbriden Jenn Briden made changes -
          Status Resolved [ 5 ] Closed [ 6 ]

            People

            • Assignee:
              imeredith Ivan Meredith
              Reporter:
              tscherler Thorsten Scherler
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: