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

EC2Cloud crashes with NPE when instanceCapStr is not passed from JCasC

    XMLWordPrintable

    Details

    • Similar Issues:
    • Released As:
      ec2-1.44

      Description

      It is a minor issue in the EC2 Plugin implementation. 
      instanceCapStr in https://github.com/jenkinsci/ec2-plugin/blob/master/src/main/java/hudson/plugins/ec2/EC2Cloud.java is designed to be optional, and it supports empty strings in the constructor. But it still fails with NPE when a field is not passed from the JCasC configuration.
       
      I consider it is as an important issue, because the field is not exported by default.

      The issue can be fixed by a small patch to properly handle nulls in the constructor arguments

        Attachments

          Issue Links

          duplicates

          Bug - A problem which impairs or prevents the functions of the product. JENKINS-56684 Config as Code Nullpointer when instanceCapStr is not initialized

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

            Activity

            Ascending order - Click to sort in descending order
            oleg_nenashev Oleg Nenashev created issue -
            oleg_nenashev Oleg Nenashev made changes -
            Field Original Value New Value
            Description It is a minor issue in the EC2 Plugin implementation. 
            instanceCapStr in [https://github.com/jenkinsci/ec2-plugin/blob/master/src/main/java/hudson/plugins/ec2/EC2Cloud.java] is designed to be optional, and it supports empty strings in the constructor. But it still fails with NPE when a field is not passed from the JCasC configuration.
             
            I consider it is as an important issue, because the field is not exported by default             		It is a minor issue in the EC2 Plugin implementation. 
             instanceCapStr in [https://github.com/jenkinsci/ec2-plugin/blob/master/src/main/java/hudson/plugins/ec2/EC2Cloud.java] is designed to be optional, and it supports empty strings in the constructor. But it still fails with NPE when a field is not passed from the JCasC configuration.
              
             I consider it is as an important issue, because the field is not exported by default.

            The issue can be fixed by a small patch to properly handle nulls in the constructor arguments
            Hide
            Permalink
            oleg_nenashev Oleg Nenashev added a comment -

            Duplicates JENKINS-56684

            Show
            oleg_nenashev Oleg Nenashev added a comment - Duplicates  JENKINS-56684
            oleg_nenashev Oleg Nenashev made changes -
            Link This issue duplicates JENKINS-56684 [ JENKINS-56684 ]
            Hide
            Permalink
            nisarg14 Nisarg Shah added a comment -

            Oleg Nenashev

            Is this issue still available to work on? If available, I would like to have a look on it.

            Show
            nisarg14 Nisarg Shah added a comment - Oleg Nenashev Is this issue still available to work on? If available, I would like to have a look on it.
            Hide
            Permalink
            timja Tim Jacomb added a comment -

            I believe fixed in https://github.com/jenkinsci/ec2-plugin/pull/358

            although unreleased

            Show
            timja Tim Jacomb added a comment - I believe fixed in  https://github.com/jenkinsci/ec2-plugin/pull/358 although unreleased
            timja Tim Jacomb made changes -
            Resolution Fixed [ 1 ]
            Status Open [ 1 ] Fixed but Unreleased [ 10203 ]
            Hide
            Permalink
            nisarg14 Nisarg Shah added a comment -

            Okay 

            Show
            nisarg14 Nisarg Shah added a comment - Okay 
            oleg_nenashev Oleg Nenashev made changes -
            Released As ec2-1.44
            Status Fixed but Unreleased [ 10203 ] Resolved [ 5 ]

              People

              Assignee:
              thoulen FABRIZIO MANFREDI
              Reporter:
              oleg_nenashev Oleg Nenashev
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: