-
Notifications
You must be signed in to change notification settings - Fork 631
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
First pass at: #2
Conversation
duftler
commented
Feb 18, 2017
- Credentials handling
- Google configuration
- Stackdriver configuration
- /fetch entrypoint (mainly a placeholder at the moment)
@lwander please review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
public interface MetricsService { | ||
boolean servicesAccount(String accountName); | ||
Map queryMetrics(String accountName, | ||
String instanceNamePrefix, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is Kayenta going to be restricted to only canarying instances, or is this just part of the first pass?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just first pass stuff.
kayenta-google/lombok.config
Outdated
@@ -0,0 +1,2 @@ | |||
lombok.nonNull.exceptionType = IllegalArgumentException |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any reason why we aren't configuring lombok for the whole project, but rather just the submodules?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No good reason, just evolved that way. Created just one in the root dir and deleted the others.
String jsonKey = googleManagedAccount.getJsonKey(); | ||
GoogleCredentials googleCredentials = | ||
StringUtils.hasLength(jsonKey) | ||
? new GoogleJsonCredentials(project, jsonKey) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sigh
- Credentials handling - Google configuration - Stackdriver configuration - /fetch entrypoint (mainly a placeholder at the moment)