I have reviewed this and will sponsor it.
---------- Forwarded message --------- From: Alessandro Lorenzi pagure@pagure.io Date: Sat, 16 Mar 2019 at 17:24 Subject: [fedora-infrastructure] Issue #7635: Add monitoring for httpd on certgetter To: smooge@gmail.com
neron added a new comment to an issue you are following: `` Hi,
first patch in fedora-infrastructure!
``` From 191cd95874e5f2f0141032cc211f2b405255df6d Mon Sep 17 00:00:00 2001 From: Alessandro Lorenzi alorenzi@alorenzi.eu Date: Sat, 16 Mar 2019 22:10:12 +0100 Subject: [PATCH] Monitoring: add service certgetter http
Refs: #7635 --- roles/nagios_server/files/nagios/services/certgetter.cfg | 6 ++++++ 1 file changed, 6 insertions(+) create mode 100644 roles/nagios_server/files/nagios/services/certgetter.cfg
diff --git a/roles/nagios_server/files/nagios/services/certgetter.cfg b/roles/nagios_server/files/nagios/services/certgetter.cfg new file mode 100644 index 000000000..2b143ed23 --- /dev/null +++ b/roles/nagios_server/files/nagios/services/certgetter.cfg @@ -0,0 +1,6 @@ +define service { + host_name certgetter01.phx2.fedoraproject.org + service_description certgetter-http + check_command check_http!certgetter01.phx2.fedoraproject.org + use defaulttemplate +} -- 2.17.1
``` ``
To reply, visit the link below or just reply to this email https://pagure.io/fedora-infrastructure/issue/7635
On Wed, Mar 20, 2019 at 07:56:10AM -0400, Stephen John Smoogen wrote:
I have reviewed this and will sponsor it.
---------- Forwarded message --------- From: Alessandro Lorenzi pagure@pagure.io Date: Sat, 16 Mar 2019 at 17:24 Subject: [fedora-infrastructure] Issue #7635: Add monitoring for httpd on certgetter To: smooge@gmail.com
neron added a new comment to an issue you are following: `` Hi,
first patch in fedora-infrastructure!
From 191cd95874e5f2f0141032cc211f2b405255df6d Mon Sep 17 00:00:00 2001 From: Alessandro Lorenzi <alorenzi@alorenzi.eu> Date: Sat, 16 Mar 2019 22:10:12 +0100 Subject: [PATCH] Monitoring: add service certgetter http Refs: #7635 --- roles/nagios_server/files/nagios/services/certgetter.cfg | 6 ++++++ 1 file changed, 6 insertions(+) create mode 100644 roles/nagios_server/files/nagios/services/certgetter.cfg diff --git a/roles/nagios_server/files/nagios/services/certgetter.cfg b/roles/nagios_server/files/nagios/services/certgetter.cfg new file mode 100644 index 000000000..2b143ed23 --- /dev/null +++ b/roles/nagios_server/files/nagios/services/certgetter.cfg @@ -0,0 +1,6 @@ +define service { + host_name certgetter01.phx2.fedoraproject.org + service_description certgetter-http + check_command check_http!certgetter01.phx2.fedoraproject.org + use defaulttemplate +} -- 2.17.1
+1 for me
Pierre
On 3/20/19 12:56 PM, Stephen John Smoogen wrote:
I have reviewed this and will sponsor it.
---------- Forwarded message --------- From: Alessandro Lorenzi pagure@pagure.io Date: Sat, 16 Mar 2019 at 17:24 Subject: [fedora-infrastructure] Issue #7635: Add monitoring for httpd on certgetter To: smooge@gmail.com
neron added a new comment to an issue you are following: `` Hi,
first patch in fedora-infrastructure!
From 191cd95874e5f2f0141032cc211f2b405255df6d Mon Sep 17 00:00:00 2001 From: Alessandro Lorenzi <alorenzi@alorenzi.eu> Date: Sat, 16 Mar 2019 22:10:12 +0100 Subject: [PATCH] Monitoring: add service certgetter http Refs: #7635 --- roles/nagios_server/files/nagios/services/certgetter.cfg | 6 ++++++ 1 file changed, 6 insertions(+) create mode 100644 roles/nagios_server/files/nagios/services/certgetter.cfg diff --git a/roles/nagios_server/files/nagios/services/certgetter.cfg b/roles/nagios_server/files/nagios/services/certgetter.cfg new file mode 100644 index 000000000..2b143ed23 --- /dev/null +++ b/roles/nagios_server/files/nagios/services/certgetter.cfg @@ -0,0 +1,6 @@ +define service { + host_name certgetter01.phx2.fedoraproject.org + service_description certgetter-http + check_command check_http!certgetter01.phx2.fedoraproject.org + use defaulttemplate +} -- 2.17.1``
Hi Stephen,
I don't know if this enough urgent to breack the freeze, but I trust you :)
thanks, Alessandro
On Wed, 20 Mar 2019 at 14:32, Alessandro Lorenzi alorenzi@alorenzi.eu wrote:
On 3/20/19 12:56 PM, Stephen John Smoogen wrote:
I have reviewed this and will sponsor it.
Hi Stephen,
I don't know if this enough urgent to breack the freeze, but I trust you :)
So I didn't cover my thinking on why it deserved a freeze break. If the system were to reboot or stop, we could end up with certs not getting renewed and outage of items during the final freeze break if it went overly long. While that is a low risk, it is a high work to debug and fix during a freeze or release. This was a low risk and should cover the problem so I weighed the two and decided to sponsor this.
thanks, Alessandro
-- Alessandro Lorenzi
+1 but note that this will need to be added to tasks/main.yml too so it gets copied over.
-re
On 3/20/19 7:56 AM, Stephen John Smoogen wrote:
I have reviewed this and will sponsor it.
---------- Forwarded message --------- From: Alessandro Lorenzi pagure@pagure.io Date: Sat, 16 Mar 2019 at 17:24 Subject: [fedora-infrastructure] Issue #7635: Add monitoring for httpd on certgetter To: smooge@gmail.com
neron added a new comment to an issue you are following: `` Hi,
first patch in fedora-infrastructure!
From 191cd95874e5f2f0141032cc211f2b405255df6d Mon Sep 17 00:00:00 2001 From: Alessandro Lorenzi <alorenzi@alorenzi.eu> Date: Sat, 16 Mar 2019 22:10:12 +0100 Subject: [PATCH] Monitoring: add service certgetter http Refs: #7635 --- roles/nagios_server/files/nagios/services/certgetter.cfg | 6 ++++++ 1 file changed, 6 insertions(+) create mode 100644 roles/nagios_server/files/nagios/services/certgetter.cfg diff --git a/roles/nagios_server/files/nagios/services/certgetter.cfg b/roles/nagios_server/files/nagios/services/certgetter.cfg new file mode 100644 index 000000000..2b143ed23 --- /dev/null +++ b/roles/nagios_server/files/nagios/services/certgetter.cfg @@ -0,0 +1,6 @@ +define service { + host_name certgetter01.phx2.fedoraproject.org + service_description certgetter-http + check_command check_http!certgetter01.phx2.fedoraproject.org + use defaulttemplate +} -- 2.17.1``
To reply, visit the link below or just reply to this email https://pagure.io/fedora-infrastructure/issue/7635
woops. well that made it even safer to apply
On Wed, 20 Mar 2019 at 15:58, Rick Elrod codeblock@elrod.me wrote:
+1 but note that this will need to be added to tasks/main.yml too so it gets copied over.
-re
On 3/20/19 7:56 AM, Stephen John Smoogen wrote:
I have reviewed this and will sponsor it.
---------- Forwarded message --------- From: Alessandro Lorenzi pagure@pagure.io Date: Sat, 16 Mar 2019 at 17:24 Subject: [fedora-infrastructure] Issue #7635: Add monitoring for httpd on certgetter To: smooge@gmail.com
neron added a new comment to an issue you are following: `` Hi,
first patch in fedora-infrastructure!
From 191cd95874e5f2f0141032cc211f2b405255df6d Mon Sep 17 00:00:00 2001 From: Alessandro Lorenzi <alorenzi@alorenzi.eu> Date: Sat, 16 Mar 2019 22:10:12 +0100 Subject: [PATCH] Monitoring: add service certgetter http Refs: #7635 --- roles/nagios_server/files/nagios/services/certgetter.cfg | 6 ++++++ 1 file changed, 6 insertions(+) create mode 100644 roles/nagios_server/files/nagios/services/certgetter.cfg diff --git a/roles/nagios_server/files/nagios/services/certgetter.cfg b/roles/nagios_server/files/nagios/services/certgetter.cfg new file mode 100644 index 000000000..2b143ed23 --- /dev/null +++ b/roles/nagios_server/files/nagios/services/certgetter.cfg @@ -0,0 +1,6 @@ +define service { + host_name certgetter01.phx2.fedoraproject.org + service_description certgetter-http + check_command check_http!certgetter01.phx2.fedoraproject.org + use defaulttemplate +} -- 2.17.1``
To reply, visit the link below or just reply to this email https://pagure.io/fedora-infrastructure/issue/7635
infrastructure mailing list -- infrastructure@lists.fedoraproject.org To unsubscribe send an email to infrastructure-leave@lists.fedoraproject.org Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/infrastructure@lists.fedorapro...
Ever safer but less useful.
Here the commit:
From 3ce22e3cae01adafe895292ee9a7ea4858dc84b8 Mon Sep 17 00:00:00 2001 From: Alessandro Lorenzi alorenzi@alorenzi.eu Date: Thu, 21 Mar 2019 09:11:55 +0100 Subject: [PATCH] role nagios_server: add certgetter to services
--- roles/nagios_server/tasks/main.yml | 1 + 1 file changed, 1 insertion(+)
diff --git a/roles/nagios_server/tasks/main.yml b/roles/nagios_server/tasks/main.yml index e91a07539..969800e77 100644 --- a/roles/nagios_server/tasks/main.yml +++ b/roles/nagios_server/tasks/main.yml @@ -185,6 +185,7 @@ with_items: - autocloud.cfg - basset.cfg + - certgetter.cfg - copr.cfg - db_backups.cfg - disk.cfg
Thanks it was applied and was put in place. We found that the default directory had no data in it and would return a 403. I did a touch index.html and that solved the problem so the check now works.
On Thu, 21 Mar 2019 at 04:15, Alessandro Lorenzi alorenzi@alorenzi.eu wrote:
Ever safer but less useful.
Here the commit:
From 3ce22e3cae01adafe895292ee9a7ea4858dc84b8 Mon Sep 17 00:00:00 2001 From: Alessandro Lorenzi alorenzi@alorenzi.eu Date: Thu, 21 Mar 2019 09:11:55 +0100 Subject: [PATCH] role nagios_server: add certgetter to services
roles/nagios_server/tasks/main.yml | 1 + 1 file changed, 1 insertion(+)
diff --git a/roles/nagios_server/tasks/main.yml b/roles/nagios_server/tasks/main.yml index e91a07539..969800e77 100644 --- a/roles/nagios_server/tasks/main.yml +++ b/roles/nagios_server/tasks/main.yml @@ -185,6 +185,7 @@ with_items: - autocloud.cfg - basset.cfg
- certgetter.cfg
- copr.cfg
- db_backups.cfg
- disk.cfg
-- 2.17.1
On 03/20/2019 09:00 PM, Stephen John Smoogen wrote:
woops. well that made it even safer to apply
On Wed, 20 Mar 2019 at 15:58, Rick Elrod codeblock@elrod.me wrote:
+1 but note that this will need to be added to tasks/main.yml too so it gets copied over.
-re
On 3/20/19 7:56 AM, Stephen John Smoogen wrote:
I have reviewed this and will sponsor it.
---------- Forwarded message --------- From: Alessandro Lorenzi pagure@pagure.io Date: Sat, 16 Mar 2019 at 17:24 Subject: [fedora-infrastructure] Issue #7635: Add monitoring for httpd on certgetter To: smooge@gmail.com
neron added a new comment to an issue you are following: `` Hi,
first patch in fedora-infrastructure!
From 191cd95874e5f2f0141032cc211f2b405255df6d Mon Sep 17 00:00:00 2001 From: Alessandro Lorenzi <alorenzi@alorenzi.eu> Date: Sat, 16 Mar 2019 22:10:12 +0100 Subject: [PATCH] Monitoring: add service certgetter http Refs: #7635 --- roles/nagios_server/files/nagios/services/certgetter.cfg | 6 ++++++ 1 file changed, 6 insertions(+) create mode 100644 roles/nagios_server/files/nagios/services/certgetter.cfg diff --git a/roles/nagios_server/files/nagios/services/certgetter.cfg b/roles/nagios_server/files/nagios/services/certgetter.cfg new file mode 100644 index 000000000..2b143ed23 --- /dev/null +++ b/roles/nagios_server/files/nagios/services/certgetter.cfg @@ -0,0 +1,6 @@ +define service { + host_name certgetter01.phx2.fedoraproject.org + service_description certgetter-http + check_command check_http!certgetter01.phx2.fedoraproject.org + use defaulttemplate +} -- 2.17.1``
To reply, visit the link below or just reply to this email https://pagure.io/fedora-infrastructure/issue/7635
infrastructure mailing list -- infrastructure@lists.fedoraproject.org To unsubscribe send an email to infrastructure-leave@lists.fedoraproject.org Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/infrastructure@lists.fedorapro...
infrastructure@lists.fedoraproject.org
-
Alessandro Lorenzi -
Kevin Fenzi -
Pierre-Yves Chibon -
Rick Elrod -
Stephen John Smoogen