Fix agent apps #1724

Merged
merged 3 commits into from Aug 20, 2015

Projects

None yet

2 participants

@tuxis-ie
Contributor

The unix-agent-mechanism can use some love. This is a first bit of love.

Application-pollers were run double. Once by the unix-agent include and afterwards by the application-poller. That is now fixed.

@f0o f0o self-assigned this Aug 20, 2015
@f0o f0o added the Polling label Aug 20, 2015
@f0o
Member
f0o commented Aug 20, 2015

@tuxis-ie this contains syntax errors.

$ php -l includes/polling/unix-agent.inc.php
PHP Parse error:  syntax error, unexpected '' (T_ENCAPSED_AND_WHITESPACE), expecting identifier (T_STRING) or variable (T_VARIABLE) or number (T_NUM_STRING) in includes/polling/unix-agent.inc.php on line 107
@tuxis-ie
Contributor

Oh, crap. I'll fix and test.

Mark Schouten
Tuxis Internet Engineering
mark@tuxis.nl / 0318 200208

On 20 Aug 2015, at 13:04, "Daniel Preussker" notifications@github.com wrote:

@tuxis-ie this contains syntax errors.

$ php -l includes/polling/unix-agent.inc.php
PHP Parse error: syntax error, unexpected '' (T_ENCAPSED_AND_WHITESPACE), expecting identifier (T_STRING) or variable (T_VARIABLE) or number (T_NUM_STRING) in includes/polling/unix-agent.inc.php on line 107

Reply to this email directly or view it on GitHub.

@tuxis-ie tuxis-ie Fix typos (Sorry about that)
Replace if($debug){..} with d_echo
47c87cb
@f0o f0o merged commit 9844abb into librenms:master Aug 20, 2015

1 check passed

Scrutinizer No new issues
Details
@tuxis-ie tuxis-ie deleted the tuxis-ie:fix-agent-apps branch Aug 24, 2015
@tuxis-ie tuxis-ie added a commit to tuxis-ie/librenms that referenced this pull request Aug 24, 2015
@tuxis-ie tuxis-ie PR #1724 might have broken memcached detection. I think this should f…
…ix it
dd0fab1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment