Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cdktf-cli diff output empty #2089

Closed
mjrlee opened this issue Sep 5, 2022 · 5 comments
Closed

cdktf-cli diff output empty #2089

mjrlee opened this issue Sep 5, 2022 · 5 comments
Assignees
Labels
cdktf-cli enhancement New feature or request priority/important-soon High priority, to be worked on as part of our current release or the following one. size/small estimated < 1 day ux/cli
Milestone

Comments

@mjrlee
Copy link

mjrlee commented Sep 5, 2022

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 debug
language: go
cdktf-cli: 0.12.2
node: v14.18.1
cdktf: 0.12.2
constructs: 10.1.94
jsii: 1.66.0
terraform: 1.2.8
arch: x64
os: linux 4.18.0-348.20.1.el8_5.x86_64
go: go version go1.19 linux/amd64

Affected Resource(s)

Debug Output

Expected Behavior

Plan with actions should produce output on cdktf diff or cdktf apply

Actual Behavior

Output plan never appears.

I have tried with/without tmux, with multiple different terminal emulators and with FORCE_COLOR=0 set. Also with redirecting output to a file.

✗ cdktf diff
aws-ec2-dns  Creating Terraform Cloud configuration version
aws-ec2-dns  Created Terraform Cloud configuration version cv-413pLi61FHyrh4bh
aws-ec2-dns  Zipping up the directory cdktf.out/stacks/aws-ec2-dns
aws-ec2-dns  Uploading the directory to Terraform Cloud
aws-ec2-dns  Uploaded the directory to Terraform Cloud
aws-ec2-dns  Waiting for configuration version to become ready...
aws-ec2-dns  Creating speculative Terraform Cloud run
aws-ec2-dns  Created speculative Terraform Cloud run: https://terraform..../runs/run-2kZe8unmbRFzSWiy
aws-ec2-dns  Speculative Terraform Cloud run done
aws-ec2-dns  Getting plan output

➜ 

If I check the plan in the web UI I can see there are 2 resources to be created

Steps to Reproduce

Important Factoids

We use a self hosted terraform enterprise instance

References

@mjrlee mjrlee added bug Something isn't working new Un-triaged issue labels Sep 5, 2022
@DanielMSchmidt
Copy link
Contributor

The plan is in the TFC run above, but we could make this clearer that we don't intend to show it in the CLI

@DanielMSchmidt DanielMSchmidt added enhancement New feature or request cdktf-cli priority/important-soon High priority, to be worked on as part of our current release or the following one. ux/cli size/small estimated < 1 day and removed bug Something isn't working new Un-triaged issue labels Sep 5, 2022
@mjrlee
Copy link
Author

mjrlee commented Sep 5, 2022

Could we show it in the CLI? It would be very useful in cdktf deploy. I understand an edge use case for production but it would be very useful for rapid development.

@digitalrager
Copy link

I'm using Terraform Cloud and am unable to fully deploy from the CLI when I'm a team member in TC that has write permission because of this issue wont allow the plan to be displayed in the CLI. Using the terraform CLI seems to work ok. It seems these should line up ideally.

@DanielMSchmidt DanielMSchmidt added this to the 0.15 milestone Dec 5, 2022
@DanielMSchmidt DanielMSchmidt self-assigned this Dec 15, 2022
@DanielMSchmidt
Copy link
Contributor

Fixed in #1955, now it should behave exactly the same as with Terraform CLI (since we are using Terraform CLI under the hood)

@github-actions
Copy link
Contributor

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.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 15, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cdktf-cli enhancement New feature or request priority/important-soon High priority, to be worked on as part of our current release or the following one. size/small estimated < 1 day ux/cli
Projects
None yet
Development

No branches or pull requests

3 participants