Skip to content
This repository has been archived by the owner on Oct 2, 2023. It is now read-only.

Fix/sonar multiple servers #2973

Closed

Conversation

alzafacon
Copy link
Contributor

Fix #2972 by moving metric strings and version numbers out of properties file and into collector
the Config from admin page overrides any config from properties file
Fix #2971 by supporting multiple usernames and passwords for multiple sonar instances

}
}
// extract unique collector item IDs from components
// (in this context collector_items are sonar projects)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@rvema rvema self-assigned this Jun 28, 2019
@rvema
Copy link
Contributor

rvema commented Jun 29, 2019

@fidelcoria-aa When I try to run the collector locally, I get errors especially around metrics section. I am moving this repository to a new location this weekend. Can we setup a call to discuss this PR?

@alzafacon
Copy link
Contributor Author

We are using Sonar versions 6.0 and 6.7. I don't see any metrics errors. What version are you testing with?
@rvema, send me an email with the call details.

@rvema
Copy link
Contributor

rvema commented Jul 4, 2019

@fidelcoria-aa I will setup a call next week and send you the details of the meeting. Meanwhile we have moved the sonar collector to its own repository can you please close this PR and create it against the new repo? Here is the link to the same - https://github.com/Hygieia/hygieia-codequality-sonar-collector

@alzafacon
Copy link
Contributor Author

Yes, sir!

@alzafacon alzafacon closed this Jul 5, 2019
@alzafacon alzafacon deleted the fix/sonar-multiple-servers branch July 11, 2019 15:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
2 participants