Branch: refs/heads/master Home: https://github.com/jenkinsci/pipeline-model-definition-plugin Commit: d07d3501b898cf3087716e12004021d8c703efea https://github.com/jenkinsci/pipeline-model-definition-plugin/commit/d07d3501b898cf3087716e12004021d8c703efea Author: Andrew Bayer <andrew.ba...@gmail.com> Date: 2017-06-01 (Thu, 01 Jun 2017)
Changed paths: M pipeline-model-definition/src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/Utils.groovy M pipeline-model-definition/src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/Options.groovy M pipeline-model-definition/src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/Parameters.groovy M pipeline-model-definition/src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/Triggers.groovy A pipeline-model-definition/src/main/java/org/jenkinsci/plugins/pipeline/modeldefinition/actions/JobPropertyTrackerAction.java M pipeline-model-definition/src/main/resources/org/jenkinsci/plugins/pipeline/modeldefinition/ModelInterpreter.groovy M pipeline-model-definition/src/test/java/org/jenkinsci/plugins/pipeline/modeldefinition/OptionsTest.java M pipeline-model-definition/src/test/java/org/jenkinsci/plugins/pipeline/modeldefinition/ParametersTest.java M pipeline-model-definition/src/test/java/org/jenkinsci/plugins/pipeline/modeldefinition/TriggersTest.java Log Message: ----------- [FIXED JENKINS-44621] Only remove job properties from Jenkinsfiles Rather than actually using the properties(...) step directly, we instead will now do this behind the scenes by interacting with the job directly. We'll also record what job properties, triggers, and parameters were defined in the Jenkinsfile for a given run, so that we can refer back in the next build to see what existing job properties, triggers, or parameters were defined through the Jenkinsfile vs defined outside of the Jenkinsfile. This way, we can preserve the defined-outside-Jenkinsfile properties et al. Note that this won't *quite* work right if the first run of a job after upgrading removes triggers/properties/etc, because there is no record for builds beforehand, so all properties etc on the job at the time of upgrade will be treated as if they were defined externally. However, after that first build, you can remove properties etc from the Jenkinsfile with the expected behavior. Commit: b82b19a5b48286dceda14de822ef8aea3b2237c9 https://github.com/jenkinsci/pipeline-model-definition-plugin/commit/b82b19a5b48286dceda14de822ef8aea3b2237c9 Author: Andrew Bayer <andrew.ba...@gmail.com> Date: 2017-06-07 (Wed, 07 Jun 2017) Changed paths: M pipeline-model-definition/src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/Utils.groovy M pipeline-model-definition/src/main/java/org/jenkinsci/plugins/pipeline/modeldefinition/actions/JobPropertyTrackerAction.java Log Message: ----------- Review comments-related cleanup Commit: c5083a16f51b24434fe730172531c56c185c7000 https://github.com/jenkinsci/pipeline-model-definition-plugin/commit/c5083a16f51b24434fe730172531c56c185c7000 Author: Andrew Bayer <andrew.ba...@gmail.com> Date: 2017-06-07 (Wed, 07 Jun 2017) Changed paths: M pipeline-model-definition/src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/Utils.groovy A pipeline-model-definition/src/main/java/org/jenkinsci/plugins/pipeline/modeldefinition/actions/DeclarativeJobPropertyTrackerAction.java R pipeline-model-definition/src/main/java/org/jenkinsci/plugins/pipeline/modeldefinition/actions/JobPropertyTrackerAction.java M pipeline-model-definition/src/test/java/org/jenkinsci/plugins/pipeline/modeldefinition/ParametersTest.java Log Message: ----------- Renamed tracker action, simplified tests a wee bit Commit: b68f6a96d881fc5377fa2ad35d6e952a9846ea79 https://github.com/jenkinsci/pipeline-model-definition-plugin/commit/b68f6a96d881fc5377fa2ad35d6e952a9846ea79 Author: Andrew Bayer <andrew.ba...@gmail.com> Date: 2017-06-08 (Thu, 08 Jun 2017) Changed paths: M pipeline-model-definition/src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/Utils.groovy M pipeline-model-definition/src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/Options.groovy M pipeline-model-definition/src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/Parameters.groovy M pipeline-model-definition/src/main/groovy/org/jenkinsci/plugins/pipeline/modeldefinition/model/Triggers.groovy A pipeline-model-definition/src/main/java/org/jenkinsci/plugins/pipeline/modeldefinition/actions/DeclarativeJobPropertyTrackerAction.java M pipeline-model-definition/src/main/resources/org/jenkinsci/plugins/pipeline/modeldefinition/ModelInterpreter.groovy M pipeline-model-definition/src/test/java/org/jenkinsci/plugins/pipeline/modeldefinition/OptionsTest.java M pipeline-model-definition/src/test/java/org/jenkinsci/plugins/pipeline/modeldefinition/ParametersTest.java M pipeline-model-definition/src/test/java/org/jenkinsci/plugins/pipeline/modeldefinition/TriggersTest.java Log Message: ----------- Merge pull request #160 from abayer/jenkins-44621 [FIXED JENKINS-44621] Only remove job properties from Jenkinsfiles Compare: https://github.com/jenkinsci/pipeline-model-definition-plugin/compare/f0dcd95e0238...b68f6a96d881 -- You received this message because you are subscribed to the Google Groups "Jenkins Commits" group. To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-commits+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.