-
Notifications
You must be signed in to change notification settings - Fork 51
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
Bring back sonar build to jenkins file #312
Conversation
I see that I'm not the only one suffering with the Jenkinsfile syntax :P |
And as the jenkins build tells us we are not there yet. I asked fredrik if he has a tip for me. |
It looks like But I think it can be chosen arbitrarily, so perhaps it's enough if you just set it as e.g. |
Ah thx, changed that lets see. |
We are a step further. Now we only need to fix how the secret is coming in. As I'm totally new to jenkins files and doing this trial and error based I would be happy for tips. if it helps I can provide a screenshot how it is done in a Jenkins ui setup. |
Thx for all the explanation. I'l gave it a try lets see! |
Believe me, I've been there. 😅 Trying to figure out how anything works on Jenkins is always a real treat... |
I think I completely misunderstood the purpose of the
Obviously, the error message doesn't show which installations are available... |
@azoitl Can you access https://ci.eclipse.org/gef/configureTools? There should normally be an entry of the installed SonarQube Scanner instance. |
Just grasping at straws here, but the other job includes something like "GEF_Classic_SonarQube.io". Can you try out that one as installation? |
And if that doesn't work, perhaps simply "SonarCloud.io"...
|
16c6e53
to
4ce353f
Compare
And now I'm out of ideas :( |
Unfortunately not. I tried now: GEF_Classic_SonarQube.io, SonarCloud.io, My SonarQube Server will ask in Eclipse helpdesk. Keep you updated. |
On the bright side, this seems to be the final hurdle. I already consider the fact that the other job works a small victory 😄 |
@ptziegler @fredg02 was so kind and provided a patch fixing our problem in PR #318. So finally we have sonar analysis running. |
@ptziegler in theory this should do it.