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

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

    XMLWordPrintable

Details

    • 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

          blocks

          Bug - A problem which impairs or prevents the functions of the product. JENKINS-39647 Failed to write defaultParameterValue: IAE from ExtensibleChoiceParameterDefinition.createValueCommon

          • Critical - Crashes, loss of data, severe memory leak.
          • Resolved
          duplicates

          Bug - A problem which impairs or prevents the functions of the product. JENKINS-39647 Failed to write defaultParameterValue: IAE from ExtensibleChoiceParameterDefinition.createValueCommon

          • Critical - Crashes, loss of data, severe memory leak.
          • Resolved
          relates to

          Bug - A problem which impairs or prevents the functions of the product. JENKINS-40083 Blueocean fails to show pipelines due to analysis-core failure

          • Critical - Crashes, loss of data, severe memory leak.
          • Resolved

          Bug - A problem which impairs or prevents the functions of the product. JENKINS-41459 Loading never finishes on blue ocean dashboard when clicked show more button.

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Closed
          links to

          Web Link CloudBees Internal OSS-1824

          Web Link core PR 2722

          Web Link Stapler PR 89

          Web Link White screen of death

          Activity

            Ascending order - Click to sort in descending order
            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.
            michaelneale Michael Neale added a comment -

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

            michaelneale Michael Neale added a comment - jamesdumay 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 -
            Resolution Duplicate [ 3 ]
            Status Open [ 1 ] Resolved [ 5 ]
            michaelneale Michael Neale added a comment -

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

            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 ]
            jglick Jesse Glick added a comment -

            Linking to a proposed fix, or prototype of fix.

            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 ]
            jglick Jesse Glick added a comment -

            JENKINS-40083 was an instance of this IIUC?

            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
            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 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 ]
            jamesdumay James Dumay added a comment -

            jglick 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.

            jamesdumay James Dumay added a comment - jglick 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 ]
            jglick Jesse Glick added a comment -

            Agreement with vivek and michaelneale 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 added a comment - Agreement with vivek and michaelneale 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 ]
            jamesdumay James Dumay added a comment -

            vivek this is the WSOD issue

            jamesdumay James Dumay added a comment - vivek 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 ]
            jamesdumay James Dumay added a comment -

            Another incidence of this problem JENKINS-41459

            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 -
            Resolution Fixed [ 1 ]
            Status Open [ 1 ] Resolved [ 5 ]
            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.

            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.
            jamesdumay James Dumay added a comment -

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

            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 -
            Resolution Fixed [ 1 ]
            Status In Review [ 10005 ] Resolved [ 5 ]
            olivergondza Oliver Gondža made changes -
            Labels blueocean lts-candidate rest robustness stapler blueocean rest robustness stapler
            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.

            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.
            oleg_nenashev Oleg Nenashev added a comment -

            Additional fix has been integrated towards Jenkins 2.74

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

            Not a fix, exactly; a new API.

            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

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

              Dates

                Created:
                Updated:
                Resolved: