Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(85)

Issue 69980048: replaces charm-helpers-sh call with nslookup

Can't Edit
Can't Publish+Mail
Start Review
Created:
7 years, 9 months ago by lazypower
Modified:
7 years, 8 months ago
Reviewers:
mp+209137, mbruzek
Visibility:
Public.

Description

replaces charm-helpers-sh call with nslookup https://code.launchpad.net/~lazypower/charms/precise/nrpe/fix-lp-1287393/+merge/209137 (do not edit description out of merge proposal)

Patch Set 1 #

Total comments: 1

Patch Set 2 : replaces charm-helpers-sh call with nslookup #

Unified diffs Side-by-side diffs Delta from patch set Stats (+191 lines, -1 line) Patch
A [revision details] View 1 1 chunk +2 lines, -0 lines 0 comments Download
M hooks/nrpe-relation-joined View 1 1 chunk +1 line, -1 line 0 comments Download
A lib/net.sh View 1 1 chunk +188 lines, -0 lines 0 comments Download

Messages

Total messages: 3
lazypower
Please take a look.
7 years, 9 months ago (2014-03-03 22:05:51 UTC) #1
mbruzek
I have a comment/question about the new command you are using. https://codereview.appspot.com/69980048/diff/1/hooks/nrpe-relation-joined File hooks/nrpe-relation-joined (right): ...
7 years, 8 months ago (2014-03-17 14:00:29 UTC) #2
lazypower
7 years, 8 months ago (2014-03-19 13:03:58 UTC) #3
Please take a look.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b