elasticloadbalancingv2: fix to be able to set deregistrationDelay

[cohalz]

Note: for support questions, please first reference our documentation, then use Stackoverflow. This repository's issues are intended for feature requests and bug reports.

• I'm submitting a ...

  • 🪲 bug report
  • 🚀 feature request
  • 📚 construct library gap
  • ☎️ security issue or vulnerability => Please see policy
  • ❓ support request => Please see note at the top of this template.

• What is the current behavior?
  If the current behavior is a 🪲bug🪲: Please provide the steps to reproduce

deregistrationDelay causes the following error

    const targetGroup = new ApplicationTargetGroup(this, "TargetGroup", {
      deregistrationDelay: Duration.seconds(10),
      vpc,
    });

/Users/cohalz/example/node_modules/@aws-cdk/core/lib/private/intrinsic.ts:25
      throw new Error(`Argument to Intrinsic must be a plain value object, got ${value}`);
            ^
Error: Argument to Intrinsic must be a plain value object, got () => {
            throw new Error(`Duration.toString() was used, but .toSeconds, .toMinutes or .toDays should have been called instead`);
        }
    at new Intrinsic (/Users/cohalz/example/node_modules/@aws-cdk/core/lib/private/intrinsic.ts:25:13)
    at Function.asAny (/Users/cohalz/example/node_modules/@aws-cdk/core/lib/token.ts:74:48)
    at Function.asString (/Users/cohalz/example/node_modules/@aws-cdk/core/lib/token.ts:51:53)
    at Duration.toString (/Users/cohalz/example/node_modules/@aws-cdk/core/lib/duration.ts:137:18)
    at new TargetGroupBase (/Users/cohalz/example/node_modules/@aws-cdk/aws-elasticloadbalancingv2/lib/shared/base-target-group.ts:208:95)
    at new ApplicationTargetGroup (/Users/cohalz/example/node_modules/@aws-cdk/aws-elasticloadbalancingv2/lib/alb/application-target-group.ts:81:5)

• What is the expected behavior (or behavior of feature suggested)?

No errors occurred

• What is the motivation / use case for changing the behavior or adding this feature?

• Please tell us about your environment:

  • CDK CLI Version: 0.36.0
  • Module Version: xx.xx.xx
  • OS: [all | Windows 10 | OSX Mojave | Ubuntu | etc... ]
  • Language: [all | TypeScript | Java | Python ]

• Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. associated pull-request, stackoverflow, gitter, etc)

[mhuebner]

Have a look at the duplicate/related issue: #3080