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

TFS plugin broken since 2.264 (tables to divs)

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      If I want to save my multi project job I get the error/callstack from attached "exception.txt". Please find attached the project file (minimalized to get this error). If I open it and press "apply" or "save" I got this error. This issue starts with 2.264. looks similar to issue https://issues.jenkins.io/browse/JENKINS-5118

       

      BTW pressing "apply" opens a new frame with error where the link to get callstack is not working

       

        Attachments

        1. config.xml
          0.9 kB
        2. exception.txt
          14 kB
        3. jenkinsPlugins.txt
          8 kB
        4. jenkinsPluginsWithCurl.txt
          3 kB
        5. screenshot_apply.png
          screenshot_apply.png
          28 kB
        6. screenshot_save.png
          screenshot_save.png
          11 kB

          Issue Links

            Activity

            arbeita Ronny Borchert created issue -
            arbeita Ronny Borchert made changes -
            Field Original Value New Value
            Description If I want to save my multi project job I get the error/callstack from attached "exception.txt". Please find attached the project file. If I open it and press "apply" or "save" I got this error.

             

            BTW pressing "apply" opens a new frame with error where the link to get callstack is not working

             
            If I want to save my multi project job I get the error/callstack from attached "exception.txt". Please find attached the project file (minimalized to get this error). If I open it and press "apply" or "save" I got this error.

             

            BTW pressing "apply" opens a new frame with error where the link to get callstack is not working

             
            arbeita Ronny Borchert made changes -
            Description If I want to save my multi project job I get the error/callstack from attached "exception.txt". Please find attached the project file (minimalized to get this error). If I open it and press "apply" or "save" I got this error.

             

            BTW pressing "apply" opens a new frame with error where the link to get callstack is not working

             
            If I want to save my multi project job I get the error/callstack from attached "exception.txt". Please find attached the project file (minimalized to get this error). If I open it and press "apply" or "save" I got this error. This issue starts with 2.264.

             

            BTW pressing "apply" opens a new frame with error where the link to get callstack is not working

             
            arbeita Ronny Borchert made changes -
            Description If I want to save my multi project job I get the error/callstack from attached "exception.txt". Please find attached the project file (minimalized to get this error). If I open it and press "apply" or "save" I got this error. This issue starts with 2.264.

             

            BTW pressing "apply" opens a new frame with error where the link to get callstack is not working

             
            If I want to save my multi project job I get the error/callstack from attached "exception.txt". Please find attached the project file (minimalized to get this error). If I open it and press "apply" or "save" I got this error. This issue starts with 2.264. looks similar to issue https://issues.jenkins.io/browse/JENKINS-5118

             

            BTW pressing "apply" opens a new frame with error where the link to get callstack is not working

             
            Hide
            oleg_nenashev Oleg Nenashev added a comment -

            Looks to be a table to divs issue

            Show
            oleg_nenashev Oleg Nenashev added a comment - Looks to be a table to divs issue
            oleg_nenashev Oleg Nenashev made changes -
            Labels tables-to-divs-regression
            Hide
            timja Tim Jacomb added a comment -

            Hi please see https://issues.jenkins.io/browse/JENKINS-64072

            Without a full plugins list and a screenshot we can't reproduce this

            Show
            timja Tim Jacomb added a comment - Hi please see https://issues.jenkins.io/browse/JENKINS-64072 Without a full plugins list and a screenshot we can't reproduce this
            arbeita Ronny Borchert made changes -
            Attachment jenkinsPlugins.txt [ 53445 ]
            arbeita Ronny Borchert made changes -
            Attachment screenshot_save.png [ 53446 ]
            arbeita Ronny Borchert made changes -
            Attachment screenshot_apply.png [ 53447 ]
            Hide
            arbeita Ronny Borchert added a comment -

            I added screenshots and plugin list. The issue appears only on press "Save" or "Apply".

            Show
            arbeita Ronny Borchert added a comment - I added screenshots and plugin list. The issue appears only on press "Save" or "Apply".
            Hide
            timja Tim Jacomb added a comment -

            Can you use the script please:

            JENKINS_HOST=username:password@myhost.com:port
            curl -sSL "http://$JENKINS_HOST/pluginManager/api/xml?depth=1&xpath=/*/*/shortName|/*/*/version&wrapper=plugins" | perl -pe 's/.*?<shortName>([\w-]+).*?<version>([^<]+)()(<\/\w+>)+/\1 \2\n/g'|sed 's/ /:/'
            
            Show
            timja Tim Jacomb added a comment - Can you use the script please: JENKINS_HOST=username:password@myhost.com:port curl -sSL "http: //$JENKINS_HOST/pluginManager/api/xml?depth=1&xpath= /*/*/ shortName| /*/*/ version&wrapper=plugins" | perl -pe 's/.*?<shortName>([\w-]+).*?<version>([^<]+)()(<\/\w+>)+/\1 \2\n/g' |sed 's/ /:/'
            arbeita Ronny Borchert made changes -
            Attachment jenkinsPluginsWithCurl.txt [ 53464 ]
            Hide
            arbeita Ronny Borchert added a comment -

            I attached it now. Having a ! in password is bad for seldom bash user , thats why I did it with the jenkins groovy script via console.

            Show
            arbeita Ronny Borchert added a comment - I attached it now. Having a ! in password is bad for seldom bash user , thats why I did it with the jenkins groovy script via console.
            Hide
            timja Tim Jacomb added a comment -

            Ronny Borchert creating an API token is probably a safer way, but thanks

            Show
            timja Tim Jacomb added a comment - Ronny Borchert creating an API token is probably a safer way, but thanks
            timja Tim Jacomb made changes -
            Labels tables-to-divs-regression tables-to-divs-regression tfs-plugin
            Hide
            timja Tim Jacomb added a comment -

            This appears to be the TFS plugin which has been suspended due to licensing issues and is not available in new installations (I had to build it from source to test it as I couldn't download it)

            see https://issues.jenkins.io/browse/INFRA-2751

            I don't intend to patch this myself but you or someone else could provide a patch and build from source or CI.
            It won't be possible for a release to be done until the licensing issue is addressed

            Show
            timja Tim Jacomb added a comment - This appears to be the TFS plugin which has been suspended due to licensing issues and is not available in new installations (I had to build it from source to test it as I couldn't download it) see https://issues.jenkins.io/browse/INFRA-2751 I don't intend to patch this myself but you or someone else could provide a patch and build from source or CI. It won't be possible for a release to be done until the licensing issue is addressed
            timja Tim Jacomb made changes -
            Labels tables-to-divs-regression tfs-plugin tables-to-divs-regression
            timja Tim Jacomb made changes -
            Component/s tfs-plugin [ 15521 ]
            Component/s core [ 15593 ]
            Hide
            arbeita Ronny Borchert added a comment -

            I could give it a try, my problem is only that if I would create a pull request nobody will merge it. My merge request is still open for a simple none code change.

            Show
            arbeita Ronny Borchert added a comment - I could give it a try, my problem is only that if I would create a pull request nobody will merge it. My merge request is still open for a simple none code change.
            timja Tim Jacomb made changes -
            Link This issue is duplicated by JENKINS-64797 [ JENKINS-64797 ]
            timja Tim Jacomb made changes -
            Summary JSONObject["hudson-triggers-SCMTrigger"] is not a JSONObject TFS plugin broken since 2.263
            timja Tim Jacomb made changes -
            Link This issue is duplicated by JENKINS-64286 [ JENKINS-64286 ]
            timja Tim Jacomb made changes -
            Link This issue is duplicated by JENKINS-64980 [ JENKINS-64980 ]
            basil Basil Crow made changes -
            Link This issue is duplicated by JENKINS-65141 [ JENKINS-65141 ]
            markewaite Mark Waite made changes -
            Link This issue is duplicated by JENKINS-65164 [ JENKINS-65164 ]
            markewaite Mark Waite made changes -
            Summary TFS plugin broken since 2.263 TFS plugin broken since 2.264 (tables to divs)
            basil Basil Crow made changes -
            Link This issue is duplicated by JENKINS-65180 [ JENKINS-65180 ]
            markewaite Mark Waite made changes -
            Link This issue is duplicated by JENKINS-65276 [ JENKINS-65276 ]
            arbeita Ronny Borchert made changes -
            Labels tables-to-divs-regression regression tables-to-divs-regression
            Hide
            arbeita Ronny Borchert added a comment -

            I could figure out that on save/apply the send JSON is different than before. 

            hudson-triggers-SCMTrigger":true

            is no array

            This is the root cause, but could not find any jelly issue by my self. Is there now a maintainer for tfs plugin? Since the license things seems resolved. But nobody merge in pull requests on github.

            Show
            arbeita Ronny Borchert added a comment - I could figure out that on save/apply the send JSON is different than before.  hudson-triggers-SCMTrigger":true is no array This is the root cause, but could not find any jelly issue by my self. Is there now a maintainer for tfs plugin? Since the license things seems resolved. But nobody merge in pull requests on github.
            Hide
            markewaite Mark Waite added a comment -

            Ronny Borchert I've not seen anyone asking to adopt the TFS plugin.

            Since you're submitting pull requests and you have access to a TFS environment where you can test it, it seems like a good case for you to offer to adopt the plugin. If you're comfortable submitting pull requests, you'll probably be comfortable as the maintainer. Thanks in advance if you choose to adopt it. I'm sure there are many users that will thank you.

            Show
            markewaite Mark Waite added a comment - Ronny Borchert I've not seen anyone asking to adopt the TFS plugin. Since you're submitting pull requests and you have access to a TFS environment where you can test it, it seems like a good case for you to offer to adopt the plugin . If you're comfortable submitting pull requests, you'll probably be comfortable as the maintainer. Thanks in advance if you choose to adopt it. I'm sure there are many users that will thank you.
            Hide
            arbeita Ronny Borchert added a comment -

            Thanks for your honor. But I have not enough spare time to fix it. Since as you can see, I try it since december 2020, 4 month, no real result.

            Show
            arbeita Ronny Borchert added a comment - Thanks for your honor. But I have not enough spare time to fix it. Since as you can see, I try it since december 2020, 4 month, no real result.
            markewaite Mark Waite made changes -
            Link This issue is related to JENKINS-65304 [ JENKINS-65304 ]
            Hide
            glevy Gilad added a comment -

            Did any one by chance looked at the code and can suggest the required changes?

            Being able to build an unofficial fixed version will help remove some pressure on setups like ours where this plugin is mandatory for production.

            I'm sure it will be appreciated by many users. 

             

            Show
            glevy Gilad added a comment - Did any one by chance looked at the code and can suggest the required changes? Being able to build an unofficial fixed version will help remove some pressure on setups like ours where this plugin is mandatory for production. I'm sure it will be appreciated by many users.   
            Hide
            arbeita Ronny Borchert added a comment - - edited

            Somehow this issue has been resolved by new jenkins version or plugins (except Tfs plugin) . Can anybody confirm?

            Show
            arbeita Ronny Borchert added a comment - - edited Somehow this issue has been resolved by new jenkins version or plugins (except Tfs plugin) . Can anybody confirm?
            Hide
            heldain Mikhail Platonov added a comment -

            Ronny Borchert Yes, I can confirm that issue has been resolved after upgrade to Jenkins 2.289.1 (plugins have been updated as well)

            Show
            heldain Mikhail Platonov added a comment - Ronny Borchert  Yes, I can confirm that issue has been resolved after upgrade to Jenkins 2.289.1 (plugins have been updated as well)
            basil Basil Crow made changes -
            Link This issue is duplicated by JENKINS-65867 [ JENKINS-65867 ]
            Hide
            caseydaniell Casey added a comment -

            I am confirming that this is fixed in 2.303.1 too. We can now upgrade our Jenkins instances to newer versions and keep using the (insecure, but mandatory and needed) TFS plugins.

            Show
            caseydaniell Casey added a comment - I am confirming that this is fixed in 2.303.1 too. We can now upgrade our Jenkins instances to newer versions and keep using the (insecure, but mandatory and needed) TFS plugins.
            Hide
            glevy Gilad added a comment -

            Actually, it is totally broken on 2.303.1 because of the removal of Apache Commons Digester from the Jenkins core.
            https://www.jenkins.io/doc/upgrade-guide/2.303/

            I had to fix and build it locally to make it work.

            Show
            glevy Gilad added a comment - Actually, it is totally broken on 2.303.1 because of the removal of Apache Commons Digester from the Jenkins core. https://www.jenkins.io/doc/upgrade-guide/2.303/ I had to fix and build it locally to make it work.
            Hide
            ianw Ian Williams added a comment -

            While it would seem MS has completely abandoned support for this plugin, the impact of "removal of Apache Commons Digester from the Jenkins core" should be a separate issue from this. Gilad, could you raise a separate issue, and perhaps even a PR, or at least document any fixes you needed to address in the JIRA?

            Still hoping someone might step up to be a maintainer (and MS formally put it up for adoption).

            Show
            ianw Ian Williams added a comment - While it would seem MS has completely abandoned support for this plugin, the impact of "removal of Apache Commons Digester from the Jenkins core" should be a separate issue from this. Gilad , could you raise a separate issue, and perhaps even a PR, or at least document any fixes you needed to address in the JIRA? Still hoping someone might step up to be a maintainer (and MS formally put it up for adoption).
            Show
            glevy Gilad added a comment - Thanks Ian Williams , I found a related issue and added a link to my fix in a new comment: https://issues.jenkins.io/browse/JENKINS-65867?focusedCommentId=413483&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-413483

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              arbeita Ronny Borchert
              Votes:
              11 Vote for this issue
              Watchers:
              17 Start watching this issue

                Dates

                Created:
                Updated: