Re: [FBR] Add kerneltest.fp.o reverse proxy + post-fact FBR

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





On Tue, 14 Apr 2020 at 16:38, Pierre-Yves Chibon <pingou@xxxxxxxxxxxx> wrote:
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@xxxxxxxxxxxx>
>    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@xxxxxxxxxxxx>
>      Date:   Tue Apr 14 11:25:17 2020 +0200
>
>          proxies-website: Add kerneltest to the proxies
>
>          Signed-off-by: Pierre-Yves Chibon <[4]pingou@xxxxxxxxxxxx>
>
>      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@xxxxxxxxxxxx>
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@xxxxxxxxxxxx>

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@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to infrastructure-leave@xxxxxxxxxxxxxxxxxxxxxxx
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@xxxxxxxxxxxxxxxxxxxxxxx
_______________________________________________
infrastructure mailing list -- infrastructure@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to infrastructure-leave@xxxxxxxxxxxxxxxxxxxxxxx
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@xxxxxxxxxxxxxxxxxxxxxxx

[Index of Archives]     [Fedora Development]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]

  Powered by Linux