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

New Rover features from user feature requests #32

Merged
merged 14 commits into from
Sep 28, 2021
Merged

New Rover features from user feature requests #32

merged 14 commits into from
Sep 28, 2021

Conversation

im2nguyen
Copy link
Owner

@im2nguyen im2nguyen commented Sep 28, 2021

Rover v0.2.0 will address most of the issues as of Sept 27, 2020

sujithq and others added 3 commits September 26, 2021 09:17
make use of multistage builds
update versions
@im2nguyen im2nguyen self-assigned this Sep 28, 2021
@im2nguyen
Copy link
Owner Author

im2nguyen commented Sep 28, 2021

Commit fixes #24. Users are able to specify rover ip and port

$ rover -ipPort "0.0.0.0:9001"
2021/09/27 21:46:54 Starting Rover...
2021/09/27 21:46:54 Initializing Terraform...
2021/09/27 21:46:55 Generating plan...
2021/09/27 21:46:55 Parsing configuration...
2021/09/27 21:46:55 Generating resource overview...
2021/09/27 21:46:55 Generating resource map...
2021/09/27 21:46:55 Generating resource graph...
2021/09/27 21:46:55 Done generating assets.
2021/09/27 21:46:55 Rover is running on 0.0.0.0:9001

@im2nguyen
Copy link
Owner Author

Fixes #9 and #20.

There are now two functions to generate a map, one that relies on parsed Terraform configuration, the other one without. This enables rover to consume plan files only -- allowing rover to work in scenarios where it doesn't have direct access to the config (Terraform Cloud, Terragrunt, auto-generated HCL, CDKTF, etc...)

The trade off is that you don't have the granularity of the resources being separated by file name.

Rover visualization with parsed configuration (notice that resources are grouped by file name first)

rover-config

Rover visualization without parsed configuration (notice how everything is grouped together)

rover-no-config

Still need to update the UI since no-config introduces a small bug in displaying/selecting resource details

@im2nguyen
Copy link
Owner Author

Fixes #29. The new showSensitive defaults to false and sensitive variable and output values to "Sensitive Value".

$ rover -showSensitive=true

@im2nguyen
Copy link
Owner Author

im2nguyen commented Sep 28, 2021

Fixes #18. The new workspaceName flag allows you to select an existing Terraform CLI workspace.

$ rover  -workspaceName "development"

This flag assumes that the workspace already exists. If it doesn't, rover will throw an error.

$ rover   -workspaceName "production"
2021/09/28 02:33:46 Starting Rover...
2021/09/28 02:33:46 Initializing Terraform...
2021/09/28 02:33:46 Running in production workspace...
2021/09/28 02:33:46 Unable to parse Plan: Unable to select workspace (production): workspace "production" does not exist
exit status 1

@im2nguyen
Copy link
Owner Author

Fixes #28.

There was a bug whenever an output or a resource referenced another resource's attribute.

@im2nguyen
Copy link
Owner Author

Fixes #35. The current/proposed state for output was wonky if the value was a string.

@im2nguyen
Copy link
Owner Author

Fixes #32 (comment) UI bug

Still need to update the UI since no-config introduces a small bug in displaying/selecting resource details

@im2nguyen im2nguyen merged commit be42b54 into main Sep 28, 2021
@im2nguyen im2nguyen deleted the v0.2.0 branch September 28, 2021 11:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants