Skip to content

Replace echo | nc with simple-graphite #16

Closed
wants to merge 1 commit into from

2 participants

@tomvachon

Use simple-graphite instead of echo | nc in a subshell. It is more toleratant to failures. Add conditional logic to not execute thecommand unless the graphite host is defined

@tomvachon tomvachon Use simple-graphite instead of echo | nc in a subshell. It is more
toleratant to failures.  Add conditional logic to not execute the
command unless the graphite host is defined
a66edc1
@jgoulah

seems to be a little silly to add a dep like this for 3 lines of code

  s = TCPSocket.new(Deployinator.graphite_host,Deployinator.graphite_port)
  s.write("deploylong.#{stack}.#{type} #{duration} #{Time.now.to_i}")
  s.close

that should contain a "\n" too but you get the idea

@tomvachon

Going to do jgoulah's approach, its cleaner.

@tomvachon tomvachon closed this Feb 24, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.