inverse-inc / packetfence-monitoring-scripts

PacketFence Monitoring Scripts
2 stars 5 forks source link

Fix logrotate check #8

Open dwlfrth opened 7 years ago

dwlfrth commented 7 years ago

Check if status file exists rather than assuming it exists and fails with an awk error

dwlfrth commented 7 years ago

Review me and regenerate monit script once merged

dwlfrth commented 7 years ago

Bump

jrouzierinverse commented 7 years ago

https://www.youtube.com/watch?v=A_af256mnTE

dwlfrth commented 7 years ago

TIL @jrouzierinverse listen to Ed Sheeran

jrouzierinverse commented 7 years ago

Not by choice I do have a daughter

dwlfrth commented 7 years ago

Of course, blame it on the daughter since she don't have a Github account which means she can't defend herself !

jrouzierinverse commented 7 years ago

Your point is?

James Rouzier jrouzier@inverse.ca :: +1.514.447.4918 (x115) :: http://www.inverse.ca Inverse inc. :: Leaders behind SOGo (http://www.sogo.nu) and PacketFence (http://www.packetfence.org)

On 2016-12-16 11:06 AM, Derek Wuelfrath wrote:

Of course, blame it on the daughter since she don't have a Github account which means she can't defend herself !

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/inverse-inc/packetfence-monitoring-scripts/pull/8#issuecomment-267628116, or mute the thread https://github.com/notifications/unsubscribe-auth/ACmeMYh1uG-AgF6iJ1iu5vyskopDfPaqks5rIrcbgaJpZM4LJGdf.

dwlfrth commented 7 years ago

I saw your Spotify history ;)

jrouzierinverse commented 7 years ago

That is funny I don't even have spotify

James Rouzier jrouzier@inverse.ca :: +1.514.447.4918 (x115) :: http://www.inverse.ca Inverse inc. :: Leaders behind SOGo (http://www.sogo.nu) and PacketFence (http://www.packetfence.org)

On 2016-12-16 11:09 AM, Derek Wuelfrath wrote:

I saw your Spotify history ;)

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/inverse-inc/packetfence-monitoring-scripts/pull/8#issuecomment-267628676, or mute the thread https://github.com/notifications/unsubscribe-auth/ACmeMTxwiWiV9TJaGf7oViayjhvlpiygks5rIreggaJpZM4LJGdf.

julsemaan commented 7 years ago

@jrouzierinverse

Then imagine how good Derek is to find intel!

jrouzierinverse commented 7 years ago

So that is where Trump got all his facts from.

Well Derek and the Russians

James Rouzier jrouzier@inverse.ca :: +1.514.447.4918 (x115) :: http://www.inverse.ca Inverse inc. :: Leaders behind SOGo (http://www.sogo.nu) and PacketFence (http://www.packetfence.org)

On 2016-12-16 11:17 AM, Julien Semaan wrote:

@jrouzierinverse https://github.com/jrouzierinverse

Then imagine how good Derek is to find intel!

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/inverse-inc/packetfence-monitoring-scripts/pull/8#issuecomment-267630862, or mute the thread https://github.com/notifications/unsubscribe-auth/ACmeMVdUQVVHtTBqKbZuT_Da3ca5uqLFks5rIrmvgaJpZM4LJGdf.

dwlfrth commented 7 years ago

Chuuut. Don't tell. "Russians" ...

dwlfrth commented 7 years ago

Soo I think that PR got the attention it was waiting for. What about reviewing it now ? ;)

julsemaan commented 7 years ago

@dwlfrth

Soo I think that PR got the attention it was waiting for. What about reviewing it now ? ;)

Ha! you're funny. That's not what Github is for...

dwlfrth commented 7 years ago

https://www.youtube.com/watch?v=HyTpu6BmE88

julsemaan commented 7 years ago

@dwlfrth

Can you perform the requested changes to this as everytime github cdn has capacity issues, this triggers a false positive alert

dwlfrth commented 7 years ago

Which changeS are we talking about ?

julsemaan commented 7 years ago

Its a change sorry, I just blindly said changes and don't bother speeding this up as it doesn't fix the curl return code issue

CLAassistant commented 5 years ago

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


Derek Wuelfrath seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.