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

Add a configuration flag in radiator view settings to show or not disabled jobs

    Details

    • Similar Issues:

      Description

      If we disabled these jobs, often we aren't interested anymore by their results

        Attachments

          Activity

          Hide
          asalasa asalasa added a comment -
          Show
          asalasa asalasa added a comment - See http://wiki.jenkins-ci.org/display/JENKINS/Sectioned+View+Plugin for sample implementation
          Hide
          aheritier Arnaud Héritier added a comment -

          Reading the code, I'm just seeing that disable job shouldn't be displayed in the radiator view, but there is a bug. I will try fix it and to add the option

          Show
          aheritier Arnaud Héritier added a comment - Reading the code, I'm just seeing that disable job shouldn't be displayed in the radiator view, but there is a bug. I will try fix it and to add the option
          Hide
          aheritier Arnaud Héritier added a comment -

          It's not bugged. It was added in version 1.7 which was released by olamy but not published on update center (I don't know why)

          Show
          aheritier Arnaud Héritier added a comment - It's not bugged. It was added in version 1.7 which was released by olamy but not published on update center (I don't know why)
          Hide
          aheritier Arnaud Héritier added a comment -

          Patch to add an option to display or not deactivated jobs. Use grey background if they are displayed

          Show
          aheritier Arnaud Héritier added a comment - Patch to add an option to display or not deactivated jobs. Use grey background if they are displayed
          Hide
          kohsuke Kohsuke Kawaguchi added a comment -

          I think the code will result in NPE when loading data from existing configuration, as the Boolean field is initialized to null.

          I suggest either using boolean or handle the case where the field is null.

          Show
          kohsuke Kohsuke Kawaguchi added a comment - I think the code will result in NPE when loading data from existing configuration, as the Boolean field is initialized to null. I suggest either using boolean or handle the case where the field is null.
          Hide
          aheritier Arnaud Héritier added a comment -

          Ok . Thx for this feedback. It was clear for me how this sort of object was managed in hudson lifecycle.
          I will fix it to ensure we don't initialize it with null.
          I didn't understand why previous committers used Boolean objects instead of types. I looked at several plugins, but it seems there is no "official/recommended" usage of Objects vs types.

          Show
          aheritier Arnaud Héritier added a comment - Ok . Thx for this feedback. It was clear for me how this sort of object was managed in hudson lifecycle. I will fix it to ensure we don't initialize it with null. I didn't understand why previous committers used Boolean objects instead of types. I looked at several plugins, but it seems there is no "official/recommended" usage of Objects vs types.
          Hide
          nkjensen Niels Kristian Jensen added a comment -

          I'm using 1.444 with radiator plugin 1.13 and disabled projects do not appear in the Radiator view. I suggest closing this issue.

          Show
          nkjensen Niels Kristian Jensen added a comment - I'm using 1.444 with radiator plugin 1.13 and disabled projects do not appear in the Radiator view. I suggest closing this issue.
          Hide
          david_resnick David Resnick added a comment -

          As mentioned in the previous comment, disabled jobs are not shown. So this bug is no longer relevant.

          Show
          david_resnick David Resnick added a comment - As mentioned in the previous comment, disabled jobs are not shown. So this bug is no longer relevant.

            People

            • Assignee:
              Unassigned
              Reporter:
              aheritier Arnaud Héritier
            • Votes:
              1 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: