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

Allow stashing/unstashing empty sets

    Details

    • Similar Issues:

      Description

      The stash step fails if there's no files are selected to be stashed. Allowing that to optionally work would make it easier to make Jenkinsfile more generic, by stashing files that might not get generated on some repositories, instead of passing information around with fields, or catching failure when unstashing (which still leaves the corresponding log marked as Failure in the build).

        Attachments

          Issue Links

            Activity

            Hide
            maxzilla Max Wahler added a comment -

            Moreover, stash doesn't include empty directories that might be used after an unstash.

            Show
            maxzilla Max Wahler added a comment - Moreover, stash doesn't include empty directories that might be used after an unstash .
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Iain Adams
            Path:
            src/main/java/org/jenkinsci/plugins/workflow/flow/StashManager.java
            http://jenkins-ci.org/commit/workflow-api-plugin/9e3a0d00ce97280a906018a82a5b1cf3b745f62c
            Log:
            JENKINS-37327 allow empty stash.

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Iain Adams Path: src/main/java/org/jenkinsci/plugins/workflow/flow/StashManager.java http://jenkins-ci.org/commit/workflow-api-plugin/9e3a0d00ce97280a906018a82a5b1cf3b745f62c Log: JENKINS-37327 allow empty stash.
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Jesse Glick
            Path:
            src/main/java/org/jenkinsci/plugins/workflow/flow/StashManager.java
            http://jenkins-ci.org/commit/workflow-api-plugin/b19922c71d411c3d0032862dee3a971cc03d36f0
            Log:
            Merge pull request #35 from iwarapter/master

            JENKINS-37327 allow empty stash.

            Compare: https://github.com/jenkinsci/workflow-api-plugin/compare/8c74d216226c...b19922c71d41

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Jesse Glick Path: src/main/java/org/jenkinsci/plugins/workflow/flow/StashManager.java http://jenkins-ci.org/commit/workflow-api-plugin/b19922c71d411c3d0032862dee3a971cc03d36f0 Log: Merge pull request #35 from iwarapter/master JENKINS-37327 allow empty stash. Compare: https://github.com/jenkinsci/workflow-api-plugin/compare/8c74d216226c...b19922c71d41
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Iain Adams
            Path:
            pom.xml
            src/main/java/org/jenkinsci/plugins/workflow/support/steps/stash/StashStep.java
            src/main/resources/org/jenkinsci/plugins/workflow/support/steps/stash/StashStep/config.jelly
            src/test/java/org/jenkinsci/plugins/workflow/support/steps/stash/StashTest.java
            http://jenkins-ci.org/commit/workflow-basic-steps-plugin/9b2d10a70074d4768047f7c63a8b555a883e81d7
            Log:
            JENKINS-37327 allow empty stash.

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Iain Adams Path: pom.xml src/main/java/org/jenkinsci/plugins/workflow/support/steps/stash/StashStep.java src/main/resources/org/jenkinsci/plugins/workflow/support/steps/stash/StashStep/config.jelly src/test/java/org/jenkinsci/plugins/workflow/support/steps/stash/StashTest.java http://jenkins-ci.org/commit/workflow-basic-steps-plugin/9b2d10a70074d4768047f7c63a8b555a883e81d7 Log: JENKINS-37327 allow empty stash.
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Jesse Glick
            Path:
            pom.xml
            src/main/java/org/jenkinsci/plugins/workflow/support/steps/stash/StashStep.java
            src/main/resources/org/jenkinsci/plugins/workflow/support/steps/stash/StashStep/config.jelly
            src/test/java/org/jenkinsci/plugins/workflow/support/steps/stash/StashTest.java
            http://jenkins-ci.org/commit/workflow-basic-steps-plugin/e4266ac53595cb31f22df8c3d53fa781d8e108f1
            Log:
            Merge pull request #42 from iwarapter/master

            JENKINS-37327 allow empty stash.

            Compare: https://github.com/jenkinsci/workflow-basic-steps-plugin/compare/37d4589bc6bb...e4266ac53595

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Jesse Glick Path: pom.xml src/main/java/org/jenkinsci/plugins/workflow/support/steps/stash/StashStep.java src/main/resources/org/jenkinsci/plugins/workflow/support/steps/stash/StashStep/config.jelly src/test/java/org/jenkinsci/plugins/workflow/support/steps/stash/StashTest.java http://jenkins-ci.org/commit/workflow-basic-steps-plugin/e4266ac53595cb31f22df8c3d53fa781d8e108f1 Log: Merge pull request #42 from iwarapter/master JENKINS-37327 allow empty stash. Compare: https://github.com/jenkinsci/workflow-basic-steps-plugin/compare/37d4589bc6bb...e4266ac53595

              People

              • Assignee:
                Unassigned
                Reporter:
                dcsobral Daniel Sobral
              • Votes:
                1 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: