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

Missing Tofu Binary in Docker Image #12

Merged
merged 3 commits into from
Sep 25, 2024
Merged

Conversation

SOHAN-cyber
Copy link
Contributor

@SOHAN-cyber SOHAN-cyber commented Sep 24, 2024

I encountered an issue with the tofuutils/pre-commit-opentofu Docker image. Currently, the image contains the tofu package as a ZIP file (tofu_1.6.1_linux_amd64.zip), but it does not include the extracted tofu binary, which prevents the command from running as expected.

Details:

Image Name: tofuutils/pre-commit-opentofu
ZIP File Location: /usr/bin/tofu_1.6.1_linux_amd64.zip
Missing Binary: The extracted tofu binary is not available in the expected path.
Secreenshot of zip tofu binary:
image
image

Suggested Solution:

Please consider updating the Docker image to include the extracted tofu binary instead of the ZIP file. This would allow users to run the tofu command without needing to manually extract the binary.

… commit introduces logic to gracefully handle the case when the tofu binary is not found in the Docker image, improving the overall user experience. BREAKING CHANGE: The previous behavior of the application when the tofu binary was missing may have caused unexpected crashes.

git commit -m "feat(tofu): add handling for missing tofu binary in Docker image

This commit introduces logic to gracefully handle the case when the tofu binary is not found in the Docker image, improving the overall user experience.

BREAKING CHANGE: The previous behavior of the application when the tofu binary was missing may have caused unexpected crashes."
@kvendingoldo
Copy link
Collaborator

LGTM!

@kvendingoldo kvendingoldo merged commit d167eb9 into tofuutils:main Sep 25, 2024
0 of 5 checks passed
@kvendingoldo
Copy link
Collaborator

This PR is included in version 2.0.0 🎉

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.

2 participants