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

Publish Over CIFS job config mishandles publishWhenFailed in job's config.xml

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Open (View Workflow)
    • Priority: Minor
    • Resolution: Unresolved
    • Labels:
      None
    • Environment:
      Jenkins 2.263.4, Publish Over CIFS 0.16, openjdk version "1.8.0_282"
    • Similar Issues:

      Description

      Publish Over CIFS job config mishandles publishWhenFailed in job's config.xml

      If you open the job config page, set the publishWhenFailed attribute to true, and save the job config, the attribute value true is written in the job's config.xml file, as expected. 

      However, if you open the job config page again and look at the publishWhenFailed attribute, the value show up as false.  Then if you save the job config then, the attribute value in the job config.xml file is false. 

      As a result, you are going to end up with a publishWhenFailed value of false, unless you remember to reset it to true every time you edit the job config for any reason. 

      Examining the jobs config.xml file, I notice the publishWhenFailed attribute does not look like it is in the right place. 

      More info follows as comments and/or attachments.

        Attachments

          Activity

          Hide
          hushp1pt Tony Wallace added a comment -

          config.xml snippet: note the location of  publishWhenFailed 

                      <jenkins.plugins.publish__over__cifs.CifsPublisherPlugin plugin="publish-over-cifs@0.16">
                        <consolePrefix>CIFS: </consolePrefix>
                        <delegate plugin="publish-over@0.22">
                          <publishers>
                            <jenkins.plugins.publish__over__cifs.CifsPublisher plugin="publish-over-cifs@0.16">
                              <configName>Eco-Jenkins</configName>
                              <verbose>false</verbose>
                              <transfers>
                                <jenkins.plugins.publish__over__cifs.CifsTransfer>
                                  <remoteDirectory>${JCI_DEVICE_CLASS_PLUS}\${UJCI_TAG}\${BUILD_TAG}</remoteDirectory>
                                  <sourceFiles>logcat-*.log, crash.log</sourceFiles>
                                  <excludes>logcat.brief*</excludes>
                                  <removePrefix/>
                                  <remoteDirectorySDF>false</remoteDirectorySDF>
                                  <flatten>false</flatten>
                                  <cleanRemote>true</cleanRemote>
                                  <noDefaultExcludes>false</noDefaultExcludes>
                                  <makeEmptyDirs>false</makeEmptyDirs>
                                  <patternSeparator>[, ]+</patternSeparator>
                                </jenkins.plugins.publish__over__cifs.CifsTransfer>
                                <jenkins.plugins.publish__over__cifs.CifsTransfer>
                                  <remoteDirectory>${JCI_DEVICE_CLASS_PLUS}\${UJCI_TAG}\${BUILD_TAG}\state</remoteDirectory>
                                  <sourceFiles>jci/state/*.*</sourceFiles>
                                  <excludes>**/.gitignore</excludes>
                                  <removePrefix/>
                                  <remoteDirectorySDF>false</remoteDirectorySDF>
                                  <flatten>true</flatten>
                                  <cleanRemote>false</cleanRemote>
                                  <noDefaultExcludes>false</noDefaultExcludes>
                                  <makeEmptyDirs>false</makeEmptyDirs>
                                  <patternSeparator>[, ]+</patternSeparator>
                                </jenkins.plugins.publish__over__cifs.CifsTransfer>
                              </transfers>
                              <useWorkspaceInPromotion>false</useWorkspaceInPromotion>
                              <usePromotionTimestamp>false</usePromotionTimestamp>
                              <retry class="jenkins.plugins.publish_over_cifs.CifsRetry">
                                <retries>2</retries>
                                <retryDelay>15000</retryDelay>
                              </retry>
                            </jenkins.plugins.publish__over__cifs.CifsPublisher>
                          </publishers>
                          <continueOnError>false</continueOnError>
                          <failOnError>false</failOnError>
                          <alwaysPublishFromMaster>true</alwaysPublishFromMaster>
                          <hostConfigurationAccess class="jenkins.plugins.publish_over_cifs.CifsPublisherPlugin" reference="../.."/>
                        </delegate>
                        <publishWhenFailed>true</publishWhenFailed>
                      </jenkins.plugins.publish__over__cifs.CifsPublisherPlugin>
          
          Show
          hushp1pt Tony Wallace added a comment - config.xml snippet: note the location of  publishWhenFailed  <jenkins.plugins.publish__over__cifs.CifsPublisherPlugin plugin="publish-over-cifs@0.16"> <consolePrefix>CIFS: </consolePrefix> <delegate plugin="publish-over@0.22"> <publishers> <jenkins.plugins.publish__over__cifs.CifsPublisher plugin="publish-over-cifs@0.16"> <configName>Eco-Jenkins</configName> <verbose>false</verbose> <transfers> <jenkins.plugins.publish__over__cifs.CifsTransfer> <remoteDirectory>${JCI_DEVICE_CLASS_PLUS}\${UJCI_TAG}\${BUILD_TAG}</remoteDirectory> <sourceFiles>logcat-*.log, crash.log</sourceFiles> <excludes>logcat.brief*</excludes> <removePrefix/> <remoteDirectorySDF>false</remoteDirectorySDF> <flatten>false</flatten> <cleanRemote>true</cleanRemote> <noDefaultExcludes>false</noDefaultExcludes> <makeEmptyDirs>false</makeEmptyDirs> <patternSeparator>[, ]+</patternSeparator> </jenkins.plugins.publish__over__cifs.CifsTransfer> <jenkins.plugins.publish__over__cifs.CifsTransfer> <remoteDirectory>${JCI_DEVICE_CLASS_PLUS}\${UJCI_TAG}\${BUILD_TAG}\state</remoteDirectory> <sourceFiles>jci/state/*.*</sourceFiles> <excludes>**/.gitignore</excludes> <removePrefix/> <remoteDirectorySDF>false</remoteDirectorySDF> <flatten>true</flatten> <cleanRemote>false</cleanRemote> <noDefaultExcludes>false</noDefaultExcludes> <makeEmptyDirs>false</makeEmptyDirs> <patternSeparator>[, ]+</patternSeparator> </jenkins.plugins.publish__over__cifs.CifsTransfer> </transfers> <useWorkspaceInPromotion>false</useWorkspaceInPromotion> <usePromotionTimestamp>false</usePromotionTimestamp> <retry class="jenkins.plugins.publish_over_cifs.CifsRetry"> <retries>2</retries> <retryDelay>15000</retryDelay> </retry> </jenkins.plugins.publish__over__cifs.CifsPublisher> </publishers> <continueOnError>false</continueOnError> <failOnError>false</failOnError> <alwaysPublishFromMaster>true</alwaysPublishFromMaster> <hostConfigurationAccess class="jenkins.plugins.publish_over_cifs.CifsPublisherPlugin" reference="../.."/> </delegate> <publishWhenFailed>true</publishWhenFailed> </jenkins.plugins.publish__over__cifs.CifsPublisherPlugin>
          Hide
          hushp1pt Tony Wallace added a comment -

          the job config page, screen 1

          Show
          hushp1pt Tony Wallace added a comment - the job config page, screen 1
          Hide
          hushp1pt Tony Wallace added a comment -

          Show
          hushp1pt Tony Wallace added a comment -

            People

            Assignee:
            Unassigned Unassigned
            Reporter:
            hushp1pt Tony Wallace
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Dates

              Created:
              Updated: