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

Docker Image doesn't get compiled #217

Closed
ivankorn opened this issue Sep 6, 2021 · 1 comment · Fixed by #219
Closed

Docker Image doesn't get compiled #217

ivankorn opened this issue Sep 6, 2021 · 1 comment · Fixed by #219
Assignees
Labels
area/docker bug Something isn't working documentation Improvements or additions to documentation estimate/1h Need 1 hour to be done

Comments

@ivankorn
Copy link

ivankorn commented Sep 6, 2021

Archive:  tflint.zip
  inflating: tflint
curl: (3) <url> malformed
The command '/bin/sh -c add-apt-repository ppa:deadsnakes/ppa &&     apt install -y python3.7 python3-pip &&     pip3 install pre-commit==${PRE_COMMIT_VERSION} &&     curl -L "$(curl -s https://api.github.com/repos/terraform-docs/terraform-docs/releases | grep -o -E "https://.+?${TERRAFORM_DOCS_VERSION}-linux-amd64.tar.gz")" > terraform-docs.tgz && tar xzf terraform-docs.tgz && chmod +x terraform-docs && mv terraform-docs /usr/bin/ &&     curl -L "$(curl -s https://api.github.com/repos/terraform-linters/tflint/releases | grep -o -E "https://.+?/${TFLINT_VERSION}/tflint_linux_amd64.zip")" > tflint.zip && unzip tflint.zip && rm tflint.zip && mv tflint /usr/bin/ &&     curl -L "$(curl -s https://api.github.com/repos/tfsec/tfsec/releases | grep -o -E "https://.+?/${TFSEC_VERSION}/tfsec-linux-amd64")" > tfsec && chmod +x tfsec && mv tfsec /usr/bin/ &&     python3.7 -m pip install -U checkov==${CHECKOV_VERSION}' returned a non-zero code: 3
✘-3 ~/Soft/no-rpms/pre-commit-terraform [master|✔]
~                                                     

The command

'/bin/sh -c add-apt-repository ppa:deadsnakes/ppa && apt install -y python3.7 python3-pip && pip3 install pre-commit==${PRE_COMMIT_VERSION} && curl -L "$(curl -s https://api.github.com/repos/terraform-docs/terraform-docs/releases | grep -o -E "https://.+?${TERRAFORM_DOCS_VERSION}-linux-amd64.tar.gz")" > terraform-docs.tgz && tar xzf terraform-docs.tgz && chmod +x terraform-docs && mv terraform-docs /usr/bin/ && curl -L "$(curl -s https://api.github.com/repos/terraform-linters/tflint/releases | grep -o -E "https://.+?/${TFLINT_VERSION}/tflint_linux_amd64.zip")" > tflint.zip && unzip tflint.zip && rm tflint.zip && mv tflint /usr/bin/ && curl -L "$(curl -s https://api.github.com/repos/tfsec/tfsec/releases | grep -o -E "https://.+?/${TFSEC_VERSION}/tfsec-linux-amd64")" > tfsec && chmod +x tfsec && mv tfsec /usr/bin/ && python3.7 -m pip install -U checkov==${CHECKOV_VERSION}' returned a non-zero code: 3

steps to reproduce:

  1. git clone [email protected]:antonbabenko/pre-commit-terraform.git
  2. cd pre-commit-terraform
  3. docker build -t pre-commit .
@MaxymVlasov MaxymVlasov self-assigned this Sep 8, 2021
MaxymVlasov added a commit that referenced this issue Sep 8, 2021
Also, add `terrascan` and update base image
@MaxymVlasov
Copy link
Collaborator

MaxymVlasov commented Sep 8, 2021

@mavogel can you please describe how to use dockerized pre-commit-terraform? Where code should be mounted and so on.

@ivankorn Can you please check that changes in GH-217/fix_docker works for you?
Also, if you can provide the command that you use to check the code - I would be very grateful.

UPD. Needed docs added/clarified

@MaxymVlasov MaxymVlasov added the documentation Improvements or additions to documentation label Sep 8, 2021
MaxymVlasov added a commit that referenced this issue Sep 9, 2021
Also, add `terrascan` and update base image
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/docker bug Something isn't working documentation Improvements or additions to documentation estimate/1h Need 1 hour to be done
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants