Skip to content
This repository has been archived by the owner on Jun 22, 2020. It is now read-only.

Define a default command or entrypoint for the Image #37

Open
hansingt opened this issue Jan 8, 2019 · 4 comments
Open

Define a default command or entrypoint for the Image #37

hansingt opened this issue Jan 8, 2019 · 4 comments

Comments

@hansingt
Copy link

hansingt commented Jan 8, 2019

There is no default command and no entrypoint defined in the Dockerfile.
Thus, everybody not using the .gitlab-ci.yml using this image needs to explicitly call it with:

docker run ciricihq/gitlab-sonar-scanner gitlab-sonar-scanner

As the gitlab-sonar-scanner script is the main purpose of this image. We could define it as the default command or maybe even as the default entrypoint.

@elboletaire
Copy link
Contributor

Do you use it outside of gitlab? Because we don't, and that's why we didn't even think about making it available for other purposes. Although, if you think it's useful, we're open to PR.

@hansingt
Copy link
Author

hansingt commented Jan 8, 2019

I do not use it outside of GitLab-CI. But I can imagine usecases in which one might need to explicitly call it.

I did a PR (#34) which included a fix for this issue as well as for !36. But it has been closed in favour of another PR, which in fact beaks things (as mentioned at #38)

@elboletaire
Copy link
Contributor

#34 was not accepted because there were changes not related to the binary upgrade. Also, the upgrade error is due to the sonarqube scanner binary itself, so the problem would be the same in all the other PR.

@hansingt
Copy link
Author

hansingt commented Jan 8, 2019

I understand why the PR was closed. Just wanted to mention this.

Offtopic, but just my two cents:
I think, that the problems with #38 exist, because you are using the "-linux-zip" which includes a pre-compiled JRE which is used by default. This JRE is compiled for GLIBC. But you are using Alpine, which uses MUSL. Because of this, this does not work.

Acutally, this brings me to another issue with the other PR: We don't need this additional JRE. Just use the plain distribution Mentioned as "any" here https://docs.sonarqube.org/display/SCAN/Analyzing+with+SonarQube+Scanner

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants