R. Tyler Croy rtyler

Organizations

@lookout @jenkinsci @jenkins-infra @jruby-gradle @fosdem-ruby
rtyler commented on issue lookout/metrics-datadog#3
@rtyler

One thing I missed, based on the discussion @polynomial and I have been having: An operator should be able to overwrite a threshold value with a y…

@rtyler
Programmatic support for setting up "Monitors" (aka thresholds)
rtyler commented on pull request lookout/deploydb#122
@rtyler

Fixed Deployment hashCode function causing HibernateExpcetion. In the future, please send notable changes like that in a separate pull request.

rtyler commented on pull request lookout/deploydb#122
@rtyler

Does output need to be flushed? I would hope that the caller of execute() would do that for us

rtyler commented on pull request lookout/deploydb#122
@rtyler

I don't understand the use of Session or Transactions below, we're not making any database related queries in this process (as far as I can tell/ho…

rtyler commented on pull request lookout/deploydb#122
@rtyler

I think this should include some javadocs on why Workflow is necessary here

rtyler commented on pull request lookout/deploydb#122
@rtyler

Can you expand on why this is the case? Also does this mean every deployment must be either COMPLETED or CREATED in order for config reload to work?

@rtyler

So all the feature file examples I could find for the reporting webhook URL and its payload seem to be related to deployment creation. i.e. There …

@rtyler
  • @rtyler 74f0091
    Merge pull request #121 from maheshkelkar/issues/120-define-reload-api
  • @maheshkelkar 1045648
    Change the config reload should be an admin task
  • 1 more commit »
rtyler merged pull request lookout/deploydb#121
@rtyler
Defined REST API for configuration reload
2 commits with 19 additions and 0 deletions
rtyler commented on pull request lookout/deploydb#121
@rtyler

I think we need to find a way to get this onto the admin servlet, but the API is fine as far as I'm concerned right now

rtyler commented on pull request lookout/Hermann#86
@rtyler

Isn't it annoying that this thing won't go away?

rtyler commented on pull request lookout/Hermann#86
@rtyler

Curator was in the Jarfile more to allow for local testing using Curator. Curator isn't actually a requirement for Hermann to be run or used of the…

rtyler pushed to master at lookout/leafy
@rtyler
  • @rtyler 72a2adb
    Add travis badge to the README
rtyler pushed to master at lookout/leafy
@rtyler
rtyler merged pull request lookout/leafy#37
@rtyler
Datadog example
3 commits with 138 additions and 29 deletions
rtyler edited the lookout/leafy wiki
rtyler pushed to master at lookout/leafy
@rtyler
rtyler merged pull request lookout/leafy#36
@rtyler
Gauge API
2 commits with 29 additions and 29 deletions
rtyler commented on issue lookout/Hermann#85
@rtyler

For additional context, since @sdaswani-lookout and I discussed this over IM. He can in fact build librdkafka 0.8.4 standalone on his machine

@rtyler
@rtyler
Webhooks should have a `type` field to differentiate event types
rtyler merged pull request lookout/deploydb#119
@rtyler
Issue 106 webhook content type for events
3 commits with 117 additions and 38 deletions
rtyler commented on pull request lookout/deploydb#119
@rtyler

The string concatenation looks brittle but I don't have better suggestions right now

rtyler commented on issue lookout/metrics-datadog#2
@rtyler

published in 0.1.4

@rtyler
  • @rtyler e17cc15
    Pin our metrics-datadog dependencies to avoid goofy poms
  • @rtyler 0dcb8f4
    Publish javadoc and sources jars when we publish artifacts
  • 1 more commit »
@rtyler
invalid metrics-datadog-0.1.3.pom
rtyler commented on issue lookout/metrics-datadog#2
@rtyler

Silly gradle, I'll fix it

@rtyler
  • @rtyler 80df66d
    Starting to implement LocalGemDependency to provide a Local gem into …
  • @rtyler b015b46
    Add the groovydocs jar to the archives published to bintray
  • 1 more commit »