Re: [PATCH] proxies: redirect apps.fp.o/calendar to calendar.fp.o in openshift

2021-04-07 Thread Mark O'Brien
+1 LGTM

On Tue, Apr 6, 2021 at 9:42 PM Kevin Fenzi  wrote:

> On Tue, Apr 06, 2021 at 09:42:35PM +0200, Pierre-Yves Chibon wrote:
> > Signed-off-by: Pierre-Yves Chibon 
> > ---
> >  playbooks/include/proxies-redirects.yml | 6 ++
> >  1 file changed, 6 insertions(+)
> >
> > diff --git a/playbooks/include/proxies-redirects.yml
> b/playbooks/include/proxies-redirects.yml
> > index 17a536149..49f2bb418 100644
> > --- a/playbooks/include/proxies-redirects.yml
> > +++ b/playbooks/include/proxies-redirects.yml
> > @@ -111,6 +111,12 @@
> >  regex: /voting
> >  target: https://elections.fedoraproject.org/
> >
> > +  - role: httpd/redirectmatch
> > +shortname: calendar
> > +website: apps.fedoraproject.org
> > +regex: /calendar
> > +target: https://calendar.fedoraproject.org/
> > +
> >- role: httpd/redirectmatch
> >  shortname: mailman
> >  website: admin.fedoraproject.org
> > --
> > 2.30.2
>
> Looks ok to me. +1
>
> kevin
> ___
> infrastructure mailing list -- infrastructure@lists.fedoraproject.org
> To unsubscribe send an email to
> infrastructure-le...@lists.fedoraproject.org
> Fedora Code of Conduct:
> https://docs.fedoraproject.org/en-US/project/code-of-conduct/
> List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
> List Archives:
> https://lists.fedoraproject.org/archives/list/infrastructure@lists.fedoraproject.org
> Do not reply to spam on the list, report it:
> https://pagure.io/fedora-infrastructure
>
___
infrastructure mailing list -- infrastructure@lists.fedoraproject.org
To unsubscribe send an email to infrastructure-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/infrastructure@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


Re: [PATCH] proxies: redirect apps.fp.o/calendar to calendar.fp.o in openshift

2021-04-06 Thread Kevin Fenzi
On Tue, Apr 06, 2021 at 09:42:35PM +0200, Pierre-Yves Chibon wrote:
> Signed-off-by: Pierre-Yves Chibon 
> ---
>  playbooks/include/proxies-redirects.yml | 6 ++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/playbooks/include/proxies-redirects.yml 
> b/playbooks/include/proxies-redirects.yml
> index 17a536149..49f2bb418 100644
> --- a/playbooks/include/proxies-redirects.yml
> +++ b/playbooks/include/proxies-redirects.yml
> @@ -111,6 +111,12 @@
>  regex: /voting
>  target: https://elections.fedoraproject.org/
>  
> +  - role: httpd/redirectmatch
> +shortname: calendar
> +website: apps.fedoraproject.org
> +regex: /calendar
> +target: https://calendar.fedoraproject.org/
> +
>- role: httpd/redirectmatch
>  shortname: mailman
>  website: admin.fedoraproject.org
> -- 
> 2.30.2

Looks ok to me. +1

kevin


signature.asc
Description: PGP signature
___
infrastructure mailing list -- infrastructure@lists.fedoraproject.org
To unsubscribe send an email to infrastructure-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/infrastructure@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


Re: [PATCH] proxies: redirect apps.fp.o/calendar to calendar.fp.o in openshift

2021-04-06 Thread Nick Bebout
This looks good to me.  +1

On Tue, Apr 6, 2021 at 2:43 PM Pierre-Yves Chibon 
wrote:

> Signed-off-by: Pierre-Yves Chibon 
> ---
>  playbooks/include/proxies-redirects.yml | 6 ++
>  1 file changed, 6 insertions(+)
>
> diff --git a/playbooks/include/proxies-redirects.yml
> b/playbooks/include/proxies-redirects.yml
> index 17a536149..49f2bb418 100644
> --- a/playbooks/include/proxies-redirects.yml
> +++ b/playbooks/include/proxies-redirects.yml
> @@ -111,6 +111,12 @@
>  regex: /voting
>  target: https://elections.fedoraproject.org/
>
> +  - role: httpd/redirectmatch
> +shortname: calendar
> +website: apps.fedoraproject.org
> +regex: /calendar
> +target: https://calendar.fedoraproject.org/
> +
>- role: httpd/redirectmatch
>  shortname: mailman
>  website: admin.fedoraproject.org
> --
> 2.30.2
> ___
> infrastructure mailing list -- infrastructure@lists.fedoraproject.org
> To unsubscribe send an email to
> infrastructure-le...@lists.fedoraproject.org
> Fedora Code of Conduct:
> https://docs.fedoraproject.org/en-US/project/code-of-conduct/
> List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
> List Archives:
> https://lists.fedoraproject.org/archives/list/infrastructure@lists.fedoraproject.org
> Do not reply to spam on the list, report it:
> https://pagure.io/fedora-infrastructure
>
___
infrastructure mailing list -- infrastructure@lists.fedoraproject.org
To unsubscribe send an email to infrastructure-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/infrastructure@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[PATCH] proxies: redirect apps.fp.o/calendar to calendar.fp.o in openshift

2021-04-06 Thread Pierre-Yves Chibon
Signed-off-by: Pierre-Yves Chibon 
---
 playbooks/include/proxies-redirects.yml | 6 ++
 1 file changed, 6 insertions(+)

diff --git a/playbooks/include/proxies-redirects.yml 
b/playbooks/include/proxies-redirects.yml
index 17a536149..49f2bb418 100644
--- a/playbooks/include/proxies-redirects.yml
+++ b/playbooks/include/proxies-redirects.yml
@@ -111,6 +111,12 @@
 regex: /voting
 target: https://elections.fedoraproject.org/
 
+  - role: httpd/redirectmatch
+shortname: calendar
+website: apps.fedoraproject.org
+regex: /calendar
+target: https://calendar.fedoraproject.org/
+
   - role: httpd/redirectmatch
 shortname: mailman
 website: admin.fedoraproject.org
-- 
2.30.2
___
infrastructure mailing list -- infrastructure@lists.fedoraproject.org
To unsubscribe send an email to infrastructure-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/infrastructure@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[PATCH] proxies: redirect apps.fp.o/calendar to calendar.fp.o in openshift

2021-04-06 Thread Pierre-Yves Chibon
Signed-off-by: Pierre-Yves Chibon 
---
 playbooks/include/proxies-redirects.yml | 6 ++
 1 file changed, 6 insertions(+)

diff --git a/playbooks/include/proxies-redirects.yml 
b/playbooks/include/proxies-redirects.yml
index 17a536149..4d5223ad5 100644
--- a/playbooks/include/proxies-redirects.yml
+++ b/playbooks/include/proxies-redirects.yml
@@ -111,6 +111,12 @@
 regex: /voting
 target: https://elections.fedoraproject.org/
 
+  - role: httpd/redirectmatch
+shortname: calendar
+website: admin.fedoraproject.org
+regex: /calendar
+target: https://calendar.fedoraproject.org/
+
   - role: httpd/redirectmatch
 shortname: mailman
 website: admin.fedoraproject.org
-- 
2.30.2
___
infrastructure mailing list -- infrastructure@lists.fedoraproject.org
To unsubscribe send an email to infrastructure-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/infrastructure@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure