This things warns about not being able to log and it emails pingou every half hour.
Can I get two +1s to apply it?
--- roles/fedimg/files/cron/kill_ec2_nodes.py | 3 +++ 1 file changed, 3 insertions(+)
diff --git a/roles/fedimg/files/cron/kill_ec2_nodes.py b/roles/fedimg/files/cron/kill_ec2_nodes.py index 7e654f7..4bfb96c 100644 --- a/roles/fedimg/files/cron/kill_ec2_nodes.py +++ b/roles/fedimg/files/cron/kill_ec2_nodes.py @@ -4,12 +4,15 @@ # NOTE this is taken from the github repo # https://github.com/fedora-infra/fedimg/blob/develop/bin/kill_ec2_nodes.py
+import logging +logging.basicConfig()
from libcloud.compute.types import Provider from libcloud.compute.providers import get_driver import datetime import fedimg
+ EC2_ACCESS_ID = fedimg.AWS_ACCESS_ID EC2_SECRET_KEY = fedimg.AWS_SECRET_KEY
On Wed, Oct 21, 2015 at 10:34:51AM -0400, Ralph Bean wrote:
roles/fedimg/files/cron/kill_ec2_nodes.py | 3 +++ 1 file changed, 3 insertions(+)
diff --git a/roles/fedimg/files/cron/kill_ec2_nodes.py b/roles/fedimg/files/cron/kill_ec2_nodes.py index 7e654f7..4bfb96c 100644 --- a/roles/fedimg/files/cron/kill_ec2_nodes.py +++ b/roles/fedimg/files/cron/kill_ec2_nodes.py @@ -4,12 +4,15 @@ # NOTE this is taken from the github repo # https://github.com/fedora-infra/fedimg/blob/develop/bin/kill_ec2_nodes.py
+import logging +logging.basicConfig()
from libcloud.compute.types import Provider from libcloud.compute.providers import get_driver import datetime import fedimg
EC2_ACCESS_ID = fedimg.AWS_ACCESS_ID EC2_SECRET_KEY = fedimg.AWS_SECRET_KEY
+1 for me :)
Pierre
Found the patch and it looks good (but I see pingou +1'd in the other thread.) +1 in case it is needed in this one.
On 21 October 2015 at 08:34, Ralph Bean rbean@redhat.com wrote:
This things warns about not being able to log and it emails pingou every half hour.
Can I get two +1s to apply it? _______________________________________________ infrastructure mailing list infrastructure@lists.fedoraproject.org http://lists.fedoraproject.org/admin/infrastructure@lists.fedoraproject.org
On Wed, Oct 21, 2015 at 10:34:50AM -0400, Ralph Bean wrote:
This things warns about not being able to log and it emails pingou every half hour.
The email has changed but keeps showing up as:
```` Subject: Cron fedmsg@fedimg01 /usr/share/fedimg/cronjobs/kill_ec2_nodes.py
WARNING:moksha.hub:Cannot find qpid python module. Make sure you have python-qpid installed. ````
I would like to try the following patch to fix this issue:
```` From ffa57ef08d4ca1d6693bbc64ca7ce282c1cfc68b Mon Sep 17 00:00:00 2001 From: Pierre-Yves Chibon pingou@pingoured.fr Date: Thu, 29 Oct 2015 11:56:26 +0100 Subject: [PATCH] Skip warning about missing qpid python module
--- roles/fedimg/files/cron/kill_ec2_nodes.cron | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/roles/fedimg/files/cron/kill_ec2_nodes.cron b/roles/fedimg/files/cron/kill_ec2_nodes.cron index b684748..ccb5d33 100644 --- a/roles/fedimg/files/cron/kill_ec2_nodes.cron +++ b/roles/fedimg/files/cron/kill_ec2_nodes.cron @@ -1,2 +1,2 @@ MAILTO=sysadmin-fedimg-members@fedoraproject.org -0 * * * * fedmsg /usr/share/fedimg/cronjobs/kill_ec2_nodes.py +0 * * * * fedmsg /usr/share/fedimg/cronjobs/kill_ec2_nodes.py | grep -v 'Cannot find qpid python'
Hi,
+1, but you might need to add 2>&1 as well, since I'm not sure if that warning is on stdout or stderr. I'm +1 for either version.
----- Original Message -----
On Wed, Oct 21, 2015 at 10:34:50AM -0400, Ralph Bean wrote:
This things warns about not being able to log and it emails pingou every half hour.
The email has changed but keeps showing up as:
Subject: Cron <fedmsg@fedimg01> /usr/share/fedimg/cronjobs/kill_ec2_nodes.py WARNING:moksha.hub:Cannot find qpid python module. Make sure you have python-qpid installed.
I would like to try the following patch to fix this issue:
From ffa57ef08d4ca1d6693bbc64ca7ce282c1cfc68b Mon Sep 17 00:00:00 2001 From: Pierre-Yves Chibon <pingou@pingoured.fr> Date: Thu, 29 Oct 2015 11:56:26 +0100 Subject: [PATCH] Skip warning about missing qpid python module --- roles/fedimg/files/cron/kill_ec2_nodes.cron | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/roles/fedimg/files/cron/kill_ec2_nodes.cron b/roles/fedimg/files/cron/kill_ec2_nodes.cron index b684748..ccb5d33 100644 --- a/roles/fedimg/files/cron/kill_ec2_nodes.cron +++ b/roles/fedimg/files/cron/kill_ec2_nodes.cron @@ -1,2 +1,2 @@ MAILTO=sysadmin-fedimg-members@fedoraproject.org -0 * * * * fedmsg /usr/share/fedimg/cronjobs/kill_ec2_nodes.py +0 * * * * fedmsg /usr/share/fedimg/cronjobs/kill_ec2_nodes.py | grep -v 'Cannot find qpid python' -- 2.1.0
+1/-1/comments?
Thanks, Pierre
infrastructure mailing list infrastructure@lists.fedoraproject.org http://lists.fedoraproject.org/admin/infrastructure@lists.fedoraproject.org
Sure, +1.
kevin
infrastructure@lists.fedoraproject.org