-
Notifications
You must be signed in to change notification settings - Fork 606
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
Official docker image #1025
Comments
I have some internal tasks to get this going but wanted to let you know I have started this work. |
This is a bit of a challenge, as in some environments using docker images becomes a tricky challenge for security reasons, as we have to pick them all apart to assess if its safe. |
IMHO this is a good alternative to the current pip installation method, you can always choose to install through pip if you have other requirements to fulfill. :) |
Initial Docker file was merged #1361 |
How would upload to https://hub.docker.com/ happen? PS There is an unofficial version at https://hub.docker.com/r/aztek/cfn-lint |
Please publish to DockerHub 🙏 |
I agree, an official image on DockerHub (with tags for each release) would be great. Currently, there are many unofficial images out there, but for all of them we cannot know how they are build and how long they will be updated: Thus, an official image would be greatly appreciated. |
I note that there is an official docker image for the AWS CLI now (see aws/aws-cli#3553 (comment)), does this set a precedent that might see an official cfn-lint image? |
I added my 👍🏻 to the original post but the scope has changed slightly. I would raise a new issue for publishing to Docker Hub or Github Container Registry, but this is still open. I would like to see this Dockerfile built and published automatically as part of your CI so that we can rely on first-party implementations rather than maintaining our own and/or using random images in our pipelines. |
Would it be easier for developers to publish it to the ECR Public Gallery (I am assuming that the users of CloudFormation use AWS anyway)? I can see multiple cfn-lint Docker images in the ECR Public Gallery but none is published by a "Verified account". |
Also added my👍, having to manually create and update a docker image for our build pipelines is quite a hassle. |
+10000 on this. This is a linting tool, using it in CI/CD seems like the main use-case and it's recommended by AWS themselves! Lack of an official docker image is shocking. |
As a public service we have created a public GitHub repository which uses GitHub Actions to check every day if a new release of cfn-lint is available and builds an image for it, stored in both: a public ECR repository and Docker Hub: GitHub repo: https://github.com/MysteriousCode/cfn-lint-docker If you're not sure you can try this repo, check out the workflow file that builds the image: https://github.com/MysteriousCode/cfn-lint-docker/blob/main/.github/workflows/ Hope this helps people. |
This will put some extra burden on the developers, especially during the release process, but this will help out the following scenarios:
Please let me know what you think, and please feel free to turn this down. I'm happy to create a PR if you guys want to go down this path. :)
The text was updated successfully, but these errors were encountered: