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

Virtual thread dump uses phony filenames

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      Discussion

      Currently thread dumps show WorkflowScript.groovy even when you are using Jenkinsfile (or some other CpsScmScriptDefinition filename), and Script1 etc. even when you used load to load a named file. This makes the thread dump confusing for a user, since the line numbers are correct but the filenames are meaningless.

        Attachments

          Issue Links

            Activity

            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Jesse Glick
            Path:
            cps/src/main/java/org/jenkinsci/plugins/workflow/cps/rerun/RerunAction.java
            http://jenkins-ci.org/commit/workflow-plugin/0b1b639e1b14feb1d488e18814487af33be2aec0
            Log:
            Noting JENKINS-31838.

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Jesse Glick Path: cps/src/main/java/org/jenkinsci/plugins/workflow/cps/rerun/RerunAction.java http://jenkins-ci.org/commit/workflow-plugin/0b1b639e1b14feb1d488e18814487af33be2aec0 Log: Noting JENKINS-31838 .
            Hide
            jglick Jesse Glick added a comment -

            Even more desirable for JENKINS-32727: we would like to refer to Jenkinsfile and servers.groovy, not Main script and Script1.

            Show
            jglick Jesse Glick added a comment - Even more desirable for JENKINS-32727 : we would like to refer to Jenkinsfile and servers.groovy , not Main script and Script1 .
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Jesse Glick
            Path:
            cps/src/main/java/org/jenkinsci/plugins/workflow/cps/rerun/RerunAction.java
            http://jenkins-ci.org/commit/workflow-cps-plugin/5cb03cabae608e761c6dd9f97cea29ae69b51427
            Log:
            Noting JENKINS-31838.
            Originally-Committed-As: 0b1b639e1b14feb1d488e18814487af33be2aec0

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Jesse Glick Path: cps/src/main/java/org/jenkinsci/plugins/workflow/cps/rerun/RerunAction.java http://jenkins-ci.org/commit/workflow-cps-plugin/5cb03cabae608e761c6dd9f97cea29ae69b51427 Log: Noting JENKINS-31838 . Originally-Committed-As: 0b1b639e1b14feb1d488e18814487af33be2aec0
            Hide
            abayer Andrew Bayer added a comment -

            So legit questions about how much of this we actually would want to implement - Declarative has been making assumptions about Jenkinsfile coming up as WorkflowScript. That'll change as of 1.2, but any earlier version of Declarative has that hardcoded. So changing this could have ramifications.

            That said, handling Script1 for load may make more sense. I don't think anything's depending on that naming.

            Show
            abayer Andrew Bayer added a comment - So legit questions about how much of this we actually would want to implement - Declarative has been making assumptions about Jenkinsfile coming up as WorkflowScript . That'll change as of 1.2, but any earlier version of Declarative has that hardcoded. So changing this could have ramifications. That said, handling Script1 for load may make more sense. I don't think anything's depending on that naming.

              People

              • Assignee:
                kohsuke Kohsuke Kawaguchi
                Reporter:
                jglick Jesse Glick
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated: