Provider generation doesn't work with v0.0.5

Community Note

• Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
• Please do not leave “+1” or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request
• If you are interested in working on this issue or have submitted a pull request, please leave a comment

cdktf & Language Versions

• cdktf v0.0.5
• cdktf-cli v0.0.5

Affected Resource(s)

TerraformProvider?

cdktf.json

{
  "language": "typescript",
  "app": "node main.js",
  "terraformProviders": ["aws@~> 2.0"]
}

main.ts

import { Construct } from 'constructs';
import { App, TerraformStack } from 'cdktf';
import { Instance } from './.gen/providers/aws';

class MyStack extends TerraformStack {
  constructor(scope: Construct, name: string) {
    super(scope, name);

    new Instance(this, 'Hello', {
      ami: "ami-2757f631",
      instanceType: "t2.micro"
    });

  }
}

const app = new App();
new MyStack(app, 'terraform-cdk-example');
app.synth();

Expected Behavior

After running yarn compile and cdktf synth, we should see the following in the terraformcdkexample.tf.json file in cdktf.out folder.

{
  "terraform": {
    "required_providers": {
      "aws": "~> 2.0"
    }
  },
  "resource": {
    "aws_instance": {
      "terraformcdkexampleHello8B822F74": {
        "ami": "ami-2757f631",
        "instance_type": "t2.micro"
      }
    }
  }
}

Actual Behavior

terraformcdkexample.tf.json

{
  "resource": {
    "aws_instance": {
      "terraformcdkexampleHello8B822F74": {
        "ami": "ami-2757f631",
        "instance_type": "t2.micro"
      }
    }
  }
}

[joatmon08]

I think this is a consequence of stabilizing provider generation. We moved this to the TerraformProvider implementation rather than within get. If we try to synthesize provider versioning on the stack level, I think we’re back to the problems we had before.

If our expected behavior is to generate a terraform block on cdktf get, then we will need to do a refactor to generate that at the top-level. I’m more inclined to make it a requirement to declare AwsProvider, etc. and document that it is required in the guides in order to render the top-level terraform block with required_provider.

[github-actions[bot]]

I’m going to lock this issue because it has been closed for 30 days. This helps our maintainers find and focus on the active issues. If you’ve found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.