Avoid deeply nested control flow statements.

Open

              if wait >= 30
                MU.log "User #{user} has been created in LDAP, but local system can't see it. Are PAM/LDAP configured correctly?", MU::ERR
                return false
              end

Avoid deeply nested control flow statements.

Open

                if public
                  # Make sure we have a public IP address to register. Use the private one if we don't
                  server.cloud_desc.public_ip_address ? server.cloud_desc.public_ip_address : server.cloud_desc.private_ip_address
                else
                  # If we specifically requested to register the private IP lets use that

Avoid deeply nested control flow statements.

Open

              ok = false if !insertKitten(siblingvpc, "vpcs", overwrite: overwrite)

Avoid deeply nested control flow statements.

Open

                if File.exist?(example_path)
                  example = "#\n# Examples:\n#\n"
                  # XXX these variables are all parameters from the BoKs in
                  # modules/tests. A really clever implementation would read
                  # and parse them to get default values, perhaps, instead of

Avoid deeply nested control flow statements.

Open

              if mu_acct
                attr[:userAccountControl] = (attr[:userAccountControl].to_i & AD_PW_ATTRS['pwdNeverExpires']).to_s
              end

Method create has a Cognitive Complexity of 78 (exceeds 75 allowed). Consider refactoring.

Open

        def create
          # RDS is picky, we can't just use our regular node names for things
          # like the default schema or username. And it varies from engine to
          # engine.
          basename = @config["name"].to_s

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

                    if public
                      { "Fn::GetAtt" => [@dependencies["server"][dnsrec['target']].cloudobj.cfm_name, "PublicDnsName"] }
                    else
                      { "Fn::GetAtt" => [@dependencies["server"][dnsrec['target']].cloudobj.cfm_name, "PrivateDnsName"] }
                    end

Avoid deeply nested control flow statements.

Open

            hashcfg.delete("id") if hashcfg["name"]

Avoid deeply nested control flow statements.

Open

                    if policy['target_tracking_configuration']["preferred_target_group"] and
                       lb["targetgroups"][policy['target_tracking_configuration']["preferred_target_group"]]
                      lb["arn"].split(/:/)[5].sub(/^loadbalancer\//, "")+"/"+lb["targetgroups"][policy['target_tracking_configuration']["preferred_target_group"]].split(/:/)[5]
                    else
                      if policy['target_tracking_configuration']["preferred_target_group"]

Avoid deeply nested control flow statements.

Open

                MU.log "No base image for platform #{platform} in cloud #{cloud} region #{region} found", MU::WARN if !quiet

Avoid deeply nested control flow statements.

Open

                if sib_by_name.size == 1
                  @vpc = sib_by_name.values.first
                  MU.log "Single VPC match for #{self}", loglevel, details: @vpc.to_s
                else
# XXX ok but this is the wrong place for this really the config parser needs to sort this out somehow

Avoid deeply nested control flow statements.

Open

                  my_cidr = nil if my_cidr.to_s.match(/^10\.255\./)

Avoid deeply nested control flow statements.

Open

                  server.cloud_desc.public_dns_name.empty? ? server.cloud_desc.private_dns_name : server.cloud_desc.public_dns_name

Avoid deeply nested control flow statements.

Open

              if disable
                attr[:userAccountControl] = (attr[:userAccountControl].to_i & AD_PW_ATTRS['disable']).to_s
              end

Avoid deeply nested control flow statements.

Open

                  raise MuError, "Couldn't find #{dnsrec['mu_type']} #{dnsrec["target"]}" if found.nil? || found.empty?

Avoid deeply nested control flow statements.

Open

            rescue StandardError => e
              MU.log "Failed to fetch #{cryptfile} from Cloud Storage bucket #{MU.adminBucketName}", MU::ERR, details: e.inspect
              %x{/bin/dd if=/dev/urandom of=#{temp_dev} bs=1M count=1 > /dev/null 2>&1}
              raise e

Avoid deeply nested control flow statements.

Open

            if ![:removed].include?(tier[:action])
              myreport["slack"] += ". New #{tier[:field] ? "`"+tier[:field]+"`" : :value}: \*#{tier[:value]}\*"
            else
              myreport["slack"] += " (was \*#{tier[:value]}\*)"
            end

Avoid deeply nested control flow statements.

Open

          elsif cfg.id
            littermate = deploy.findLitterMate(type: cfg.type, cloud_id: cfg.id, habitat: cfg.habitat)
            if littermate and littermate.config['name']
              hashcfg['name'] = littermate.config['name']
              hashcfg.delete("id") if hashcfg["name"]

Avoid deeply nested control flow statements.

Open

                unless policy['type']
                  MU.log "You must specify a 'type' when 'policy_type' is set to 'StepScaling'", MU::ERR
                  ok = false
                end

Avoid deeply nested control flow statements.

Open

                  elsif dnsrec["type"] == "A"
                    if public
                      { "Fn::GetAtt" => [@dependencies["server"][dnsrec['target']].cloudobj.cfm_name, "PublicIp"] }
                    else
                      { "Fn::GetAtt" => [@dependencies["server"][dnsrec['target']].cloudobj.cfm_name, "PrivateIp"] }