-
Notifications
You must be signed in to change notification settings - Fork 88
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
Discussion: official SonarSource Docker image vs. this repo's image #42
Comments
I tried out the official Sonar Scanner image using the following commands, which align generally with the "try it out locally" commands I have in my README: docker run -d --name sonarqube -p 9000:9000 -p 9092:9092 sonarqube
SONARQUBE_URL=sonarqube:9000 && \
docker run \
--rm \
--link sonarqube \
-e SONAR_HOST_URL="http://${SONARQUBE_URL}" \
-v "$PWD:/usr/src" \
sonarsource/sonar-scanner-cli \
sonar-scanner -Dsonar.projectKey=myproject I ran that image on some local source (python/javascript) of my own followed immediately by the image from this repo on the same source code. Both produced identical results in that one test (not definitive, but "something"). A few things to note about the official source image that differs from the one in this repo:
So I'm interested in the thoughts / feedback from the community about how best to proceed. If you're not using the official image, why not? |
I migrated easily, I just replaced the image name. I think your image is well ranked on google, that's it. |
original image is missing curl and their busybox-wget is very basic regarding TLS and no HTTP auth support... therefore unusable. |
SonarScanner has an official Docker image available, and has for a while. See here and here for details. Although I'm happy to keep my project alive, I'd love to hear from consumers and contributors of this repo about whether the official image is the Better Way™ or if what I've done here has some distinct and specific value that the official image doesn't.
Please leave responses to this issue with your thoughts on using either image.
The text was updated successfully, but these errors were encountered: