On Tue, Apr 14, 2020 at 04:03:59PM +0200, Clement Verna wrote:
> On Tue, 14 Apr 2020 at 14:33, Pierre-Yves Chibon <[1]pingou@pingoured.fr>
> wrote:
>
> Good Morning,
>
> While working on [2]https://pagure.io/fedora-infrastructure/issue/8035
> to help
> Kevin I ended up adding the following commit to the proxies and running
> the
> proxy playbook for it:
>
> commit d6fe2e08a6897470cdaf3ecd191c5329dca49b3c
> Author: Pierre-Yves Chibon <[3]pingou@pingoured.fr>
> Date: Tue Apr 14 11:25:17 2020 +0200
>
> proxies-website: Add kerneltest to the proxies
>
> Signed-off-by: Pierre-Yves Chibon <[4]pingou@pingoured.fr>
>
> diff --git a/playbooks/include/proxies-websites.yml
> b/playbooks/include/proxies-websites.yml
> index a18f7a3a3..5159ecdff 100644
> --- a/playbooks/include/proxies-websites.yml
> +++ b/playbooks/include/proxies-websites.yml
> @@ -996,6 +996,13 @@
> cert_name: "{{wildcard_cert_name}}"
> tags: calendar
>
> + - role: httpd/website
> + site_name: [5]kerneltest.fedoraproject.org
> + sslonly: true
> + server_aliases: [[6]kerneltest.stg.fedoraproject.org]
> + cert_name: "{{wildcard_cert_name}}"
> + tags: kerneltest
> +
> # fedorahosted is retired. We have the site here so we can redirect it.
>
> - role: httpd/website
>
> I had totally forgot proxies were frozen, so my apologies for pushing
> this
> through and I'd like to ask for a post-fact FBR.
>
> With this I got kerneltest mostly working expect that it doesn't send
> back the
> expected html page.
> Clément took a look at it and pointed me to the reverse proxy which
> haven't been
> configured for kerneltest.
> So I'd like to apply the following patch and run the corresponding
> playbook
>
> I think there is a copy/paste error, the commit below is the same as the
> one above :-)
Indeed, here is the correct one:
commit ad1224c3f09d75f02362e721110a565676575e99 (HEAD -> master)
Author: Pierre-Yves Chibon <pingou@pingoured.fr>
Date: Tue Apr 14 14:19:30 2020 +0200
proxies: add kerneltest.fp.o reverse proxy so it points to openshift
Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>
diff --git a/playbooks/include/proxies-reverseproxy.yml b/playbooks/include/proxies-reverseproxy.yml
index dd80b5e34..1bb5cf87a 100644
--- a/playbooks/include/proxies-reverseproxy.yml
+++ b/playbooks/include/proxies-reverseproxy.yml
@@ -234,6 +234,15 @@
balancer_name: app-os
targettype: openshift
keephost: true
+
+ - role: httpd/reverseproxy
+ website: kerneltest.fedoraproject.org
+ destname: kerneltest
+ balancer_name: app-os
+ targettype: openshift
+ keephost: true
+ tags: kerneltest
+ header_scheme: true
I think this was from the previous block, so the block should be added after that :)
when: env == "staging"
- role: httpd/reverseproxy
one small thing to fix, but +1 from me.
Pierre
_______________________________________________
infrastructure mailing list -- infrastructure@lists.fedoraproject.org
To unsubscribe send an email to infrastructure-leave@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