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

mode should default to NORMAL when absent

    XMLWordPrintable

    Details

    • Sprint:
      Evergreen - Milestone 2
    • Similar Issues:
    • Released As:
      ec2-plugin 1.40

      Description

      When configuring an EC2 cloud using CasC, one must explicitly set mode so that the Cloud actually is usable.

      Here is the working yaml

      ---
      jenkins: 
        clouds: 
          - amazonEC2:
              cloudName: "ec2"
              instanceCapStr: 20
              zone: "us-east-1"
              useInstanceProfileForCredentials: true
              privateKey: "${PRIVATE_KEY}"
              templates: 
                - description: "EC2 Agent"
                  ami: "ami-0c6bb742864ffa3f3"
                  labels: "agent"
                  type: "T2Xlarge"
                  remoteFS: "/home/ec2-user"
                  remoteAdmin: "ec2-user"
                  ################            
                  # FIXME: without this, the whole setup will look like it's not even configured
                  mode: NORMAL 
      

      Expected behaviour

      If mode is not specified, it should default to NORMAL. Or if really we want to make it required, this should fail fast when absent, not stay null and make the whole settings broken.

      I had to run the code through a debugger to actually understand what was going on.

        Attachments

          Activity

          Hide
          ndeloof Nicolas De Loof added a comment -

          Would require ec2 plugin to handle null value on mode constructor parameter to use NORMAL as default
          https://github.com/jenkinsci/ec2-plugin/blob/master/src/main/java/hudson/plugins/ec2/SlaveTemplate.java#L173

          Show
          ndeloof Nicolas De Loof added a comment - Would require ec2 plugin to handle null value on mode constructor parameter to use NORMAL as default https://github.com/jenkinsci/ec2-plugin/blob/master/src/main/java/hudson/plugins/ec2/SlaveTemplate.java#L173

            People

            • Assignee:
              ndeloof Nicolas De Loof
              Reporter:
              batmat Baptiste Mathus
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: