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

SONARQUBE_JDBC_* var env ignored #58

Open
gbonnefille opened this issue Oct 9, 2020 · 3 comments · May be fixed by #59
Open

SONARQUBE_JDBC_* var env ignored #58

gbonnefille opened this issue Oct 9, 2020 · 3 comments · May be fixed by #59
Labels
bug Something isn't working

Comments

@gbonnefille
Copy link
Contributor

Describe the bug

When setting variable environment like SONARQUBE_JDBC_* or like sonar.* (as allowed in recent release of sonarqube docker) they are ignored.

To reproduce

Run the docker-compose.yml script provided.

Expected behavior

Variables like SONARQUBE_JDBC_* or like sonar.* should be passed to bin/run.sh.

@gbonnefille gbonnefille added the bug Something isn't working label Oct 9, 2020
@gbonnefille
Copy link
Contributor Author

After investigation, it is certainly due to the su sonarqube... instruction in the /tmp/init.sh.

@gbonnefille
Copy link
Contributor Author

Possible fix:

  1. install sudo in docker-cat image
  2. declare sonarqube user in sudoer
  3. extract user/group modifications made in /tmp/init.sh in a dedicated script
  4. run this script with sudo
  5. replace su ... with simple exec
  6. restore USER sonarqube in Dockerfile

@gbonnefille
Copy link
Contributor Author

Or simply explicitly pass SONARQUBE_* in the su command line.

@gbonnefille gbonnefille linked a pull request Oct 9, 2020 that will close this issue
13 tasks
@begarco begarco added this to the 2.3.0 milestone Oct 30, 2020
@diegorodriguez31 diegorodriguez31 modified the milestones: 3.0.0, 3.1.0 Aug 23, 2023
@Topin2001 Topin2001 removed this from the 3.1.0 milestone Dec 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants