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

fix(terraform_prompt): modify prompt to have a better control on reso… #58

Closed
wants to merge 2 commits into from

Conversation

mohammadll
Copy link
Contributor

…urces parameters

@mohammadll
Copy link
Contributor Author

If you select docker_container as the base_config, you should see these parameters in modules/docker_container/main.tf only, and no other parameters: name, image, hostname, restart.

If you select docker_image, you should see these parameters in modules/docker_image/main.tf only, and no other parameters: name, force_remove, and build.

As you probably know, docker_image has a dependency on docker_container. Therefore, when you select docker_image as the base_config, you should see two resources in modules/docker_image/main.tf because it wants to create and run the container based on the image it creates

@abolfazl8131
Copy link
Contributor

abolfazl8131 commented Nov 13, 2024

Thanks for your contribution. but it doesn't seem to work very well on the docker container. most of the time it doesn't create 2 resources as you mentioned. please check them out. also the main.tf in both root directory needs to be modifyed

@mohammadll mohammadll closed this Nov 13, 2024
@mohammadll mohammadll deleted the terraform branch November 13, 2024 14:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants