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

JSON serialization should not fail if a plugins action failed to serialize

    Details

    • Similar Issues:
    • Epic Link:
    • Sprint:
      frank, tethys, christmas, iapetus, 1.0

      Description

      Context
      If there is a problem with the serialization of an Action then the rest response should not fail. We are unable to control 1000s of plugins and it would be a shame if a single plugin failed and made Blue Ocean inaccessible.
      A good example of this happening is JENKINS-40083.

      If the serialization of an action fails, the logs should identify what plugin failed and link to its wiki page (I believe all of this info is available via the Plugin Manager API) and ask them to report the stack trace as a bug for that project.

      Solution

        Attachments

          Issue Links

            Activity

            jamesdumay James Dumay created issue -
            jamesdumay James Dumay made changes -
            Field Original Value New Value
            Description If there is a problem with the serialization of an Action then the rest response should not fail. We are unable to control 1000s of plugins and it would be a shame if a single plugin failed and made Blue Ocean inaccessible.

            A good example of this happening is JENKINS-40083.

            If the serialization of an action fails, the logs should identify what plugin failed and link to its wiki page (I believe all of this info is available via the Plugin Manager API).
            If there is a problem with the serialization of an Action then the rest response should not fail. We are unable to control 1000s of plugins and it would be a shame if a single plugin failed and made Blue Ocean inaccessible.

            A good example of this happening is JENKINS-40083.

            If the serialization of an action fails, the logs should identify what plugin failed and link to its wiki page (I believe all of this info is available via the Plugin Manager API) and ask them to report the stack trace as a bug for that project.
            jamesdumay James Dumay made changes -
            Epic Link JENKINS-35759 [ 171771 ]
            jamesdumay James Dumay made changes -
            Link This issue blocks JENKINS-39647 [ JENKINS-39647 ]
            jamesdumay James Dumay made changes -
            Sprint tethys [ 161 ] frank [ 156 ]
            jamesdumay James Dumay made changes -
            Rank Ranked higher
            jamesdumay James Dumay made changes -
            Rank Ranked higher
            jamesdumay James Dumay made changes -
            Description If there is a problem with the serialization of an Action then the rest response should not fail. We are unable to control 1000s of plugins and it would be a shame if a single plugin failed and made Blue Ocean inaccessible.

            A good example of this happening is JENKINS-40083.

            If the serialization of an action fails, the logs should identify what plugin failed and link to its wiki page (I believe all of this info is available via the Plugin Manager API) and ask them to report the stack trace as a bug for that project.
            If there is a problem with the serialization of an Action then the rest response should not fail. We are unable to control 1000s of plugins and it would be a shame if a single plugin failed and made Blue Ocean inaccessible.
            A good example of this happening is JENKINS-40083.

            If the serialization of an action fails, the logs should identify what plugin failed and link to its wiki page (I believe all of this info is available via the Plugin Manager API) and ask them to report the stack trace as a bug for that project.
            Hide
            michaelneale Michael Neale added a comment -

            James Dumay do you want to write up to the mailing list what your proposed fix could be for this?

            Show
            michaelneale Michael Neale added a comment - James Dumay do you want to write up to the mailing list what your proposed fix could be for this?
            jamesdumay James Dumay made changes -
            Description If there is a problem with the serialization of an Action then the rest response should not fail. We are unable to control 1000s of plugins and it would be a shame if a single plugin failed and made Blue Ocean inaccessible.
            A good example of this happening is JENKINS-40083.

            If the serialization of an action fails, the logs should identify what plugin failed and link to its wiki page (I believe all of this info is available via the Plugin Manager API) and ask them to report the stack trace as a bug for that project.
            *Context*
            If there is a problem with the serialization of an Action then the rest response should not fail. We are unable to control 1000s of plugins and it would be a shame if a single plugin failed and made Blue Ocean inaccessible.
            A good example of this happening is JENKINS-40083.

            If the serialization of an action fails, the logs should identify what plugin failed and link to its wiki page (I believe all of this info is available via the Plugin Manager API) and ask them to report the stack trace as a bug for that project.

            jamesdumay James Dumay made changes -
            Description *Context*
            If there is a problem with the serialization of an Action then the rest response should not fail. We are unable to control 1000s of plugins and it would be a shame if a single plugin failed and made Blue Ocean inaccessible.
            A good example of this happening is JENKINS-40083.

            If the serialization of an action fails, the logs should identify what plugin failed and link to its wiki page (I believe all of this info is available via the Plugin Manager API) and ask them to report the stack trace as a bug for that project.

            *Context*
            If there is a problem with the serialization of an Action then the rest response should not fail. We are unable to control 1000s of plugins and it would be a shame if a single plugin failed and made Blue Ocean inaccessible.
            A good example of this happening is JENKINS-40083.

            If the serialization of an action fails, the logs should identify what plugin failed and link to its wiki page (I believe all of this info is available via the Plugin Manager API) and ask them to report the stack trace as a bug for that project.

            *Solution*

            jamesdumay James Dumay made changes -
            Priority Major [ 3 ] Critical [ 2 ]
            jamesdumay James Dumay made changes -
            Remote Link This issue links to "White screen of death (Web Link)" [ 15149 ]
            jamesdumay James Dumay made changes -
            Sprint frank [ 156 ] frank, christmas [ 156, 176 ]
            jamesdumay James Dumay made changes -
            Rank Ranked lower
            jamesdumay James Dumay made changes -
            Rank Ranked higher
            jamesdumay James Dumay made changes -
            Link This issue duplicates JENKINS-39647 [ JENKINS-39647 ]
            jamesdumay James Dumay made changes -
            Status Open [ 1 ] Resolved [ 5 ]
            Resolution Duplicate [ 3 ]
            Hide
            michaelneale Michael Neale added a comment -

            This can't be closed as it is linked to as the resolving issue

            Show
            michaelneale Michael Neale added a comment - This can't be closed as it is linked to as the resolving issue
            michaelneale Michael Neale made changes -
            Resolution Duplicate [ 3 ]
            Status Resolved [ 5 ] Reopened [ 4 ]
            michaelneale Michael Neale made changes -
            Sprint frank, christmas [ 156, 176 ] frank, tethys, christmas [ 156, 161, 176 ]
            michaelneale Michael Neale made changes -
            Priority Critical [ 2 ] Major [ 3 ]
            Hide
            jglick Jesse Glick added a comment -

            Linking to a proposed fix, or prototype of fix.

            Show
            jglick Jesse Glick added a comment - Linking to a proposed fix, or prototype of fix.
            jglick Jesse Glick made changes -
            Remote Link This issue links to "Stapler PR 89 (Web Link)" [ 15234 ]
            Hide
            jglick Jesse Glick added a comment -

            JENKINS-40083 was an instance of this IIUC?

            Show
            jglick Jesse Glick added a comment - JENKINS-40083 was an instance of this IIUC?
            jglick Jesse Glick made changes -
            Link This issue relates to JENKINS-40083 [ JENKINS-40083 ]
            jglick Jesse Glick made changes -
            Labels rest robustness stapler
            Hide
            jglick Jesse Glick added a comment -

            Filing under core as there (inside Stapler) is where the problem lies; whether or not Blue Ocean plugins should be contributing to the exact fix is under discussion. Moved lts-candidate label from JENKINS-40083; it missed 2.32.2, but if we act fast it might still make it into 2.32.3.

            Show
            jglick Jesse Glick added a comment - Filing under core as there (inside Stapler) is where the problem lies; whether or not Blue Ocean plugins should be contributing to the exact fix is under discussion. Moved lts-candidate label from JENKINS-40083 ; it missed 2.32.2, but if we act fast it might still make it into 2.32.3.
            jglick Jesse Glick made changes -
            Component/s core [ 15593 ]
            Component/s blueocean-plugin [ 21481 ]
            Labels rest robustness stapler blueocean lts-candidate rest robustness stapler
            jamesdumay James Dumay made changes -
            Component/s blueocean-plugin [ 21481 ]
            Hide
            jamesdumay James Dumay added a comment -

            Jesse Glick adding back the blueocean-plugin component so we can track this in our backlog more easily, but yes I agree this is a core issue.

            Show
            jamesdumay James Dumay added a comment - Jesse Glick adding back the blueocean-plugin component so we can track this in our backlog more easily, but yes I agree this is a core issue.
            jglick Jesse Glick made changes -
            Status Reopened [ 4 ] Open [ 1 ]
            jglick Jesse Glick made changes -
            Status Open [ 1 ] In Progress [ 3 ]
            jglick Jesse Glick made changes -
            Status In Progress [ 3 ] In Review [ 10005 ]
            jglick Jesse Glick made changes -
            Issue Type Task [ 3 ] Bug [ 1 ]
            jglick Jesse Glick made changes -
            Assignee Vivek Pandey [ vivek ] Jesse Glick [ jglick ]
            Hide
            jglick Jesse Glick added a comment -

            Agreement with Vivek Pandey and Michael Neale was to start with the simple defense against having the entire response end with a 500, and then have a follow-up PR (best tracked in a linked issue I think) to create an SPI for customized serialization that Blue Ocean would implement to ensure that its own classes are not tolerant of errors.

            Show
            jglick Jesse Glick added a comment - Agreement with Vivek Pandey and Michael Neale was to start with the simple defense against having the entire response end with a 500, and then have a follow-up PR (best tracked in a linked issue I think) to create an SPI for customized serialization that Blue Ocean would implement to ensure that its own classes are not tolerant of errors.
            jglick Jesse Glick made changes -
            Remote Link This issue links to "core PR 2722 (Web Link)" [ 15284 ]
            jamesdumay James Dumay made changes -
            Sprint frank, tethys, christmas [ 156, 161, 176 ] frank, tethys, christmas, pannonian [ 156, 161, 176, 211 ]
            Hide
            jamesdumay James Dumay added a comment -

            Vivek Pandey this is the WSOD issue

            Show
            jamesdumay James Dumay added a comment - Vivek Pandey this is the WSOD issue
            jamesdumay James Dumay made changes -
            Assignee Jesse Glick [ jglick ] Vivek Pandey [ vivek ]
            jamesdumay James Dumay made changes -
            Status In Review [ 10005 ] In Progress [ 3 ]
            jamesdumay James Dumay made changes -
            Status In Progress [ 3 ] Open [ 1 ]
            jamesdumay James Dumay made changes -
            Sprint frank, tethys, christmas, pannonian [ 156, 161, 176, 211 ] frank, tethys, christmas [ 156, 161, 176 ]
            jamesdumay James Dumay made changes -
            Sprint frank, tethys, christmas [ 156, 161, 176 ] frank, tethys, christmas, iapetus [ 156, 161, 176, 216 ]
            jamesdumay James Dumay made changes -
            Rank Ranked higher
            jamesdumay James Dumay made changes -
            Priority Major [ 3 ] Critical [ 2 ]
            jamesdumay James Dumay made changes -
            Link This issue relates to JENKINS-41459 [ JENKINS-41459 ]
            Hide
            jamesdumay James Dumay added a comment -

            Another incidence of this problem JENKINS-41459

            Show
            jamesdumay James Dumay added a comment - Another incidence of this problem JENKINS-41459
            jamesdumay James Dumay made changes -
            Rank Ranked higher
            jglick Jesse Glick made changes -
            Status Open [ 1 ] Resolved [ 5 ]
            Resolution Fixed [ 1 ]
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: Jesse Glick
            Path:
            changelog.html
            http://jenkins-ci.org/commit/jenkins/2f057b4edde450bb2e6fe86bcd02fd731c8dbc88
            Log:
            @daniel-beck’s 56da425 merging #2722 implied that JENKINS-40088 was fixed in 2.42; in fact it is toward 2.46.

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Jesse Glick Path: changelog.html http://jenkins-ci.org/commit/jenkins/2f057b4edde450bb2e6fe86bcd02fd731c8dbc88 Log: @daniel-beck’s 56da425 merging #2722 implied that JENKINS-40088 was fixed in 2.42; in fact it is toward 2.46.
            Hide
            jamesdumay James Dumay added a comment -

            We are still tracking this one to fix in Blue Ocean.

            Show
            jamesdumay James Dumay added a comment - We are still tracking this one to fix in Blue Ocean.
            jamesdumay James Dumay made changes -
            Resolution Fixed [ 1 ]
            Status Resolved [ 5 ] Reopened [ 4 ]
            vivek Vivek Pandey made changes -
            Status Reopened [ 4 ] In Progress [ 3 ]
            vivek Vivek Pandey made changes -
            Status In Progress [ 3 ] In Review [ 10005 ]
            jamesdumay James Dumay made changes -
            Sprint frank, tethys, christmas, iapetus [ 156, 161, 176, 216 ] frank, tethys, christmas, iapetus, 1.0 [ 156, 161, 176, 216, 221 ]
            jamesdumay James Dumay made changes -
            Rank Ranked lower
            vivek Vivek Pandey made changes -
            Status In Review [ 10005 ] Resolved [ 5 ]
            Resolution Fixed [ 1 ]
            olivergondza Oliver Gondža made changes -
            Labels blueocean lts-candidate rest robustness stapler blueocean rest robustness stapler
            Hide
            olivergondza Oliver Gondža added a comment -

            Removing lts-candidate label as the core part seems done and made it to 2.46, witch is the LTS baseline.

            Show
            olivergondza Oliver Gondža added a comment - Removing lts-candidate label as the core part seems done and made it to 2.46, witch is the LTS baseline.
            Hide
            oleg_nenashev Oleg Nenashev added a comment -

            Additional fix has been integrated towards Jenkins 2.74

            Show
            oleg_nenashev Oleg Nenashev added a comment - Additional fix has been integrated towards Jenkins 2.74
            Hide
            jglick Jesse Glick added a comment -

            Not a fix, exactly; a new API.

            Show
            jglick Jesse Glick added a comment - Not a fix, exactly; a new API.
            cloudbees CloudBees Inc. made changes -
            Remote Link This issue links to "CloudBees Internal OSS-1824 (Web Link)" [ 18548 ]
            jbriden Jenn Briden made changes -
            Status Resolved [ 5 ] Closed [ 6 ]

              People

              • Assignee:
                vivek Vivek Pandey
                Reporter:
                jamesdumay James Dumay
              • Votes:
                0 Vote for this issue
                Watchers:
                7 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: