-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
improve jest/sonar integration. #19531
Conversation
I saw some warnings, will try to fix. |
bf9c006
to
8d99f43
Compare
8d99f43
to
037966c
Compare
Testing this now... |
Looks like there are some new issues:
|
@mshima Any ideas on how to fix the errors I experienced above? |
@mraible I will test, have you run gradle test |
I did not use
Our Sonar docs also use this command, albeit with an additional I just tried
|
Looks like jest-sonar is not compatible with |
Related: I had to add the following to the Sonar image for it to work on M1.
Should I create a PR for this? |
Hmmm, Docker shows it started, but it's unreachable, so adding
|
Sonar docker containter doesn’t support arm64 and fails bad with amd64 image at apple silicon. |
I just came across your PR. As Marcelo pointed out, the official SonarQube image available on Docker Hub does not support the execution on Apple M1 yet. However, I can give you some info on how to get the latest SonarQube running as a container on M1. SonarQube uses a base image (currently Please let me know if this comment solves your problems with SonarQube and how we can help further on this 😃 |
Hi @carminevassallo, Thanks for providing this information. Do you know when your official image will support Apple M1? Our users expect everything to work out-of-the-box and we'll have to provide this workaround for them (both in my JHipster Mini-Book and in our docs). We currently have a workaround for Keycloak documented, but it's no longer needed as of Keycloak v18. Do you have a similar list of steps we could use? FWIW, here are our current versions of SonarQube: https://github.com/jhipster/generator-jhipster/blob/main/generators/generator-constants.js#L88-L89 And the Docker Compose file we distribute with new apps: https://github.com/jhipster/generator-jhipster/blob/main/generators/server/templates/src/main/docker/sonar.yml.ejs |
Hi @mraible, We do not have a date for supporting Apple M1 yet, but this is already under discussion as it requires some adaptations in our docker release process. I’ll follow up on this when I know more. In the meantime, I see that you are using the sonarqube 9.6.0 community edition. We can use a workaround similar to the Keycloak one. To build and run the latest sonarqube 9 community edition, I’d suggest these steps:
Depending on your docker setup, you should be able to access sonarqube at http://localhost:9000/. Please let me know if this workaround is useful in your case. |
@carminevassallo Is there a way to checkout a particular version like we can with Keycloak?
We're currently using |
I think I figured it out:
Then, I ran the following in my JHipster app's directory:
Then, I ran |
Indeed, we have a tag for each release, so it's possible to do a checkout similar to keycloak. Anyways, 9.6.1 contains important bug fixes, you might want to move to that version soon :) |
@carminevassallo I agree we should upgrade. I'll work on that. Here's a PR to add the workaround to our docs. jhipster/jhipster.github.io#1228 |
Hi @mraible, I have good news for you on this topic. With the new SonarQube LTS launched last month, we do support Apple M1 in our docker image. Therefore, if you set your docker-compose file to use the |
@carminevassallo It looks like we might already be using the version you recommend: https://github.com/jhipster/generator-jhipster/blob/main/generators/server/templates/Dockerfile#L49 @DanielFran Can you confirm? If so, I can update our docs. |
@mraible Yes, I confirm! |
This warning is [no longer necessary](jhipster/generator-jhipster#19531 (comment)).
I created a PR to remove this warning in our docs. jhipster/jhipster.github.io#1269 |
This warning is [no longer necessary](jhipster/generator-jhipster#19531 (comment)).
Fixes #19433
Switch from archived jest-sonar-reporter to jest-sonar
Please make sure the below checklist is followed for Pull Requests.
When you are still working on the PR, consider converting it to Draft (below reviewers) and adding
skip-ci
label, you can still see CI build result at your branch.